🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
JavaScript

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

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

Q&A

解決済

2回答

1283閲覧

onclick属性に指定したJavaScript関数が実行しない

kinsncn

総合スコア34

JavaScript

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

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

0グッド

1クリップ

投稿2021/01/09 10:28

localStorageにデータを保存と読み取る関数を作りましたが、なぜか保存ボタンは機能していますが、
読取るボタンが機能しません。 原因を教えていただけますでしょうか。?
console.logには以下のメッセージが出ます。
index.html:11 Uncaught ReferenceError: load is not defined

<!DOCTYPE html> <html lang="en"> <head> <meta charset="UTF-8"> <meta name="viewport" content="width=device-width, initial-scale=1.0"> <title>Document</title> </head> <body> <input type="text", id="dataIn" value=""> <input type="button", id="savebut" value="save" onclick="save()"> <input type="button", id="loadbut" value="load" onclick="load()"> <div id="output"></div> <script> function save () { let item = document.querySelector('#dataIn').value; localStorage.setItem('item', item); function load () { let mydate = ''; mydata = localStorage.getItem('item'); document.querySelector('output') = mydata; } } </script> </body> </html>

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

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

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

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

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

guest

回答2

0

save関数の中にload関数があるのでそのままのコードでは呼び出せません。
内部関数にした意図も含めて考えてみてください。
あとはそれぞれの機能(querySelector()など)が何を返すのか確認して、適切なプロパティを呼び出してください。
ご自身で#dataInでやってるので出来ると思います。

投稿2021/01/09 10:51

編集2021/01/09 10:55
m.ts10806

総合スコア80875

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

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

kinsncn

2021/01/09 11:14

ありがとうございました。
guest

0

ベストアンサー

document.querySelector('output').innerText = mydata;

では?

let mydate = '';

let mydata = '';のミス?

投稿2021/01/09 10:46

編集2021/01/09 10:51
arta

総合スコア20

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

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

arta

2021/01/09 10:53 編集

function save () { let item = document.querySelector('#dataIn').value; localStorage.setItem('item', item); } function load () { let mydata = ''; mydata = localStorage.getItem('item'); document.querySelector('#output').innerText = mydata; }
m.ts10806

2021/01/09 10:56

変数名はコードには影響ないですけど、よろしくはないですね。
kinsncn

2021/01/09 11:16

ありがとうございました。 解決できました、#が足りなかったですね。
kinsncn

2021/01/09 11:18

@m.ts10806 初心者なので、まだ変数名などの付け方がよくわからないままやっているところですが、 変数名に対して、具体的に提案していただけますでしょうか?
m.ts10806

2021/01/09 11:26

いえ、変数名の指摘ではないです。 mydateと変数を定義しておきながら mydataという変数で初期化してるので、コード実行に問題ないものの無駄なメモリ使ってるという話です。 型付け含めて「宣言」には気を遣うようにしてください。 「変数名」については「役割を名称だけで誰が見てもわかるように」ですね。 1週間見なかったら自分で書いたコードも忘れますよ。
kinsncn

2021/01/09 11:38

ご指摘通りですね。確かに無駄な一行ですね、ありがとうござました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.36%

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

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

質問する

関連した質問