【Unity C#】if (isA != newIsA) をどこに入れるべきか分かりません。

Question

例えばこんなようなメソッドがあったとします。
「引数 newIsA と isA を比較、異なる場合は代入し、その値に応じて処理をする」という感じのものです。

void ToggleIsA(bool newIsA) {
　　if (isA != newIsA) {
　　　　isA = newIsA;
　　
　　　　if (isA) {
　　　　　　Debug.Log("Aになりました！");
　　　　} else {
　　　　　　Debug.Log("Aではなくなりました！");
　　　　}
　　}
　　
}

そしてこのメソッドを毎フレーム実行するとします。
例えば、レイが当たっている間は ToggleIsA(true); 、当たっていない間は ToggleIsA(false); を実行するとします。

これで期待する動作はするのですが、
レイが当たっている間、ずーっと ToggleIsA(true); が実行されているのが気持ち悪いので、
メソッドを呼ぶ部分にこのようなif分を追加します。

「レイが当たっているのに isA == false なら ToggleIsA(true); 、当たっていないのに isA == true なら ToggleIsA(false); 」という感じです。

//レイが当たっている間
if (isA == false) {
　　ToggleIsA(true);
}

//レイが当たっていない間
if (isA == true) {
　　ToggleIsA(false);
}

でもそうすると、 ToggleIsAメソッド の、この部分が必要なくなります。

void ToggleIsA(bool newIsA) {
　　if (isA != newIsA) { //←◆◆◆ココ◆◆◆
　　　　isA = newIsA;
　　
　　　　if (isA) {
　　　　　　Debug.Log("Aになりました！");
　　　　} else {
　　　　　　Debug.Log("Aではなくなりました！");
　　　　}
　　}
　　
}

というような感じで、
要はif (isA != newIsA)をどこに入れるべきなのかを悩んでおります。

選択肢としては、
0. メソッドを呼ぶ部分、メソッド内の両方に入れる

1. メソッドを呼ぶ部分のみに入れる
2. メソッド内のみに入れる
3. そのほかのやり方？

私の今の感覚としては、先ほども書いた通り、3の場合は毎フレームToggleIsA(bool newIsA)が実行されるのが気持ち悪く、また処理速度的な観点からも重くならないのかな？と思っているのですが、かと言ってメソッドを呼ぶ部分に必要、となると、「そのメソッドを使うときの決まり事」が増えて残念とも思います。

ご教授いただければ幸いです。
よろしくお願いいたします。

Answer 1

こんにちは。

感覚に関する質問ということで、個人的な感覚で回答しますが、
これは単に ToggleIsA という名前が良くない気がします。
結局やっていることは「引数に渡した値で IsA を上書きする (ただし既に同じ値のときはなにもしない)」なので、
例えばこれを SetIsA(bool newValue) などとすれば、呼び出し側で毎フレーム実行しても違和感はなくなるものと思います。
こうすることで、呼び出し側では毎フレーム新しい値をセットする、呼ばれた側では値が変化した場合のみ何かをする、とそれぞれの担当範囲が分かれるので、この場合では質問の選択肢「3」が適当になります。

なお、質問のコード例では名前付けが意味を持っていないのでこのような回答になりますが、メソッドの名前によっては「呼ぶ側が呼ぶかどうか考えなければならない」ことにもなるので、全ては名前付け次第です。
「書いて読んで違和感がない」ことはとても重要なプログラミングの要素なのです。

Comments

[catdoglion]

ご回答いただきありがとうございます。
なるほど！

もろもろ、すごくなるほど！となりました。

ひとつお聞きしたいのですが、
この 「SetIsAメソッドが毎フレーム呼び出される」という点について、
処理速度的に重くなったりということはないでしょうか？

毎フレーム呼び出された結果実行されるのが「if文での値比較のみ」ならば気にすることはないでしょうか？
また、メソッドが別のスクリプトにある場合、それの参照に負荷がかかるとか、そんなようなこともいかがでしょうか。

もちろんケースバイケースで、具体的にすみずみまで確認しないと明言はできないと思いますので、「可能性はある」というような曖昧な表現でも大丈夫です。
これも肌感覚といいますか、印象でお答えいただけると嬉しいです。

よろしくお願いいたします。

[tamoto]

メソッド呼び出し一つ程度なら誤差未満なので気にしなくて良いです。
特に、質問のコードは入った直後引数をチェックして条件を満たさなければ即終了するので、この場合メソッド呼び出し分のコストはゼロと想定しても問題ないでしょう。

Answer 2

「IsAをプロパティとして実装すること」と「IsAプロパティ変更をイベントとして抽出すること」が出来ると良さそうですね。

private bool _isA;
public bool IsA
{
  // IsAプロパティのgetter
  get => _isA;

  // IsAプロパティのsetter
  set 
  {
    // 値が異なる場合だけ
    if (_isA != value)
    {
      // 値の変更を適応し
      _isA = value;
      // 変更時の処理を呼び出す
      IsAPropertyChanged(_isA);
    }
  }
}

// IsAが変更された時だけ呼び出される処理
private void IsAPropertyChanged(bool isA)
{
   if (isA) {
     Debug.Log("Aになりました！");
   } else {
     Debug.Log("Aではなくなりました！");
   }
}

ここで if (isA != newIsA) に該当するのは setter内の if (_isA != value) です。

プロパティのsetterが定義されている場合、代入演算子(=)によってプロパティのsetterが呼び出されて、変数の書き換え処理をカスタマイズできます。詳しくは以下のリンクを確認してみてください。

プロパティ（C#プログラミング入門） - 未確認飛行 C

実利用時としても

IsA = true;
or 
IsA = false;

とするだけでIsAプロパティのsetterが呼ばれて 値が異なる場合だけ変更を受け付け、変更があった時だけIsAPropertyChanged(_isA);が呼ばれます。

Comments

[catdoglion]

丁寧にご回答いただきありがとうございます。
まったく知らなかったやり方で、先ほど試したところとても綺麗にできました。
リンクも貼っていただきありがとうございます。リンク先ページの説明もすごく分かりやすかったです。
ありがとうございます m( _ _ )m

ご回答をくださった他の皆様も本当にありがとうございます。
特に命名の部分はあまり考えずしていた節があり、ご回答によって気づくことができました。
ありがとうございました m( _ _ )m

Answer 3

個人的にToggleIsA(bool newIsA)のメソッド名と引数がすごく気になります。

Toggleって名前だと、スイッチを切り替えるように、値の変化は担保された上で切り替わるようなイメージがあります。
引数で明示的に値をセットするなら、Set～～みたいな名前かなと思いましたが。どうでしょうか。

選択肢に関しては3が良いかと思います
呼び出し側で色々やるより、メソッド側で処理させるのが良いでしょう。折角メソッドで分離しているのに呼び出す側で色々制約をかけなきゃいけないのは、非常にめんどくさいです。

もちろん明確な理由があるなら呼び出し側で弾くのもありです。

Answer 4

そういうのはあなた自身が、それぞれいろいろやってみて、あなた自身が都合がいい方にすればいいです。

あなたのいう「気持ち悪い」のは各人それぞれ違いますんで、他の誰かのいうことを聞いたところで解決はしないかと。