convert from if else to switch statement

Question

У меня есть следующее if, else if, else construct и мне просто интересно, как я мог бы преобразовать такую ​​конструкцию в оператор 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 

} 

Мои мысли, но я не думаю, что это правильно:

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

Или

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

Answer 1

Ваш пример кода не может быть легко преобразован в распределительном заявление в большинстве языков, ни если это так. switch предназначен для сравнения одной переменной с диапазоном постоянных значений, тогда как ваша логика требует сравнения с непостоянными значениями, без переменной, с которой они сравниваются. if/else if - это правильная конструкция для вашего корпуса.

Answer 2

Вы можете использовать только случай проверить значение:

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

В противном случае вы должны использовать, если/иначе

Answer 3

Как упоминалось выше, если/еще лучше для того, что у вас есть.

Если, однако, вы искали фактические целые сюжетные линии, вместо слов в пределах сюжетных линий, вы могли бы сделать что-то вроде:

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

Answer 4

конструктов, как это обычно взывает полиморфизм ...

Играть с ним здесь: 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

Подобно тому, как совет: обернуть свой код в функции и возвращает значение матча. (В этом случае вам не нужно использовать другое.) Если вы захотите, вы можете вернуть код для соответствия (например, int) и использовать переключатель/case для выполнения действия.

Answer 6

Я только что встретил это в дикой природе, и я не могу не поделиться им, но не делайте этого.

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