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

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

ただいまの
回答率

90.50%

  • C#

    7116questions

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

  • .NET Framework

    457questions

    .NET Framework は、Microsoft Windowsのオペレーティングシステムのために開発されたソフトウェア開発環境/実行環境です。多くのプログラミング言語をサポートしています。

C# foreachの処理で配列に要素を追加したい

解決済

回答 3

投稿

  • 評価
  • クリップ 1
  • VIEW 646

go-kawano

score 7

C#で以下のような処理を実装しようとしています。

処理内容
文字列の配列を持つリストに対して、ある文字列で検索をかけ、ヒットすれば、
ヒットした配列に対して要素を追加するというものです。

サンプルプログラムとしては、

        static void Main(string[] args)
        {
            int i;
            int j;

            string[] searchlist = new string[3] { "犬", "猫", "ゴリラ" };

            string[] item1 = new string[3] { "ワニ", "猫", "犬" };
            string[] item2 = new string[3] { "コアラ", "キリン", "猫" };

            List<string[]> list = new List<string[]>();

            list.Add(item1);
            list.Add(item2);

            foreach(var item in list)
            {
                for (i = 0; i < searchlist.Length; i++)
                {
                    if (item.Contains(searchlist[i]) == true)
                    {
                        for (j = 0; j < searchlist.Length; j++)
                        {
                            if (item.Contains(searchlist[j]) == false)
                            {
                                //itemに要素を追加したい
                                string[] temp = item;
                                Array.Resize(ref temp, temp.Length + 1);
                                temp[temp.Length - 1] = searchlist[j];

                                list.Remove(item);  //foreachの中でlistを変わっているので例外が発生
                                list.Add(temp);

                                break;
                            }
                        }
                    }
                }
            }
        }

listが持っている配列全てに対して検索を書けるために、foreachで処理をしていますが、
検索がヒットした際に、要素を追加する処理を入れると例外が発生しています。

例外の原因は理解しているのですが、他の方法が思いつかず、質問させて頂いている次第です。

ご教示の程よろしくお願い致します。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

+3

「foreachでList<T>の追加削除をしたい」のであれば、resultlistを用意するということになりますが、やりたいことは何でしょうか?
C#はやりたいことが素直に実装できる言語なので、本当にやりたいことを日本語で書くとよりよいヒントがteratailでは得られると思います。

コードサンプルを私は「searchlistのいずれか一つでも含む配列に対して、searchlistの含まれていない要素を足し込む」だと理解しました。それがやりたいことならUnionを使って下記のように書くことができます(item2も同じように書けます)。

  static void Main(string[] args)
  {
      string[] searchlist = new string[3] { "犬", "猫", "ゴリラ" };

      string[] item1 = new string[3] { "ワニ", "猫", "犬" };
      string[] item2 = new string[3] { "コアラ", "キリン", "猫" };
      var item1Merge = (item1.Intersect(searchlist).Any()) ? item1.Union(searchlist).ToArray() : item1; // searchlistを1つでも含んでいたら、searchlistのうち、持っていないものを足す
      Console.WriteLine(string.Join(",", item1Merge));
  }


追記>>
今回の目的でIntersectはやりすぎなのかもしれない。

var item2Merge = item2.Amy(x=>searchlist.Contains(x)) ? item2.Union(searchlist).ToArray() : item2;


<<追記

