¿Por qué JSLint se queja de "Inesperado" else después de "devolver"?

Question

JSLint queja de que el (ejemplo inútil) código siguiente es válido:

(function (x) { 
    "use strict"; 
    if (x === 1) { 
     return 1; 
    } else if (x === 2) { 
     return -1; 
    } 
    return 0; 
}(1)); 

Error: Problem at line 4 character 9: Unexpected 'else' after 'return'.

return 1;

¿Es en serio lo que sugiere que es malo para utilizar sentencias de retorno dentro de una estructura if/else?

Se cree que esta versión está bien:

(function (x) { 
    "use strict"; 
    var returnval = 0; 
    if (x === 1) { 
     returnval = 1; 
    } else if (x === 2) { 
     returnval = -1; 
    } 
    return returnval; 
}(1)); 

Answer 1

Es apenas le dice que else después return es superfluo. El siguiente está bien:

(function (x) { 
    "use strict"; 
    if (x === 1) { 
     return 1; 
    } 
    if (x === 2) { 
     return -1; 
    } 
    return 0; 
}(1)); 

Answer 2

Es mejor tener una función siempre devuelve algo, ya que agrega consistencia. Se sabe que JSLint es bastante estricto y duele los sentimientos de los programadores. No puedo ayudarlo. Personalmente, creo que la versión 1 está bien

Answer 3

Lo que he encontrado con jslint es que si sigues las reglas, el 50% es ridículo pero no tiene un impacto negativo en tu código. El otro 50% (más o menos) le dará un buen beneficio. Entonces hazlo por el otro 50%. Este ejemplo particular te obliga a ser explícito sobre el inverso de una condición o similar ... en lugar de dejarla implícita con un else ... lo mismo se aplica a if/else.