Announcement

**Old Pedant** · Sep 9, 2011, 02:53 PM

Looks right to me.

You might want to add break; at the end of your "do domehting" code, else if there is more than one <div> with that class name the "do something" code will execute once for each such <div>. The break; says "exit the surrounding for loop".

**turpentyne** · Sep 9, 2011, 03:11 PM

hmmm... the break; didn't make a difference. I did find a jquery solution that works:

if($('div').hasClass('single-journal-entry-wrapper')) {do something}

But as I learn more, that has me wondering what the problems with using jquery are. I know it's obviously cheating me out of learning to code properly. But other than that?

**Old Pedant** · Sep 9, 2011, 04:17 PM

Teach me not to read carefully!

You omitted a very important variable assignment and then misused a variable.

Code:

<html>
<head>
<script>
function doit() 
{
    // you MUST assign the collection of <div>s to a variable!
    [B][COLOR="Red"]var divs [/COLOR][/B]= document.getElementsByTagName("div");
    // you must search *IN* that collection!
    for ( var x = 0; x < [B][COLOR="Red"]divs[/COLOR][/B].length; x++ )
    {
        // and must get one member of that collection:
        var div = [B][COLOR="Red"]divs[/COLOR][/B][x];
        if ( div.className.indexOf("single-journal-entry-wrapper") >= 0 )
        { 
            alert("Found this: " + div.innerHTML );
            // try un-commenting next line to see the difference
[B][COLOR="Red"]            // break; [/COLOR][/B]
        }
    }
}
window.onload = doit;
</script>
</head>
<body>
<div class="whompit">
   should not find this
</div>
<div class="whompit single-journal-entry-wrapper">
   should find this first
</div>
<div class="single-journal-entry-wrapper">
   should ALSO find this if you omit the [b]break[/b]
</div>
</body>
</html>

Announcement

is this script opener right? getting element class

is this script opener right? getting element class

Comment

Comment

Comment