前提・実現したいこと
とある二つのスクリプトがあるのですが、呼び出したいときに予め呼び出し先のインスタンスを保持して呼び出しています。
ですが、呼び出し先の関数が含まれているクラスは規模が大きいクラスで尚且つ呼び出し元のクラスはかなり規模の小さいクラスです。
パフォーマンス的にこれってどうなんだ...?と思ったので質問いたしました。
どうにかインスタンスではなく関数そのものを保持しておけるものは無いでしょうか?
・前提
呼び出し先のBananaクラスは複数ある
該当のソースコード
※ あくまでコードは一例です
C#
1public class Apple 2{ 3 private float remainTime = 20f; 4 [SerializeField] private Banana[] banana; 5 6 private void Update() 7 { 8 // remainTimeを減らす処理 9 // ... 10 11 if (remainTime < 10) banana[0].Fire(); 12 if (remainTime < 0) banana[1].Fire(); 13 } 14}
C#
1public class Banana 2{ 3 private int fireCount = 0; 4 5 public void Fire() 6 { 7 fireCount++; 8 } 9 10 // 以下数百行に渡るコード 11 // ... 12}
関数のみ参照するのは不可能なように思います。
それにもしそのクラスがGameObjectに付いたコンポーネントの場合、参照があろうと無かろうとインスタンスが存在している間はメモリを食うので、参照の仕方は関係無いかもしれません。とりあえずProfiler等で現状のパフォーマンスを確認してみてください(参照の有無で変わるかどうか等)。
出来ないのですね...ありがとうございます。
となると外から参照させる関数だけ纏めたファイルを作っても良いかもしれないですね。
profilerで確認したところ特に問題はないようです。
また新たに疑問が浮かんだのですが
> 参照があろうと無かろうとインスタンスが存在している間はメモリを食う
これは同じインスタンスであれば極端な話1つのクラスで保持、あるいは存在する全てのクラスに保持してもパフォーマンス面では特に変化無いのでしょうか?
>同じインスタンスであれば極端な話1つのクラスで保持、あるいは存在する全てのクラスに保持
試したことは無いですが参照を保持するだけであれば特に変化は無いと思います。
但し例えばUpdateの中でif文も挟まずパラメータにアクセスし続けるコードが沢山ある等、何らかの動作を短時間に大量に行う場合は負荷が上がるかもしれません。
この辺は実際の作り方による部分が大きいので、実際に計測して確認するのが一番確実かと思います。
> Updateの中でif文も挟まずパラメータにアクセスし続けるコードが沢山
試してみましたが、参照回数が等しければ保持しているクラス数によって処理速度が変化することはありませんでした。
1Update 500000回 1クラス 0.131ms
1Update 50000回 10クラス 0.128ms
1Update 10000回 50クラス 0.129ms
ひとまず安心です、ありがとうございました。
...かといって相互参照まみれにならないよう気を配りながらコーディングしようと思います。
もし宜しければ、上記の文章を回答として投稿してはもらえないでしょうか?
是非ともベストアンサーにしてこの質問を共有したいので。
回答1件
あなたの回答
tips
プレビュー