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

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

ただいまの
回答率

87.96%

$.each()文からfor文への書き換えが上手くいかない

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 670

score 17

 私はJavaScriptを始めたばかりで、文法が分かっていなかったり、記法が汚いかもしれませんがご容赦ください!

個人情報入力フォーム的なものを作りながらJavaScriptの勉強をしています。
現在は、学歴の入力フォームを作っています。
仕様は、「中学」「高校」「大学」「行ってない」の4つのラジオボタンと、学校名が入ったセレクトボックスです。
セレクトボックスのデータは、ラジオボタンで学歴をクリックすると学歴ごとにJSONデータを読み込む感じです。

ラジオボタンを押してない状態だと何も出ないようにしてます。
イメージ説明

ラジオボタンで学歴を選択すると、それぞれのデータが出ます
イメージ説明
イメージ説明

 質問

https://q-az.net/without-jquery-each/
このページを参考にしながら、$.each()をfor文で書き換えたのですが上の写真で示したように動作しません。。
$.each文とfor文は以下です
コード全体は下部に記載しています。
for文で書くと下の写真のようになります。

イメージ説明

検証を使ってもエラー文は出ませんでした。

$.each()文のコード

        case 1: //高校
            $.getJSON("school.json", {name: "chara"}, function(data) {
                var list = data.highschool;
                $.each(list, function(i) {
                    document.form_signin.school.options[i] = new Option(list[i], list[i]);
                });
            });

for文に書き換えたコード

case 1: //高校
            $.getJSON("school.json", {name: "chara"}, function(data) {
                var list = data.highschool;      
                for(var i=1; i<=list.length; i++ ) {
                    document.form_signin.school.options[i] = new Option(list[i], list[i]);
                }
            });

何が間違っているかも分かりません。
どなたかアドバイスよろしくお願いします!!!

以下は、コードの全体です。

function schoolSelect(value) {
    //セレクトボックスの初期化
    arrayNum = document.form_signin.school.options.length;
    for(var i=1; i<arrayNum; i++) {
      document.form_signin.school.options[1] = null;
    }

    //選択した学校のリストをjsonファイルからAJAXで取得し、セレクトボックスに追加
    switch(parseInt(value, 10)) {
        case 0: //中学
            $.getJSON("school.json", {name: "chara"}, function(data) {  
                var list = data.juniorhighschool;
                $.each(list, function(i) { 
                    document.form_signin.school.options[i] = new Option(list[i], list[i]);
                });
            });
            break;

        case 1: //高校
            $.getJSON("school.json", {name: "chara"}, function(data) {
                var list = data.highschool;
                $.each(list, function(i) {
                    document.form_signin.school.options[i] = new Option(list[i], list[i]);
                });
            });

            // for(var i=1; i<=list.length; i++ ) {
            //     document.form_signin.school.options[i] = new Option(list[i], list[i]);
            // };

            break;

        case 2: //大学
            $.getJSON("school.json", {name: "chara"}, function(data) {
                var list = data.university;
                $.each(list, function(i) {
                    document.form_signin.school.options[i] = new Option(list[i], list[i]);
                });
            });
            break;

        default: break;
    }
}
<form class="form-signin" name="form_signin"> 
                <tr>
                    <th>最終学歴<span style="color:red;">*</span></th>
                    <td>
                        <input type="radio" name="career" id="junior" value="0" onchange="schoolSelect(this.value)" ><label for="junior">中学</label>
                        <input type="radio" name="career" id="high-school" value="1" onchange="schoolSelect(this.value)" ><label for="high-school">高校</label>
                        <input type="radio" name="career" id="university" value="2" onchange="schoolSelect(this.value)" ><label for="university">大学</label>
                        <input type="radio" name="career" id="no-career" value="3" onchange="schoolSelect(this.value)" ><label for="no-career">行ってない</label>
                        <br>
                        学校名:
                        <select id="school" class="w250" onblur="careerCheck()" >
                            <option value="0" >-選択-</option>
                        </select>
                        <br>
                        <span id="careerError">*必須項目です</span>
                    </td>
                </tr>
