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

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

ただいまの
回答率

89.99%

JavaScript アラート表示の動作エラー

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,477

a-_.

score 131

・ラジオボタン「投稿」がチェックされている状態の場合はチェックボックス非表示    
・ラジオボタン「更新」「削除」がチェックされた場合のみチェックボックス表示    
・チェックボックス未チェックのままsubmitボタンを押下した場合はエラー(javascript:alert)を表示し、投稿を中止    
という条件をJavaScriptのみで処理したいんですが、ブラウザ表示してラジオボタンのps・up・deを選択し、psとupだったら条件以外の場合ERROR1~4を表示、upとdeを選択中chkid[](チェックボックス)をチェックしなかったらERROR5を表示という動作を追加したいんです
今のコードだと一応動作は出来るけど、チェックボックスにチェックをいれてもERROR5が表示されないため今ここで何が処理されてるのか確認したいんです
あと解読しきれていないconsole.logのコードがあるんですが、私の解釈としては
console.log(cde[bde]);
console.log(bde);
cde[bde].disabled = false;
return false;
・cde(=obj)はbdeの数だけ出力
・bdeをループ出力
・cde[bde]が選択できなかったらfalse
・submit処理を止めるためfalseとして返す
という解釈をしてるんですが、その考え方は合ってるんでしょうか

<html>
    <head>
        <meta charset="utf-8">
        <script>
            function condition() {
                var ps = document.getElementById("r1");
                var up = document.getElementById("r2");
                var de = document.getElementById("r3");
                if (ps.checked == true || up.checked == true) {
                    if (document.getElementById('name').value.length >= 10) {
                        window.alert("ERROR1");
                        return false;
                    } else if (document.getElementById('name').value.length == "") {
                        window.alert("ERROR2");
                        return false;
                    } else if (document.getElementById('comment').value.length >= 300) {
                        window.alert("ERROR3");
                        return false;
                    } else if (document.getElementById('comment').value.length == "") {
                        window.alert("ERROR4");
                        return false;
                    }
                }
                if (up.checked == true || de.checked == true) {
                    cde = document.getElementsByName('chkid[]');
                    for(var bde = 0; bde < cde.length; bde++) {
                        console.log(cde[bde]);
                        console.log(bde);
                        cde[bde].disabled = false;
                        return false;
                    }
                    if (bde == 0) {
                        window.alert ("ERROR5");
                    }
                }
            }
            function change(obj) {
                console.log(obj.id);
                if (obj.id == "r1") {
                    document.getElementById("send").value = "投稿";
                    chkchange(true);
                } else if (obj.id == "r2") {
                    document.getElementById("send").value = "更新";
                    chkchange(false);
                } else if (obj.id == "r3") {
                    document.getElementById("send").value = "削除";
                    chkchange(false);
                }
            }
            function chkchange(bool) {
                chk = document.getElementsByName('chkid[]');
                for(var box = 0; box < chk.length; box++) {
                    console.log(chk[box]);
                    console.log(box);
                    chk[box].disabled = bool;
                }
            }
        </script>
    </head>
    <body>
        <form method="post" action="" onsubmit="return condition()">
            <table>
                <tr>
                    <td>
                        名前:<input type="text" name="name" id="name">
                        内容:<textarea name="comment" cols="30" rows="3" id="comment"></textarea>
                        <input type="radio" name="bbs" id="r1" value="post" onChange="change(this)">投稿
                        <input type="radio" name="bbs" id="r2" value="update" onChange="change(this)">更新
                        <input type="radio" name="bbs" id="r3" value="delete" onChange="change(this)">削除
                        <input type="submit" value="投稿" id="send">
                    </td>
                </tr>
            </table>

<?php
            while($row = mysqli_fetch_array($result, MYSQLI_ASSOC)) {
?>

            <table>
                <input type="checkbox" name="chkid[]" value="<?=$row['id'] ?>">
                <tr>
                    <td>名前:
                        <?php echo $row['name'] ?>
                    </td>
                </tr>
                <tr>
                    <td>内容:
                        <?php echo $row['comment'] ?>
                    </td>
                </tr>
            </table>

<?php
            }
            mysqli_free_result($result);

            mysqli_close($link);
?>

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

bdeはループの変数なので、0にならないですよね。
なので
if (bde == 0) 
の部分はかならずfalseになりますね。だからError5は表示されません。

ループ文の中でチェックボックスのチェックOn/Offをif文でしらべて、
全部OffならError5を表示、
もしくは、1個でもOnならError5を表示せずにスルー。
といった処理を書けばOKだとおもいます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/09/01 11:11

    ラジオボタンのtrueじゃなくて、チェックボックスのtrueをみるんじゃないんですか?
    チェックボックスがoffならエラーというif文を書けばOKです。
    forループの中で、チェックボックスがoffかどうか処理を入れて、
    全部offならError5を表示とかけばOKです。

    キャンセル

  • 2016/09/01 15:30

    教えていただいたサイトと他に参考にしていたhttp://www.pori2.net/js/form/2.htmlのコードを元に再訂正してみました
    気になったんですが、2つ目のサイトのコードに書かれているvalueは例文に動物名が各valueが定義され、そのvalueを変数に代入してアラート表示となっていますが、strの数によってアラート表示となるのであれば、私の場合だとそのvalueの箇所を書き換えてchkid[]にすることで、if文で数がない場合アラート表示になるのではと考えたんですが、この解釈だとどうでしょうか
    if (up.checked == true || de.checked == true) {
    cde = document.getElementsByName('chkid[]');
    var str = "";
    for(var bde = 0; bde < cde.length; bde++) {
    if (document.FM.elements[chkid[]][bde].checked) {
    str = str + document.FM.elements[chkid[]][bde].value;
    }
    }
    if (str == "") {
    window.alert("ERROR5");
    return false;
    }
    }

    キャンセル

  • 2016/09/01 16:11

    document.FM.elementsは参考HPのコードなので違いますよ。
    そのままもってくればいいってもんじゃないです。
    っていうか、cdeをループしているのになぜそうなるw

    全部offならError5としたいなら、
    逆に1つでもONならreturnすればいいとおもいます。
    for文の中に、cdeのcheckedがtrueならreturnして、
    forのあとにwindow.alert("ERROR5")を書けばいいかと。

    っていうか、本当にJavaScript基礎から学んだほうがいいと思います。

    キャンセル

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

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

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