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

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

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

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

Q&A

解決済

1回答

560閲覧

コンストラクタに設定した要素が取得できない

Lave

総合スコア18

JavaScript

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

0グッド

0クリップ

投稿2021/11/08 02:15

仕事でとあるサイトに無限スクロールを実装しようとしているのですが、コードの整備性を改善するためクラス構文を使用したところプログラムが動かなくなりました。

javascript

1class InfScr { 2 constructor(url) { 3 this.entryMore = document.getElementsByClassName('entry-more'); 4 this.entryLoading = document.getElementsByClassName('entry-loading'); 5 this.entryItems = document.getElementsByClassName('entry-items'); 6 this.documentHeight = document.documentElement.clientHeight; 7 this.windowsHeight = window.innerHeight; 8 this.url = url; 9 this.postNumNow = 9; 10 this.postNumAdd = 45; 11 this.flag = false; 12 this.postData = { 13 post_num_now: this.postNumNow, 14 post_num_add: this.postNumAdd 15 } 16 } 17 18 load() { 19 for (let item of this.entryMore) { 20 item.addEventListener("click", this.process); 21 item.addEventListener("touch", this.process); 22 } 23 } 24 25 unload() { 26 for (let item of this.entryMore) { 27 item.removeEventListener("click", this.process); 28 item.removeEventListener("touch", this.process); 29 } 30 } 31 32 process() { 33 for (let item of this.entryMore) { 34 item.classList.add('is-hide'); 35 } 36 for (let item of this.entryLoading) { 37 item.classList.add('is-show'); 38 } 39 this.flag = true; 40 fetch(this.url, { 41 method: 'POST', 42 headers: { 43 'Accept': 'application/json', 44 'Content-Type': 'application/json; charset=utf-8' 45 }, 46 body: JSON.stringify(this.postData) 47 }).then( 48 Response => Response.json() 49 ).then(data => { 50 for (let item of this.entryItems) { 51 item.innerHTML = item.innerHTML + data[0]; 52 } 53 for (let item of this.entryLoading) { 54 item.classList.remove('is-show'); 55 } 56 if (data[1] > 0) { 57 for (let item of this.entryMore) { 58 item.classList.remove('is-hide'); 59 } 60 } 61 this.documentHeight = document.documentElement.clientHeight; 62 this.postNumNow += this.postNumAdd; 63 this.flag = false; 64 }).catch(error => { 65 console.error(error); 66 }); 67 } 68}

発生している問題

  • 一部のメソッド内でthis.hogeと指定した要素がundefinedになる

この一点です。
load()内にあるthis.entryMoreを使用したfor文は問題なく実行されますが、
process()内のthis.~は全てundefinedになり、一番上のfor文から反復不能とエラーが返ってきます。

試したこと

process()に引数をつける

load内for文の変数itemをprocess()に継承できるよう引数を付けましたが、process()自体は問題なく実行されるものの読み込み時に発火してしまいます。イベントリスナーの記述がある通り特定の要素をクリックした時に実行されるようにしたいです。

よろしくお願いします。

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

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

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

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

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

guest

回答1

0

ベストアンサー

js

1 item.addEventListener("click", this.process); 2 item.addEventListener("touch", this.process);

addEventListener()で登録されたコールバック内では、this はイベントターゲット item になります。

item.addEventListener("click", this.process.bind(this)) のように this を固定すれば期待通りの動作になると思います。

投稿2021/11/08 02:20

int32_t

総合スコア21012

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

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

Lave

2021/11/08 02:25

ありがとうございます。すんなりと解決しました。 インスタンス化されたオブジェクトのthisにはならないのですね。勉強になります。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.46%

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

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

質問する

関連した質問