時間関数が時々動作しない場合

Question

だから、カウントダウンする関数を作ろうとしています。それは私がここで見たものに基づいています。変数は異なる関数から与えられます。時間がなくなると、変数finishは1に変わり、関数を終了します。 この関数は時々動作し、時にはうまくいきません。例えば、11秒の入力を与えてもうまく動作しますが、1分に与えると動作しません。誰かがコードに何が間違っているか教えてもらえますか？

if (time1 == 0 && time2 == 0 && time3 == 0 && time4 == 0) 
//if all the time is 0 finish the sequence 
    finish = 1; 

if (time1 != 0) //Checking to see if the first digit is NOT at 0 
    time1 = time1 - 1; // subtract time 1 by 1 
else { 
    time2 = time2 - 1; //When time1 is 0 
    time1 = 9; 
} //Time1 going back to it's original value 

if (time2 == 0 && time1 == 0) { //if time1 and time2 are 0s 
    if (time3 != 0) { //The minute value (time3) 
     time2 = 5; //60 SECONDS 
     time3 = time3 - 1; 
     time1 = 9; 
    }    
} //Put time 1 to its original value 
if (time2 <= 0 && time1 <= 0 && time3 <= 0) { 
    if (time4 != 0) { //The minute value (time3) 
     time2 = 5; //60 SECONDS 
     time3 = 9; 
     time4 = time4 - 1; 
     time1 = 9; 
    } 
} //Put time 1 to its original value 

TIME4 = 3、Time3を= 2、タイム2 = 1、時間1 = 0。これは、時間はあなたがちょうどあなたが必要とする、非ゼロに対してチェックすることはできません32:10分

Answer 1

であることを意味します指定された時刻が正であるかどうかをチェックし、そうでない場合は負の値でカウントされ、カウンタがオーバーフローする可能性があります。

if (time1 > 0) 
    time1 -= 1; 
if (time3 > 0) 
    time3 -= 1; 

もう1つの考えは、分と秒の各桁でカウントダウンしています。なぜなら秒を秒単位で変換するだけではないからです。例えば、1：23にカウントダウンする：

int minutes = 1; 
int seconds = 23; 
int timer = minutes * 60 + seconds; 

// in your timer function 
if (seconds == 0) { 
    finish = 1; 
} else if (seconds > 0) { 
    seconds -= 1; 
} else { 
     // error 
} 

あなたが時間を処理したい場合は、単にsecondsにhours * 3600を追加し、何も拡張可能になります。この道を、あなたは日、月を処理するために簡単にこれを行うことができます均一。あなたのアプローチでは、それらの結果が多すぎるケースを追加します。正しく処理することはほとんど不可能です。

Answer 2

問題は、数値をゼロ以外に変更した後でゼロと比較していることです。

は、独自のロジックに従うことができ一時が

time1 = 0 
time2 = 0 
time3 = 1 

としてエンコードされると仮定すると：

if (time1 != 0) // Nope 
    time1 = time1 - 1; 
else { // Yes 
    time2 = time2 - 1; 
    time1 = 9; 
} 

は今、あなたは

time1 == 9 
time2 == 0 
time3 == 1 

if (time2 == 0 && time1 == 0) { // Nope, time1 is 9 
    if (time3 != 0) { 
     time2 = 5; 
     time3 = time3 - 1; 
     time1 = 9; 
    }    
} 

を持っているとあなたはまだ

を持っています

time1 == 9 
time2 == 0 
time3 == 1 

し、最終的に

if (time2 <= 0 && time1 <= 0 && time3 <= 0) { // Nope 
    if (time4 != 0) { 
     time2 = 5; 
     time3 = 9; 
     time4 = time4 - 1; 
     time1 = 9; 
    } 
} 

ので、あなたは、1：09である

time1 == 9 
time2 == 0 
time3 == 1 

で終わります。

時間 _K-1がゼロに「交差」したときにあなたは時間 _Kを変更したいだけの時間があります。
これは、条件文の巣を行うことができます。

if (time1 > 0 || time2 > 0 || time3 > 0 || time4 > 0) 
{  
    time1 -= 1; 
    if (time1 < 0) 
    { 
     time1 = 9; 
     time2 -= 1; 
     if (time2 < 0) 
     { 
      time2 = 5; 
      time3 -= 1; 
      if (time3 < 0) 
      { 
       // ... 
      } 
     } 
    } 
}