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

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

ただいまの
回答率

89.10%

JavaScript エラーの原因が分からない

解決済

回答 2

投稿 編集

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

a-_.

score 133

下記コードのelse if (up.checked == true || de.checked == true) の箇所に何やらエラーがある様なんですが、for文、if文、アラートをそれぞれ確認しても何がエラーなのか原因は把握できません

for文でboxにchkid[]を代入し、チェックボックスにチェックが入った場合はtrue、未チェックの場合は falseとしてアラート表示となるよう書いてみたんですが、こちらも動作確認できませんでした
if文の条件分岐に書き換え、更新と削除のラジオボタン、アラート表示、アラート表示の条件の処理を変更することで、アラート表示及びその条件の処理の元正しく動作されると推測しましたが、どの点に誤りがあるのでしょうか
因みにelse ifのコードを全て消して再度ブラウザ表示してみると指示通りの動作になるのを確認しました

<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 == "") {
                        window.alert("ERROR2");
                        return false;
                    } else if (document.getElementById('comment').value.length >= 300) {
                        window.alert("ERROR3");
                        return false;
                    } else if (document.getElementById('comment').value == "") {
                        window.alert("ERROR4");
                        return false;
                    }
                } else if (up.checked == true || de.checked == true) {
                    ca = document.getElementsByName('chkid[]');
                    var box = "";
                    for(var cb = 0; cb < ca.length; cb++) {
                        if (document.chkid[].elements[cb].checked) {
                            box = box + document.chkid[].elements[cb].value;
                        }
                        return true;
                    }
                    if (box == "") {
                        window.alert("ERROR5");
                        return false;
                    }
                }
            }
            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=""  name="fn" 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ページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • date

    2016/09/02 11:10

    質問者さんはよくある程度質問したらまとめてベストアンサーを選んでいますが、質問が解決するたびに行ってください

    キャンセル

回答 2

+2

document.chkid[].elements[cb].checked

document.chkid[]という参照の仕方はありません。
ご自身でも書いている通り
chk = document.getElementsByName('chkid[]');
のようにして一度変数に受けた上で処理したほうがよいでしょう

追記

とりあえず以下参考にしてください
ここまで書いて気がつきましたが、複数選択して削除はよいとして
複数選択して更新は仕様としてどうなんでしょう?

<script src="https://code.jquery.com/jquery-1.12.4.min.js"></script>
<script>
$(function(){
  $('form[name=fn]').submit(function(e){
    if(!condition()){
      e.preventDefault();
    }
  });
  $('input[name=bbs]').click(function(e){
    change($(this).prop('id'));
  });
  change($('input[name=bbs]:checked').prop('id'));
});
function condition() {
  if($('#r1').prop('checked')||$('#r2').prop('checked')){
    with($('#name')){
      if(val().length >= 10) {
        alert("ERROR1");
        return false;
      }
      if (val() === "") {
        alert("ERROR2");
        return false;
      }
    }
    with($('#comment')){
      if(val().length >= 300) {
        alert("ERROR3");
        return false;
      }
      if (val() === "") {
        alert("ERROR4");
        return false;
      }
    }
  }
  if($('#r2').prop('checked')||$('#r3').prop('checked')){
    if($('[name="chkid[]"]:checked').length==0){
      window.alert("ERROR5");
      return false;
    }
  }    
  return true;
}
function change(myid) {
  var sendlist={
    "r1":{"text":"投稿",checkflg:true},
    "r2":{"text":"更新",checkflg:false},
    "r3":{"text":"削除",checkflg:false},
    };
  $('#send').val(sendlist[myid].text);
  $('[name="chkid[]"]').prop('disabled',sendlist[myid].checkflg);
}
</script>
<form method="post" name="fn">
名前:<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" checked>投稿
<input type="radio" name="bbs" id="r2" value="update">更新
<input type="radio" name="bbs" id="r3" value="delete">削除
<input type="submit" value="投稿" id="send">
<table>
<tr>
<td><input type="checkbox" name="chkid[]" value="1"></td>
<td>名前:name1</td>
<td>内容:content1</td>
</tr>
<tr>
<td><input type="checkbox" name="chkid[]" value="2"></td>
<td>名前:name2</td>
<td>内容:content2</td>
</tr>
<tr>
<td><input type="checkbox" name="chkid[]" value="3"></td>
<td>名前:name3</td>
<td>内容:content3</td>
</tr>
</table>
</form>

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/09/02 11:54

    いや、複数の書き込みを更新するという自体があまりないということです。
    更新というのはたいてい1件1葉でデータをあたえるものなので
    ラジオボタンで処理します。
    ただ質問者さんが複数のデータをすべて同じ値に書き換えたいという
    仕様を決めたのであればそれで構いません。

    キャンセル

  • 2016/09/02 12:00

    jQueryについては好き嫌いがあるので必ずしもおすすめしませんが
    毎回トラブっている再帰処理あたりが驚くほど楽になりますので
    いずれ検討されるとよいでしょう

    キャンセル

  • 2016/09/02 13:07 編集

    複数のデータをすべて同じ値に書き換える仕様にしているので、その点に関しては違和感を感じてらっしゃるのかもしれませんが、そのままにしておきます

    すいません
    コメントの一部に誤りがありました
    訂正いたしましたので、再度コメントにお答えいただければ幸いです

    キャンセル

checkベストアンサー

+1

前の質問が未解決のまま、同じ質問を出さないでください。
https://teratail.com/questions/46227

if (document.chkid[].elements[cb].checked) {
   box = box + document.chkid[].elements[cb].value;
}


↑の部分、
ca でループをまわしてるので、ca で取得しましょう。
っていうか、すぐ下では

console.log(chk[box]);

ってやっていますね。それと同じです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/09/02 11:20 編集

    ご回答ありがとうございます

    if文のchkid[]だった箇所をcaに訂正したらfunction chkchange(bool)のchk[box].disabled = bool;が動作するようになりました
    ですが、アラート表示の処理にまだ不備があるようです
    box = box + document.ca.elements[cb].value;のvalue値を変数boxに入れる
    為に書いてるのに、そもそもこのvalueがどこのvalue値を指してるのかが定義されてない為にエラーが発生してると考えたんですが、考え方は合ってるんでしょうか

    キャンセル

  • 2016/09/02 11:33

    document.chkid[].elements[cb]がどこからでてきたんですか…。
    ループしているのはcaですから、ca をみればOKです。

    キャンセル

  • 2016/09/02 13:09

    すいません
    コメントの一部に誤りがありました
    訂正いたしましたので、再度お答えいただければ幸いです

    キャンセル

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

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

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