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

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

ただいまの
回答率

90.47%

  • JavaScript

    17006questions

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

  • jQuery

    6926questions

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

【jQuery】引数にthisを渡せない

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 5,449

k499778

score 501

現在開発をしていてjqueryで
clickイベントの処理を行っていました。

そのクリックfunction内ではthisを使って、
クリックしたセレクタを指定することはできたのですが、
そのクリックfunction内にfunction(関数)を書き、その引数にthisを渡すとエラーになりました。

もちろんそのクリックfunction外に使いたい関数を定義し、その引数にもthisと記述しています。

このやり方は間違っているのでしょうか?
なぜエラーになったのか教えていただけるとありがたいです。


追記

例えばこのような場合です。

<!DOCTYPE html>
<html>

<head>
  <meta charset="UTF-8">
  <script src="http://ajax.googleapis.com/ajax/libs/jquery/1.7.2/jquery.min.js"></script>
  <style>
    .aaa {
      width: 150px;
      height: 50px;
      background-color: #EFE;
      border: solid 1px #CCC;
      position: absolute;
    }

    .ddd {
      width: 150px;
      height: 50px;
      background-color: pink;
      border: solid 1px pink;
      position: absolute;
    }
  </style>
</head>

<body>
  <label class="aaa">
    <span class="bbb"></span>
    <input type="hidden" class="ccc">りんご
  </label>
  <br>
  <br>
  <br>
  <label class="aaa">
    <span class="bbb"></span>
    <input type="hidden" class="ccc">いちご
  </label>
  <br>
  <br>
  <br>
  <label class="aaa">
    <span class="bbb"></span>
    <input type="hidden" class="ccc">ぶどう
  </label>
  <br>
  <br>
  <br>
  <label class="ddd">
    <span class="bbb"></span>
    <input type="hidden" class="ccc">りんご
  </label>
  <script>
    $(function() {
      $(".aaa").on("click", function() {
        //処理1。関数に切り出さず、直接書くとエラーにならない
        // this.ownerDocument.querySelector('.ddd').lastChild.data = this.lastChild.data;
        test(this);

        //処理2。関数に切り出さず、直接書くとエラーにならない
        // var txt = $(this).text();
        // $(".ddd").contents().filter(function(){ return this.nodeType === this.TEXT_NODE }).remove();
        // $(".ddd").append(txt);
        test2(this);
      });
    });

    //関数に切り出すとエラーになる
    function test(this){
      this.ownerDocument.querySelector('.ddd').lastChild.data = this.lastChild.data;
    }

    //関数に切り出すとエラーになる
    function test2(this){
      var txt = $(this).text();
      $(".ddd").contents().filter(function(){ return this.nodeType === this.TEXT_NODE }).remove();
      $(".ddd").append(txt);
    }
  </script>
</body>

</html>

追記2

処理1は動く。処理2は想定の動きにならない。
「直接書いたコード」と「関数を呼び出した場合」で動きが変わる。
文字をremoveできていない。

<!DOCTYPE html>
<html>

<head>
  <meta charset="UTF-8">
  <script src="http://ajax.googleapis.com/ajax/libs/jquery/1.7.2/jquery.min.js"></script>
  <style>
    .aaa {
      width: 150px;
      height: 50px;
      background-color: #EFE;
      border: solid 1px #CCC;
      position: absolute;
    }

    .ddd {
      width: 150px;
      height: 50px;
      background-color: pink;
      border: solid 1px pink;
      position: absolute;
    }
  </style>
</head>

