質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.06%

バリデーションチェックに関して

解決済

回答 2

投稿

  • 評価
  • クリップ 1
  • VIEW 601
退会済みユーザー

退会済みユーザー

下記コードにおきまして、年齢の入力ボックスに数字を入力しているにも関わらず、エラーメッセージ「年齢は数字のみ入力してください」が表示されてしまいます。
バリデーションチェックに問題があるということなのでしょうか?こちらの現象を解決したい為、アドバイス頂けましたら幸いです。

<!DOCTYPE html>
<html lang="ja">
<head>
  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <title>Document</title>
  <style>
    .err_msg{
      color:red;
      margin-bottom:0;
    }
    #name_box{
      margin:0;
    }
    .err_box{
      margin-top:0;
    }

  </style>
</head>
<body>
  <h1>お問合せ画面</h1>
  <h2>お問合せ内容を入力してください</h2>
  <form method="post" action="" name="input_form" class="form" onsubmit="return false">

    <p id="name_box">(必須)名前<input type="text"  name="name"></p>
    <p id="age_box">(必須)年齢<input type="text" size="2" name="age"></p>
    <p id="color_list">好きな色
      <label><input type="checkbox" name="color" value="red"></label>
      <label><input type="checkbox" name="color" value="green"></label>
      <label><input type="checkbox" name="color" value="blue"></label>
      <label><input type="checkbox" name="color" value="other">それ以外</label>
      <label><input type="checkbox" name="color" value="nothing">特になし</label>
    </p>
    <input type="submit" id="btn" value="送信">
  </form>
  <script>


    document.addEventListener('DOMContentLoaded',function(){
      document.forms.input_form.addEventListener('submit',function(e){
        // if(user_name)

      let valid = false;
      let form = document.getElementsByClassName('form');
      let name_box = document.getElementById('name_box');
      let name_err_msg = document.createElement('p');
      let age_err_msg = document.createElement('p');
      let err_msg = document.getElementsByClassName('err_msg');
      let age_box = document.getElementById('age_box');
      let color_list = document.getElementById('color_list');
      let is_err_msg = document.getElementsByClassName('name_check_err');
      let node_err_msg = is_err_msg.item(0);
      let age_err_box = document.getElementsByClassName('age_err_box');
      let node_age_err = age_err_box.item(0);
      console.log(is_err_msg);

      name_err_msg.classList.add('err_msg');
      let str_num = '';

      let user_name = this.name.value;
      let user_age = this.age.value;
        console.log(user_name);

      if(user_name == ''){

        valid = false;

        if (is_err_msg.length === 0) {
          name_err_msg.classList.add(('err_box','name_check_err'));
          name_err_msg.textContent  =  '名前が入力されていません';
          name_box.parentNode.insertBefore(name_err_msg, name_box.parentNode.firstChild);

        }

      }else if(node_err_msg){
        node_err_msg.parentNode.removeChild(node_err_msg);

      }

      if (str_num.match(/[^0-9]/g)
         // "", " ", " 1", "1a" をはじく。
         // 単体では,"01" を通してしまう
        ||

        parseInt(str_num, 10) + "" != str_num
         // "01", "1 " など0付き・スペース付き文字列をはじく。
         // 単体では,スペースのみの文字列は通してしまう
      )
      {

        valid = false;


        if(age_box.classList.contains('err_box') == false){
            console.log('ng');
          age_box.classList.add(('err_box','age_err_box'));
          name_box.insertAdjacentHTML('afterend','<p class="err_msg">年齢は数字のみ入力してください</p>');

        }





      }else{
        if(node_age_err){
          node_age_err.parentNode.removeChild(node_age_err);
        }

      }

      let checked_flag = false;

      for(let i = 0; i < document.input_form.color.length; i++){
        if(document.input_form.color[i].checked){
          checked_flag = true;
        }
      }

      if(!checked_flag){
        valid = false;

        if(color_list.classList.contains('err_box') == false){
          color_list.classList.add(('err_box'));
          age_box.insertAdjacentHTML('afterend','<p class="err_msg">チェックされていません</p>');

        }
      }

      if(!valid){
        e.preventDefault();
      }

      });
    });


  </script>
</body>
</html>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

+2

