If Statement Funkiness

Hey All,

I was happy to find this forum, I didn't know there was a new one since the site update at Moo.

I have some strangeness that I'm wondering if someone can help with.

I have my js functions separated into 2 functions in my control file (which holds all the js calls to mootools) as follows:

function pageVariables ()

and

function loadVariables ()

then I implement these functions accordingly via window.addEvent ('domready', pageVariables) etc.

It makes a nice easy to understand set up where I can simply nest functions inside one or the other to ensure they are called at the right time.

As part of this, since all functions become available on either load or domready I've always used "if" statements to stop errors since a given object may not appear on a given page and would produce an error.

For example:



This way the code within will only execute if there's a link with the class of myClass on the page.

This has always worked perfectly till today

Is there some reason this would be failing? or perhaps a better or more elegant way to do the same thing?

Thanks

Josh

From the docs:

If an expression doesn't find any elements, an empty array will be returned.

You actually can remove the if condition:

Right,

But that's the whole purpose of putting the expression into the if statement so that the empty array isn't thrown at all.

Let me give you a working example.

I have 2 pages for instance active.html and error.html

On the active.html page I want to implement some MultiBox links that will display my terms, privacy etc.

On the error.html page I don't want these to open at all, so I use a .stop() action by class selector.

So, no MultiBox links mean no need to add the excess overhead of loading multibox.js.

My code in the pageVariable function looks like so:



The objective being obviously to allow the MultiBox object to be implemented ONLY when the .mb links appear on a page, and to kill those links when they're class equals .da

But, this doesn't work, it throws an error on the error.html page saying that MultiBox is not defined.

Hope that makes sense

Josh

Of course you can only remove the if condition if you use the elements from $$() in a loop.

Ah, see ... there's always a simple solution

Curious though that I haven't run into this before now, probably just a change in my technique. I used to load everything, compress it and cache it. I'm trying now to get away from that and load only what's absolutely necessary and ONLY when it's needed.

Anyway, thanks a bunch, I really appreciate the help

Josh