<body>
  <label class="aaa">
    <span class="bbb"></span>
    <input type="hidden" class="ccc">りんご
  </label>
  <br>
  <br>
  <br>
  <label class="aaa">
    <span class="bbb"></span>
    <input type="hidden" class="ccc">いちご
  </label>
  <br>
  <br>
  <br>
  <label class="aaa">
    <span class="bbb"></span>
    <input type="hidden" class="ccc">ぶどう
  </label>
  <br>
  <br>
  <br>
  <label class="ddd">
    <span class="bbb"></span>
    <input type="hidden" class="ccc">りんご
  </label>
  <script>
    $(function() {
      $(".aaa").on("click", function() {
        //処理1。関数に切り出さず、直接書くとエラーにならない
        // this.ownerDocument.querySelector('.ddd').lastChild.data = this.lastChild.data;
        // test(this);

        //処理2。関数に切り出さず、直接書くとエラーにならない
        // var txt = $(this).text();
        // $(".ddd").contents().filter(function(){ return this.nodeType === this.TEXT_NODE }).remove();
        // $(".ddd").append(txt);
        test2(this);
      });
    });

    //関数に切り出すとエラーになる
    function test(_this){
      _this.ownerDocument.querySelector('.ddd').lastChild.data = _this.lastChild.data;
    }

    //関数に切り出すとエラーになる
    function test2(_this){
      var txt = $(_this).text();
      $(".ddd").contents().filter(function(){ return _this.nodeType === _this.TEXT_NODE }).remove();   //ここがうまくいっていない
      $(".ddd").append(txt);
    }
  </script>
</body>

</html>
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+5

