javascript If statement not working

Question

I'm trying to show an alert if the value of the input (inp_val) isn't between minvalue and maxvalue but it doesn't work, it shows the alert even if the value is between minvalue and maxvalue.

function compare(e){
    this_id = e.id.slice(10,15);
    inp_val = e.value;
    minvalue = document.getElementById('min_' + this_id).value;
    maxvalue = document.getElementById('max_' + this_id).value;
    console.log(maxvalue);
    console.log(minvalue);
    console.log(inp_val);
    if(inp_val < minvalue){
        alert('Minimum Value: ' + minvalue + 'Input Value: ' + inp_val);
    }
    if(inp_val > maxvalue){
        alert('Maximum Value: ' + maxvalue + ' Input Value: ' + inp_val);
    }
}

I'm calling the function compare with an onBlur():

echo '<input class="result_fields" autocomplete="off" onBlur="compare(this)" id="result_inp' . $idc . '" type="text" name="test_res' . $idc . '" />'

I've tried to do this in jQuery but is the same error. Minvalue and maxvalue are input type hidden and its values are stored in a DB on MySQL and retrieved with php like this:

echo '<input type="hidden" value="' . $val_test . '" name="test_name' . $idc . '" id="test_name' . $idc . '" class="testN" />';

The php variable $idc is an incremental function $idc++.

Also , it seems that chrome (26) already handle this. parseInt('010')=10 and parseInt(010)=8. — Royi Namir
– Royi Namir, Commented Apr 5, 2013 at 20:34

Wryte · Accepted Answer · 2013-04-05 20:25:08Z

1

You need to parse your integers or floats from the values you get from the text box

if they are integers:

minvalue = document.getElementById('min_' + this_id).value;
maxvalue = document.getElementById('max_' + this_id).value;
minvalue = parseInt(minvalue,10);
maxvalue = parseInt(maxvalue,10);

Or if they are floats:

minvalue = document.getElementById('min_' + this_id).value;
maxvalue = document.getElementById('max_' + this_id).value;
minvalue = parseFloat(minvalue);
maxvalue = parseFloat(maxvalue);

answered Apr 5, 2013 at 20:25

Wryte

9153 gold badges10 silver badges24 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

gen_Eric Over a year ago

You should really use parseInt(minvalue, 10).

fernandosavio Over a year ago

There is the Unary Operator Plus that can help too...

Sachin · Accepted Answer · 2013-04-05 20:23:50Z

1

You need to apply < and > operators on integer value so parse string value to integer and then compare.

if(parseInt(inp_val,10) < parseInt(minvalue,10)){
        alert('Minimum Value: ' + minvalue + 'Input Value: ' + inp_val);
    }

answered Apr 5, 2013 at 20:23

Sachin

41.1k7 gold badges93 silver badges107 bronze badges

1 Comment

gen_Eric Over a year ago

You should really use parseInt(inp_val, 10).

palaѕн · Accepted Answer · 2013-04-05 20:24:09Z

0

Try this:

if(!(inp_val >= minvalue && inp_val <= maxvalue)){
     alert('Max Val: ' + maxvalue + ' Min Val: ' + minvalue + ' Input Value: ' + inp_val);

answered Apr 5, 2013 at 20:24

palaѕн

74.2k17 gold badges122 silver badges140 bronze badges

Collectives™ on Stack Overflow

javascript If statement not working

3 Answers 3

2 Comments

1 Comment

Comments

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

1 Comment

Comments

Related