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

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

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

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

Unity

Unityは、Unity Technologiesが開発・販売している、IDEを内蔵するゲームエンジンです。主にC#を用いたプログラミングでコンテンツの開発が可能です。

Q&A

解決済

4回答

5608閲覧

Start()内でreadonlyなフィールドを初期化したい

Takaya901

総合スコア181

C#

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

Unity

Unityは、Unity Technologiesが開発・販売している、IDEを内蔵するゲームエンジンです。主にC#を用いたプログラミングでコンテンツの開発が可能です。

1グッド

1クリップ

投稿2019/01/10 07:06

編集2019/01/10 08:01

前提・実現したいこと

MonoBehaviourを継承したクラスで,Start()内で一度だけ初期化され,それ以降の変更を許さないフィールドを作りたいです.Start()内で初期化したい理由は,Start()内で取得した他の値を使って初期化したいからです.

自分の書いたコードを他者がメンテナンスする場合を考えると,コメントで「このフィールドはStart()以外でset禁止」と書いてもよいのですが,もしコードで表現できるならその方がよいと考えています.

問題点

フィールドをreadonlyで宣言すると,MonoBehaviourを継承したクラスはコンストラクタを使えないため,宣言時にしか初期化できません.

ネットで調べた結果

Unity Answersに以下のような投稿があったのですが,ベストアンサーが何を言っているのかわかりません.readonlyは忘れろ,というところまではわかるのですが...
readonly C# keyword not so usefull in Unity

他の回答はコードが例示されているため理解できたのですが,それら以上にスマートな方法があるでしょうか.

bochan2👍を押しています

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

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

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

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

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

takabosoft

2019/01/10 07:14

普通にprivateなフィールドを置いておくだけだと何かまずいのでしょうか?
Takaya901

2019/01/10 07:25

readonlyを指定せずに,ということですよね.コードの保守性やチーム開発の観点から,一度だけ初期化されるフィールドはコードレベルでそうわかるようにしておきたいと考えています.コメントで書いてもよいのですが,もしコードで表現できるならその方がよいと思って質問いたしました. 考えすぎでしょうか.
takabosoft

2019/01/10 07:49

Unityは使ったことがないのですが、少なくとも UnityEngine.Objectの派生クラスではreadonlyは使えないようですね。(外部から見て)読み取り専用のプロパティなら作成可能かと思いますが、期待されているのはメンバ変数の方ですよね・・・c#的になかなか厳しい気がします。他の方の回答に期待します。
ozwk

2019/01/10 08:05

unityって自動実装プロパティ初期化子使えましたっけ? public T prop{get;}= initialValue; こんな感じの
Takaya901

2019/01/11 05:21

使えるのですが,Unityだとコンストラクタによる初期化ができないので...
guest

回答4

0

非MonoBehaviourな内部クラスにフィールドを持たせて、そのコンストラクタでのみ値をセットできるようにしてはいかがでしょうか。
たとえば以下のような感じで。

C#

1using UnityEngine; 2 3public class InitReadOnlyField : MonoBehaviour 4{ 5 // 内部クラス 6 private class MyField 7 { 8 private int myInt; 9 public int MyInt => myInt; 10 11 // コンストラクタでのみ値をセットできる 12 public MyField ( int val ) 13 { 14 myInt = val; 15 } 16 } 17 18 private void Start () 19 { 20 var a = new MyField(3); 21 Debug.Log ( a.MyInt ); 22 } 23}

投稿2019/01/10 11:21

編集2019/01/10 11:34
negitama

総合スコア943

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

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

Takaya901

2019/01/11 05:24

たしかにこれが一番簡単かもしれませんね,ありがとうございます!
guest

0

ベストアンサー

一度だけ書き込めるジェネリッククラスを作っている方もいらっしゃるようです。

Is there a way of setting a property once only in C#
https://stackoverflow.com/questions/839788/is-there-a-way-of-setting-a-property-once-only-in-c-sharp

投稿2019/01/10 08:01

takabosoft

総合スコア8356

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

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

Takaya901

2019/01/11 05:28

なるほど,これなら簡単に使い回せますね.ありがとうございます!
guest

0

意味があるかわかりませんが以前似たようなことしようとして悪あがきしたことがあるのでその時のコードを載せておきます

C#

1using UnityEngine; 2using System.Collections; 3 4public class ReadOnlyTest : MonoBehaviour { 5 6 public readonly string readonlyStr = "初期化"; 7 8 public class ReadOnlyWrapper { 9 public readonly string readonlyInnerStr; 10 11 public ReadOnlyWrapper(string str) { 12 readonlyInnerStr = str; 13 } 14 15 public override string ToString() { 16 return readonlyInnerStr; 17 } 18 } 19 20 private ReadOnlyWrapper readonlyData; 21 public ReadOnlyWrapper ReadonlyData{ get { return readonlyData; } } 22 23 void Start () { 24 25 if (readonlyData == null) { 26 readonlyData = new ReadOnlyWrapper("Startで初期化しました"); 27 } 28 29 Debug.Log(ReadonlyData.ToString()); 30 } 31}

MonoBehaviourを継承していないReadOnlyWrapperクラスを作ってそこでreadonlyな値を管理。
readonlyを使いたかったMonoBehaviour継承側(ここならReadOnlyTest)にはreadonlyな変数の代わりにReadOnlyWrapperを保持させて、必要であればReadOnlyWrapperのコンストラクタ経由で初期化させます。
こんなことさせるくらいならreadonlyじゃなくていいやって結論になったですが・・・

質問のリンク先はちょっとわかりませんでしたがリフレクションで書き換えるかdiコンテナを使えっていってるんですかね?

投稿2019/01/10 07:40

koronatail

総合スコア433

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

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

Takaya901

2019/01/11 05:31

なにか簡単な方法があると思ったのですが,なかなかないようですね... リフレクションもDIコンテナもあまり使ったことがないので,また調べてみようと思います. ありがとうございます!
guest

0

readonly とはしないで、
プロパティとして定義して、set{}をprivateにすればいいかと

投稿2019/01/10 07:36

編集2019/01/10 07:37
y_waiwai

総合スコア88024

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

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

Takaya901

2019/01/10 07:45

ありがとうございます. その場合クラス内では何度もsetできてしまうと思うのですが,クラス内の処理にそこまで気を配る必要はないのでしょうか.
y_waiwai

2019/01/10 07:48

自分の作ったクラス内なら自分で管理できるでしょう 自分が信用出来ないと言うなら、しようがないですが。。
Takaya901

2019/01/10 08:08 編集

言葉足らずで申し訳ありませんが,自分のコードを他者がメンテナンスする場合を考えておりました. やはり考えすぎでしょうか.
y_waiwai

2019/01/10 08:12

ならどう小細工したところでどうしようもないような気がしますが。 コメントでしっかり残しておく、ぐらいしか。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.37%

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

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

質問する

関連した質問