Ah, yes, I see now. If the number is outside the range the alert pops up, but the next line is executed nonetheless.
Try sticking this line below the alert: code: return false; Or, preferably, adding an else clause: code: function getChosenNumber()
{
numberChosen=
window.prompt('Player ' + currentPlayer + ' please choose a number between 1 and 3.','');
if (numberChosen < 1 || numberChosen > 3)
{
alert('please choose a number between 1 and 3');
return false;
} else
{
return numberChosen;
}
}
bastard by name, bastard by nature |