CodeSOD: Functional IsFunction

Julio S recently had to attempt to graft a third-party document viewer onto an internal web app. The document viewer was from a company which specialized in enterprise "document solutions", which can be purchased for enterprise-sized licensing fees.

Gluing the document viewer onto their internal app didn't go terribly well. While debugging, and browsing through the vendor's javascript, he saw a lot of calls to a function called IsFunction. It was loaded from a "utilities.js"-type do-everything library file. Curious, Julio pulled up the implementation.

function IsFunction ( func ) { var bChk=false; if (func != "undefined") bChk=true; else bChk=false; return bChk;}

I cannot emphasize enough how beautiful this block of code is, by the standards of bad code. There's so much there. One variable, bChk uses Hungarian notation. Nothing else seems to. It's a totally superfluous variable, as we could just do return func != "undefined".

Then again why would we even do that? The real beauty, though, is how the name of the function and its implementation have no relationship to each other, and the implementation is utterly useless. For example:

IsFunction("Hello World"); //trueIsFunction({spam: "eggs"}); //trueIsFunction(function() {}); //true, but it was probably an accidentIsFunction(undefined); //trueIsFunction("undefined"); //false

Yes, the only time this function returns false is the specific case where you pass it the string "undefined". Everything else IsFunction apparently. The useless function sounds important. Someone wrote it, probably as a quick attempt at vaguely defensive programming. "I should make sure my inputs are valid". They didn't test it. The certainly didn't think about it. But they wrote it. And then someone else saw the function in use, and said, "Oh" I should probably use that, too." Somewhere, there's probably a "Style Guide", which mandates that, before attempting to invoke a variable that should contain a function, you use IsFunction to confirm it does. It comes up in code reviews, and code has been held from going into production because someone didn't use IsFunction.

And Julio probably is the first person to actually check the implementation since it was first written.

[Advertisement] Manage IT infrastructure as code across all environments with Puppet. Puppet Enterprise now offers more control and insight, with role-based access control, activity logging and all-new Puppet Apps. Start your free trial today!