The more I delve into javascript, the more I think about the consequences of certain design decisions and encouraged practices. In this case, I am observing anonymous functions, a feature which is not only JavaScript-provided, but I see strongly used.
I think we can all agree on the following facts:
- the human mind does not deal with more than 7 plus minus two entities (Miller’s law)
- deep indentation is considered bad programming practice, and generally points out at design issues if you indent more than three or four levels. This extends to nested entities, and it’s well presented in the python Zen entry “Flat is better than nested.”
- the idea of having a function name is both for reference, and for easy documentation of the task it performs. We know, or can expect, what a function called removeListEntry() does. Self-documenting, clear code is important for debugging and readability.
While anonymous functions appears to be a very nice feature, its use leads to deeply nested code design. The code is quick to write, but difficult to read. Instead of being forced to invent a named context for a functionality, and flatten your hierarchy of callable objects, it encourages a “go deep one level”, pushing your brain stack and quickly overflowing the 7 +/- 2 rule. A similar concept is expressed in Alan Cooper’s “About Face“, quoting loosely “people don’t understand hierarchies”. As programmers we do understand hierarchies, but our biology still limits our grasping of deep nesting.
I’d like to hear you on this point. Should anonymous functions be considered harmful, an apparent shiny syntactic sugar which we find later on to be salt, or even rat poison ?
CW as there’s no correct answer.
As I see it, the problem you’re facing is not anonymous functions, rather an unwillingness to factor out functionality into useful and reusable units. Which is interesting, because it’s easier to reuse functionality in languages with first-class functions (and, necessarily, anonymous functions) than in languages without.
If you see a lot of deeply nested anonymous functions in your code, I would suggest that there may be a lot of common functionality that can be factored out into named higher-order functions (i.e. functions that take or return (“build”) other functions). Even “simple” transformations of existing functions should be given names if they are used often. This is just the DRY principle.