convertir à partir d'if else pour changer l'instruction

Question

J'ai le suivant if, else if, else construire et je suis juste curieux de savoir comment je pourrais convertir tel que construct en une instruction switch.

var emailSubject = email.subject.toLowerCase(); 
if(emailSubject.indexOf("account request") >= 0){ 
    //do acct req 
}else if(emailSubject.indexOf("accounts pending removal for") >= 0){ 
    //do account removal 
}else if(emailSubject.indexOf("listserv application") >= 0){ 
    //do listserv app 
}else if(emailSubject.indexOf("student organization webmaster transfer request") >= 0){ 
    //do webmaster xfer 
}else{ 
    //do default 

} 

Mes pensées sont mais je ne pense pas que ce soit correct:

switch(emailSubject){ 
    case this.indexOf("account request"): 
     //do acct request 
     break; 
    default: 
     //do default 
} 

Ou

switch(0){ 
    case emailSubject.indexOf("accounts pending removal"): 
    //process account pending removal 
    break; 
    default: 
    //do default behavior 
} 

Answer 1

Votre code exemple ne peut pas facilement être converti en une instruction switch dans la plupart des langues, ni devrait-il. switch est pour comparer une seule variable contre une gamme de valeurs constantes, tandis que votre logique nécessite une comparaison avec des valeurs non constantes, sans aucune variable pour les comparer. if/else if est la construction correcte pour votre cas.

Answer 2

Vous ne pouvez utiliser le cas pour vérifier une valeur:

switch(emailSubject){ 
    case "Subject1": //(emailSubject == "Subject1") 
     //do acct request 
     break; 
    case "Subject2": //(emailSubject == "Subject2") 
     //do something else 
     break; 
    default: 
     //do default 
} 

Sinon, vous devriez utiliser if/else

Answer 3

Comme mentionné précédemment, if/else est le mieux pour ce que vous avez.

Cependant, si vous recherchez des lignes entières de sujets réels, au lieu des mots dans les lignes de sujet, vous pouvez faire quelque chose comme:

var a = ["account request", "listserv application", "student organization webmaster transfer request"]; 
switch(a.indexOf(emailSubject)) { 
    // ... 
} 

Answer 4

Constructs comme celui-ci sont généralement criant pour le polymorphisme ...

Jouez avec ici: http://jsbin.com/utilu4/3

var mailHandlers = [ 

    { 
    CanHandleEmail : function(email) { 
     return email.subject.toLowerCase().indexOf("account request") >= 0; 
    }, 

    HandleEmail : function(email) { 
     alert("do acct req"); 
    } 
    }, 

    { 
    CanHandleEmail : function(email) { 
     return email.subject.toLowerCase().indexOf("account pending removal for") >= 0; 
    }, 

    HandleEmail : function(email) { 
     alert("do account removal"); 
    } 
    }, 

    { 
    CanHandleEmail : function(email) { 
     return email.subject.toLowerCase().indexOf("listserv application") >= 0; 
    }, 

    HandleEmail : function(email) { 
     alert("do listserv app"); 
    } 
    }, 

    { 
    CanHandleEmail : function(email) { 
     return email.subject.toLowerCase().indexOf("student organization webmaster transfer request") >= 0; 
    }, 

    HandleEmail : function(email) { 
     alert("do webmaster xfer"); 
    } 
    }, 

    { 
    CanHandleEmail : function(email) { 
     return true; 
    }, 

    HandleEmail : function(email) { 
     alert("do default"); 
    } 
    } 
]; 

function HandleEmail(email) { 
    for(i=0; i< mailHandlers.length; i++) { 
    if(mailHandlers[i].CanHandleEmail(email)){ 
     mailHandlers[i].HandleEmail(email); 
     break; 
    } 
    } 
}; 

Answer 5

Tout comme un conseil: envelopper votre code dans une fonction et retourner la valeur du match. (Vous n'avez pas besoin d'utiliser autre chose dans ce cas.) Si vous le souhaitez, vous pouvez renvoyer un code pour la correspondance (par exemple un int) et utiliser switch/case pour effectuer l'action.

Answer 6

Je viens de rencontrer cela dans la nature et je ne peux pas m'empêcher de le partager, mais ne le fais pas.

var emailSubject = email.subject.toLowerCase(); 
switch (true) { 
    case (emailSubject.indexOf("account request") >= 0): 
     //do acct req 
     break; 
    case (emailSubject.indexOf("accounts pending removal for") >= 0): 
     //do account removal 
     break; 
    case (emailSubject.indexOf("listserv application") >= 0): 
     //do listserv app 
     break; 
    case (emailSubject.indexOf("student organization webmaster transfer request") >= 0): 
     //do webmaster xfer 
     break; 
    default: 
     //do default 
     break; 
}