javascript for loop checking shows error

Question

function check()
{
    var rowl=document.getElementsByClassName("tbody")[0].rows;
    for(var r=0;r<rowl.length;r++)
    {
        ch=document.getElementsByClassName("tbody")[0].rows[r].cells[2].childNodes[0].value;
        
        var tiderow=document.getElementsByClassName("tide")[r];//rows[0].cells[1].childNodes[0].value;
        for(var t=0;t<tiderow.rows.length;t++)
        {
            var noo=tiderow.rows[t].cells[1].childNodes[0].value;
            console.log(noo);
            

        }
        
    }
}

Whenever I run this code, this prints correctly the subtable values.

And I am checking each subtable value with the Main table cells2 value. If it is greater, then I am showing the error message. But even though it is not greater it shows the error

function check()
{
    var rowl=document.getElementsByClassName("tbody")[0].rows;
    for(var r=0;r<rowl.length;r++)
    {
        ch=document.getElementsByClassName("tbody")[0].rows[r].cells[2].childNodes[0].value;
        
        var tiderow=document.getElementsByClassName("tide")[r];//rows[0].cells[1].childNodes[0].value;
        for(var t=0;t<tiderow.rows.length;t++)
        {
            var noo=tiderow.rows[t].cells[1].childNodes[0].value;
            console.log(noo);
            if(noo>ch)
            {
                console.log("Error occured");
            }

        }
        
    }
}

AuxTaco · Accepted Answer · 2017-12-31 12:10:10Z

2

Ah, but it is greater. You're comparing "5" to "10". "5", the string, is greater than "10", the string. If you want to compare these as numbers, convert them to numbers before the comparison:

ch = Number(rowl[r].cells[2].childNodes[0].value);
var noo = Number(tiderow.rows[t].cells[1].childNodes[0].value);

answered Dec 31, 2017 at 12:10

AuxTaco

5,2011 gold badge17 silver badges28 bronze badges

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

3 Comments

Angel Politis Over a year ago

What is this thing you're doing? Just use the unary operator +.

AuxTaco Over a year ago

@AngelPolitis A matter of style. It always takes me just a little longer to figure out what's going on with the unary plus.

Angel Politis Over a year ago

Surely, but since the only thing it does is convert something to a number, I believe it's just a matter of getting used to it. Great answer nonetheless! +1

TheOni · Accepted Answer · 2017-12-31 12:09:53Z

1

you should do

parseInt(noo)>parseInt(ch)

else javascript will treat this variables as string so 5 is greater than 10

answered Dec 31, 2017 at 12:09

TheOni

81810 silver badges28 bronze badges

Collectives™ on Stack Overflow

javascript for loop checking shows error

2 Answers 2

3 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related