Javascript全局变量不能正常工作

我在JavaScript中遇到全局变量问题。我试过的是在函数外声明变量，然后在函数内改变它，然后调用另一个函数。从我读过的内容来看，这应该是有效的，但它只是给了我不确定的东西。这是我制作的卡片绘制游戏的代码。Javascript全局变量不能正常工作

var randSuit; 
function getRandCard() { 
    var randNum; 
    var randSuit; 
    var randVal; 
    randNum = Math.floor(Math.random()*13)+1; 
    if (randNum == 1) { 
     randVal = "2"; 
    } else if (randNum == 2) { 
     randVal = "3"; 
    } else if (randNum == 3) { 
     randVal = "4"; 
    } else if (randNum == 4) { 
     randVal = "5"; 
    } else if (randNum == 5) { 
     randVal = "6"; 
    } else if (randNum == 6) { 
     randVal = "7"; 
    } else if (randNum == 7) { 
     randVal = "8"; 
    } else if (randNum == 8) { 
     randVal = "9"; 
    } else if (randNum == 9) { 
     randVal = "10"; 
    } else if (randNum == 10) { 
     randVal = "Jack"; 
    } else if (randNum == 11) { 
     randVal = "Queen"; 
    } else if (randNum == 12) { 
     randVal = "King"; 
    } else if (randNum == 13) { 
     randVal = "Ace"; 
    } 

    randNum = randNum = Math.floor(Math.random()*4)+1; 

    if (randNum == 1) { 
     randSuit = "Hearts"; 
    } else if (randNum == 2) { 
     randSuit = "Clubs"; 
    } else if (randNum == 3) { 
     randSuit = "Spades"; 
    } else if (randNum == 4) { 
     randSuit = "Diamonds"; 
    } 

    console.log(randSuit); 
    var randCard = (randVal + " of " + randSuit); 
    //Return the Value of the randomly chosen Card. 
    return (randCard); 
} 

//This function calls the random card from the function above, then applies logic to see if it's the same, then outputs the result. 
$(function() { 
    $('#drawCard').click(function() { 

     var e = document.getElementById("faceValue"); 
     var faceValue = e.options[e.selectedIndex].text; 
     var e = document.getElementById("suit"); 
     var suit = e.options[e.selectedIndex].text; 

     $('#oneCardContainer').slideDown('slow'); 
     var pickedCard = (faceValue + " of " + suit); 

     var randCard = getRandCard(); 
     console.log (randSuit); 

     if (pickedCard == randCard) { 
      $("#oneCardResults").val("You Chose a " + pickedCard + " and got a " + randCard + ". \nYou Win!"); 
     } else if (pickedCard != randCard) { 

      $("#oneCardResults").val("You Chose a " + pickedCard + " and got a " + randCard + ". \nYou Lose!"); 

     } 
    }); 
});

这是我试过的代码和我试图通过的变量是randSuit。我究竟做错了什么？

来源

2012-04-19 Jack Davis

您正在定义一个名为randSuit的全局变量，但是也是的一个具有相同名称的局部变量。当你做randSuit = randSuit;时，实际上没有任何反应，因为左侧和右侧都引用了局部变量。你需要以不同的方式命名它们。

来源

2012-04-19 02:06:10 jli

你在函数里面重新声明randSuit;该声明隐藏全局randSuit，所以你的函数正在修改它自己的局部变量而不是全局变量。

来源

2012-04-19 02:06:52

试试这个，这是你的代码的简化版本，其应该工作：

var randSuit; 
function getRandCard() { 
    var randNum, randVal; 

    var upperCards = ["Jack", "Queen", "King", "Ace"]; 
    var suits = ["Hearts", "Clubs", "Spades", "Diamonds"]; 

    randNum = Math.floor(Math.random()*13)+1; 
    (randNum < 10) ? randVal++ : randVal = upperCards[randNum-10]; 

    randSuit = suits[Math.floor(Math.random()*4)]; 

    // return the value of the randomly chosen card. 
    return (randVal + " of " + randSuit); 
}

我用了几个阵列和三元运营商大大缩短你的代码。我也消除了不必要的局部randSuit变量，它的作用域和覆盖了你的全局对象。

来源

2012-04-19 02:12:18

当你重新声明函数内部randsuit，它私有化该函数

var randSuit = 5; // not shared 
function getRandCard() { 
    var randSuit = 3; // not shared 
    console.log(randSuit); 
} 
getRandCard(); 
console.log(randSuit);

