Archive for the FAIL Category

if($state_check) {
  $selected_state=$state_code;
} else {
  $selected_state=$state_code;
}

… too bad $state_check was not defined…

Its use is obvious!

| December 23rd, 2008

Here’s a JavaScript function, buried deep within a 5000+ line JavaScript file with absolutely no documentation, let alone variables that might indicate what the function’s use is.

function checkCheck(ct) {
        var v1;
        var v2 = false;
        var x;
        for(x = 1; x <= ct; x++) {
                v1 = $('check_' + x);
                if(v1) {
                        if(v1.checked == 1) {
                                return true;
                                break;
                        }
                        else {
                                continue;
                        }
                }
        }
        return v2;
}

Aside from the weirdness of the return/break/continue statements, v2 is never even used other than just to return false!