🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
C#

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

Q&A

解決済

3回答

5215閲覧

c# LIST型カスタム配列の編集(値変更)について

Qoo

総合スコア1249

C#

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

0グッド

0クリップ

投稿2019/10/01 00:40

下記のように時刻表のようなLIST型のカスタム配列を作成しており
ステータスがfalseの時にtrueに変更したいのですが、
コンパイルエラー「変数ではないためList<TimeTable>.this[int]の戻り値を変更できません」
となってしまいます。定義自体はbool型なのですが、this[int]の戻り値?とはどういうことなのでしょうか。

c#

1 2public struct TimeTable 3{ 4 public DateTime datetime; 5 public string itemno; 6 public string itemname; 7 public bool departureflag; 8 public bool status; 9} 10 11----- 12 13void Start(){ 14 15 timetable.Add(new TimeTable { 16 datetime = DateTime.Parse(departure.Value), //時間 17 itemno = itemno,    //アイテム番号 18 itemname = itemname, //アイテム名 19 departureflag = false, //出発到着フラグ 20 status = false //動作中フラグ 21 }); 22 23} 24 25void ContentsManager() 26{ 27  28 for (int i = 0; i < timetable.Count(); i++) { 29  if (timetable[i].status == false) { 30   Debug.Log(timetable[i].itemno + "-" + timetable[i].itemname); 31  timetable[i].status = true; // ← コンパイルエラー 32  } 33 } 34 35} 36

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

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

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

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

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

guest

回答3

0

ベストアンサー

確かめていませんが、struct だからじゃないかなと思います。class に変えてみたらどうでしょうか?

struct は値型なので、取得されたコピーのメンバーを変更しても元の値には反映されません。従って、無意味なコードになっていることを警告しているのではないでしょうか。

投稿2019/10/01 01:08

編集2019/10/01 01:23
Zuishin

総合スコア28669

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

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

Qoo

2019/10/01 01:13

出来ました!ありがとうございました!
Zuishin

2019/10/01 01:18

クラスと構造体のどちらを選ぶべきかについては、ほとんどの場合クラスを選んでおけば無難です。特殊なケースで構造体の方がパフォーマンスが良くなることがあります。 大まかに次のガイドラインを頭に入れておいてください。 https://docs.microsoft.com/ja-jp/dotnet/standard/design-guidelines/choosing-between-class-and-struct この質問のケースは、サイズが大きくミュータブルなのでクラスが適切です。
guest

0

こちらが参考になるかと思います。

投稿2019/10/01 01:02

YAmaGNZ

総合スコア10469

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

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

0

csharp

1void ContentsManager() 2{ 3  4 for (int i = 0; i < timetable.Count(); i++) { 5  if (timetable[i].status == false) { 6   Debug.Log(timetable[i].itemno + "-" + timetable[i].itemname); 7 var cur = timetable[i]; 8  cur.status = true; // ← コンパイルエラー 9 timetable[i] = cur; 10  } 11 } 12 13}

質問者さまがもし初学者だったら。
Selectで射影とるか、少なくともforeachを使うかを学ぶことを勧めます。

投稿2019/10/01 00:59

編集2019/10/01 01:08
papinianus

総合スコア12705

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

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

YAmaGNZ

2019/10/01 01:06

curを戻してやらないとListの中身が変わらないのではないですか?
papinianus

2019/10/01 01:09

雑でした。ご指摘感謝します。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.36%

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

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

質問する

関連した質問