color_list のクラスを操作しているのがそのif文の中だけだから、必ずfalseになる。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/29 06:21 編集

    前の質問でuser_age使ってないって指摘してたんですが、そこも関係ありますね。
    他のアドバイスも聞きたいことに直接関係ない内容は一切スルーされてるので時間だけ過ぎそう。
    (この質問出るって分かってた)

    キャンセル

  • 2019/08/29 08:28

    ご回答ありがとうございます。
    「そのif文の中だけ」とあるのですが、この「if文」はどこのif文のことを指しているのでしょうか?

    キャンセル

  • 2019/08/29 08:38 編集

    > 「そのif文の中だけ」とあるのですが、この「if文」はどこのif文のことを指しているのでしょうか?

    "color_list" で検索したら、候補は一つだけでしたが、どうやって探していますか?

    キャンセル

  • 2019/08/29 08:46 編集

    >"color_list" で検索したら、候補は一つだけでしたが、どうやって探していますか?
    color_listの記載のあるifとは限らないのではないかと思っていました。
    color_listが含まれているif文ですと「if(color_list.classList.contains('err_box') == false){」ここの部分のことでしょうか?
    だとしても疑問な点があるのですが、「必ずfalseになる」とはどういう意味でしょうか?
    デフォルトではcolor_listにerr_boxクラスは設定されていないため、一度目の条件分岐ではerr_boxは「false」にはならないのではないでしょうか?

    キャンセル

checkベストアンサー

+1

前の質問の回答の後半に書いてます。

引用:
メッセージずっと出っ放し+追記され続けるという現象はありますし、
user_ageを何も使ってないとか

年齢をチェックしているであろう箇所は下記

      if (str_num.match(/[^0-9]/g)
         // "", " ", " 1", "1a" をはじく。
         // 単体では,"01" を通してしまう
        ||

        parseInt(str_num, 10) + "" != str_num


str_numの定義場所は下記

let str_num = '';

コメントの通り、''弾きますね。

「なぜlet user_age = this.age.value;を利用しない?」
という指摘を既にしていたのですが、スルーされてたのでずっと残るわけです。

で、NGとなりますし、初回はage_boxにerr_boxなんてクラスはないので必ずif(age_box.classList.contains('err_box') == false){は成立します。

他についても同様のことが言えるのでは。

そこだけ直してももう1つ書いた「メッセージずっと出っ放し+追記され続けるという現象」は起きてるので、これについては3回目の指摘なので現象は認識されているものとこれを見ている人は思っているのでちゃんと調べて自己解決してください。
イメージ説明

insertAdjacentHTML()の仕様を重々確認しておくと良いのでは。私ならバリデーション処理の前に全エラーを削除しますけどね。
「エラーあったら残し続けておく」というのは非常にナンセンスな作りです。無駄な分岐が増えるだけの話。

それに前の質問の別の回答にあるように要素や入力内容の参照や取得の仕方もきちんと見直したほうが良いでしょう。
それで動くモノが作れたとしても評価されるものではなく、ほぼ使えないものです(古いので、業務では使わない)

作り直したコード

自分で調べてほしいのであえてコメントは付けていません。
ただし、不明点はこの場で全部解決すること
これを以て質問しても結局「そのコード、前の質問でもらったもんでしょ?その回答者に聞けば?」とほぼ間違いなく言われます。

HTMLもそれなりに調整していますのでJavaScriptだけ持っていかないようご注意を。

<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Document</title>
<style>
.err_msg {
    color: red;
    margin-bottom: 0;
}

.input_box {
    margin-top: 0;
}

</style>
</head>
<body>
    <h1>お問合せ画面</h1>
    <h2>お問合せ内容を入力してください</h2>
    <form method="post" action="" name="input_form" class="form">
        <span class="err_msg" id="name_err"></span>
        <p id="name_box" class="input_box">
            (必須)名前<input type="text" name="name" id="input_name">
        </p>
        <span class="err_msg" id="age_err"></span>
        <p id="age_box" class="input_box">
            (必須)年齢<input type="number" size="2" name="age" id="input_age"></p>
        <span class="err_msg" id="color_err"></span>
        <p id="color_list" class="input_box">
            好きな色 
            <label><input type="checkbox" name="color[]" value="red"></label>
            <label><input type="checkbox" name="color[]" value="green"></label>
            <label><input type="checkbox" name="color[]" value="blue"></label>
            <label><input type="checkbox" name="color[]" value="other">それ以外</label>
            <label><input type="checkbox" name="color[]" value="nothing">特になし</label>
        </p>
        <input type="button" id="submit_btn" value="送信">
    </form>
    <script>
    document.addEventListener('DOMContentLoaded', function() {
        document.getElementById('submit_btn').addEventListener('click', function(e) {
            let valid = true;

            let error_msg_class = document.getElementsByClassName('err_msg');
            for(er=0;er<error_msg_class.length;er++){
                error_msg_class[er].textContent = "";
            }

            let name_value = document.getElementById('input_name').value;

            if (name_value.trim() == '') {
                valid = false;
                document.getElementById('name_err').textContent = '名前が入力されていません';
            }

            let age_value = document.getElementById('input_age').value;
            if (age_value.match(/[^0-9]/g)
                ||
                parseInt(age_value, 10) + "" != age_value
            ) {
                valid = false;
                document.getElementById('age_err').textContent = '年齢は数字のみ入力してください';
            }

            let color_list = document.querySelectorAll("input[type='checkbox'][name='color[]']");

            let checked_flag = false;
            for(cl=0;cl<color_list.length;cl++){
                if (color_list[cl].checked) {
                    checked_flag = true;
                    break;
                }
            }
            if(!checked_flag){
                valid = false;
                document.getElementById('color_err').textContent = 'チェックされていません';
            }

            if (valid) {
                document.input_form.submit();
            }

        });
    });
  </script>
</body>
</html>

参考まで。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/29 20:33

    解決済みになっているので別途質問たてられた方が良いです。
    その際はこの質問の回答を参考にしたことと、あと自身がどういう考えて作ったかをきちんと記載してください(もちろん、「うまくいかない」では何も伝わらないので具体的に現象を記載してくださいね)

    キャンセル

  • 2019/08/29 20:37

    分かりました。

    キャンセル

  • 2019/08/29 20:40

    ちなみになぜそうするかというとここでクローズディスカッションすると私の考え方で一部しか伝わらないので新規質問にすることで広くアドバイスをもらえ、広い見解・知見を得られる可能性があるからです。
    (オフトピではもったいないし、私が必ず回答するとも限りませんので)

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.06%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る