Trim all rows in an array

Question

I have a line that comes as:

string email = "[e-mail], [e-mail], [e-mail]"; 

I want to split it into an array of strings

If I do this:

 string[] emails = email.Split(','); 

I get spaces before each email address (after the first):

 emails[0] = "[e-mail]" emails[1] = " [e-mail]" emails[2] = " [e-mail]" 

What is the best way to get this (the best way to parse or the way to trim all rows in an array)?

 emails[0] = "[e-mail]" emails[1] = "[e-mail]" emails[2] = "[e-mail]" 

Answer 1

You can also replace all occurrences of spaces, and so avoid the foreach loop:

 string email = "[e-mail], [e-mail], [e-mail]"; string[] emails = email.Replace(" ", "").Split(','); 

Answer 2

 emails.Split(',').Select(email => email.Trim()).ToArray() 

Answer 3

One of the following will work. I would recommend the first, as it expresses the connection string more accurately.

 string[] emails = email.Split(new string[] { ", " }, StringSplitOptions.None); string[] emails = email.Split(new char[] { ' ', ',' }, StringSplitOptions.RemoveEmptyEntries); 

Answer 4

You can use Trim ():

 string email = "[e-mail], [e-mail], [e-mail]"; string[] emails = email.Split(','); emails = (from e in emails select e.Trim()).ToArray(); 

Answer 5

Use Regex.Split to avoid cropping

 var emails = Regex.Split(email, @",\s*"); 

Answer 6

You can use a single line solution like this:

 string[] emails = text.Split(',', StringSplitOptions.RemoveEmptyEntries); Array.ForEach<string>(emails, x => emails[Array.IndexOf<string>(emails, x)] = x.Trim()); 

Answer 7

Alternatively, you can split using a regular expression of the form:

 \s*,\s* 

i.e.

 string[] emails = Regex.Split(email, @"\s*,\s*"); 

He will use the surrounding space directly.

Regular expressions usually refer to performance, but the example you provided indicates that this is what you plan to do once in your code for a short array.

Answer 8

The answer from Brian Watts is elegant and simple. It implicitly refers to an array of strings created by Split ().

Also pay attention to its extensibility if you are reading a file and want to massage the data when building the array.

 string sFileA = @"C:\Documents and Settings\FileA.txt"; string sFileB = @"C:\Documents and Settings\FileB.txt"; // Trim extraneous spaces from the first file data string[] fileAData = (from line in File.ReadAllLines( sFileA ) select line.Trim()).ToArray(); // Strip a second unneeded column from the second file data string[] fileBData = (from line in File.ReadAllLines( sFileB ) select line.Substring( 0, 21 ).Trim()).ToArray(); 

Of course, you can use the Linq => notation if you want.

 string[] fileBData = File.ReadAllLines( sFileB ).Select( line => line.Substring( 0, 21 ).Trim()).ToArray(); 

Although my answer should have been posted as a comment, I have no comments for comments yet. But I found this discussion invaluable in determining how to massage data using ReadAllLines ().

Answer 9

Use String.Trim in a foreach or if you are using the .NET 3.5+ LINQ statement.

Answer 10

If you just need to manipulate records without returning an array:

 string[] emails = text.Split(','); Array.ForEach(emails, e => e.Trim());