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

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

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

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

WPF

Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです

Q&A

解決済

2回答

3802閲覧

クラスの中のインデクサへの書き込みのエラー

退会済みユーザー

退会済みユーザー

総合スコア0

C#

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

WPF

Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです

0グッド

0クリップ

投稿2016/09/21 12:49

###質問
インデクサを使った以下のコードでエラーが発生します。
普通の変数だとOKなのですが、インデクサだと上手くいきません。

インデクサの行列の大きさを決めてなかったり、get、setがまだよく理解できていなかったりと、
どことなく思い当たる節はたくさんあるのですが、調べてもそれらしい記述がないのでご教授のほどよろしくお願い致します。

※リストボックスの項目と関連付けられたPersonクラスに、月と日付ごとの身長体重をインデクサを作って代入していくプログラム(のつもり)です。

###該当のソースコード

C#

1//-----省略------ 2 3public class Body 4{ 5 public int height{ get; set; } 6 public int weight{ get; set; } 7} 8 9public class Person 10{ 11 public string Name { get; set;} 12 public Body[,] DayMonth{ get; set;} 13} 14 15public class List 16{ 17 public List() 18 { 19 List_Items = new ObservableCollection<Person> { }; 20 } 21 public ObservableCollection<Person> List_Items { get; set; } 22 public Person List_SelectedItem { get; set; } 23} 24 25//-----省略------ 26 27var vm = this.DataContext as MainWindowViewModel; 28vm.List.List_SelectedItem.Name = "Tanaka";//←OK 29vm.List.List_SelectedItem.DayMonth[20,8].height = 170;//←エラーになる 30

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

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

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

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

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

tabamotch

2016/09/21 13:02

エラーのスタックトレースは取得できませんか? 提示された情報だけだと、エラーになりそうな箇所が多くて特定しづらいです…
guest

回答2

0

ベストアンサー

DayMonthの所定の位置のインスタンスが初期化されていない場合はNullReferenceException、配列外の場合([20,8]なんて長さを持っていない)場合はIndexOutOfRangeException、あとは、配列自体が初期化されていない可能性もあるかもしれません。発生しているエラーが何なのか分かれば原因が特定出来ると思いますが…

投稿2016/09/21 13:17

tabamotch

総合スコア53

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

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

退会済みユーザー

退会済みユーザー

2016/09/21 13:21

型 'System.NullReferenceException' のハンドルされていない例外が WpfApplication3.exe で発生しました 追加情報:オブジェクト参照がオブジェクト インスタンスに設定されていません。 がエラーの内容です。 他の方に貼って頂いたリンクの内容と症状が似ている(気がする)ので、それを今参考にしています。
tabamotch

2016/09/21 13:27

であれば、DayMonthを初期化していないのかもしれません。 以下のような処理は入っていますか? vm.List.List_SelectedItem.DayMonth = new Body[21, 9]; vm.List.List_SelectedItem.DayMonth[20, 8] = new Body();
退会済みユーザー

退会済みユーザー

2016/09/21 13:54

その処理をはさんだところ、エラーが出ずに上手くいきました! 上記の1行目でサイズを指定して 2行目で、行と列を指定して、そこに対応するBodyクラスのインスタンスを生成してるという解釈でいいのですよね。 代入させるごとではなく、サイズを指定した時点でループなどを使って一気にすべての行と列に対応したインスタンスを作るのが実際のコードの書き方でしょうか?
tabamotch

2016/09/21 14:03

回答としては「好み&データ量による」…でしょうか。 最初に配列内を一気に初期化しておけばNullReferenceExceptionは出にくくなるかもしれませんが、代わりにメモリを消費します。 かといって、必要な時にいちいち初期化するのもメンドクサイものですし… 少し冷たい言い方で申し訳ないですが、状況に応じた選択になると思います。 (個人的な感覚です。選択は人それぞれ異なると思います)
退会済みユーザー

退会済みユーザー

2016/09/21 14:07

ケースバイケースですか。どちらがいいか少し考えてみます。 丁寧な回答ありがとうございました。
guest

0

DayMonthの初期化が行われていないためエラーを吐いているのだと思います。
また、DayMonth内の各Body要素も初期化が必要だと思います。

以下が参考になるのではないでしょうか。
C#でクラスのインスタンスが初期化されてない

投稿2016/09/21 12:58

carimatics

総合スコア740

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問