なぜJSLintは "返品"後に "Unexpected" else 'について不平を言うのですか？

Question

JSLintは（役に立たない例）次のコードは無効であることを訴える：

(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;

それは真剣にそれがある場合/他の構造内でreturnステートメントを使用して悪いことを示唆していますか？

それは、このバージョンは大丈夫だと考えて：

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

Answer 1

それはちょうどelse後returnは不要であることを知らせるています。次は大丈夫です：

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

Answer 2

そのより良い、それは一貫性を追加すると、常に何かを返す機能を持っています。 JSLintは非常に厳しく、プログラマーの気持ちを傷つけることが知られています。それを助けてください。個人的に私はバージョン1が良いと思います

Answer 3

私がjslintで見つけたことは、あなたがルールに従えば、50％があなたのコードに悪影響を及ぼしていないことです。残りの50％（またはそれ以上）はあなたに良い利益を与えます。他の50％のためにそれをしなさい。この特定の例では、条件などの逆数を明示的に指定する必要があります。elseを暗黙的にするのではなく... if/elseにも同じことが言えます。