No, no, no, no. Did I mention no? I inherited a Javascript application that was maybe 1500 lines of code. Every function was 3-5 lines. Most were like this:

No, no, no, no. Did I mention no? I inherited a Javascript application that was maybe 1500 lines of code. Every function was 3-5 lines. Most were like this: process_foos = function(my_list) { for (i=0; i<my_list.lengh; i++) { process_foo(my_list[i]); } } process_foo = function(foo) { foo.some_method(); } And foo.some_method (in another file) was similar. What it meant was it was impossible to get a view of what was going on in the system. It took months to accomplish an understanding that should have taken weeks. I really think the original authors heard your advice and decided comprehension didn't matter, only function length. Function should accomplish a task. Sometimes (very rarely) a task may be two hundred lines; usually, it is much shorter. Sometimes three lines, but often 15-20.