質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.37%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Q&A

解決済

2回答

20220閲覧

[C#] Array.Foreach(T) で、先頭の要素もしくは最後の要素と、途中の要素で処理を分けることは可能でしょうか?

tonitakekawa

総合スコア44

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

0グッド

0クリップ

投稿2014/12/01 06:22

最近ラムダ式を勉強しています。

たとえば、string の配列 items があるとして、
その要素をカンマ区切りで結合した msg を取得したいとします。

間違ったコードとして、こんなものを作ってみました。

lang

1 var items = new string[]{ "Ringo", "Banana", "Melon", "Kaki" }; 2 var msg = ""; 3 4 Array.ForEach(items, item => { 5 msg += item + ","; 6 }); 7

このコードを実行すると、msg の中身は、

Ringo,Banana,Melon,Kaki,

となり、最後のカンマが余分になります。

この場合、配列の最後の要素に関してカンマをつけない あるいは配列の先頭の要素に関してカンマをつけない という場合分けするのが良いと思われるのですが、

何かエレガントな解法はないでしょうか?

itemsを、stringとデリゲートの配列にするのがいいのかな、と思ったりもするのですが・・・

ご教授お願いしたいです。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

シンプルに書くなら

lang

1var items = new string[] { "Ringo", "Banana", "Melon", "Kaki" }; 2var msg = string.Join(",", items);

でどうでしょう。

あえてラムダ式(LINQ)を使うなら

lang

1var items = new string[] { "Ringo", "Banana", "Melon", "Kaki", "Melon" }; 2 var msg = ""; 3items.Select((item, index) => index == (items.Count() - 1) ? item : item + ",") 4 .ToList().ForEach(x => msg += x);

投稿2014/12/01 06:42

sho_cs

総合スコア3541

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

tonitakekawa

2014/12/01 07:27

こんな便利なメソッドがあったのですね! 個人的にやりたいことは string.Join でできてしまいました。 しかし「Array.Foreach(T) で、先頭の要素もしくは最後の要素と、途中の要素で処理を分けることは可能か」どうかはまた別件で知りたいのです・・・ 解答ありがとうございます!
tonitakekawa

2014/12/01 07:28

おお!求めていたのはまさしくコレです。 解答ありがとうございました。
guest

0

join的なものがある言語ではすでに回答にある通り、joinを使ったほうがいいのですが、言語に依存しない方法としては、msgが空文字列でなければ、カンマのみを先に結合すると言う方法があります。

投稿2014/12/01 07:48

TaMaMhyu

総合スコア1356

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

tonitakekawa

2014/12/01 08:02

そのアプローチでもいけそうですね。 ```lang-<C#> var items = new [] { "Ringo", "Banana", "Melon", "Kaki", "Melon" }; var msg = ""; items.ToList().ForEach(s => { if (msg.Length == 0) { msg += s ; } else { msg += "," + s; } }); ``` こんな感じでしょうか。
TaMaMhyu

2014/12/01 08:07

もう少し補足すると、 msg += ","; だけを空文字列の時に実行して、 そのあとに無条件で msg += s; を実行します。
TaMaMhyu

2014/12/01 08:08

あえてjoinを使わないという時もあります。同時に別の処理もしたい場合です。
TaMaMhyu

2014/12/01 08:09

空文字列のところは条件が逆でした。失礼しました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.37%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問