静的int変数のstd :: out_of_range

Question

小さなDOSベースのゲーム（コースプロジェクト）でメークシフトタイマーとして静的変数を使用しています。この変数は、状態効果が消える前のターン数を記録します。

for (auto &i : v) // <-- code that calls enemyAttack 
    enemyAttack(i, p, str, i.attack); 
break; 

void enemyAttack(Enemy &e, playerObject &p, std::array<std::string, NUM_MESSAGES> &str, void(*a)(playerObject &p, std::array<std::string, NUM_MESSAGES> &str)) { 
    int die = rand() % 100 + 1; 
    int d = 1; 

    a(p, str); // <-- Call function which causes the error 

    ... 
} 

void batAttack(playerObject &p, std::array<std::string, NUM_MESSAGES> &str) { 
    static int time = 2; 
    static bool bit = false; 

    if (rand() % 10 < CHANCE_OF_STATUS_EFFECT && !bit) { 
     p.damage /= 2; 
     str[STATUS] += "WEAKENED "; 
     bit = true; 
    } 
    else if (time == 0) { 
     p.damage *= 2; 
     str[STATUS].replace(str[STATUS].find("WEAKENED ", 0), 9, ""); 

     time = 2; // <-- error 
     bit = false; 
    } 
    else if (bit) { 
     time--; 
    } 
} 

が、私は第二の条件内側の線 time = 2;でのstd :: out_of_rangeエラーが表示されます。ここでは、コードです。この関数は、プライマリ攻撃関数から関数ポインタを介して呼び出されます。エラーはランダムであるように見え、MSVSはエラーが発生したときに必要な値を持つすべての変数を報告します。

Answer 1

ライン

str[STATUS].replace(str[STATUS].find("WEAKENED ", 0), 9, ""); 

が起こるのを待っているだけの災害です。最初にインナーを見てみましょう。あなたが十分な頻度であなたがミスを作るチャンスがないので、ここでは名前付きの値を使用するために、その優れたので、スペルミスがありますこれを行うとき

str[STATUS].find("WEAKENED ", 0) 

あなたの値を使用しては二回、短いプログラムでは「弱体化」。その後、

constexpr const char *WeakenedStr = "WEAKENED "; 

と第二に、これは失敗する可能性が

str[STATUS].find(WeakenedStr , 0) 

を使用した文字列が見つからない場合、それは（現在である-1）「NPOの」を返します。だから我々はあまりにも

auto pos = str[STATUS].find("WEAKENED ", 0); 
if (pos != std::string::npos) 
    str[STATUS].replace(pos, WeakenedStrLen, ""); 

注意を与えるという名前の値

constexpr const char *WeakenedStr = "WEAKENED "; 
const int WeakenedStrLen = strlen(WeakenedStr); // strlen is sadly not constexpr. 

でなければならないこと、あまりにも

auto pos = str[STATUS].find("WEAKENED ", 0); 
if (pos != std::string::npos) 
    str[STATUS].replace(pos, 9, ""); 

次はマジックナンバーが「9」であることをテストする必要があります。テストされていないコードは、バグが発生します。