エレガントな検索の場合

Question

もっとエレガントな方法がありますか？

if (err.code === 'CONFLICT-GROUP-GENERAL' || 
    err.code === 'CONFLICT-USER-GENERAL' || 
    err.code === 'CONFLICT-FORM-GENERAL' || 
    err.code === 'CONFLICT-PROJECT-GENERAL' || 
    err.code === 'CONFLICT-TEMPLATE-GENERAL') {} 

Answer 1

このトリックは、私にはよりエレガントなようです（アレイとindexOfを使用して）：

var conflicts = ['CONFLICT-GROUP-GENERAL', 
        'CONFLICT-USER-GENERAL', 
        'CONFLICT-FORM-GENERAL', 
        'CONFLICT-PROJECT-GENERAL', 
        'CONFLICT-TEMPLATE-GENERAL']; 

if (conflicts.indexOf(err.code) !== -1) { 
    doSomething(); 
} 

あなたはES7を使用している場合、あなたはincludes()代わりのindexOfを使用することができます。これは、より多くの「表現力」になります。

var conflicts = ['CONFLICT-GROUP-GENERAL', 
        'CONFLICT-USER-GENERAL', 
        'CONFLICT-FORM-GENERAL', 
        'CONFLICT-PROJECT-GENERAL', 
        'CONFLICT-TEMPLATE-GENERAL']; 

if (conflicts.inclues(err.code)) { 
    doSomething(); 
} 

注includes()ことは、すべてのブラウザでsuportedされることはありません。

EDIT：

別の方法：switchを使用します。この方法：err.code各caseで指定した文字列の1に等しい場合

switch (err.code) { 
    case 'CONFLICT-GROUP-GENERAL',: 
    case 'CONFLICT-USER-GENERAL',: 
    case 'CONFLICT-FORM-GENERAL',: 
    case 'CONFLICT-PROJECT-GENERAL',: 
    case 'CONFLICT-TEMPLATE-GENERAL': 
     doSomething(); 
     break; 
} 

上記のコードはdoSomething()機能を実行します。

Answer 2

私はすべてのコードで配列を持つほうが良いと、それは-1よりも大きいのですかどうかを確認するためにindexOfを使用します。

if (['CONFLICT-GROUP-GENERAL', 'CONFLICT-USER-GENERAL', 'CONFLICT-FORM-GENERAL', 'CONFLICT-PROJECT-GENERAL', 'CONFLICT-TEMPLATE-GENERAL'].indexOf(err.code) > -1) { 
}