construct comma separated string from "name" property of objects stored in a list

Question

On each looping, I get a value for that field. I want to store all values using comma separated in the variable (String).

var result= string.Empty;
foreach (var details in Response)
{
    result+= details.name;
}

Example data I am want:

result = "abc, de123, 15R2W" ;

while current code collapses all into one big string: "abcde12315R2W"

score 1 · Accepted Answer · answered Dec 03 '18 at 03:51

1

You can use string.Join and some Linq,

var result = string.Join(",", details.Select(x => x.name));

The Select will pull out the name string values and Join will concatenate them with a comma separating them.

answered Dec 03 '18 at 03:51

juharr

Do you mean `Response.Select(...`? – Alex Kudryashev Dec 03 '18 at 03:53
@juharr thank you buddy for timely help :) God bless you for helping me :) – Jasmine Dec 03 '18 at 03:54
you can also use `IEnumerable.Aggregate((a,b)=> $"{a}, {b}");` – Minijack Dec 03 '18 at 03:55
one quick question - Is there a way to use only distinct? As I anticipate some repeating values, but not always. – Jasmine Dec 03 '18 at 03:56
@Minijack: Too complex for me to understand – Jasmine Dec 03 '18 at 03:56
@Learner `var result = Response.Select(x=>x.name).Aggregate((a,b) => a + ", " + b);` is that a bit more clear? :) – Minijack Dec 03 '18 at 03:57
To do distinct look at: https://stackoverflow.com/questions/1300088/distinct-with-lambda – Minijack Dec 03 '18 at 04:00
1

@Minijack The problem with using `Aggregate` like this is that it's not as efficient because you're doing string concatenation in a loop. `String.Join` uses a `StringBuilder` thus avoiding the creation of a bunch of intermediate strings. – juharr Dec 03 '18 at 13:27
1

@Learner If you just want distinct names then you can put `.Distinct()` after the `Select`. – juharr Dec 03 '18 at 13:30

score -2 · Answer 2 · answered Dec 03 '18 at 04:01

-2

Try this:

var result = "";
foreach (var details in Response)
{
    result+= details.name + ",";
}
result = result.Substring(0,result.length-1)

answered Dec 03 '18 at 04:01

A.Bazan

For a useful answer this reaction needs to be extended. Explain why this is an answer to the question. – Jeroen Heier Dec 03 '18 at 04:58

2 Answers2