for文で二次元配列の値の正誤について

Question

for文を使い、二次元配列の値の正誤を取得したいのですが、上手く反映されないです。初心者なので、ネットで調べながらやっているのですがわかりません。どうか教えてください。

function logincheck(){
const id = document.getElementById('id').value;
const pw = document.getElementById('pas').value;

//ユーザー仮ナンバー
const usernum=[
['ss', '2222'],
['df', '4444'],
['we', '5555'],
];

//ID PW未入力時
if(id === "" && pw === ""){
  alert("IDとパスワードは必須入力です");
  
//ID未入力時
}else if(id === ""){
  alert("IDは必須入力です");
  
//PW未入力時
}else if(pw === ""){
  alert("パスワードは必須入力です");
}else{
//ID PW一致チェック
  for(let i=0; i<usernum.length; i++){
		let element = usernum[i];
		if(element[0] === id && element[1] === password){
        window.location.href='html トップ画面.html';
    }else{
      alert("IDとパスワードが一致しません");
    
    }
  });
}
}

Comments

[int32_t]

https://teratail.com/questions/342800 この質問と何が違うのでしょうか?

[1T2R3M4]

https://ja.stackoverflow.com/questions/77406
https://qiita.com/x9jdzhcc72mxib2azgcyo/questions/e5543d63e83cf3ba05d7

以下ご対応ください。
https://teratail.com/help#posted-otherservice

Answer 1

入力されたパスワードを格納している変数名を間違えてますね。

(追記)
よく見たらそもそも文法エラーになっています。デベロッパーツールで確認すると

Uncaught SyntaxError: Unexpected token ')'

がログに出力されています。

   for(let i=0; i<usernum.length; i++){
         let element = usernum[i];
-        if(element[0] === id && element[1] === password){
+        if(element[0] === id && element[1] === pw){
         window.location.href='html トップ画面.html';
     }else{
       alert("IDとパスワードが一致しません");
     }
-  });
+  }

Comments

[tomokia]

ご指摘ありがとうございます。直しましたがまだ動きそうにないです(-_-;)頑張って突き止めます

[draq]

文法エラーが発生していたので回答に指摘を追記しました。Diffの２箇所目の変更点も確認してください。

[tomokia]

ご指摘ありがとうございます

[draq]

それ以外にも、usernum配列の要素数文だけループしてますが、毎回アラートが出るのもまずい仕様です。

Answer 2

作成されたものベースで修正した場合、こんな感じかと思います。

function logincheck(){
    const id = document.getElementById('id').value;
    const pw = document.getElementById('pas').value;

    //ユーザー仮ナンバー
    const usernum=[
      ['ss', '2222'],
      ['df', '4444'],
      ['we', '5555'],
    ];

    //ID PW未入力時
    if(id === "" && pw === ""){
        alert("IDとパスワードは必須入力です");
    //ID未入力時
    }else if(id === ""){
        alert("IDは必須入力です");

    //PW未入力時
    }else if(pw === ""){
        alert("パスワードは必須入力です");
    }else{
    //ID PW一致チェック
        for(let i=0; i<usernum.length; i++){
            let element = usernum[i];
            if(element[0] === id && element[1] === pw){//◆pwに
                window.location.href='html トップ画面.html';
            }else if(element[0] === id && element[1] !== pw){//◆ここがないと繰り返しアラートが発生します。&&以降はなくてもよいとおもいます。　
                alert("IDとパスワードが一致しません");
                break;//◆同じIDがないことを前提にループを打ち切ります
            }
        }//◆　):を取ります
    }
}

Answer 3

<script>
function logincheck(){
  const id = document.getElementById('id').value;
  const pw = document.getElementById('pas').value;
  const usernum=[
    ['ss', '2222'],
    ['df', '4444'],
    ['we', '5555'],
    ];
  let errortext="";
  if(id === "" && pw === ""){
    errortext="IDとパスワードは必須入力です";
  }else if(id === ""){
    errortext="IDは必須入力です";
  }else if(pw === ""){
    errortext="パスワードは必須入力です";
  }else if(!usernum.map(x=>x.join("\t")).includes([id,pw].join("\t"))){
    errortext="IDとパスワードが一致しません";
  }
  if(errortext){
    alert(errortext);
    return false;
  }
}
</script>
<form>
id:<input type="text" name="id" id="id"><br>
pas:<input type="text" name="pas" id="pas"><br>
<input type="submit" value="send" onclick="return logincheck()">
</form>

Answer 4

idとpwに何が入ってくるのか分からないので憶測になりますが、厳密等価演算子がどういう比較の仕方をしているのか理解するとうまくいくかもしれません。
もし違ったらごめんなさい。
https://qiita.com/PianoScoreJP/items/e43d70ec188c6fed73ed