teratail header banner
teratail header banner
質問するログイン新規登録
HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

解決済

1回答

707閲覧

JS内でJSを読み込むとdocument.getElementById()でエラーが出ます

bicLighter

総合スコア21

HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

0クリップ

投稿2022/01/31 06:20

0

0

前提・実現したいこと

現在、自分が作成しているサイトにカレンダーを埋め込もうと思い、他サイトからJS,css,htmをコピーして埋め込んでいます。
前提として、index.js内のmakeCalendar関数でカレンダーのhtmlを作成して、そこにcalendar.jsを読み込んでいる状況です。

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

calendar.js:32 Uncaught TypeError: Cannot set properties of null (setting 'innerHTML') at calendar.js:32:52

calendar.js内のdocument.getElementById("year").innerHTML = createYearにエラーが出て、カレンダーが表示されません。

該当のソースコード

index.js

1function makeCalendar(){ 2 const res = document.createElement('div') 3 res.innerHTML = 4 ` 5 <div class="container-calendar"> 6 <h4 id="monthAndYear"></h4> 7 <div class="button-container-calendar"> 8 <button id="previous" onclick="previous()">‹</button> 9 <button id="next" onclick="next()">›</button> 10 </div> 11 12 <table class="table-calendar" id="calendar" data-lang="ja"> 13 <thead id="thead-month"></thead> 14 <tbody id="calendar-body"></tbody> 15 </table> 16 17 <div class="footer-container-calendar"> 18 <label for="month">日付指定:</label> 19 <select id="month" onchange="jump()"> 20 <option value=0>1月</option> 21 <option value=1>2月</option> 22 <option value=2>3月</option> 23 <option value=3>4月</option> 24 <option value=4>5月</option> 25 <option value=5>6月</option> 26 <option value=6>7月</option> 27 <option value=7>8月</option> 28 <option value=8>9月</option> 29 <option value=9>10月</option> 30 <option value=10>11月</option> 31 <option value=11>12月</option> 32 </select> 33 <select id="year" onchange="jump()"></select> 34 </div> 35 <script src="js/calendar.js" type="text/javascript"></script> 36 ` 37 return res 38}

calendar.js

1 2function generate_year_range(start, end) { 3 var years = ""; 4 for (var year = start; year <= end; year++) { 5 years += "<option value='" + year + "'>" + year + "</option>"; 6 } 7 return years; 8 } 9 10 var today = new Date(); 11 var currentMonth = today.getMonth(); 12 var currentYear = today.getFullYear(); 13 var selectYear = document.getElementById("year"); 14 var selectMonth = document.getElementById("month"); 15 16 var createYear = generate_year_range(1970, 2200); 17 //↓ここでエラーが出てしまいます。 18 document.getElementById("year").innerHTML = createYear;

試したこと

不可解なのが、エラーが出るコードの上の var selectYear = document.getElementById("year");はエラーが出ずに読み込めている点です。
また、calendar.jsはmakeCalendar()の最後に読み込んでいるため、jsが読み込まれているときにはすでに"year"は存在していると考えています。

皆さんのお力を貸してください。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

index.js内のmakeCalendar関数でカレンダーのhtmlを作成して、そこにcalendar.jsを読み込んでいる状況です。

innerHTML への代入で作られた<script>は、実行されません。makeCalendar() が呼ばれるより前にどこかで calendar.js をロードしているはずです。

calendar.js の内容を関数に入れて、maekCalendar() を呼んだ後にその関数を呼ぶのはいかがでしょうか。

投稿2022/01/31 06:34

int32_t

総合スコア21933

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

bicLighter

2022/01/31 07:50

回答ありがとうございます。勉強になりました。 回答を参考にmakeCalendar関数の最後にscriptを指定することで解決できました。 ``` const el = document.createElement('script') el.src = 'js/calendar.js' res.append(el) ```
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.30%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問