JavaScript IF statement with conditions from input field

Question

I'm trying to make a simple calc usign values from input fileds, but I have a problem with passing values to if statement. It looks like this:

$('#confirmSila').click(function(){
var dlugosc = $('#sila').val();
var silaOd = $('.silaOd').val();
var silaDo = $('.silaDo').val();
if ((dlugosc > silaOd) && (dlugosc < silaDo))
{
//$('#orderWymiar').attr('value', dlugosc);
$('#sila').css('border', '1px solid green');
$('.silaInfo').text('Statement true');
}else{
$('#wymiar').css('border', '1px solid red');
$('.silaInfo').text('silaOd must be more than' + silaOd + ', and less than ' + silaDo);
}
});

And, the wird thing is that, when force is 20, JavaScript says that it passed the statemenst, when in input .silaOd is typed 100. (silaOd meand minForce, and sildaDo means maxForce in Poland)

I dont have any idea, why that is happening. Please help! :)

There's a working example: http://jsfiddle.net/rJA6U/

I hope those variables are understandable in a language you speak, because if not that's just confusing... — davin
– davin, Commented Mar 31, 2011 at 14:09

n00dle · Accepted Answer · 2011-03-31 14:11:00Z

3

You need to run parseInt() on the input values, like so:

var dlugosc = parseInt($('#sila').val());
var silaOd = parseInt($('.silaOd').val());
var silaDo = parseInt($('.silaDo').val());

answered Mar 31, 2011 at 14:11

n00dle

6,0632 gold badges40 silver badges49 bronze badges

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

1 Comment

user529649 Over a year ago

you might consider this as well, stackoverflow.com/questions/2240490/….

Mark · Accepted Answer · 2011-03-31 14:09:20Z

1

You are comparing strings so < or > does not work. You need to cast them as int.

example:

parseInt(dlugosc) > parseInt(sila0d)

answered Mar 31, 2011 at 14:09

Mark

5,7302 gold badges28 silver badges26 bronze badges

Comments

andres descalzo · Accepted Answer · 2011-03-31 14:16:27Z

try this:

$.fn.tryParseInt = function(defaultValue) {

    var 
        retValue = defaultValue,
        actualValue = $(this).val();

    if (actualValue != null) {
        if (actualValue.length > 0) {
            if (!isNaN(actualValue)) {
                retValue = parseInt(actualValue);
            }
        }
    }

    return retValue;
}


var 
  dlugosc = $('#sila').tryParseInt(0),
  silaOd = $('.silaOd').tryParseInt(0),
  silaDo = $('.silaDo').tryParseInt(0);

based on this link

Collectives™ on Stack Overflow

JavaScript IF statement with conditions from input field

3 Answers 3

1 Comment

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Linked

Related