check if number is inside the range or beyond the range

Question

i have an a range number coming from server as below.

var max = 50;
var min = 25;
var uservalue = document.getElementById("userinput").value;// 35

User enters value in input box and i need to compare if entered value is inside the range or above the range. I have written code as below but not working.

if(uservalue<min || uservalue>max ){
alert("Error");
}else if (uservalue>min && uservalue<max)
alert("success");

What is the recommended conditions to see if user entered value is inside the rang or outside the range EDIT: user enters value inside and out side the range, i am looking for a condition to change the button state to yellow, red , green accordingly of user input value. Now it always turns to be out of range and hitting 1st condition

you just need one of the two conditions. I'd go for the second (check if it is between) but use `<=` and `>=` if you want to include the case when uservalue == min or max — Lelio Faieta, Nov 17 '16 at 09:48
Possible duplicate of [Check a range of numbers in an if condition](http://stackoverflow.com/questions/6454198/check-a-range-of-numbers-in-an-if-condition) — Dominick, Nov 17 '16 at 09:49

score 2 · Answer 1 · answered Nov 17 '16 at 09:47

2

You are reading uservalue in as a string, you need to convert it to an int:

uservalue = parseInt(uservalue)

Also, this should be enough:

if(uservalue < min || uservalue > max ){
   alert("Error");
}
else {
   alert("success");
}

answered Nov 17 '16 at 09:47

Joe

2,500
1
14
12

score 1 · Answer 2 · answered Nov 17 '16 at 09:48

.value will be a string value, not an integer. JavaScript's type-coercion is... complicated, so you'd be best-off doing explicit integer conversion with parseInt and isNaN:

var max = 50;
var min = 25;
var userValueStr = document.getElementById('userInput').value;
var userValueNum = parseInt( userValueStr );
if( !isNaN( userValueNum ) ) {

    if( min <= userValueNum && userValueNum < max ) {

        alert("Success");
    }
    else {

        alert("Outside of range");
    }
}
else {

    alert("Value was not a number.");
}

score 1 · Answer 3 · answered Nov 17 '16 at 09:48

1

I would did it upside-down

if(uservalue>=min && uservalue<=max ){
    alert("In range");
}else if (uservalue>min){
    alert("Too high");
}else alert("Too low:);
alert("success");

answered Nov 17 '16 at 09:48

Alex Bykov

718
6
13

score 1 · Answer 4 · answered Nov 17 '16 at 09:49

1

if(uservalue < min || uservalue > max ){
   alert("Error");
}
else {
   alert("success");
}

This must work.

answered Nov 17 '16 at 09:49

Deep Devi

37
6

score 1 · Answer 5 · answered Nov 17 '16 at 09:49

1

use:

var uservalue = parseInt(document.getElementById("userinput").value);

or

if(+uservalue<min || +uservalue>max ){
    ...
}else if (+uservalue>min && +uservalue<max){
    ...
}

answered Nov 17 '16 at 09:49

Sean W

527
6
15

prasanth · Answer 6 · 2016-11-17T09:57:17.950

0

If input type="text" use with parseInt()

function check(){
var uservalue = parseInt(document.getElementById("userinput").value);
var max = 50;
var min = 25;
if (uservalue>min && uservalue<max){
console.log("success");
}else 
  {
    console.log("Error");

  }
}

<input type="text" id="userinput" onchange="check()">
click me..!

edited Nov 17 '16 at 09:57

answered Nov 17 '16 at 09:52

prasanth

22,145
4
29
53

check if number is inside the range or beyond the range

6 Answers6