Linqを使えないというのでは、C#を使う意味がないのですが、どうしてもforeachでやらないと気がすまないとしたら

  static void Main(string[] args)
  {
    string[] searchlist = new string[3] { "犬", "猫", "ゴリラ" };

    string[] item1 = new string[3] { "ワニ", "猫", "犬" };
    string[] item2 = new string[3] { "コアラ", "キリン", "猫" };

    List<string[]> list = new List<string[]> {item1, item2};
    var result = new List<string[]>();

    foreach(var item in list)
    {
      var hashset = new HashSet<string>(item);
      for (var i = 0; i < searchlist.Length; i++) //foreach(var matchItem in searchlist)とすべき
      {
        if (item.Contains(searchlist[i])) //item.Contains(matchItem)
        {
          for (var j = 0; j < searchlist.Length; j++) //foreach(var addItem in searchlist)とすべき
          {
            hashset.Add(searchlist[j]); // HashSetは重複要素を追加してもなかったことにしてくれる(HashSet.Add()の返り値がfalseになるがエラーにならない) //hashset.Add(addItem)
          }
        }
      }
      result.Add(hashset.ToArray());
    }
    Console.WriteLine(string.Join(",", result.First()));
    Console.WriteLine(string.Join(",", result.Last()));


のようにHashSet<T>を使えば、arrayのresizeとか、searhlist[j]が含まれて*いない*ことの確認とかいう無駄な処理をとばせます。

ついでながら、boolに対して== trueとするのはやめたほうがいいです。item.Contains() == trueのところです。falseも!item.Contains()というのを早急に身に付けたほうがいいです。
それと、listをforeachしているのに、何の必然性もないのにsearchlistをforにしているのもやめたほうがいいです。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/27 09:43 編集

    ご回答ありがとうございます。
    サンプルプログラムの意図は仰る通りです。
    var item1Merge = (item1.Intersect(searchlist).Any()) ? item1.Union(searchlist).ToArray() : item1;
    上記コードも理解しました。linq構文の凄さを実感しました。勉強します。

    foreachを使っている理由は、実際には、itemが1000以上存在し、listに格納されています。
    listの各itemに対して、「searchlistのいずれか一つでも含む配列に対して、searchlistの含まれていない要素を足し込む」という処理を行うために、listをforeachで処理しました。
    itemが多数ある場合も、linqで実現できるのであれば、是非教えて頂きたいです。
    また、多数のご指摘もありがとうございます。

    キャンセル

  • 2018/06/27 16:35 編集

    linqに件数の上限はありませんし、たかだか1000件でしょ?searchlistが1000件だとしても2000件。List<T>をいくつも作ったり、何回も追加削除したり、Array.Resizeを1000回もやるほうが負荷が高いのではないでしょうか。
    いちおう、Intersectは「searchlistを1つでも含んでいるか」というだけでやるには過剰な仕組みなので、追記>><<追記のところに別のやりかたとして、item2Mergeを追加しました。ただコンパイルや実際に実行したときにIntersectしてからAnyするのがこれに比べてはっきり遅くなるかというと微妙かなと思っています。
    ただ、速さという観点でいうと、早過ぎる最適化という問題を調べてみてください。コードを動作するドキュメントと捉えたときに「item1とsearchlistの共通部分Intersectがあったときに、和集合Unionを求める、というほうが後々理解しやすいと思います。また速さを考えるのであればPLINQやasync/awaitといった非同期処理や並列処理を考えるべきでしょう。

    キャンセル

  • 2018/06/29 09:06

    Linq構文の知識が足りず、foreachに頼ってしまう部分がありました...
    大変勉強になりました!ご指摘ありがとうございました。

    キャンセル

checkベストアンサー

+1

HashSet使えば良いです。(ただし、順序がいれかわるのでそれを解決したければもうすこし手を加える必要がありますが)

using System.Linq;
using System.Collections.Generic;

var item1Set = new HashSet<string>(item1);
var item2Set = new HashSet<string>(item2);

// 要素が増えてくるとHashSetに一旦入れたほうが処理が早くなります
var allItemSet = new HashSet<string>(item1);
foreach(var item in item2) {
    allItemSet.Add(item);
}

foreach( var s in searchlist) {
    if(!allItemSet.Contains(s)) continue;

    item1Set.Add(s); // 重複要素は追加されないので、なにもチェックする必要なし
    item2Set.Add(s);
}
return new List<string[]>(){
  item1Set.ToArray(), 
  item2Set.ToArray()
};

// itemXが複数バージョン
var itemLists = new List<string[]>(){item1, item2, ...}
  .Select(items => new HashSet<string>(items))
  .ToList();
var allItemSet = new HashSet<string>(itemLists.SelectMany(_ => _));

foreach(var s in serachlist) {
    if(!allItemSet.Contains(s)) continue;

    foreach(var itemSet in itemLists) {
        itemSet.Add(s);
    }
}
return itemLists.Select(itemSet => itemSet.ToArray()).toList();

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/27 16:39

    質問をどう捉えるかの解釈問題かもしれませんが、`if (item.Contains(searchlist[i]) == true)`を無視しているので、item1やitem2、searchlistの内容を変えたときに、得られる結果が異なることになります。

    キャンセル

  • 2018/06/27 16:55

    失礼、その部分を失念していました。修正します。

    キャンセル

  • 2018/06/29 09:03

    ご回答ありがとうございます!大変勉強になりました。

    キャンセル

0

itemはいじらずに、
結果格納用のlistを追加すればいけますな

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 90.50%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • C#

    7116questions

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

  • .NET Framework

    457questions

    .NET Framework は、Microsoft Windowsのオペレーティングシステムのために開発されたソフトウェア開発環境/実行環境です。多くのプログラミング言語をサポートしています。