</form>
{"juniorhighschool":{
"1":"大阪市立桜宮中学校",
"2":"大阪市立高倉中学校",
"3":"大阪市立友渕中学校",
"4":"大阪市立都島中学校",
"5":"大阪市立淀川中学校",
"6":"大阪市立下福島中学校",
"7":"大阪市立野田中学校",
"8":"大阪市立八阪中学校",
"9":"大阪市立春日出中学校",
"10":"大阪市立此花中学校",
"11":"大阪市立梅香中学校",
"12":"大阪市立西中学校",
"13":"大阪市立花乃井中学校
},
"highschool":{
"1":"大阪市立都島工業高等学校",
"2":"大阪市立都島第二工業高等学校",
"3":"大阪市立東高等学校",
"4":"大阪市立桜宮高等学校",
"5":"大阪府立西野田工科高等学校",
"6":"大阪市立咲くやこの花高等学校",
"7":"大阪市立西高等学校",
"8":"大阪府立市岡高等学校",
"9":"大阪府立港高等学校",
"10":"大阪府立泉尾高等学校",
},
"university":{
"1":"大阪大学",
"2":"大阪教育大学",
"3":"大阪市立大学",
"4":"大阪府立大学",
"5":"藍野大学",
"6":"LCA大学院大学",
"7":"追手門学院大学",
"8":"大阪青山大学",
"9":"大阪医科大学",
"10":"大阪大谷大学"
}
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • kei344

    2018/08/10 13:39

    「うまくできない」とは「何をしたときに」「どうなると思って」「どうなったのか」を、出ているエラーなどと併せて、具体的に記述されたほうが回答を得られやすいと思います。

    キャンセル

  • m.ts10806

    2018/08/10 13:42

    「$.each()をfor文で書き換えた」ところがありますがコメントアウトされています。うまくいっていないソースで結構ですのでご提示ください。あと「school.json」の内容もサンプルデータで結構ですので提示いただくと再現確認がとれてアドバイスもしやすくなります。

    キャンセル

  • Beginnerrr

    2018/08/10 14:24

    ご指摘有難うございます!質問内容文を訂正いたしました!

    キャンセル

  • kei344

    2018/08/10 15:04

    エラーは出ていませんか?デベロッパーツールで確認してみてください。http://eng-entrance.com/javascript-display-error

    キャンセル

回答 2

checkベストアンサー

+3

data.highschoolはオブジェクトなのでlengthをもっていません、
forをつかうならinでまわせばいいでしょう

for(var i in data.highschool){
console.log(data.highschool[i]);
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/10 15:25 編集

    for in だと、学校名ではなく、1, 2, 3のようなキーが取得されませんか?

    キャンセル

  • 2018/08/10 15:27

    iが1,2,3・・・と変化して、data.highschool[i]が学校名です

    キャンセル

  • 2018/08/10 15:28

    あーほんとですね。data.highschool[i]でアクセスしてました。失礼しました。

    キャンセル

  • 2018/08/10 18:20

    ありがとうございます!!!
    オブジェクトへの理解が甘すぎました、、、

    キャンセル

+3

上手くいかないとは、どう上手くいかないのでしょうか?

パッと見て思ったのは、ループインデックスは0で初期化すべきということと
i <= list.length ではなく i < list.length ではないかと思いました。

for (var i = 0; i < list.length; i++) {
  document.form_signin.school.options[i] = new Option(list[i], list[i]);
};

 追記

highschoolは配列ではなく、オブジェクトですね。
以下のような感じでやるとどうでしょうか?

var values = Object.values(list);
for(var i = 0; i < values.length; i++) {
  document.form_signin.school.options[i] = new Option(values[i], values[i]);
}

とか

for(var value of Object.values(list)) {
  document.form_signin.school.options[i] = new Option(value, value);
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/10 15:05

    質問内容が曖昧でしたので、訂正しました!
    上手くいかないとは、データが詠み込まれないということです!

    そちらのコードで実行してもデータが詠み込まれませんでした。。。

    キャンセル

  • 2018/08/10 15:12

    追記しました。

    キャンセル

  • 2018/08/10 18:21

    ありがとうございます!!!
    お陰様で解決できました!

    キャンセル

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

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

関連した質問

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