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

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

ただいまの
回答率

88.20%

functionを活用した時に、「Uncaught ReferenceError: value is not defined」と表示される

解決済

回答 1

投稿

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

tenten11055

score 66

やりたいこと

JavaScriptを勉強中です。
本の内容を応用して、同じ内容の処理をfunctionで定義して呼び出したいのですが、
ブラウザのコンソールに(index):117 Uncaught ReferenceError: idName is not defined
出てしまいます。
原因が分からないため、教えていただければ幸いです。
テーマは非同期通信です。

ソース

[
  {"id":"js", "crowded":"yes"},
  {"id":"security", "crowded":"no"},
  {"id":"aiux", "crowded":"middle"}
]
        <section>
          <ul class = 'list'>
            <li class = 'seminar' id = 'js'>
              <h2>JavaScript勉強会</h2>
              <p class = 'check'>空き席状況を確認</p>
            </li>
            <li class = 'seminar' id = 'security'>
              <h2>セキュリティ対策講座</h2>
              <p class = 'check'>空き席状況の確認</p>
            </li>
            <li class = 'seminar' id = 'aiux'>
              <h2>AIを利用したUX設計</h2>
              <p class = 'check'>空き席状況を確認</p>
            </li>
          </ul>
        </section>
//装飾用ーーーーーーーーーーーーーーーーーーーーーー
      .list{
        overflow: hidden;
        margin: 0;
        padding: 0;
        list-style-type: none;
      }
      .list h2{
        margin: 0 0 2em 0;
        font-size: 16px;
        text-align: center;
      }
      .seminar{
        float: left;
        margin: 10px 10px 10px 0;
        border: 1px solid #23628f;
        padding: 4px;
        width: 25%;
      }
      .check{
        margin: 0;
        padding: 8px;
        font-size: 12px;
        color: #ffffff;
        background-color: #23628f;
        text-align: center;
        cursor: pointer;
      }

//下記をプログラムで使用するーーーーーーーーーーーーーーーーーーーーーーーーーーー
      .red{
        background-color: #e33a6d;
      }
      .green{
        background-color: #7bc52e;
      }
      .yellow{
        background-color: yellow;
      }
      'use strict';

      $(document).ready(function(){
        //ファイルの読み込み
        $.ajax({url: 'data.json', dataType: 'json'})
        .done(function(data){
          data.forEach(function(item, index){
            if(item.crowded === 'yes'){
              // const idName = '#' + item.id; <- 応用前
              createIdName(item); //<- 応用後
              $(idName).find('.check').addClass('crowded');
            }else if(item.crowded === 'middle'){
              // const idName = '#' + item.id; <- 応用前
              createIdName(item);//<- 応用後
              $(idName).find('.check').addClass('middle');
            }
          });
        })
        .fail(function(){
          window.alert('読み込みエラー');
        });
      });

      //クリックされたら空き席状況を確認
      $('.check').on('click', function(){
        if($(this).hasClass('crowded')){
          $(this).text('残席わずか').addClass('red');
        }else if($(this).hasClass('middle')){
          $(this).text('微妙!').addClass('yellow');
        }else{
          $(this).text('お席あります').addClass('green');
        }
      });

      //応用のため関数を定義
      function createIdName(item){
        const idName = '#' + item.id;
      };

定義した関数、createIdNameを使わずに、応用前の状態にするとエラーが出ずに正しく動作します。
宜しくお願い致します。

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+2

こんにちは

以下の関数

function createIdName(item){
  const idName = '#' + item.id;
};


の本体で宣言されている変数 idName は、関数の外からは参照できないです。なので、 

$(idName).find('.check').addClass('crowded');


のところで、ご質問にある、idName is not defined というエラーになります。
修正の一案としては、関数createIdName で、

function createIdName(item){
  const idName = '#' + item.id;
  return idName; // この行を追加
};


上記のように、 idName を return するようにしておいて、createIdName を使っている下記

createIdName(item);
$(idName).find('.check').addClass('crowded');


を、

$(createIdName(item)).find('.check').addClass('crowded');


のように修正するといかがでしょうか?

参考になれば幸いです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/11/04 00:25

    ご指摘の通り試したところ、思うような結果になりました。
    解説がわかりやすく大変助かりました!
    ありがとうございました。

    キャンセル

  • 2019/11/04 00:26

    どういたしまして。
    > 思うような結果になりました。
    とのことでよかったです👏

    キャンセル

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

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

関連した質問

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