復帰のベストプラクティス後のJavascript警告到達不能コード

Question

私は主題に関する他の質問を見て、特に私の主題については何も見ていませんでした。

このタイプのコードを使用するルールはありますか。私はこの場合のように感じます。ifはすべてreturn文を持っているので、elseを使うのは無駄です。

var __sortBySortType = function(x, y, index, type) { 
    if (type === "name") { 
     x[0]["sortname"] = x[0]["sortname"] || x[0]["name"]; 
     y[0]["sortname"] = y[0]["sortname"] || y[0]["name"]; 
     type = "sortname"; 
    } 
    if (typeof global !== "undefined" && typeof global.locale !== "undefined" && global.locale.compareString !== "undefined" && index === 0) { 
     return sim.locale.compareString(x[index][type], y[index][type]); 
    } 
    if (x[index][type] > y[index][type]) { 
     return 1; 
    } 
    if (x[index][type] < y[index][type]) { 
     return -1; 
    } 
    //If sorting by grades return sorting by whatever is diplayed (name, login or id) 
    return (index === 0)? 0 : __sortBySortType(x, y, 0, this.displayType); 
} 

おかげで

Answer 1

あなたは、各if文のリターンを持って考えると、違いがあってはなりません。将来の保守性のためにif-elseif-elseを実行することがベストプラクティスです。一見して読むのが一層容易であることがわかります。それはすべてが1つの選択肢のブロックとして関連していることを知っています。