I generate a random number with an interval and then I check if the number chooses by the user is the same, but it always returns false
JavaScript
x
22
22
1
const checkButton = document.getElementById("checkButton");
2
var rndNumber = RandomNumberGenerator();
3
checkButton.onclick = CheckNumber;
4
console.log("rndNumber", rndNumber);
5
6
function CheckNumber(rndNumber) {
7
var numberChoosed = document.getElementById("numberChoosed").value;
8
console.log("rndNumber", rndNumber)
9
console.log("numberChoosed", numberChoosed);
10
if (numberChoosed == rndNumber) {
11
console.log("true");
12
} else {
13
console.log("False");
14
}
15
16
}
17
18
function RandomNumberGenerator() {
19
var min = document.getElementById("bottomNumber").value;
20
var max = document.getElementById("topNumber").value;
21
return Math.round(Math.random() * (max - min) + min);
22
}
JavaScript
1
5
1
<button type="button" id="checkButton">Click</button>
2
3
<input id="numberChoosed" />
4
<input id="bottomNumber" value="1"/>
5
<input id="topNumber" value="110"/>
Advertisement
Answer
Remove the rndNumber
from function CheckNumber(rndNumber) {
because that is actually the click event on the checkButton
Also a good idea to cast to number since .value
is a string.
The ==
will cast for you but it is no longer up to you then
JavaScript
1
18
18
1
const checkButton = document.getElementById("checkButton");
2
var rndNumber = RandomNumberGenerator();
3
checkButton.onclick = CheckNumber;
4
console.log("rndNumber", rndNumber);
5
6
function CheckNumber() {
7
var numberChoosed = document.getElementById("numberChoosed").value;
8
console.log("rndNumber", rndNumber)
9
console.log("numberChoosed", numberChoosed);
10
console.log(+numberChoosed === +rndNumber); // cast to number
11
12
}
13
14
function RandomNumberGenerator() {
15
var min = document.getElementById("bottomNumber").value;
16
var max = document.getElementById("topNumber").value;
17
return Math.round(Math.random() * (max - min) + min);
18
}
JavaScript
1
5
1
<button type="button" id="checkButton">Click</button>
2
3
<input id="numberChoosed" />
4
<input id="bottomNumber" value="1"/>
5
<input id="topNumber" value="110"/>