[Westa]Login,Mainページコメント

7930 ワード

3つの条件演算子(重複する関数を削除)

の3つの条件演算子は、true/falseに従ったifelse文の簡単な表現であるため、ifelse文に3つの条件演算子を追加する必要はありません.
(btnOnという関数を使用してtrue/falseベースの戻り値を指定する必要はありません)

[修正前]

function allClear() {
    if (id.value !== "" && password.value !== "" && password.value.length >= 5) { 
        return true;
    } else {
        return false;
    }
}
function btnOn(allClear) { 
    const check = allClear(); 
    const yes = () => Btn.style.opacity = "1"; 
    const no = () => Btn.style.opacity = "0.3";
    return (check ? yes() : no()) 
}
id.addEventListener("keyup", ()=>{ 
    btnOn(allClear); 
});
password.addEventListener("keyup", ()=>{
    btnOn(allClear);
});

[修正後]

function validate() {
    Btn.style.opacity =
    id.value !== "" && password.value !== "" && password.value.length >= 5 ?
    "1" : "0.3"
}
id.addEventListener("keyup", ()=>{ 
    validate();
});
password.addEventListener("keyup", ()=>{
    validate();
});