Это говнокод или фича?

Question

[vasIvas]

switch (len) {
case 1: return listeners.fn.call(listeners.context), true;
case 2: return listeners.fn.call(listeners.context, a1), true;
case 3: return listeners.fn.call(listeners.context, a1, a2), true;
case 4: return listeners.fn.call(listeners.context, a1, a2, a3), true;
case 5: return listeners.fn.call(listeners.context, a1, a2, a3, a4), true;
case 6: return listeners.fn.call(listeners.context, a1, a2, a3, a4, a5), true;
}

Это говнокод или опять фича такая?

Answer 1

[Сергей Сергей]

это фичи, переваренные в говнокод.

Answer 2

[M K]

Ну, можно написать рациональнее и универсальнее:

var a=[null,a1,a2,a3,a4,a5];
var args=[];
for(var i=1;i<len;i++) args.append(a[i]);
listeners.fn.apply(listeners.context, args);

P. S. =)

Comments

[Сергей Сергей]

var a = [ a1, a2, a3, a4, a5 ];
listeners.fn.apply( listeners.context, a.slice(0,len) );

[M K]

Сергей Сергей: Ах да, я совсем забыл про slice, отлично!

[vasIvas]

Сергей Сергей: Array.prototype.slice.call(arguments, 1)

Answer 3

[Dmitriy Mozgovoy]

Вероятно, это оптимизация производительности критического участка кода, довольно часто используется разработчиками соответствующего уровня при написании библиотек и фреймворков. В некоторых случаях switch производительнее условных конструкций, тем более любых других реализаций, с более читабельным либо "рациональным" видом.

Answer 4

[WebDiez]

это говнокод))