Two return statements in Mustache.getHelper

[daffl]

Just curious as to why there are two return statements in the afore-mentioned function??

[daffl]

If you mean https://github.com/bitovi/canjs/blob/master/view/mustache/mustache.js#L1653, this is a common JS practise called early return. This avoids going through the whole loop because the function returns as soon as a matching helper has been found. I generally recommend using it. The easiest case looks like this:

function early(condition) {
    if(condition) {
        return 'true';
    }

    return 'false';
}

[djdev]

So, when will it go to the FOR loop.

And is it advisable to return NULL from a function.

On Fri, Aug 23, 2013 at 2:19 PM, David Luecke notifications@github.comwrote:

If you mean
https://github.com/bitovi/canjs/blob/master/view/mustache/mustache.js#L1653,
this is a common JS practise called early return. This avoids going through
the whole loop because the function returns as soon as a matching helper
has been found. I generally recommend using it. The easiest case looks like
this:

function early(condition) {
if(condition) {
return 'true';
}

return 'false';}

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/465#issuecomment-23181642
.

[daffl]

Oh now I see what you mean! That looks pretty wrong ;)