Comparison Operators

[smartic]

I'm having problem with this code:

function check(){
    element1 = 2;
    element2= 110;
    if(element1 > element2){
        alert('Sorry, element1 should be less than element2');
    }
}

what is the problem in this code ???

[gits]

?? where is the problem ?? ... 2 isn't greater then 110 so you don't get the alert ...

[smartic]

i know that 2 isn't greater than 110 but i get the alert message how can i solve this problem I'm getting mad that is my real code:

function is_numeric(mixed_var){
    return !isNaN(mixed_var);
}
if(is_numeric(value) && is_numeric(compv)){
    if(condi == '<' && (compv < value)){
        forError(this.elemID,'Sorry, '+elemID+' should be less than '+compn);
    }else if(condi == '>' && (compv > value)){
        forError(this.elemID,'Sorry, '+elemID+' should be greater than '+compn);
    }else if(condi == '>=' && (compv >= value)){
        forError(this.elemID,'Sorry, '+elemID+' should be greater than or equal '+compn);
    }else if(condi == '<=' && (compv <= value)){
        forError(this.elemID,'Sorry, '+elemID+' should be less than or equal '+compn);
    }
}else{
    forError(this.elemID,'Sorry, numeric letters only');
}

[gits]

so you just posted the wrong code ... but i think your problem is that you in fact compare strings like that:

element1 = '2';
element2 = '110';
 
if (element1 > element2) {
    alert('Sorry, element1 should be less than element2');
}
 

so you just need to parseInt() or parseFloat() the values.

kind regards

[smartic]

thank you for your replay gits that was my problem.

[gits]

no problem ... you're welcome ...

kind regards