Dictionaryを使い配列を作る方法

前提・実現したいこと

初心者なので色々アドバスください。
コレクション型のDictionaryを使い配列作っています。

Dictionaryの配列を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

１、System.NullReferenceException: Object reference not set to an instance of an object
２、NullReferenceException: Object reference not set to an instance of an object

該当のソースコード

C#
1        string[] KeyTable = new string[4] { "UP", "DOWN", "RIGHT", "LEFT" };
2        int[,] ValueTable = new int[4, 4] {
3        {0,0,0,0},
4        { 0,1,2,0},
5        { 0,3,2,0},
6        { 0,0,0,0},
7        };
8        
9        Dictionary<string, int>[] move = new Dictionary<string, int>[4];
10        //値の数
11        for (int i = 0; i < 4; i++)
12        {
13            //キーの数
14            for (int j = 0; j < 4; j++)
15            {
16                //Dictionaryにキーと値を入れる
17                move[i].Add(KeyTable[i], ValueTable[i, j]);
18
19            }
20        }
21

試したこと

moveの要素の削除

補足情報（FW/ツールのバージョンなど）

エラーの場所は、for文内のイカの場所です。

//Dictionaryにキーと値を入れる
move[i].Add(KeyTable[i], ValueTable[i, j]);

行動規範の内容に同意します

回答3件

ベストアンサー

Dictionary自体のインスタンスが生成されていません。
以下のように値を入れる前に生成するようにしてください。

C#
1        for (int i = 0; i < 4; i++)
2        {
3            move[i] = new Dictionary<string, int>(); //これが必要
4
5            //キーの数
6            for (int j = 0; j < 4; j++)
7            {
8                //Dictionaryにキーと値を入れる
9                move[i].Add(KeyTable[i], ValueTable[i, j]);
10
11            }
12        }

投稿2018/08/24 17:46

f-miyu

総合スコア1625

SaharaDesert

2018/08/24 18:09 編集

回答ありがとうございます。実装してみましたら以下のエラーが出ました。これは、どう直せばよいのでしょうか。 ArgumentException: An element with the same key already exists in the dictionary. /*変更後のソース*/ for (int i = 0; i < 4; i++) { move[i] = new Dictionary<string, int>(); //これが必要 //キーの数 for (int j = 0; j < 4; j++) { //Dictionaryにキーと値を入れる move[i].Add(KeyTable[i], ValueTable[i, j]); } }

f-miyu

2018/08/24 18:11

Addは同じキーを追加することはできません。おそらく、KeyTable[i]ではなく、KeyTable[j]にするべきところだと思います。

SaharaDesert

2018/08/25 05:22

返信遅くなり申し訳ありません。 KeyTable[i]ではなく、KeyTable[j]にしましたらできました。ありがとうございます。完成したソース for (int i = 0; i < 4; i++) { move[i] = new Dictionary<string, int>(); //これが必要 //キーの数 for (int j = 0; j < 4; j++) { //Dictionaryにキーと値を入れる move[i].Add(KeyTable[j], ValueTable[i, j]); } }

行動規範の内容に同意します

返信遅くなり申し訳ありません。
KeyTable[i]ではなく、KeyTable[j]にしましたらできました。
ありがとうございます。

完成したソース
for (int i = 0; i < 4; i++)
{
move[i] = new Dictionary<string, int>(); //これが必要

//キーの数
for (int j = 0; j < 4; j++)
{
//Dictionaryにキーと値を入れる
move[i].Add(KeyTable[j], ValueTable[i, j]);

}
}

投稿2018/08/25 05:19

編集2018/08/25 05:20

SaharaDesert

総合スコア16

初期化リストなしのnew演算子によって作成された参照型の配列はnullで初期化されています。

よって

c#
1        Dictionary<string, int>[] move = new Dictionary<string, int>[4];
2        //値の数
3        for (int i = 0; i < 4; i++)
4        {
5            // 空の辞書で初期化
6            move[i] = new Dictionary<string, int>();
7            //キーの数
8            for (int j = 0; j < 4; j++)
9            {
10                //Dictionaryにキーと値を入れる
11                move[i].Add(KeyTable[i], ValueTable[i, j]);
12
13            }
14        }