如果 javascript 语句有问题

Question

      <!DOCTYPE html>
  <html>
 <head>
 </head>
 <body>
 <hr>
 <div>

   <div id="myPos">

 </div>


 </div>

  <hr>
  <!--Storing my array -->
  <div id="myArray">

  </div>

  <br>
  <br/>
  <hr>

   <script type="text/javascript">
   var pos=-1;


   function randomSort(a,b){
return(parseInt(Math.random()*10)%2);
     }

   function roll(){
var myGun = new Array();
myGun[0]="First Slot";
myGun[1]="Second Slot";
myGun[2]="Third Slot";
myGun[3]="Fourth Slot";
myGun[4]="Fifth Slot";
myGun[5]="Sixth Slot";
myGun.sort(randomSort).toString();

document.getElementById("myArray").innerHTML="";



for(i=0;i<myGun.length;i++)
{
    document.getElementById("myArray").innerHTML+=myGun[i]+ "<br>"; 
}

    document.getElementById("myPos").innerHTML= "Position:"+ (pos=-1);
     }

function shot(){

    if(pos<myGun.length)
    {
        document.getElementById("myPos").innerHTML="Position:"+ (pos=pos+1) +"<br/>";


   }
    else
    {
       alert("you loose");
    }
       return pos;
}

     </script>

<footer>
    <button id="btnRoll" onclick="roll()">Roll</button>
    <button id="btnShot" onclick="shot()">Shot</button>
</footer>
    </body>
    </html>

Answer 1

尝试这个：

function shot(){
        if(pos>=myGun.length)
        {
            pos = pos +1;
            document.getElementById("myPos").innerHTML="Position:"+ pos +"<br/>";

       }
        else
        {
           alert("you loose");
        }
    }

Answer 2

您需要将 ">=" 替换为 "<"：

if( pos < myGun.length ) { ...

也就是说，您还使用了不推荐的全局变量。

编写此 function 的更好方法是：

function shot( pos, myGun ) {
  if ( pos < myGun.length ) {
    document.getElementById( "myPos" ).innerHTML="Position:"+ ++pos + "<br/>";
  } else {
    alert( "you loose" );
  }
  return pos;
}

请注意，function 在递增后返回更新的位置。

Answer 3

试试这个，它对我有用，但我输了。 :) 我已经修复了几个小错误并对 HTML 进行了一些调整，这不是问题所在。 这里的问题是数组 - 它是第二个 function 的局部变量，但不是第三个 - 我已将它设为 pos 的全局变量，但将其重置为滚动（您的原始定义看起来应该在那里重置）。

第一个 function 不需要 a 和 b arguments，因为它不使用它们。 我已经移动了页面标签中的所有内容，但您也可以将所有脚本内容放在单独的.js文件中以提高可读性：
<script type="text/javascript" src="path to file"></script> 。

告诉我这是否也适合你。

<!DOCTYPE html>
<html>
<head>
    <script type="text/javascript">
        var pos = -1;
        var myGun = new Array;

        function randomSort()
        {
            return parseInt(Math.random() * 10) % 2;
        }

        function roll()
        {
            myGun = [];
            myGun[0]="First Slot";
            myGun[1]="Second Slot";
            myGun[2]="Third Slot";
            myGun[3]="Fourth Slot";
            myGun[4]="Fifth Slot";
            myGun[5]="Sixth Slot";
            myGun.sort(randomSort).toString();

            document.getElementById("myArray").innerHTML = '';



            for(i=0; i < myGun.length; i++)
                document.getElementById("myArray").innerHTML += myGun[i] + "<br />";

            document.getElementById("myPos").innerHTML = "Position:" + (--pos);
        }

        function shot()
        {
            if(pos < myGun.length)
                document.getElementById("myPos").innerHTML = "Position:" + (pos++) + "<br />";
            else
                alert("you loose");

            return pos;
        }

     </script>
 </head>
 <body>
    <hr />
    <div>
        <div id="myPos"></div>
    </div>

    <hr />
    <!--Storing my array -->
    <div id="myArray">

    </div>

    <br />
    <br />

    <footer>
        <input type="button" id="btnRoll" onclick="roll()" value="Roll" />
        <input type="button" id="btnShot" onclick="shot()" value="Shot" />
    </footer>
</body>
</html>