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

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

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

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

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

解決済

jQueryの関数に引数を渡すとプロパティエラーになる

yunosuke
yunosuke

総合スコア18

JavaScript

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

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

2回答

0グッド

1クリップ

207閲覧

投稿2022/11/17 09:29

前提

コードのリファクターの際におきたエラーです。
初歩的なエラーかもしれませんが、回答いただけると嬉しいです。
以下のjQueryのコードですが、関数に引数を渡そうとするとエラーが出てしまいます。

エラーメッセージ

Uncaught TypeError: Cannot read properties of undefined (reading '0')

該当のソースコード

html

1<label>ファイルを選択 2<input type="file" name="file"  class="upload-1"> 3</label> 4<p class="js-upload-filename-1">ファイルを選択されていません</p> 5

JavaScript

1 2$('.upload-1').on('change',file1('.js-upload-filename-1'))//引数を渡したい 3 4 function file1(target) { 5 var file = $(this).prop('files')[0];//エラーがでてしまう 6 $(target).text(file.name); 7};

エラーがでない例

JavaScript

1 2$('.upload-1').on('change', function() { 3 var file = $(this).prop('files')[0]; 4 $('.js-upload-filename-1').text(file.name); 5}); 6

上記のエラーがでないコードをリファクターして使いまわすには関数化したいのですが、うまくいかず。。。
スコープが原因かなと思ったのですが、その辺の知識が乏しいので回答いただけると幸いです・・・!

以下のような質問にはグッドを送りましょう

  • 質問内容が明確
  • 自分も答えを知りたい
  • 質問者以外のユーザにも役立つ

グッドが多くついた質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

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

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

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

下記のような質問は推奨されていません。

  • 間違っている
  • 質問になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

適切な質問に修正を依頼しましょう。

回答2

0

ベストアンサー

bind() してやればよいですね。

js

1const upload1 = $('.upload-1'); 2upload1.on('change', file1.bind(upload1[0], '.js-upload-filename-1'));

もし <input type="file">js-upload-filename-1 のような要素の組が複数必要なら、イベントハンドラに引数を与えられるようにするよりは、属性や要素の構造で紐付けを行ったほうがよいかもしれません。

html

1<input type="file" name="file" class="upload" data-target=".js-upload-filename-1"> 2<p class="js-upload-filename-1">ファイルを選択されていません</p> 3<input type="file" name="file" class="upload" data-target=".js-upload-filename-2"> 4<p class="js-upload-filename-2">ファイルを選択されていません</p>

js

1$('.upload').on('change', function() { 2 $(this.dataset.target).text(this.files[0].name); 3});

投稿2022/11/17 11:09

編集2022/11/17 22:01
int32_t

総合スコア17097

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

下記のような回答は推奨されていません。

  • 間違っている回答
  • 質問の回答になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

このような回答には修正を依頼しましょう。

回答へのコメント

maisumakun

2022/11/17 11:14

.upload-1が複数個あった場合には想定通り動かなさそうです。
int32_t

2022/11/17 21:48

> .upload-1が複数個あった場合には想定通り動かなさそうです。 ごもっともですね。
yunosuke

2022/11/28 10:02

なんとか解決できましたありがとうございます!

0

file1('.js-upload-filename-1')のように、普通に関数を呼ぶと、thisは何もセットされません。

というより、$('.upload-1').on('change',file1('.js-upload-filename-1'))のように書くと、file1('.js-upload-filename-1')実行した結果をイベントハンドラとしてセットする、ということになってしまい、「file1という関数をハンドラとしてセットする」という意味に鳴りません。

「ハンドラにするための関数を返す関数を作る」などの、ちょっと複雑な構造を取る必要があります。

投稿2022/11/17 09:54

maisumakun

総合スコア141365

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

下記のような回答は推奨されていません。

  • 間違っている回答
  • 質問の回答になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

このような回答には修正を依頼しましょう。

回答へのコメント

yunosuke

2022/11/17 10:33

回答ありがとうございます。 やっぱりthisが原因だったんですね・・・ 「ハンドラにするための関数を返す関数を作る」 というのはコールバック関数のようなものでしょうか!?

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

ただいまの回答率
86.12%

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

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

質問する

関連した質問

同じタグがついた質問を見る

JavaScript

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

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。