C# リストからの値の取り方について

Question

C# リストからの値の取り方について

class Sample
{
  public string No;
  public string Name;
}

このようなクラスをリストで保持しているときに
Noだけを取得しカンマ区切りで一つの文字列としたいのです。
Noが2、１、3の順にあったらそのまま ”2,1,3” としたいです。

foreachでリストを回していけばいいと思っているのですが
カンマをつける際にNo + "," とか ”,” + No と書いた場合に
”2,1,3,” ",2,1,3" のようにしたくないので
最初または最後だったらとか判定しなくちゃいけいないと思うとコードが少し
ごちゃごちゃしてしまいます。

値の取得方法やカンマのつけ方など含めて簡潔にできる方法を教えて下さい。

追記

返事が遅くなってしまいましたが
回答してくださった方ありがとうございます。

コードは結局以下の形となりました（若干どこか間違っているかもしれませんが現状このような形です）
string.Join(",", sampleList.Where(!string.IsNullOrEmpty(No) && !string.IsNullOrEmpty(Name)).Select(sample => sample.No).ToArry());

Whereの条件にNoが数値(long型)かどうかのチェックを入れたいのですが
まだ試してはいませんがTryParseだとout値はいらないです、他に数値に変換できるかどうかの関数って存在するのでしょうか？
自分で作るしかないのでしょうか？

Answer 1

string.Join() を調べて下さい。

Comments

[Zuishin]

追記に対して

Parse() を使い、例外を catch してください。

Answer 2

こんにちは。

Sampleが入ったListなのでList<Sample>型のインスタンスに対する加工とします。

まず、ListからNoを抽出するにはLinqを使うと手っ取り早いです。

sampleList.Select(sample => sample.No); // IEnumerable<string> ["2", "1", "3"]

次に、stringのストリームをカンマ区切りで結合するにはstring.Joinメソッドを使用します。

string.Join(",", sampleList.Select(sample => sample.No)); // "2,1,3"

質問にあるような"やりたいこと"を、思いつく限り分解して考えることで、どういうキーワードで調べればいいのかが分かるようになってくると思います。

Answer 3

Joinメソッドを利用すれば簡単に連結できます。
下記の例では、文字型の配列をカンマ区切りで連結せよというプログラムです。

どのようにインスタンスを保持しているか分かりませんが、
基本的にはJoinしたい繰り返し可能な型（配列型などforeachで回せるクラス）に、
値を格納した上で、Joinの第２引数に渡します。

基本的に、C#ではあれをしながらこれをやる（配列を回しながら連結する）という風にプログラムを書くより、あれをしてからこれをやる（配列を作ってから連結する）という風にプログラムを書くと綺麗になります。
この書き方を極めていくとラムダ式記法などにたどり着きます。
（はじめは、とっつきにくいので気が向いたらでいいと思います。）

お試しください。

string[] arr = new string[] { "1", "2", "3" };

//カンマで配列を連結する。
string result = string.Join(",", arr );

Answer 4

基本的には、tamatoさんと同様でstring.joinでやれば色んな意味で良いかと。

var models = new List<SampleModel>
{
    new SampleModel {No = "1", Name = "Demo1"},
    new SampleModel {No = "2", Name = "Demo2"},
    new SampleModel {No = "3", Name = "Demo3"},
};

var result = string.Join(",", models.Select(m => m.No));

ただ、このコードがいろんなところで登場するのであれば、IEnumerable<SampleModel> の振る舞いとして扱われてもいいかと思うので、そんなときは以下の拡張メソッドをどっかで用意してあげて、

public static class SampleModelExtensions
{
    //拡張メソッド名はとりあえすてきとーにつけましたが、振る舞いに合った名称を...
    public static string GetNoCsv(this IEnumerable<SampleModel> models) => string.Join(",", models.Select(m => m.No));
}

以下の感じで使うことで、同じコードがクソのように濫用せずキレイに収まり、いい感じになると思います。

var models = new List<SampleModel>
{
    new SampleModel {No = "1", Name = "Demo1"},
    new SampleModel {No = "2", Name = "Demo2"},
    new SampleModel {No = "3", Name = "Demo3"},
};

var result = models.GetNoCsv();

（一応書いておくと、なんでも拡張メソッドすればいいという話ではないので、用途用法は考えて利用していくださいネ♪）

Comments

[BEACHSIDE]

あ、longのチェック読み落としてました...

Answer 5

こんなのは如何でしょう？

var sampleList =
    new List<Sample>
    {
        new Sample { No = "2", Name = "Jiro" },
        new Sample { No = "1", Name = "Taro" },
        new Sample { No = "3", Name = "Saburo" }
    };

var sampleNumbers = sampleList
    .Select(s => s.No)
    .Aggregate((l, r) => l + "," + r);

追記に対応しました。
Noが負であっても弾きます。

var sampleNumbers = sampleList
    .Select(s => s.No) // No を抽出
    .Where(n => n?.Any() == true) // No は null ではなく1桁以上
    .Where(n => n.Length <= long.MaxValue.ToString().Length - 1) // long の正の最大桁数-1を超えない
    .Where(n => !n.Where(c => c < '0' || c > '9').Any()) // [0-9] 以外の文字を含まない
    .Aggregate((l, r) => l + "," + r);

Answer 6

連結先のstringだかStrigBuilderだかに値を入れる際に、空だったらカンマを入れずに要素を追加するようにすればよいとおもう。

Answer 7

Whereの条件にNoが数値(long型)かどうかのチェックを入れたいのですが

class Sample
{
	public string No;
	public string Name;
}

void Main()
{
	var samples = new List<Sample>
	{
		new Sample { No = "2", Name = "Jiro" },
		new Sample { No = null, Name = null },
		new Sample { No = "A", Name = "A" },
		new Sample { No = "1", Name = "Taro" },
		new Sample { No = "3", Name = null },
	};

	Console.WriteLine(string.Join(",", samples
		.Where(_ => !string.IsNullOrEmpty(_.No) && !string.IsNullOrEmpty(_.Name))
		.Where(_ =>
		{
			long t;
			return long.TryParse(_.No, out t);
		})
		.Select(_ => _.No)));
}

とか。

最近の C# なら

class Sample
{
	public string No;
	public string Name;
}

void Main()
{
	var samples = new List<Sample>
	{
		new Sample { No = "2", Name = "Jiro" },
		new Sample { No = null, Name = null },
		new Sample { No = "A", Name = "A" },
		new Sample { No = "1", Name = "Taro" },
		new Sample { No = "3", Name = null },
	};

	Console.WriteLine(string.Join(",", samples
		.Where(_ => !string.IsNullOrEmpty(_.No) && !string.IsNullOrEmpty(_.Name))
		.Where(_ => long.TryParse(_.No, out var t))
		.Select(_ => _.No)));
}