this は予約語なので引数の名前で this は指定できないです
_this とかにすればいいと思います

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/03/24 00:53

    回答ありがとうございます!
    そうなんですね。_thisにするとエラーが消えうまくいきました!

    ただ処理2が「直接書いた動き」と「関数を呼びたした時の動き」で変わってしまいます。
    もしよろしければ教えていただけないでしょうか?

    追記2に実際に修正したコードを載せます。

    キャンセル

  • 2016/03/24 01:01 編集

    $(".ddd").contents().filter(function(){ return this.nodeType === this.TEXT_NODE }).remove();

    で filter に渡す無名関数ないで this を使っています
    関数の中では勝手に this が変わってしまうので「直接書いた」方では意図しない動きになってるはずです
    this は window (グローバル領域) を指しているので nodeType も TEXT_NODE も存在しなく undefined 同士で true が返ってるはずです

    _this にしたほうでは、 filter に渡した関数の中でも _this が変わらず test2 関数の引数で受け取った_this になっています

    キャンセル

  • 2016/03/24 07:49 編集

    返答ありがとうございます。

    どちらかと言うと、
    「直接書いた」方が意図している動きになっていて
    「関数に切り出した(_this)」の方が意図しない動きになってしまいます。

    前回投稿した質問の動きなのですが
    https://teratail.com/questions/30449

    「直接書いた」方はremoveされて、 「$(".ddd")のボックスの中の文字」がちゃんと変わるのですが、
    「関数に切り出した(_this)」の方では、removeされず、「$(".ddd")のボックスの中の文字」が増えていってしまいます。

    デベロッパーツールで「_this.nodeType」と「_this.TEXT_NODE」を見た限り、
    「直接書いた」方と同じ値が入っていたのでなぜうまくいかないのかわからない次第です。

    もしアドバイスいただけたらありがたいです。

    キャンセル

  • 2016/03/24 23:33

    すみません私もちゃんと読んでませんでした。
    文字の要素(テキストノード)を remove したいのですね。

    正しくはこうです

    $(".ddd").contents().filter(function(i,e){ return e.nodeType === Node.TEXT_NODE }).remove();

    説明しますと

    $(".ddd").contents()

    では、 .ddd の子要素一覧を持ってきます

    それは

    <span class="bbb"></span>
    <input type="hidden" class="ccc">
    りんご

    の3つといくつかの空白文字列のテキストノードです
    これらの中から filter をするわけですが、ここでは span や「りんご」などのそれぞれの要素に対してチェックをしないといけないです
    それらは filter に渡す関数の引数でとることができます
    2番めの引数の e に span や りんご などが入っています

    なので e.nodeType と Node.TEXT_NODE が等しい物を残せば文字列だけを選択できるので remove で削除できます
    Node.TEXT_NODE は e.TEXT_NODE でも同じ値です

    ---

    直接書いた場合でも、関数に分けた場合でも filter に渡した関数の中での `this` は window です
    なので filter の結果はすべての値がそのまま入っています

    そのため、`_this` にしない方では 全部の要素(input も含めて)が消えてから「りんご」が追加されているのでうまく行ってるように見えます

    `_this` のほうでは span や「りんご」ではなく、 .ddd の要素に対して テキストノードかのチェックを繰り返すので全部違うので filter の結果は空になり何も remove されないということになります

    ---

    ただ 文字列を置き換えたいだけなら、「りんご」などを <span class="fruit-name"></span> で書こんで、

    $(".ddd .fruit-name").text("みかん")

    としたほうが楽だと思います

    なんか凄く長くなってしまいました
    すみません

    キャンセル

  • 2016/03/25 07:46 編集

    詳しく丁寧に説明していただきありがとうございます。

    確かに
    $(".ddd").contents().filter(function(i,e){ return e.nodeType === Node.TEXT_NODE }).remove();
    だと関数の中でも同じ動きが実現できました。


    ただ少し腑に落ちない点がございます。

    1つ目
    >直接書いた場合でも、関数に分けた場合でも filter に渡した関数の中での `this` は window です
    「`_this` でない直接書いた方」では、`this` は 「window」 ではなく、「label.aaa」つまり「選択したラベル」が入っています。


    2つ目
    function(i,e)の「i」を消すと、spanタグやinputタグも消えてしまうのはなぜでしょうか?
    「i」について調べて、i番目などを指すときに使うものだと思っています。
    またこのコードで「i」を使っていないのに影響していることが不思議です。
    ちなみに、コードは「=== e.TEXT_NODE })」です。


    3つ目
    function(i,e)の「i」を消し、「=== e.TEXT_NODE })」のときと「=== Node.TEXT_NODE })」のときで処理の内容が変わるのはなぜでしょうか?

    「=== e.TEXT_NODE })」のとき
    ・spanタグやinputタグも消えてしまう

    「=== Node.TEXT_NODE })」のとき
    ・removeされず、文字がどんどんappendされてしまう


    4つ目
    そもそも一番知りたいのですが、
    関数の引数に「this」を渡したい時
    普通の場合は、「_this」のように予約語を避けた名前をつければ、直接書いたコードのまま関数に切り出せるが、
    今回のように「this」の中身が関数に切り出すと変わってしまう場合は、直接書いたコードを一部変更しなくてはいけない。
    という認識でよろしいでしょうか?


    私もかなり長くなってしまっているので、解決済みのスレに対して追加で聞きすぎているので別のスレを立てます。

    https://teratail.com/questions/30668

    よければ回答いただけたらと思います。

    キャンセル

  • 2016/03/26 12:47

    jQuery では this がbindされているのですね。これは私の勘違いでした、すみません。
    新しいスレではベストアンサーも決まってることですし私はこれまでにしておきます

    キャンセル

+2

関数の引数に this を渡すとエラーになるという事ですが、具体的にどんなコードを書いていますか。
jQuery でコールバック関数に引数を渡す方法はいくつかあります(個人的には event.data を推します)。

Re: k499778 さん

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/03/24 00:30

    回答有り難うございます!
    追記しましたので見てもらえればと思います。

    キャンセル

  • 2016/03/24 08:46

    関数オブジェクトを指定すれば引数束縛する必要がなくなりますね。
    ついでに event オブジェクトの扱いについても学んでおくといいかもしれません。
    jQuery('.aaa').on('click', test);
    function test (event){ this.ownerDocument.querySelector('.ddd').lastChild.data = this.lastChild.data; }

    キャンセル

  • 2016/03/24 12:09

    回答ありがとうございます!

    関数オブジェクトのやり方は知りませんでした。
    イベントオブジェクトの使い方についても勉強してみます。

    キャンセル

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

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

関連した質問

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

  • JavaScript

    17006questions

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

  • jQuery

    6926questions

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