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

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

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

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

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

Q&A

4回答

23867閲覧

【C#】配列の要素の削除についての質問です。

tmmy

総合スコア6

C#

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

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

0グッド

0クリップ

投稿2017/08/03 11:44

###配列の特定(null)のものを削除し、削除されたものを読みたい。
現在、データテーブルから持ってきたデータを配列の要素に代入しているのですが、
代入されていないnullの要素を削除したいと思っております。
現在行っている処理としては、

static void Main(string[] args) { //null配列は呼び出さない処理。 string[] tn = new string[10]; tn[0] = "hoge"; //nullデータがあった場合その要素を削除する for (int i = 0; i < tn.Length; ++i) { if (tn[i] != null) { } else{ List<string> numberList = new List<string>(tn); numberList.RemoveAt(i); //要素のコピー tn = numberList.ToArray(); } } for (int i = 0; i < tn.Length; ++i) { if (tn[i] == null) { Console.WriteLine("null"); } //途中で消えるため、エラー Console.WriteLine(tn[i]); }

この処理を実行すると、何故か、5つだけ要素は消えるのですが、他のものが消えてくれません。。。

ですが、

static void Main(string[] args) { //null配列は呼び出さない処理。 string[] tn = new string[10]; tn[0] = "hoge"; //nullデータがあった場合その要素を削除する for (int i = 0; i < tn.Length; ++i) { if (tn[i] != null) { List<string> numberList = new List<string>(tn); numberList.RemoveAt(i); //要素のコピー tn = numberList.ToArray(); } } for (int i = 0; i < tn.Length; ++i) { if (tn[i] == null) { Console.WriteLine("null"); } Console.WriteLine(tn[i]); }

このように、nullのもの以外消すという処理にすると、hogeは消えてくれます。

RemoveAtに削除できる上限があるということなのでしょうか??

分かる方、よろしくお願いします。

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

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

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

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

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

guest

回答4

0

C#

1using System; 2using System.Linq; 3 4namespace Test 5{ 6 class Program 7 { 8 static void Main() 9 { 10 string[] tn = new string[10]; 11 tn[0] = "hoge"; 12 13 foreach(var _t in tn.Where(e => e != null)) 14 { 15 // nullが入っていないことを知るためにあえて判定。 16 Console.WriteLine(_t == null ? "null" : _t); 17 } 18 } 19 } 20}

Linqでnull以外の要素を取り出して
配列にするという方法はどうでしょうか。

#補足
Tak1016様の回答を見て、コードを書き換えました。

投稿2017/08/03 12:17

編集2017/08/03 14:33
退会済みユーザー

退会済みユーザー

総合スコア0

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

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

0

えーっと 回してる配列をRemoveしたらだめですよ。

C#

1int[] tn= new int[] {0,1,2,3,4,5,6,7,8,9}; 2 3 4for(int i=0; i < tn.Length; i++) 5{ 6 List<string> numberList = new List<string>(tn); 7 numberList.RemoveAt(i); 8 tn = numberList.ToArray(); 9}

これを実行すると
i=0 のとき 実行後のtnは { 1,2,3,4,5,6,7,8,9 } 0番目の要素0が消えた。
i=1 のとき 実行後のtnは { 1,3,4,5,6,7,8,9 } 1番目の要素2が消えた。消えるのは1ではない。
i=2 のとき 実行後のtnは { 1,3,5,6,7,8,9 } 2番目の要素4が消えた。
i=3 のとき 実行後のtnは { 1,3,5,7,8,9 } 3番目の要素6が消えた。

となります。ループしている配列やList,Collectionから要素を削除してはいけません。

関数型プログラミングのLinqを使いましょう。

C#

1var nullItems = tn.Where(n=> n == null).ToArray(); 2var notNullItems = tn.Where(n=> n != null).ToArray();

ToArray()
を実行すると 新しい配列のインスタンスが作られますので無駄です。

必要な時にIEnumerableのままループすると遅延実行(yield return)により無駄なく処理されます。

C#

1foreach(var n in tn.Where(n=> n == null)) 2{ 3 Debug.WriteLine(n); 4}

投稿2017/08/03 13:35

Tak1016

総合スコア1408

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

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

退会済みユーザー

退会済みユーザー

2017/08/03 13:44

勉強になります。
guest

0

C#

1using System; 2using System.Collections.Generic; 3using System.Linq; 4 5namespace Answer86902 6{ 7 class Program 8 { 9 static void Main(string[] args) 10 { 11 var tn = new string[10]; 12 13 tn[0] = "hoge"; 14 15 var counter = 0; 16 17 do 18 { 19 if (tn[counter] == null) 20 { 21 var remover = new List<string>(tn); 22 remover.RemoveAt(counter); 23 tn = remover.ToArray(); 24 25 Console.WriteLine($"Nullを検知したため{counter}番目の要素を削除しました"); 26 } 27 else 28 { 29 counter++; 30 } 31 } while (counter < tn.Length); 32 33 Console.WriteLine($"配列tnの要素数は{tn.Length}です。"); 34 tn.ToList().ForEach(Console.WriteLine); 35 } 36 } 37} 38

上の皆さんが採用しているForのカウンタ弄りはやるべきではないです。あくまで順次処理なのでLINQを使わない場合はWhile文を採用すべきです。

C#

1using System; 2using System.Collections.Generic; 3using System.Linq; 4 5namespace Answer86902 6{ 7 class Program 8 { 9 static void Main(string[] args) 10 { 11 12 var tn = new string[10]; 13 tn[0] = "hoge"; 14 15 tn = tn.Where(val => val != null).ToArray(); 16 17 Console.WriteLine($"配列tnの要素数は{tn.Length}です。"); 18 tn.ToList().ForEach(Console.WriteLine); 19 } 20 } 21} 22

ただC#ならやっぱりLINQを採用すべきなのでC#的にはこっちのほうがいいと思います。

投稿2017/08/03 12:48

Argent

総合スコア59

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

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

退会済みユーザー

退会済みユーザー

2017/08/03 13:16

勉強になります。
IShix

2017/08/03 17:04

Forのカウンタ弄りはやるべきではないです。あくまで順次処理なのでLINQを使わない場合はWhile文を採用すべきです。 とのことですが、やるべきではない理由を具体的にお聞かせいただいてもよろしいでしょうか。僕自身、for文のキーをデクリメントする方法を稀に使っているので今後使用しないようにしますが、過去に書いたプログラムを修正した方が良いか判断したいです。大変お手数ですがよろしくお願いいたします。
kiichi54321

2017/08/04 09:30

バグを生む未来しかないからだよ。予期しない無限ループや配列の範囲外アクセスが簡単にできてしまう。だから、基本的に、For文ではなく、Foreachを使おうということになっている。
IShix

2017/08/04 13:01

了解しました。以後使用しないようにします。ご返答ありがとうございました。
guest

0

[追記] キーのデクリメントは推奨しないとの事でしたので使用しないでください。

消えてなくなったのでデクリメントしないといけないですね。

C#

1 string[] tn = new string[10]; 2 tn[0] = "hoge"; 3 4 5 //nullデータがあった場合その要素を削除する 6 for (int i = 0; i < tn.Length; ++i) 7 { 8 9 if (tn[i] != null) 10 { 11 } 12 else 13 { 14 List<string> numberList = new List<string>(tn); 15 numberList.RemoveAt(i); 16 //要素のコピー 17 tn = numberList.ToArray(); 18 //※ デクリメント 非推奨 ※ 19 i--; 20 } 21 22 } 23 for (int i = 0; i < tn.Length; ++i) 24 { 25 26 if (tn[i] == null) 27 { 28 Console.WriteLine("null"); 29 30 } 31 32 //途中で消えるため、エラー 33 Console.WriteLine(tn[i]); 34 35 }

###その他の方法

一旦リストにして削除

C#

1 string[] tn = new string[10]; 2 tn[0] = "hoge"; 3 4 5 List<string> _tn = new List<string>(tn); 6 _tn.RemoveAll(o => o == null); 7 tn = _tn.ToArray(); 8 9 for (int i = 0; i < tn.Length; ++i) 10 { 11 12 if (tn[i] == null) 13 { 14 Console.WriteLine("null"); 15 16 } 17 18 //途中で消えるため、エラー 19 Console.WriteLine(tn[i]); 20 21 }

投稿2017/08/03 11:51

編集2017/08/03 17:12
IShix

総合スコア1724

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問