【Unity】[C#]インスペクターやプロパティ 値の設定、取得方法のコードについて

Question

前提・実現したいこと

UnityでRPGを作成しようと思っています。
値の設定方法等を調べていくうちに
「インスペクターで設定をすると共同作業者に変えられる場合がある」とか
「どこで値が変わったのか追いづらいのでやめたほうが良い」とかいろんな情報を見かけました。
後からHPなり色々変更するのは大変な作業になると思うため、
なるべく今から一番良い状況でコードを書いていきたいと思っています。
クラスやら色々とごっちゃになってしまっているという自覚はあります…。
以下コードで問題なく参照ができるのかどうか、判断頂けないでしょうか。

該当のソースコード

参考：https://qiita.com/RyotaMurohoshi/items/b64b36009ba652b21e2c
インスペクターで値も設定したいけど，プロパティを使いたい。めんどくさいのも嫌！

[値を設定する側]

//Personというclassに記載している設定にしました。
class Person
	{
	[SerializeField]
	private int level;
	public int Level {
		 get { return level; } 
		 private set { level = value; }
		}
}

levelがプログラム上でのみ利用するもので
Levelがインスペクターで設定したものでしょうか?

[利用する側の想定]
載っていなかったので、想定になります。

//プロパティ上から100レベルに設定している?
Person player = new Person(Level=100);

//これだとインスペクターの値を取得できる?
Person player = new Person();
Debug.Log (string.Format ("{0}",player.Level));

試したこと

インスペクターとプロパティで変更できるコードについては調べました。
しかし、参照方法については基本的な事だからか記載がありません。
PCが不調の為、明日にならないとテストは出来なさそうです。
明日自分でも試行錯誤する予定です。
上記だと、エラーが出るのだろうと思うのですが
どのように組めば良いのか分からない状況になっています。
大変申し訳ありませんが、回答のほどよろしくお願いいたします。

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

Unity　2018.3.0f2
VisualStudio 2019
Windows10

Comments

[BluOxy]

「私ならこうする」と一方的に答える事は簡単ですが、それがtarokoさんにとって最適なものだとは限りません。tarokoさんにとって「一番良い状況」とは何でしょう。

もしかして、「インスペクターで設定をすると共同作業者に変えられる場合がある」とか「どこで値が変わったのか追いづらいのでやめたほうが良い」とかをされないような手段であれば良いということでしょうか。

[退会済みユーザー]

BluOxy様
質問頂きありがとうございます。
複数人でゲームを作成していく予定ですので、一番良い状況としては
見やすいソースになっていることや、急な仕様変更や仕様追加があっても対応しやすい汎用性の高いものでしょうか。バグがあった際に、値の変化を追いやすいということも重要だと思っています。
変更したい値のみインスペクターで作業できるようにし、それ以外のものはクラスにカプセル化？して分かりやすくまとめたいという気持ちがあります。
たぶん、C#をしっかり勉強すれば分かることなのだと思いますが、
書籍を数種類持っているのですがカプセル化などについてはあまり詳細に書いておらず分かっていないので、その辺でごちゃごちゃしているのだと思います。

Answer 1

質問の意図が良く分からないので、質問の回答になっているか怪しいですが、突っ込みどころがあるので記載します。
（たぶん、BluOxyさんの回答のほうが、質問の意図に近いような気もします）

まず、PersonにMonoBehaviourが継承されていません。
そうしないと、ゲームオブジェクトにアタッチすることが出来ません。
アタッチ出来ないということは、インスペクターから設定することも不可能なため、インスペクターとは全く縁のないことになります。

ただ、MonoBehaviourを継承したクラスをnewすることは、普通はしません。
代わりにInstantiate()を使います。
もし、newしたいクラスなのであれば、MonoBehaviourの関数などが使えなくなりますが、MonoBehaviourを継承しない、という判断もあり得ます。

levelがプログラム上でのみ利用するもので
Levelがインスペクターで設定したものでしょうか?

MonoBehaviourを継承した前提で話を進めますが、この場合は逆になります。
levelは変数で、Levelはプログラム上からlevelを変更するためのものです。
また、levelは[SerializeField]によって、インスペクターから変更できるようになっています。
そのlevelはprivateになっており、値を別のクラス上から変更することはできなくなっております。
よって、Levelはプログラム上で利用するもので、インスペクターで設定するのはlevel（の初期値）といった具合になります。

このプロパティを参照したい場合は（Personを代入している変数をpersonとすると）person.Levelとすればよいです。

なお、あえてプロパティを使わない場合は、以下のような感じになります。

    [SerializeField]
    private int level;
    
    public int getLevel()
    {
        return level;
    }
    private void setLevel(int value)
    {
        level = value;
    }

この場合だと、get～()やset～()と関数を呼び出さないといけないので、やや面倒です。
それを簡略化したのがプロパティだと思ってください。

Comments

[退会済みユーザー]

fiveHundred様
回答ありがとうございます。
「PersonにMonoBehaviourが継承されていません」
あぁあ、仰るとおりです。忘れておりました。
Levelの説明をとても丁寧に記載頂きありがとうございます！
複雑で頭がこんがらがったりもしましたが、
[SerializeField]の下の行はインスペクタ表示、
privateは変更不可、Publicは変更可能というのはきっちり理解できました。
level = value; のvalueはどこから出てきたのかというのが分かりませんが、それはたぶんC#の基礎知識なのでしょう。勉強してきます。

また、色々教えていただいているうちに、結局
・インスペクタに表示する必要があるのか?
・外部CSにまとめてカプセル化などして分かりやすく管理する？
など考えはじめてしまったので、少し整理したいと思います。

Answer 2

例えばPerson playerをパブリックなメンバーで定義した場合は、インスペクタ上で中の値を設定することができます。

public class Script : MonoBehavior{
    public Person player;
}

そして、外部スクリプトから変更を加えることもできます。

public class OtherScript : MonoBehavior{
    private Script s;
    void Start(){
        s = GetComponent<Script>();
        s.player = new Person(){Level = 32};
    }
}

上記のように書けてしまうことが

「インスペクターで設定をすると共同作業者に変えられる場合がある」とか
「どこで値が変わったのか追いづらいのでやめたほうが良い」

と言われている所以だと思います。

解決策として一番単純な方法はパブリックではなくプライベートなメンバーで定義することです。

public class Script : MonoBehavior{
    private Person player = new Person();
}

このように書くとインスペクタ上からは見えなくなり、外部スクリプトから値を取ることもできなくなります。

この、public、privateといったキーワードはアクセス修飾子と言われており、C#の機能の一つです。

privateをつけることで、クラス内部からのみアクセス可能になりますから、下記のように書くことができます。

public class Script : MonoBehavior{
    private Person player = new Person();
    void Start(){
        //Scriptクラス内部からはアクセス可能
        player.Level = 32;
    }
}

このように書く事でScriptクラスに定義されているplayerというメンバーがどのように変更されるかは、Scriptクラスしか知らなくなるため、「外部から変えられたくない値をインスペクターで設定できてしまう」「どこで値が変わったのか追いづらい」と言った問題を回避することができます。

余談ですが、SerializeFieldという属性をprivateなメンバーに付与することでインスペクタ上から見ることもできます。ですが、外部から弄られたくないのであれば使う必要はありません。

Comments

[退会済みユーザー]

BluOxy様、回答ありがとうございます！
具体例も出して頂き、わかりやすいです。じっくり読み込もうと思います。
また、「SerializeFieldという属性をprivateなメンバーに付与することでインスペクタ上から見ることもできます」とコメント頂きましたが
レベルデザインのためにHPや経験値をインスペクタ上から急に参照したくなった場合は、[SerializeField]と宣言すれば使えるという事でしょうか。
それであれば、自分が利用する場合のみ追記して、インスペクタで作業すれば良い気もしてきました。

[BluOxy]

>レベルデザインのためにHPや経験値をインスペクタ上から急に参照したくなった場合は、[SerializeField]と宣言すれば使えるという事でしょうか。

その認識で問題ないと思います。
こういった記事( https://qiita.com/makopo/items/8ef280b00f1cc18aec91 )もあるので参考にどうぞ。

[退会済みユーザー]

BluOxy様
丁寧にありがとうございます！
参考サイトまでいただいて、本当にありがたいです。
しっかり読み込んで理解したいと思います！ありがとうございました！