Ce 'commutateur' devrait être un 'si' - pourquoi?

Question

Je suis en cours d'exécution par mon code JSHint, et je suis frappé cette erreur:

This switch should be an if

Sur ce bloc de code:

switch(true) 
{ 
    case Major === 0 && Minor === 0 && Patch < 433: 
     upgraded = upgraded.replace(/(\s+)skill(\d)=/gm, '$1section_8_$2_body='); 
    /* falls through */ 
    case Major === 0 && Minor === 0 && Patch < 442: 
     upgraded = upgraded.replace(/test=/gm, 'void='); 
    /* falls through */ 
    case Major === 0 && Minor === 0 && Patch < 459: 
     upgraded = upgraded.replace(/another=/gm, 'void='); 
    /* falls through */ 
} 

De googler j'ai trouvé this Github issue, mais il ressemble c'était parce qu'il n'y avait que 1 case.

Comment puis-je résoudre ce problème? Je ne vois pas pourquoi ce switch devrait être un if. Est-ce que le fait que j'utilise le switch(true) a quelque chose à voir avec ça? A côté: le code se termine bien sur la version en ligne (j'utilise le plugin Notepad ++).

Answer 1

Une déclaration switch est destiné à tester une seule variable ou une expression à une liste connue finie de valeurs possibles (une énumération )

var userColor = 'red'; 

switch(userColor){ 
    case 'red': 
     alert('Stop'); 
     break; 
    case 'yellow': 
     alert('Slow'); 
     break; 
    case 'green': 
     alert('Go'); 
     break; 
} 

Ce code est essentiellement un raccourci pour:

if(userColor == 'red'){ 
    alert('Stop'); 
}else if(userColor == 'yellow'){ 
    alert('Slow'); 
}else if(userColor == 'green'){ 
    alert('Go') 
} 

Dans le code que vous avez fourni, le seul facteur déterminant est la valeur de Patch puisque le reste des variables sont toujours 0.

Je recommande refactorisation votre code à une série de if/else

if(Patch < 433){ 
    //... 
}else if(Patch < 442){ 
    //... 
}else if(Patch < 459){ 
    //... 
}else{ 
    //... fall through 
} 

Answer 2

je ferais quelque chose comme:

if(Major === 0 && Minor === 0){ 
    if(Patch < 433) { 
     upgraded = upgraded.replace(/(\s+)skill(\d)=/gm, '$1section_8_$2_body='); 
    } 
    if(Patch < 442) { 
     upgraded = upgraded.replace(/test=/gm, 'void='); 
    } 
    if(Patch < 459) { 
     upgraded = upgraded.replace(/another=/gm, 'void='); 
    } 
} 

Il maintient les états sèche et est un peu plus facile à lire imo . Un commutateur serait utile si vous faisiez de l'égalité directe sur une valeur mais le < enlève tous les "avantages" d'une instruction switch.