2016-08-04 6 views
-1

私はsecondsがxと等しいかどうかをチェックするuserInputという関数を持っています。それからユーザに警告を出し、時刻をリセットします。 userInputは秒がxに等しいかどうかをチェックしません。 forループを使うべきですか?ここに私のコードがあります:私のuserInputはseconds = 0であるかどうかをチェックしません。

var userInput = function(x) { 
    if (seconds == x) { 
    var done = alert("You're done!"); 
    h1.textContent = "00:00:00" 
    seconds = 0; minutes = 0; hours = 0; 
    } 
} 
    userInput(x); 

そして、あなたが知っているように、秒、分、および時間はすでに変数として宣言されています。

ご協力いただきありがとうございます。

+1

[最小限の、完全で検証可能な例を作成する*](http://stackoverflow.com/help/mcve)をご覧ください。ここに実行可能スニペットを投稿して問題を表示することができます。 – RobG

+0

'秒はどこから来ますか?そして、関数は 'x'という引数をとりますが、' userInput'を呼び出すと 'x'を渡しています。関数内の 'x'は関数外の' x'の[shadowing](http://stackoverflow.com/questions/11901427/an-example-of-variable-shadowing-in-javascript)です。 – kevin628

+0

上記の質問は削除されました:http://stackoverflow.com/questions/38778172/my-userinput-function-does-not-work。残りのコードとすべてのコンテキストが含まれていました。 – Xufox

答えて

-1

私のコードでは、xがぼやけていました。コメントのご協力ありがとうございます

+0

なぜそれは問題ですか?関数に 'x'を渡しているので、外側の' x'と同じ値が得られます。 – Barmar

+0

質問に対する良い答えが説明され、問題を解決する方法を示す必要があります。この答えは、同様の問題を抱える将来の読者にとっては役に立ちません。 – Barmar

関連する問題