When we say functions are "first class", we mean they are just like everyone else... so normal class (coach?). We can treat functions like any other data type and there is nothing particularly special about them - they may be stored in arrays, passed around, assigned to variables, what have you.
That is JavaScript 101, but worth mentioning as a quick code search on github will show the collective evasion, or perhaps widespread ignorance of the concept. Shall we go for a feigned example? We shall.
var hi = function(name){
return "Hi " + name;
};
var greeting = function(name) {
return hi(name);
};Here, the function wrapper around hi in greeting is completely redundant. Why? Because functions are callable in JavaScript. When hi has the () at the end it will run and return a value. When it does not, it simply returns the function stored in the variable. Just to be sure, have a look-see:
hi;
// function(name){
// return "Hi " + name
// }
hi("jonas");
// "Hi jonas"Since greeting is merely turning around and calling hi with the very same argument, we could simply write:
var greeting = hi;
greeting("times");
// "Hi times"In other words, hi is already a function that expects one argument, why place another function around it that simply calls hi with the same bloody argument? It doesn't make any damn sense. It's like donning your heaviest parka in the dead of July just to blast the air and demand an ice lolly.
It is obnoxiously verbose and, as it happens, bad practice to surround a function with another function merely to delay evaluation. (We'll see why in a moment, but it has to do with maintenance.)
A solid understanding of this is critical before moving on, so let's examine a few more fun examples excavated from npm modules.
// ignorant
var getServerStuff = function(callback){
return ajaxCall(function(json){
return callback(json);
});
};
// enlightened
var getServerStuff = ajaxCall;The world is littered with ajax code exactly like this. Here is the reason both are equivalent:
// this line
return ajaxCall(function(json){
return callback(json);
});
// is the same as this line
return ajaxCall(callback);
// so refactor getServerStuff
var getServerStuff = function(callback){
return ajaxCall(callback);
};
// ...which is equivalent to this
var getServerStuff = ajaxCall; // <-- look mum, no ()'sAnd that, folks, is how it is done. Once more then we'll see why I'm so insistent.
var BlogController = (function() {
var index = function(posts) {
return Views.index(posts);
};
var show = function(post) {
return Views.show(post);
};
var create = function(attrs) {
return Db.create(attrs);
};
var update = function(post, attrs) {
return Db.update(post, attrs);
};
var destroy = function(post) {
return Db.destroy(post);
};
return {
index: index, show: show, create: create, update: update, destroy: destroy
};
})();This ridiculous controller is 99% fluff. We could either rewrite it as:
var BlogController = {
index: Views.index,
show: Views.show,
create: Db.create,
update: Db.update,
destroy: Db.destroy
};...or scrap it altogether as it does nothing other than bundle our Views and Db together.
Okay, let's get down to the reasons to favor first class functions. As we saw in the getServerStuff and BlogController examples, it's easy to add layers of indirection that have no actual value and only increase the amount of code to maintain and search through.
In addition, if a function we are needlessly wrapping does change, we must also change our wrapper function.
httpGet('/post/2', function(json){
return renderPost(json);
});If httpGet were to change to send a possible err, we would need to go back and change the "glue".
// go back to every httpGet call in the application and explicitly pass err
// along.
httpGet('/post/2', function(json, err){
return renderPost(json, err);
});Had we written it as a first class function, much less would need to change:
// renderPost is called from within httpGet with however many arguments it wants
httpGet('/post/2', renderPost); Besides the removal of unnecessary functions, we must name and reference arguments. Names are a bit of an issue, you see. We have potential misnomers - especially as the codebase ages and requirements change.
Having multiple names for the same concept is a common source of confusion in projects. There is also the issue of generic code. For instance, these two functions do exactly the same thing, but one feels infinitely more general and reusable:
// specific to our current blog
var validArticles = function(articles) {
return articles.filter(function(article){
return article !== null && article !== undefined;
});
};
// vastly more relevant for future projects
var compact = function(xs) {
return xs.filter(function(x) {
return x !== null && x !== undefined;
});
};By using specific naming, we've seemingly tied ourselves to specific data (in this case articles). This happens quite a bit and is a source of much reinvention.
I must mention that, just like with Object-Oriented code, you must be aware of this coming to bite you in the jugular. If an underlying function uses this and we call it first class, we are subject to this leaky abstraction's wrath.
var fs = require('fs');
// scary
fs.readFile('freaky_friday.txt', Db.save);
// less so
fs.readFile('freaky_friday.txt', Db.save.bind(Db));Having been bound to itself, the Db is free to access its prototypical garbage code. I avoid using this like a dirty nappy. There's really no need when writing functional code. However, when interfacing with other libraries, you'll have to acquiesce to the mad world around us.
Some will argue this is necessary for speed. If you are the micro-optimization sort, please close this book. If you cannot get your money back, perhaps you can exchange it for something more fiddly.
And with that, we're ready to move on.