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

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

新規登録して質問してみよう
ただいま回答率
85.50%
JavaScript

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

Q&A

解決済

2回答

2381閲覧

HTMLで指定した変数をJs関数の引数に読み込ませたいです

OOO_777

総合スコア50

JavaScript

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

0グッド

0クリップ

投稿2017/08/09 11:11

###Js関数の引数がうまく読み込めない
htmlで指定した変数をjsの関数の引数に渡したいのですが、

Uncaught ReferenceError: TimeCookie is not defined

定義されていないとエラーが出てしまいます。
console.logで確認したらきちんと50と表示されます…
どこが問題が教えて頂きたいです。

###該当のソースコード

html

1<script type="text/javascript" src="./test.js"></script> 2<script type="text/javascript"> 3 var TimeCookie = 50; 4</script>

###該当のソースコード

javascript

1//test.js 2function CookieSegment(str,time) { 3 function SetCookie(str) { 4 var nowDate = new Date(); 5 var clearTime = new Date(nowDate.getTime() + time*60*1000); 6 } 7}; 8CookieSegment('文字列',TimeCookie);

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

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

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

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

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

guest

回答2

0

ベストアンサー

html

1<script type="text/javascript"> 2 var TimeCookie = 50; 3</script>

このスクリプトを

html

1<script type="text/javascript" src="./test.js"></script>

より上で定義するとエラーは表示されなくなります。

投稿2017/08/09 11:16

LanHma

総合スコア192

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

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

OOO_777

2017/08/09 14:09

ご回答ありがとうございます。 そうでしたか… ありがとうございます。
guest

0

別解

質問に対する直接的な解は LanHma さんの回答通りですが、別解もあります。

  • test.js 内では関数呼び出しせず、インラインスクリプト内で CookieSegment('文字列', 50) で関数呼び出しする

この方法はグローバル変数の数が減るというメリットがあります。

test.js

JavaScript

1//test.js 2function CookieSegment(str,time) { 3 function SetCookie(str) { 4 var nowDate = new Date(); 5 var clearTime = new Date(nowDate.getTime() + time*60*1000); 6 } 7}; 8CookieSegment('文字列',TimeCookie);

このコードではSetCookie() が呼び出されていないので、CookieSegment() で呼び出しても何も実行されないと思います。
また、コンストラクタとしての機能がないのであれば、関数名は小文字から始まる慣習があり、一般には小文字から始まる名前が推奨されます。

Re: OOO_777 さん

投稿2017/08/09 12:47

think49

総合スコア18156

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

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

OOO_777

2017/08/09 14:25

ご回答ありがとうございます。 ・インラインスクリプト内、 ・コンストラクタとしての機能がないのであれば、関数名は小文字。  等のアドバイスありがとうございます。勉強になります。 質問があります。もし上記のコードで関数を実行したい場合は、 SetCookie('文字列'); CookieSegment('文字列',TimeCookie); 上記のような形で問題ないでしょうか? 単純な発想ですが…
think49

2017/08/09 16:48 編集

CookieSegment の中に SetCookie 一つしかないのなら、SetCookie の存在意義はないので SetCookie を削除して良いと思います。 SetCookie も変数宣言だけで何も実行しているように読めないので、関数単位でコードをテストして設計を見直してみてはいかがでしょうか。
OOO_777

2017/08/09 22:47

ご回答ありがとうございます。 設計見直してみます。 ありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問