How to Remove comma if the cell is empty?

Question

1.80/5 (2 votes)

See more:

This is my code

C#

e.Row.Cells[1].Text = e.Row.Cells[1].Text +","+ e.Row.Cells[2].Text + "," + e.Row.Cells[3].Text;

here am merging three cells on gridview .
but am getting problem with comma if any cell is empty then also comma is printing

like
us,uk,(correct)
us, ,(like am getting)

Please give me solution for this problem.

Posted 8-May-13 0:51am

devchina

Updated 8-May-13 1:21am

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ)

v2

Add a Solution

3 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Pheonyx · Answer 1 · 2013-05-08T01:05:00

Solution 1

Somthing like this might do the job:

C#

private string CellCombination(IEnumerable<string> values)
{
	string result = "";
	foreach (string s in values)
	{
		if (!string.IsNullOrWhiteSpace(s)) result += string.Format("{0},", s);
	}

	return result.Trim(',');

}</string>

Or the same thing using Linq

C#

private string CellCombination(IEnumerable<string> values)
{
    string result = values.Where(s => !string.IsNullOrWhiteSpace(s)).Aggregate("", (current, s) => current + string.Format("{0},", s));

    return result.Trim(',');

 }

To use it, just pass in your cells into the function as an enumerable collection.

So something like this:

e.Row.Cells[1].Text = CellCombination(new [] {e.Row.Cells[1].Text, e.Row.Cells[2].Text, e.Row.Cells[3].Text });

Posted 8-May-13 1:05am

Pheonyx

Updated 8-May-13 1:09am

v3

Comments

[no name] 8-May-13 7:07am

Note that "IsNullOrWhiteSpace" is framework version dependent.

Pheonyx 8-May-13 7:10am

## Wrong ##Yep, that it is, but as long as you aren't using framework less than 2 it appears to be present.
http://msdn.microsoft.com/en-us/library/system.string.isnullorempty(v=vs.110).aspx ##

My mistake, wrong link.. its Framework 4 and newer.... IsNullorEmpty is framework 2 and later.

[no name] 8-May-13 7:12am

IsNullOrWhiteSpace and IsNullOrEmpty are completely different functions.

Pheonyx 8-May-13 7:18am

My understanding of the difference is that IsNullOrWhiteSpace does the same as IsNullOrEmpty, but also checks for white space characters such as new lines, tabs, etc.

Explained here: http://dotsandnets.wordpress.com/2011/07/30/isnullorempty-vs-isnullorwhitespace/

So I wouldn't say they are completely different, but IsNullOrWhiteSpace is an extended version of IsNullOrEmpty.

[no name] 8-May-13 12:27pm

Yes and? Not is the same is different is it not? Try calling IsNullOrWhiteSpace in .NET 3.5 and see how far you get :-)

Manoj S Rekya · Answer 2 · 2013-05-08T01:19:00

Solution 3

C#

e.Row.Cells[1].Text = e.Row.Cells[1].Text + (string.IsNullOrEmpty(e.Row.Cells[2].Text) ? string.empty : ",") + e.Row.Cells[2].Text + (string.IsNullOrEmpty(e.Row.Cells[3].Text) ? string.empty : ",") + e.Row.Cells[3].Text;

Posted 8-May-13 1:19am

Manoj S Rekya

Updated 8-May-13 4:47am

v3

Comments

devchina 8-May-13 8:04am

I am getting error cannot implicitly convert string to bool

Manoj S Rekya 8-May-13 10:49am

I have updated the solution by including braces for the ternary operator as + operator has a higher precedence than ? operator, error is thrown.

Aarti Meswania · Answer 3 · 2013-05-08T01:06:00

Solution 2

C#

e.Row.Cells[1].Text = e.Row.Cells[1].Text + ((e.Row.Cells[2].Text.ToString().Trim().Length>0)?  ("," + e.Row.Cells[2].Text) : string.Empty )+  ((e.Row.Cells[3].Text.ToString().Trim().Length>0) ?  ("," + e.Row.Cells[3].Text) : string.Empty );

Happy Coding!
:)

Posted 8-May-13 1:06am

Aarti Meswania

Updated 8-May-13 2:22am

v2

Comments

devchina 8-May-13 8:17am

This is not working I am getting same(uk, ,)

Aarti Meswania 8-May-13 8:22am

check updated solution

devchina 8-May-13 9:01am

many times i was tried but it is not working

here iam tried two cells its working
if(e.Row.Cells[7].Text == " " && e.Row.Cells[8].Text !=" " )
{
e.Row.Cells[7].Text = e.Row.Cells[8].Text;
}
else if(e.Row.Cells[8].Text != " " && e.Row.Cells[7].Text !=" ")
{
e.Row.Cells[7].Text = e.Row.Cells[7].Text + "," + e.Row.Cells[8].Text;
}

But in my case three cells
Plz provide the solution

Aarti Meswania 8-May-13 9:07am

have you try my latest solution?

devchina 8-May-13 10:13am

only one solution is here i tried that one but it is not working

Pheonyx 8-May-13 10:51am

Aarti updated his solution, did you try his version 2? or his version 1?

Did you even consider mine above? it is designed so that it is not limited to just 3 cells in case you choose to expand it in the future.

devchina 15-May-13 3:09am

e.Row.Cells[1].Text = e.Row.Cells[1].Text + ((e.Row.Cells[2].Text.ToString().Trim().Length>0)? ("," + e.Row.Cells[2].Text) : string.Empty )+ ((e.Row.Cells[3].Text.ToString().Trim().Length>0) ? ("," + e.Row.Cells[3].Text) : string.Empty );

This code is not working