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

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

ただいまの
回答率

88.93%

関数のイベント(e)がundefindになる

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 715

前提・実現したいこと

function 関数名(e)の’(e)’の部分を取得できるようにしたいです。

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

Uncaught TypeError: Cannot read property 'target' of undefined

該当のソースコード

var file = document.getElementsByClassName('.sell-upload-drop-file');
var canvas = document.getElementById('c1');
var canvasWidth = 400;
var canvasHeight = 300;
var uploadImgSrc;

// Canvasの準備
canvas.width = canvasWidth;
canvas.height = canvasHeight;
var ctx = canvas.getContext('2d');

function loadLocalImage(e) {
  // ファイル情報を取得
  var fileData = e.target.files[0];
  // FileReaderオブジェクトを使ってファイル読み込み
  var reader = new FileReader();
  // ファイル読み込みに成功したときの処理
  reader.onload = function() {
      // Canvas上に表示する
      uploadImgSrc = reader.result;
      canvasDraw();
  }
  // ファイル読み込みを実行
  reader.readAsDataURL(fileData);
}

// ファイルが指定された時にloadLocalImage()を実行
file.addEventListener('change', loadLocalImage(), false);

// Canvas上に画像を表示する
function canvasDraw(imgSrc) {
  // canvas内の要素をクリアする
  ctx.clearRect(0, 0, canvasWidth, canvasHeight);

  // Canvas上に画像を表示
  var img = new Image();
  img.src = uploadImgSrc;
  img.onload = function() {
      ctx.drawImage(img, 0, 0, canvasWidth, this.height * (canvasWidth / this.width));

    // Canvas上にテキストを表示
    addText();

    // canvasを画像に変換
    var data = canvas.toDataURL();

    // 画像として出力
    var outputImg = document.createElement('img');
    outputImg.src = data;
    document.getElementById('result').appendChild(outputImg);
    }
  }
});

試したこと

console.log(e);で確認したところundefindが返ってきている

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

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

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

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

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

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+5

// ファイルが指定された時にloadLocalImage()を実行
file.addEventListener('change', loadLocalImage(), false);

ここでloadLocalImage()と書くと、この関数をその場で実行して結果だけaddEventListenerにわたす、という意味になってしまいます。

カッコ無しでloadLocalImageと、関数を引数にする形としてください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

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

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

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/01/07 13:30

    自分で書いておいて申し訳ないのですが、"document.getElementsByClassName('.sell-upload-drop-file');"で書いてfileを要素のリストにすることによるメリットはあるのでしょうか?

    キャンセル

  • 2019/01/07 13:45

    同じクラスを持つ要素が複数個あった場合に、一括して取得できます。

    キャンセル

  • 2019/01/07 13:54

    なるほど。すごく勉強になりました!何度もご回答いただきありがとうございました!
    querySelectorにするとエラーが解決しました!感謝します。

    キャンセル

+3

file.addEventListener('change', loadLocalImage(), false);
の、loadLocalImage()()がいりません。

また、var file = document.getElementsByClassName('.sell-upload-drop-file');
で指定しているクラス名の、.も不要です。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

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

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

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/01/07 13:39

    ご教授ありがとうございます!助かりました。
    もっと勉強します

    キャンセル

+2

var file = document.getElementsByClassName('.sell-upload-drop-file');

classNameからひっぱるなら先頭の「.」はいらないです
(よほど変なclass名をつけていなければ)

var file = document.getElementsByClassName('sell-upload-drop-file');
もしくは
var file = document.querySelectorAll('.sell-upload-drop-file');


ただしHTMLCollectionにもNodeListにもaddEventListerは実行できないので

  • forで回して処理するか
  • 0番目(もしくは任意)の要素だけ抽出して処理するか
  • var file = document.querySelector('.sell-upload-drop-file')で1つだけ取り出すか

のいずれかで対応してください

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

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

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

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/01/07 13:41

    具体的なご教授ありがとうございます!教えていただいた内容を理解できていないので一旦勉強してから書いてみたいと思います!感謝します!

    キャンセル

  • 2019/01/07 13:44

    できれば該当するHTMLも表記してもらえるとより精度が高い回答ができると思います

    キャンセル

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

  • ただいまの回答率 88.93%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

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