C# Remove multiple char types from end of string

Question

I have a loop that builds up address fields, some of these fields may be empty at the end of the string

List<string> list = new List<string>();

//list can contain any number of values, some of which might be "" (empty string)

string returnValue = "";
for (int iRow = 1; iRow <= list.Count; iRow++)
    string += String.Format("{0}, ", list[iRow]);

returnValue = returnValue.Trim();

my output is

asd, aaa, qwe, 123123, , , , ,

How can i remove the trailing ", " from the string?

Thanks

Answer 1

string's TrimEnd static method allows to specify which characters should be trimmed.

However, in your case it would make sense to check in the for loop whether you have empty slots, and at the end put together a resulting string with string.Join(string separator, string[] parts)

Answer 2

If you want to remove something you added, don't add them in the first place. Also, the StringBuilder type is better suited for concatenating multiple strings, as it much more memory efficient.

StringBuilder sb = new StringBuilder();

for (int i = 0; i < list.Count; i++)
{
    string rowValue = list[iRow];
    if (!string.IsNullOrEmpty(rowValue))
    {
        sb.Append(rowValue);
        sb.Append(", ");
    }    
}

// use sb.ToString() to obtain result

Answer 3

You should also avoid using strings in your case, instead use StringBuilder. Avoid also using senseles formatting -just list[iRow] is a better option.

Try something like this instead:

string result = string.Join(", ", 
                  list.Where(s => !string.IsNullOrEmpty(s)).ToArray());

Answer 4

I hate to make assumptions, but I will because it seems you want preserve the "gaps" until you get to the end, in which case you should use TrimEnd. If not, then use any one of the other options to avoid adding the empty values in the first place.

More precisely if your output could look like:

asd, aaa, qwe, 123123, , , somevalue , ,

Then you'll have to loop through and use TrimEnd.

Otherwise, if you can collapse the fields then exclude the empties upfront.

asd, aaa, qwe, 123123, somevalue

Answer 5

returnValue = returnValue.TrimEnd( ' ', ',' );

Answer 6

try this:

List<string> list = new List<string>(
    new string[]{"asd", "aaa", "qwe", "123123", "", null, "", null, ""});
return String.Join(", ", list.Where(i => !String.IsNullOrEmpty(i)).ToArray());