其他函数调用函数不起作用答案

【问题标题】：Calling function by other function not working其他函数调用函数不起作用
【发布时间】：2021-03-26 08:48:45
【问题描述】：

我正在尝试使计算机与人类玩家的石头剪刀布比赛。我有函数computerplay(); 从三个可能的字符串（“Rock”、“Paper”、“Scissors”）中生成随机值作为游戏匹配值的计算机选择。使用第三个函数save(); 我允许玩家将他的选择输入到 HTML 表单中并尝试将其保存到 round()；功能。

作为结果，我希望变量 c 或 p 递增或打印字符串“draw”，以指示 round(); 获取存储在 x 中的计算机选择和作为变量播放器的播放器选择。似乎 x 和 player 都附加了它们的值，但 round(); 函数没有收到它。感谢您帮助解决它。

function computerPlay() {
       const number = Math.floor(Math.random() * 1000);
        if (number % 3 === 0) {
                return 'Rock';
       }
        if (number % 3 === 1) {
                return 'Paper';
        }
        return 'Scissors';
}


var p = 0;
var c = 0;


 var x = computerPlay();
    var player;
      player = document.getElementById("player").value;
 document.write(x);
function round ( playerSelection = player, computerSelection =  x ) {

 document.write(player);
 document.write(x);
 
                 if (playerSelection == 'Rock' && computerSelection == 'Rock') {
           
                      
                      
                     document.write (p);
                     document.write (c); 
                    
                     document.write("draw"); 
                  }

                 else if (playerSelection == 'Rock' && computerSelection == 'Paper') {
           
                      
                     c++; 
                     document.write (p); 
                     document.write (c); 
                     document.write("comp"); 
                  }
           
                 else if (playerSelection == 'Rock' && computerSelection == 'Scissors') {
           
                      
                     p++;
                     document.write (p);  
                     document.write (c); 
                     document.write("player"); 
                  }
                 else if (playerSelection == 'Scissors' && computerSelection == 'Rock') {
           
                      
                     c++;
                     document.write (p);  
                     document.write (c); 
                     document.write("comp"); 
                  }
                else if (playerSelection == 'Scissors' && computerSelection == 'Paper') {
           
                      
                     p++; 
                     document.write (p);
                     document.write (c); 
                     document.write("player"); 
                  }

                else if (playerSelection == 'Scissors' && computerSelection == 'Scissors') {
           
                      
                     document.write (p);
                     document.write (c); 
                     document.write("draw"); 
                  }

                 else if (playerSelection == 'Paper' && computerSelection == 'Rock') {
           
                      
                     p++; 
                     document.write (p);
                     document.write (c); 
                     document.write("player"); 
                  }

                else if (playerSelection == 'Paper' && computerSelection == 'Scissors') {
           
                      
                     c++; 
                     document.write (p);
                     document.write (c); 
                     document.write("computer"); 
                  }

                 else if (playerSelection == 'Paper' && computerSelection == 'Paper') {
           
                      
                     document.write (p);
                     document.write (c); 
                     document.write("draw"); 
                  }
              }









  function save() {
      var player;
      player = document.getElementById("player").value;
       document.write (player);
       document.write(x);
       computerPlay();
       var x = computerPlay();
       round();
       

     

}



document.write (p); 
    document.write (c); 

if ( c == 5 ) {
    document.write (c); 
    document.write ('computer win');
}
else if ( p == 5 ) {
   document.write ('player win');
 }

<p id="demo"></p>

<form id="form">
  <input id="player" type="text" min="1" name="player">
  <button type="button" onclick="save();">Get Value</button>
</form>

【问题讨论】：

嗨，好吧，我想我在这里看到了问题，您正在为 playerSelection 和 computerSelection 设置默认值，但这些值是在加载时设置的，那时玩家选择将为空。您需要在 save() 中的 round() 中传递玩家，这可以解决问题

标签： javascript html

【解决方案1】：

在您的脚本中，您使用了两个变量，都称为player，一个在这里：

var x = computerPlay();
  var player; 
    player = document.getElementById("player").value;

这里有一个：

function save() {
  var player;
  player = document.getElementById("player").value;

第一个变量在脚本主体中声明，这意味着它具有“全局范围”。相反，您在 save() 中定义的那个有一个“函数范围”（这意味着它对同一函数主体之外的任何人都不可见）。

由于您在没有参数的情况下调用round()，它将使用它们的默认值初始化它们，并从发生函数声明的范围（在本例中为全局范围）中获取这些值。由于全局 player 最初为空并且您永远不会更改该变量，因此 round 将始终将其第一个参数作为空字符串。有很多方法可以“解决”这个问题：

直接在round()里面获取变量：

  function round(/* ... */) {
      player = document.get...

实际上是把参数传给函数：
```
  round(player);
```

更新 player 变量的值而不是创建一个新变量：

  var player;

  function save() {
      player = document.get...
      round();
  }

有关 JavaScript 范围规则的更多信息，请阅读here

【讨论】：