http://jsfiddle.net/PbBph/

如果你想分享的randSuit输出不重新声明变量

var randSuit = 5; // shared 
function getRandCard() { 
    randSuit = 3; // shared 
    console.log(randSuit); 
} 
getRandCard(); 
console.log(randSuit);

http://jsfiddle.net/PbBph/1/

一个更好的选择将是模块化卡变量和方法

var cardStack = (function() { 
    var randSuit; // protected from global 

    return { 
     getRandSuit: function() { return randSuit; }, // but still readable 
     getRandCard: function() { .... } 
    }; 

}()); 

var card = cardStack.getRandCard(); 
    suit = cardStack.getRandSuit();

来源

2012-04-19 02:12:54 Trevor

@Elliot巴纳维亚的和@ jfriend00的回答都不错，但这里是有点落后的局部和全局变量问题的理论解释。

JavaScript与全局和局部变量一起工作的方式是当解释器遇到一个标识符时，它在当前范围内搜索它（在你的情况下：getRandCard），如果它找不到它 - 解释器上面有一个范围，如果它不能在那里找到它 - 它会在两个范围之上等等。

randSuit = randSuit;

在这一行这些标识符都将指向同一个局部变量，作为解释发现他们无论是在局部范围内，所以有效这一行什么也不做。

为了引用全局变量，你应该建立其范围参考：

var randSuit; 
var that = this;  

function getRandCard() { 
    ... 
}

然后，替换：

randSuit = randSuit;

有：

that.randSuit = randSuit;

来源

2012-04-19 02:15:47

不再是相关的，但你不能真正通过'that'在'randSuit'得到除非'那是'窗口'。 – jli 2014-05-11 00:17:49

@jli true，除非你声明你的变量是这样的：'this.randSuit = val;' – 2014-05-11 06:45:12

如果你想使用一个全局变量，然后在全局范围内定义该变量，而不是在本地重新定义它，只需在本地使用它。当您在本地重新定义它时，将创建一个新的局部变量，其名称与该范围内的全局变量相同。

其余部分是真正的评论，但由于无法在评论中有效地包含代码，因此我会将其发布为答案的一部分。您确实需要将DRY（不要重复自己）原则应用于您的代码。这是可怕的重复：

randNum = Math.floor(Math.random()*13)+1; 
if (randNum == 1) { 
    randVal = "2"; 
} else if (randNum == 2) { 
    randVal = "3"; 
} else if (randNum == 3) { 
    randVal = "4"; 
} else if (randNum == 4) { 
    randVal = "5"; 
} else if (randNum == 5) { 
    randVal = "6"; 
} else if (randNum == 6) { 
    randVal = "7"; 
} else if (randNum == 7) { 
    randVal = "8"; 
} else if (randNum == 8) { 
    randVal = "9"; 
} else if (randNum == 9) { 
    randVal = "10"; 
} else if (randNum == 10) { 
    randVal = "Jack"; 
} else if (randNum == 11) { 
    randVal = "Queen"; 
} else if (randNum == 12) { 
    randVal = "King"; 
} else if (randNum == 13) { 
    randVal = "Ace"; 
} 
randNum = randNum = Math.floor(Math.random()*4)+1; 

if (randNum == 1) { 
    randSuit = "Hearts"; 
} else if (randNum == 2) { 
    randSuit = "Clubs"; 
} else if (randNum == 3) { 
    randSuit = "Spades"; 
} else if (randNum == 4) { 
    randSuit = "Diamonds"; 
}

而且可以用这个远不重复的代码来代替：

var cards = ["1", "2", "3", "4", "5", "6", "7", "8", "9", "Jack", "Queen", "King", "Ace"]; 
randVal = cards[Math.floor(Math.random() * cards.length)]; 

var suits = ["Hearts", "Clubs", "Spades", "Diamonds"]; 
randSuit = suits[Math.floor(Math.random() * suits.length)];

来源

2012-04-19 02:18:15 jfriend00

如果你想要非常聪明，你甚至可以这样做：'[“1”，“2”，“3”，“ 4，5，6，7，8，9，Jack，Queen，King，Ace等[Math.floor（Math.random（）* 13）];'因为你只使用'cards'变量一次。 =） – 2012-04-19 02:20:18

Javascript全局变量不能正常工作

回答

相关问题