Unity #2 Advent Calendar 2019 2日目を、1週間過ぎました。。
フィールドを、public で他のクラスに公開するのは良くないよ！
public string itemName;
自動実装プロパティで、setのアクセスレベルを設定できるからこう書くといいよ！
public string ItemName { get; private set; }
という感じの書き方をよく見かけていたので
そんな感じで使っていましたが、
それより、「readonly」を使うと良さそうです。
public readonly string itemName;
うんうん。
MonoBehaviour を継承したクラスは、コンストラクタ使えないじゃん？
これは極端な思想だと思う人もいるかもしれないですが、
（そこまでしなくてもいいと思う人もいるかもしれないですが）
MonoBehaviour を継承したクラスには、[SerializeField] の変数だけあれば良いと思います。
MonoBehaviour を継承したクラスには、readonly にしたいような変数は置かない。
- 一時的な状態を格納する変数は、ViewModelとして別のクラスに持たせる。
- 最初に初期化してその後変更しない変数は、Modelなど別のクラスに持たせて「readonly」にする。
逆に、この２つを分けないと、神なビューが出来上がりますよね。
コンポーネントを細分化しても、それだけでは、神なコントローラーが出来上がりますよね。
とするとやっぱり、readonly にしたいような変数はMonoBehaviourを継承したクラスには置かないほうが良さそうです。
以上です。
readonlyを定数(const的な)位にしか使ってこなかった人に届くと嬉しいです。