Remove specific characters from string C#

Question

I have got a string ["foo","bar","buzz"] from view, i want to remove [,"&,]
I have used string x = tags.Trim(new Char[] { '[', '"', ']' }); but the output i got is foo","bar","buzz instead of foo,bar,buzz

I have tried Trimming & this but still having problem.

Solution 1

By the power of Regex!

var x = Regex.Replace(tags, @"(\[|""|\])", "");

Solution 2

As alternative, you can use a "simple" Replace

string x = tags.Replace("[","")
               .Replace("\"","")
               .Replace("]","");

It isn't fast, but it's simple.

If you need more performance you should use an alternative.

---

please note:

Solution 3

Personally, I'd switch your order of operations. For instance

String[] unformattedTags = tags.Split(',');
String[] formattedTags = unformattedTags.Select(itm => itm.Trim( '[','"',']')).ToArray();

This removes the restricted characters from each tag individually.

Comments

[Corak]

"creative" approach (not recommended): new string(tags.ToCharArray().Where(c => !"[\"]".Contains(c)).ToArray());

[Fawad Bin Tariq]

I used this but was looking for some better alternative, regex worked for me :)

[JMadelaine]

var trimmed = Regex.Replace(x, @"([|""|])", ""); Added a second doube quote in the regex due to verbatim string literal. The current answer doesn't compile.

[FCin]

To me this "simple" approach is 10x better than regex. Deciphering regex is frustrating and here you can clearly see what is going on. It is readable.

[Corak]

Also, Trim accepts params, so you can simply write String[] formattedTags = tags.Split(',').Select(item => item.Trim('"', '[', ']')).ToArray(); -- no need for new char[] { ... }

[Corak]

@A_Name_Does_Not_Matter - that would be nice, but it seems there is no overload of Replace takes char[] and string.

[A_Sk]

Yes, there is no overload method... my bad. @Stefan