2012-10-26 63 views
-5

它为什么不起作用?我现在正在尝试很长时间,似乎没有尽头。需要使它尽可能简单代码javascript快速回答

var firstInput = document.luckForm.numberBox.value; 
var secondInput = document.luckForm.numberBox2.value; 
var thirdInput = document.luckForm.numberBox3.value; 



for (firstInput = 0; firstInput <= 10; firstInput++) { 
if (firstInput = luckyNumber) { 
if (secondInput = luckyNumber2) { 
if (thirdInput = luckyNumber3) { 

{alert('Congratulations! You got all 3 numbers correct. You\'ve won £1000!'); }}}}} 

我试图改变这种代码(见下文)成一个for循环的代码,很简单,没有阵列。我不知道我该如何做到这一点。它应该是一个彩票,你可以输入三个数字,如果他们是正确的,你会得到消息,你赢了1000磅。

if (firstInput == luckyNumber && secondInput == luckyNumber2 && thirdInput == luckyNumber3 || firstInput == luckyNumber && secondInput == luckyNumber3 && thirdInput == luckyNumber2 || firstInput == luckyNumber2 && secondInput == luckyNumber3 && thirdInput == luckyNumber || firstInput == luckyNumber2 && secondInput == luckyNumber && thirdInput == luckyNumber3 || firstInput == luckyNumber3 && secondInput == luckyNumber && thirdInput == luckyNumber2 || firstInput == luckyNumber3 && secondInput == luckyNumber2 && thirdInput == luckyNumber) 
    { 
     alert('Congratulations! You got all 3 numbers correct. You\'ve won £1000!'); 
+0

它不起作用? –

+3

虽然在IF条件下比较,你应该把双等于“==” – Vivek

+0

什么不行?你想要做什么? – PaulG

回答

3

分配而不是比较

firstInput = luckyNumber 

应该是:

firstInput == luckyNumber 

或者更好的是:

parseInt(firstInput, 10) === luckyNumber 
0

看起来这种说法只是试图确保顺序并不重要。所以,如果你把第一个数字放在第二个盒子里,第二个放在第一个盒子里,但是仍然得到和乐透相同的三个数字,你仍然会赢。

(即23,35,15赢,如果结果是15,23,35)

1

没有必要for循环在这里。数字只输入一次,然后根据幸运数字进行检查。没有必要遍历一个循环并且一遍又一遍地检查相同的数字。

如果你想通过得到的数字,然后检查对幸运数字的号码,重复,如果他们不匹配,你应该使用do-while循环的全过程循环。

事情 - 得到的数字,并检查他们对幸运数字。

WHILE - 数字不等于幸运数字。