javascriptでプルダウンの表示設定がうまくいきません

Question

前提・実現したいこと

プルダウンメニューの初期表示を特定の時間にしたいのですがうまくいきません
アドバイスお願いします。

発生している問題・エラーメッセージ

0時から24時のプルダウンメニューで初期表示を9時に設定できない

該当のソースコード

/*
    今日の日付データを変数todayに格納
   */
  var optionLoop, this_day, this_month, this_year, today;
  today = new Date();
  this_year = today.getFullYear();
  this_month = today.getMonth() + 1;
  this_day = today.getDate();

 optionLoop = function (start, end, id, this_day) {
      var i, opt;

      opt = null;
      for (i = start; i <= end; i++) {
          if (i === this_day) {
              opt += "<option value='" + i + "' selected>" + i + "</option>";
          } else {
              opt += "<option value='" + i + "'>" + i + "</option>";
          }
      }
      return document.getElementById(id).innerHTML = opt;
      
  };

  /*
      ループ処理（表示id名、デフォルト数字）
  */
  timeLoop = function (id, this_time) {
      var hour, opt;
      opt = null;
      var minute = [':00', ':30'];
      var count = 0;
      for (var hour = 0; hour <= 24;) {
          if (i === this_time) {
              opt += "<option value='" + hour + "' selected>" + hour + "</option>";
          } else {
              opt += "<option value='" + hour + "'>" + hour + "</option>";
          }
          text.push(tagF + hour + minute[count] + tagE);
          count++;
          if (count == 2) {
              count = 0;
              hour++;
          }
      }
  }

<!-- 日付表示 -->
    <script type="text/javascript">
        /*
      関数設定（スタート数字、終了数字、表示id名、デフォルト数字）
     */
        $(function() {
            optionLoop(this_year - 5, this_year + 5, 'id_year', this_year);
            optionLoop(1, 12, 'id_month', this_month);
            optionLoop(1, 31, 'id_day', this_day);
        })
        
        $(function() {
            optionLoop(this_year - 5, this_year + 5, 'bar_year', this_year);
            optionLoop(1, 12, 'bar_month', this_month);
            optionLoop(1, 31, 'bar_day', this_day);
        })
    </script>
</head>


<tr>
<td>開始時間</td>
                        <td>
                           <div id="id_start">
                            <select name="time" class="sel_time">
                                <script type="text/javascript">
                                    var text = times();
                                    for (var i = 0; i < text.length; i++) {
                                        document.write(text[i]);
                                    }
                                </script>
                            </select>
                            </div>
                        </td>

                    </tr>
                    <tr>
                        <td>終了時間</td>
                        <td>
                            <select name="time" class="sel_time">
                                <script type="text/javascript">
                                    var text = times();
                                    for (var i = 0; i < text.length; i++) {
                                        document.write(text[i]);
                                    }
                                </script>
                            </select>
                        </td>
                    </tr>

Answer 1

ソース足りないので挙動確認できない＆想像になりますが、とりあえず
timeloop関数の

if (i === this_day) {

は

if (i === this_time) {

かな(厳密等価で一致するかは引数がちゃんと数値かどうかで決まるので気をつけて)

Comments

[monolith123]

回答ありがとうございます。
すいません、ソースを追加しました。

[rururu3]

あれ・・・・ベストアンサーついたけど、原因これだったのかな・・・

Answer 2

rururu3さんにご教授頂いた内容を追加し、記述を忘れていたreturn文を書き込みました。
その結果無事表示することができました。

timeLoop = function (id, this_time) {
var hour, opt;
opt = null;
var minute = [':00', ':30'];
var count = 0;
for (var hour = 0; hour <= 24;) {
if (hour === this_time) {
opt += "<option value='" + hour + minute[count] + "' selected>" + hour + minute[count] + "</option>";
} else {
opt += "<option value='" + hour + minute[count] + "'>" + hour + minute[count] + "</option>";
}
count++;
if (count == 1) {
count = 0;
hour++;
}
}
return document.getElementById(id).innerHTML = opt;
}