if文でのフラグ分岐の効率化について

Question

if文での複数のフラグ変数の宣言を無くすには

 bool flg = false;
 bool flg1 = false;
 bool flg2 = false;
 bool flg3 = false;
 bool flg4 = false;

if(!flg) 
　{
     flg = true;
     //アイテム入手後の処理のような
  }
if(!flg1) 
　{
　　 flg1 = true;
    //要素数に達した時に1回通る
  }
if(!flg2) 
　{
　　 flg2 = true;
    //要素数に達した時に1回通る
  }
if(!flg3) 
　{
　　 flg3 = true;
    //要素数に達した時に1回通る
  }
if(!flg4) 
　{
　　 flg4 = true;
    //要素数に達した時に1回通る
  }

このように無駄の多い宣言をしてしまっているような気がするとともに、管理がしにくいと感じています。
switch文での制御もしていたのですが、それでもこのような変数が必要な場面に多々遭遇してしまいます。
Flg変数を使う場合でも、このように大量に宣言しない書き方をご教授していただけると嬉しいです。

Answer 1

「Flagを効率よく管理する」というのはすでにいくつか案が出ているので、ちょっと別視点で。

ゲームを作る（というかアプリ全般にも言える話ですが）場合、フラグを極力減らす方向で考えたほうがよいです。

そして先人たちは色々な便利な手法を編み出し、それらを「デザインパターン」と呼んでいます。
もしかしたら学ぶにはまだ少し早いかもしれませんが、「フラグ管理大変」という思いが出てきたのであれば、一度調べてみてもいいかもしれません。

今回のフラグがどういった形で使われるかによるので一概にどのパターンがいいか、という話はできないのですが、「状態を管理する」という意味では「ステートパターン」あたりから調べてみるといいと思います。

繰り返しますが、もしかしたら学ぶのには少し早いかもしれません。
なので、むずかしいな、とかどう使うかまったく分からん・・となったら、まずはここで回答されている内容をヒントに（デザインパターンを使わずに）実装してみるのがいいと思います。

個人的に、デザインパターンという単語自体を知らずに苦労していた時期があるので、単語だけでも覚えておいてもらえるといいかなと思って書きました。参考になれば。

Answer 2

コメントに書いてあるような処理をするなら、
状態を保持するオブジェクトを作って、オブジェクトに聞くのでしょうかね。

Answer 3

個人開発で、明日の自分も上手くやってくれると信じてるぜ！ということなら、
bool[] flags;と配列化。（何番が何のフラグかはコメントに書いたりしておく）
if (flags[3]) { }のように使ったりforで回したり出来ます。
類型でDictionary<string, bool>も一応挙げておきます。
（宣言が減るだけでif文の分岐は減らない可能性が高いですけどね……！）

まぁマジックナンバーやらstringで指定するのはあんまり宜しくないので、**「本当にその変数が必要なのか」「本当にその型でいいのか」**を考慮した方がいいでしょうね。

例示されている「要素数に達した時に1回通る」のようなケースだと上記の配列で上手く動くかもしれませんが、別案としてboolでなくint変数１つにして「達した要素数を入れておく」でもいいかもしれません（これだとif文１つに出来そう）。

Answer 4

public bool FlagAction(bool flag,Action action)
{
  if(!flag)
  { 
     action();
     flag = true;
  }
  return flag;
}

こういう関数を作って、

flag = FlagAction(flag , ()=>{やりたいこと });

という感じで使うのも一案か。

flag変数が多くて大変なら、クラスor構造体を作ればいいじゃないとしか。

実は、flag変数なんて必要ないのもあるとかいうオチがありそうだけど。

Answer 5

フラグ変数を
unsigned int flags=0;
で定義しておいて、

flags |=1; // set
flags &= ~1; // reset
if((flags & 1)!=0) // check flag

としておけば、unsigned intなら３２個までのフラグ操作ができます
おまけに、flags の数値チェックで、３２個のフラグの状態チェックを一括でできますね

Comments

[Zuishin]

そのような使い方なら FlagsAttribute をつけた enum がいいと思います。

[y_waiwai]

おっしゃるとおりです。

まあ、タグにC/C++もあったんで、こーゆーふーに。

[yumetodo]

というかstd::bitsetありますよね、bit節約したいなら

Answer 6

Flgを減らすには状態遷移図を書いて、管理する状態を整理してみればいいと思います。
またその後に再利用しなくてすむ場合なら以下のように状態をFlgに保存しない方法もあります。

if ((fp = fopen(filename, "w")) == NULL) {
  // ファイルデスクリプタがNULLの場合=ファイルのオープンに失敗した場合
}