入力フォームのチェック機能を作りました。変な所の指摘をお願いします。

前提・実現したいこと

javasctiptでユーザー登録機能を作りました。
ユーザーの入力内容をチェックして、正しく入力されていないと送信を取り消す処理を書きました。
初学者のためこれで良いのかわかりません！評価して欲しいです。
おかしな所や、改善した方が良い書き方、もっとスマートな書き方、等ありましたら教えてくださるとありがたいです。
どんな細かい事であっても指摘して頂けたら嬉しいです。

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

<!DOCTYPE html>
<html lang="ja">
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Document</title>
</head>
<body>
        <article>
            <p id="add-user">ユーザー登録</p>
            <form method="get" id="user-text" onsubmit="return Check();">
                <input type="text" id="name" placeholder="ユーザー名">
                <p id="name_p"></p>
                <input type="text" id="mail" placeholder="メールアドレス">
                <p id="mail_p"></p>
                <input type="text" id="pass" placeholder="パスワード">
                <p id="pass_p"></p>
                <input type="text" id="pass2" placeholder="パスワード(確認)">
                <p id="pass2_p"></p>
                <input id="button" type="submit" value="ユーザー登録">
            </form>
    </article>
    <script>
        let nameFlag = false;
        let mailFlag = false;
        let passFlag = false;
        let pass2Flag = false;
     function nameCheck(){
         let name = document.getElementById('name').value;
         let nameP = document.getElementById('name_p');
         let regName = /^[^a-zA-Z0-9]/g;
         let nameResult = name.match(regName);
         if(name === ''){
             nameP.innerHTML='名前を入力して下さい';
         }else if(name <= 8){
             nameP.innerHTML = '8文字以内で入力して下さい';
         }else if(nameResult === null){
             nameP.innerHTML = '英数字以外で入力して下さい';
         }else{
             nameP.innerHTML = '入力完了';
             nameFlag = true;
         }
     }
     function mailCheck(){
         let mail = document.getElementById('mail').value;
         let mailP = document.getElementById('mail_p');
         let regMail = /^[A-Za-z0-9_.-]*@{1}[A-Za-z0-9_.-]{1,}.[A-Za-z0-9]{1,}$/g;
         let mailResult = mail.match(regMail);
         if(mail === ''){
             mailP.innerHTML='メールアドレスを入力して下さい';
         }else if(mailResult === null){
             mailP.innerHTML = '不正なメールアドレスです';
         }else{
             mailP.innerHTML = '入力完了';
             mailFlag = true;
         }
     }
     function passCheck(){
        let pass = document.getElementById('pass').value;
         let passP = document.getElementById('pass_p');
         let regPass = /^[a-zA-Z0-9]{10,18}$/g;
         let passResult = pass.match(regPass);
         if(pass === ''){
             passP.innerHTML = 'パスワードを入力して下さい';
         }else if(passResult === null){
             passP.innerHTML = '半角英数字10-18文字で入力して下さい';
         }else{
             passP.innerHTML = '入力完了';
             passFlag = true;
         }
     }
     function pass2Check(){
        let pass = document.getElementById('pass').value;
        let pass2 = document.getElementById('pass2').value;
         let pass2P = document.getElementById('pass2_p');
         if(pass2 !== pass){
            pass2P.innerHTML='パスワードをもう一度入力して下さい';
         }else{
             pass2P.innerHTML = '入力完了';
             pass2Flag = true;
         }
     }
     let button = document.getElementById('button');
        button.addEventListener('click',nameCheck,false);
        button.addEventListener('click',mailCheck,false);
        button.addEventListener('click',passCheck,false);
        button.addEventListener('click',pass2Check,false);
    function Check(){
        if(nameFlag===true && mailFlag===true && passFlag===true && pass2Flag===true){
            return true;
        }else{
            return false;
        }
    }
    </script>
    </body>
    </html>

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

m.ts10806

2021/02/08 21:23 編集

要件通り動いてるならそれでいいのでは。「とりあえず全部見て」ではやろうと思えばどこまででも出来てしまうし好みや現場のルールもある。面倒でだれも手を出さないと思います。観点を明示してください。

行動規範の内容に同意します

回答2件

ベストアンサー

「おかしなところがあるので別の質問を」と別の回答で書いたのは私ですが、前の質問文中にあった明らかな間違いは既に直っているようです。

■ 動作の不具合

あるフィールドが、一度「入力完了」になってからユーザが再編集してチェックが通らない状態になったとき、「入力完了」状態から抜けることがありません。nameFlagなど何とかFlag変数がfalseに戻ることがないからです。各チェック関数で「入力完了」ではないときに 何とかFlag = false; する必要があります。

各<input>要素にname属性がないと、フォームで値が送信されません。

メールアドレスの正規表現が厳しすぎて、規格に準拠したメールアドレスも拒絶してしまいます。たとえば、foo+bar@example.com は規格準拠です。

■ コーディングスタイルなど

インデントをきちんと揃えましょう。読みにくいです。

if の後にはスペースを1つ入れましょう。{ の前にスペースを1つ入れましょう。行末ではない } の後にスペースを1つ入れましょう。行末ではない , の後にスペースを1つ入れましょう。= と == の前後にスペースを入れましょう。

イベントハンドラ属性 onsubmit と addEventLisetner() が混在していて統一感がありません。現在は、どちらかというと後者が推奨されています。

何とかCheck() 関数の結果は Check() でしか使わないのですから、何とかCheck() 関数はCheck()の中で呼べばよいです。何とかCheck()関数は return true か return false をするように変更すると、Check() は以下のようになります。ボタンのclickイベントを使う必要はありません。

javascript
1function Check() {
2    const nameFlag = nameCheck();
3    const mailFlag = mailCheck();
4    const passFlag = passCheck();
5    const pass2Flag = pass2Check();
6    return nameFlag && mailFlag && passFlag && pass2Flag;
7}