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

Question

#やりたいこと
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を使わずに、応用前の状態にするとエラーが出ずに正しく動作します。
宜しくお願い致します。

Answer 1

こんにちは

以下の関数

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');

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

参考になれば幸いです。

Comments

[tenten11055]

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

[jun68ykt]

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