Является ли какой-либо лучший синтаксис для выполнения этого кода?

Question

Ниже мой код

if(!chineseName) { 
     alert("chineseName is not correct"); 
    } 
    else if(!IDN){ 
     alert("IDN is not correct"); 
    } 
    else if(!mobileNumber){ 
     alert("number is not correct"); 
    } 
    else if(hasAccount){ 
     if(!isAccountNumberValid) { 
      alert("account number is not correct"); 
     } 
     else { 
      check(); 
     } 
    } 
    else { 
     check(); 
    } 



    function check() { 
     if(!check1 && !check2) { 
      alert("Please read NOTE and check the read box."); 
     } 
     else { 
      alert("Everything is good to go!!"); 
     } 
    } 

Теперь этот код работает правильно. Однако я хотел бы знать, есть ли какой-нибудь лучший синтаксис для выполнения того же самого?

Спасибо.

Answer 1

Вы можете комбинировать последние два, потому что одно и то же действие, просто изменяя последнее условие if.

if (!chineseName) { 
    alert("chineseName is not correct"); 
} else if (!IDN){ 
    alert("IDN is not correct"); 
} else if (!mobileNumber){ 
    alert("number is not correct"); 
} else if (hasAccount && !isAccountNumberValid) { // Modified condition 
    alert("account number is not correct"); 
} else { 
    check(); 
} 

Answer 2

Вы, вероятно, может консолидировать все оповещения в одном:

var msg = ""; 

if (!chineseName) { 
    msg = "chineseName is not correct"; 
} else if (!IDN) { 
    msg = "IDN is not correct"; 
} else if (!mobileNumber) { 
    msg = "number is not correct"; 
} else if (hasAccount) { 
    if (!isAccountNumberValid) { 
     msg = "account number is not correct"; 
    } else { 
     msg = check(); 
    } 
} else { 
    msg = check(); 
} 

alert(msg); 


function check() { 
    if (!check1 && !check2) 
     return "Please read NOTE and check the read box."; 

    return "Everything is good to go!!"; 
}