クラス変数内のIListに追加

Question

前提・実現したいこと

クラス変数内のIListに追加をしていきたいのですが、
ソースコードの　「Startup」　の　「wk.XX.Add(wwkk);」の行　で
が発生してしまいます。
( xGTOをnewしている 変数 wk の XCDの値は「1」、XXの値は「null」でした。)
wk　の　XXのIListに　リストの追加と値を設定できる方法を教えてください。

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

「System.NullReferenceException: 'Object reference not set to an instance of an object.'
TESTPro.Startup.xGTO.XX.get が null を返しました。」

該当のソースコード

public class xGTO
 {
 public string XCD { get; set; }
 public IList<xxGTO> XX { get; set; }
 }

 public class xxGTO
 {
 public string XXCD { get; set; }
 }

 public void Start()
 {

 xGTO wk = new xGTO();
 wk.XCD = "1";

 xxGTO wwkk = new xxGTO();
 wwkk.XXCD = "2";
 wk.XX.Add(wwkk);
 }

試したこと

CASTしたりしたがエラーは変わらず

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

Comments

[Automatic9045]

xGTO.XXプロパティをList型ではなくIList型（インターフェイス）にしている理由は何かありますか？
それによって回答が変わってくるので、返信の程よろしくお願いします。

[taopaipai]

IListにしているのは詳しくは不明なのですが速度的な面の考慮だと思われます。
IListと決まっているため　IListでの話で教えていただきたいです。

[退会済みユーザー]

マルチポストのようなので他の投稿先の URL を張っておきます。

http://bbs.wankuma.com/index.cgi?mode=al2&namber=95937

質問者さん＞

わんくまの方はともかく、こちらにはルールがあります。次の機会がありましたらその時はルールを守っていただきたく。

https://teratail.com/help#posted-otherservice

[taopaipai]

了解しました。

Answer 1

IListになっている理由があるとのことでしたので、改めて回答します。

以下のように記述することで、IList型のプロパティにList型のインスタンスを代入することは出来ます。

public IList<xxGTO> XX { get; set; } = new List<xxGTO>();

IListになっている意味がよく分からないので、ここまでしか回答は出来ません。
もしかしたら今後予期せぬ例外などが発生することがあるかもしれないので、発生したときにまた改めて質問してみて下さい。

Comments

[taopaipai]

了解しました。

Answer 2

xGTO クラスの定義に以下のようなコンストラクタを追加してはいかがですか？

そうすれば同じコードで少なくともエラーにはなりません。

Comments

[taopaipai]

ありがとうございます。

Answer 3

こんにちは。

IList型になっている特段な理由は無いものとして回答します。
ただリストを作りたいだけの場合は、List型にするのが一般的です。

ListにAddするためには、まずはじめにListのインスタンスを作成しておく必要があります。
xGTOクラスを初期化する時にListのインスタンスを作成する形にすると良いでしょう。

public List<xxGTO> XX { get; set; } = new List<xxGTO>();

Comments

[taopaipai]

リストの場合は以下のように書くことで追加できることは確認しました。
IListでは無理なのでしょうか？

xGTO wk = new xGTO{
XCD = "1",
XX = new List<xxGTO> {
new xxGTO { XXCD = "2" },
new xxGTO { XXCD = "3" },
new xxGTO { XXCD = "4" }
}

[Automatic9045]

外部のライブラリで提供されているインターフェイスを実装していたり、あるいは抽象クラスをオーバーライドしている関係でIListにする必要がある…という感じですか？

[taopaipai]

Listの要素の中にさらに要素と別のListがあり、それが膨れていく形で決まっているのです。
なぜそうなっているのかは理由は不明なのです。

[Automatic9045]

その要件ではIList型にする必要性は無さそうに見えますが……
データ構造をもう少し詳しく質問文に追記して頂けますか？

[taopaipai]

以下のように要素とIListがあり、XX、YY、ZZ　の中に要素とさらにIListがあり、その中で設定しているIListにもさらに要素とIListがある形で(IList内にIList内にIList内にIList内にIList)となっています。
なぜこういう構造になっているのか理由が不明なのです。

public class xGTO
{
public string XCD { get; set; }
public IList<xxGTO> XX { get; set; }
public IList<xxGTO> YY { get; set; }
public IList<xxGTO> ZZ { get; set; }
}

[Automatic9045]

木構造になっているようですね。
これだけではIListである必要は特にありませんが、その他の何らかの理由でIListにしなければならないものとして回答します。

[taopaipai]

訂正します。　こうでした。
public class xGTO
{
public string XCD { get; set; }
public IList<xxGTO> XX { get; set; }
public IList<yyGTO> YY { get; set; }
public IList<zzGTO> ZZ { get; set; }
}

Answer 4

XXで返すべきListのインスタンスを作成していないようです。
また、プロパティとして公開する型はIList<xxGTO>でいいのですか？List<xxGTO>でもいいような感じに見えます。

Comments

[taopaipai]

ILst<xxGTO>　として決まっていて変更できないのです。
そのためILst<xxGTO>での書き方を教えてもらえるとありがたいです。

[YAmaGNZ]

IList<xxGTO>として公開するということは、IList<xxGTO>インターフェイスを実装しているものであればいいと取れるので、中身はList<xxGTO>でいいのではないでしょうか

[taopaipai]

Listでしか書けないのでしょうか？
悩んでしまって先に進めないのです。

[YAmaGNZ]

IListはあくまでインターフェイスでこれをインスタンス化することはできません。
public IList<xxGTO> XX { get; set; }
という書き方ですと、IList<xxGTO>インターフェイスが実装されていると明記されている状態です。
なので、IList<xxGTO>を実装しているものとなるとList<xxGTO>とするのが一番素直な実装になると思います。
なので
public IList<xxGTO> XX { get; set; } = new List<xxGTO>();
とすればいいのではないでしょうか。

[taopaipai]

動作して確認してみます

[taopaipai]

public IList<xxGTO> YY { get; set; } = new List<xxGTO>();
を宣言して
xxGTO YY = new xxGTO();
YY.XXCD = "2";
wk.YY.Add(wwkk);
とした際にぬるぽで怒られてしまいます。

[YAmaGNZ]

YYはどのYYなのですか？
提示された部分だけでYYが２つあり、正しく設定されているのか不明です。
今までXXというプロパティの話だったのに突然YYというものを出されても困ります。
変に省略しないで、該当の全ソースを提示できませんか？

[taopaipai]

要素がかなりあり、IListのクラスも複数あって深く、書ききれないです。

[taopaipai]

2段目までで簡略するとこうなります。
public class xGTO
{
public string XCD { get; set; }
public IList<xxGTO> XX { get; set; }
public IList<yyGTO> YY { get; set; }
public IList<zzGTO> ZZ { get; set; }
}

[YAmaGNZ]

何段階とか知りませんが、適切なところで初期化できていないのでNullReferenceExceptionとなるということです。
public class xGTO
{
public string XCD { get; set; }
public IList<xxGTO> XX { get; set; } = new List<xxGTO>();
public IList<yyGTO> YY { get; set; } = new List<yyGTO>();
public IList<zzGTO> ZZ { get; set; } = new List<zzGTO>();
}
とすれば
xGTO x = new xGTO();
x.XCD = "1";
x.XX.Add(new xxGTO());
x.YY.Add(new yyGTO());
x.ZZ.Add(new zzGTO());
このように書いてもNullReferenceExceptionは発生しません。
なので、余計なこと、足りないこと、勘違いなどがある可能性があるのでソースを提示してくださいと書きました。

[taopaipai]

ありがとうございます。