Using onblur to trigger a function

I have a page with two text boxes,

<input type="text"  id="number_one" placeholder="Please Enter a Number (1-5)">
<input type="text"  id="number_two" placeholder="Please Enter a Number (1-5)">`

and am trying to fire off a function

function checkInput(value) {
	 
	if(isNaN(value)){
	  alert("Enter a number");
	}
	if (value > 5 || value < 1) {
	  alert("It must be between 1 & 5");
	}
}

whenever either of those text boxes are filled out

document.getElementById("number_one").onblur = checkInput(first);
document.getElementById("number_two").onblur = checkInput(second);

where

var first = document.getElementById("number_one").value;
var second = document.getElementById("number_two").value;

Is this ok?

Are you sure the conditional logic is correct?
(value > 5 && value < 1)

I’m having a difficult time (maybe I’m too euclidean) imagining a number ever being able to be both greater than 5 and less than 1 at the same time.

2 Likes

I changed && to ||

1 Like

This topic was automatically closed 91 days after the last reply. New replies are no longer allowed.