javascript - 猜数字游戏中的全局变量
问题描述
我正在尝试制作 MDN 在其 JS 教程中提供的标准数字猜谜游戏。我试着稍微调整一下,为不同的场景制作不同的功能。
似乎全局变量var userGuess = parseInt(guessField.value)
不起作用,因为您之前的猜测总是出现NaN
.
此外,当游戏重置showWin()
和showLoss()
功能工作,但不是showError()
功能。
我对 JS 和一般编码非常陌生,所以很可能在某个地方出现了一个愚蠢的错误,如果有人可以帮助我解决这个问题,那将不胜感激!
var randNum = Math.floor(Math.random() * 100) + 1;
var guessField = document.querySelector('.guessField');
var guessSubmit = document.querySelector('.guessSubmit');
var guesses = document.querySelector('.guesses');
var lastResult = document.querySelector('.lastResult');
var lowOrHi = document.querySelector('.lowOrHi');
var guessCount = 1;
var resetButton;
var userGuess = parseInt(guessField.value);
function checkGuess() {
if(guessCount === 1) {
guesses.textContent = "Previous Guesses: ";
}
guesses.textContent += userGuess + ' ';
if(userGuess === randNum) {
showWin();
} else if(guessCount === 10) {
showLoss();
} else {
showError();
}
guessCount++;
guessField.value = '';
guessField.focus();
}
guessSubmit.addEventListener('click', checkGuess);
function showWin() {
lastResult.textContent = 'You won nice job schmuck';
lastResult.style.backgroundColor = 'green';
gameOver();
}
function showError() {
lastResult.textContent = 'Sorry, wrong guess';
if(userGuess > randNum) {
lowOrHi.textContent = 'Your guess was too high';
} else if(userGuess < randNum) {
lowOrHi.textContent = 'Your guess was too low';
}
}
function showLoss() {
lastResult.textContent = 'You lost, you schmuck';
lastResult.style.backgroundColor = 'red';
gameOver();
}
function gameOver() {
guessField.disabled = true;
guessSubmit.disabled = true;
resetButton = document.createElement('button');
resetButton.textContent = 'New Game';
document.body.appendChild(resetButton);
resetButton.addEventListener('click', resetGame);
}
function resetGame() {
guessCount = 1;
var resetParas = document.querySelectorAll('.resultParas');
for(i = 0; i < resetParas.length; i++) {
resetParas[i].textContent = '';
}
guessField.disabled = false;
guessSubmit.disabled = false;
resetButton.parentNode.removeChild(resetButton);
lastResult.style.backgroundColor = 'white';
randNum = Math.floor(Math.random() * 100) + 1;
}
<h1>Guessing Game</h1>
<p>Type in a number between 1 and 100 and I will tell you if it is too high or low.</p>
<form>
<label for="guessField">Enter a guess: </label>
<input type="text" id="guessField" class="guessField"/>
<input type="button" value="Submit Guess" class="guessSubmit"/>
</form>
<div class='resultParas'>
<p class="guesses"></p>
<p class="lastResult"></p>
<p class="lowOrHi"></p>
</div>
解决方案
在您的脚本中,您调用
parseInt(guessField.value) // effectively that is parseInt("") when it's empty
parseInt()
使用空字符串调用返回NaN
;
MDN 在他们的示例中使用:
var userGuess = Number(guessField.value);
Number("")
返回一个 0 数值。
您还需要在userGuess
每次调用时更新checkGuess()
. 因此,您需要的更改是:
// ... code
var userGuess = Number(guessField.value);
// ... the rest of code
function checkGuess() {
userGuess = Number(guessField.value)
// ... rest of code
}
// rest of code
当然,您不必使用Number()
,也可以进行其他一些条件检查,但这Number()
是一种接受数字或空字符串的优雅方式。
在这里更新 新的 jsbin。
对于resetGame()
部分:您选择的.resultParas
是:
var resetParas = document.querySelectorAll('.resultParas');
然后你迭代了结果并替换了.textContent
那些元素。但那些不是简单的文本节点,它们是段落节点,里面有文本节点。我将其更改为:
var resetParas = document.querySelector('.resultParas').children;
它应该工作!我在 jsfiddle 中添加了一些评论以获得更多解释。
推荐阅读
- dynamic - 来自任意表的动态 SELECT
- jenkins - Cloudbees Jenkins Bitbucket 分支源插件配置硬编码以使用云?
- c# - 如何请求用户输入(内联)
- c# - C# Windows 服务未执行某些代码
- java - Docker容器以java中的命令行开始
- python - 如何在记录 dictConfig 中配置 JsonFormatter?
- apache-spark - pyspark Dataframe 到分区的 Hive 表中
- java - 用文本替换内容控件
- mysql - 如何在laravel中检测内部服务器错误(500)
- ruby-on-rails - 在同一线程中使用辅助/自治事务