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

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

ただいまの
回答率

91.03%

  • JavaScript

    13797questions

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

  • jQuery

    5810questions

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

  • jQuery UI

    141questions

    jQuery UI はjQuery公式のインターフェースライブラリであり、対話型のウェブアプリケーションを作る際に役立ちます。

stop(jQueryUI)内にあるメソッドを外部化したい

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 210
退会済みユーザー

退会済みユーザー

前提・実現したいこと

jQueryUIのDraggableを使用しています。

以下のソースコードのstop: function () {}内に.each()を設置しているのですが、stopの度に.each()が動作するのはあまり良くないのではないか、と考えています。

そこで、stop: function () {}の外に.each()を設置しながらも、stop: function () {}の中から変数$elmを呼び出せるようにしたいのですが、可能でしょうか?

何卒よろしくお願い致します。

該当のソースコード

$($element).each(function () {
  $(this).draggable({
    drag: function () {
      var $hogehoge = $('#hogehoge');
    },

    stop: function () {
      $('#hoge').each(function () {
        var $elm =  $(this).children('.items');
        var $test =  $(this).children('.items').data('aaa'); //追記部分
        console.log($elm); //追記:eachでなくとも取得できる
        console.log($test); //追記:data-aaa="10"のみ取得できる
      });
    },
  });
});
<section id="hoge">
  <div class="items" data-aaa="10">あああ</div>
  <div class="items" data-aaa="20">いいい</div>
  <div class="items" data-aaa="30">ううう</div>
  <div class="items" data-aaa="40">えええ</div>
  <div class="items" data-aaa="50">おおお</div>
</section>


※data-aaaを追記いたしました。

追記

masaya_ohashi様のご指摘の通り、$elmの部分は.each()でなくとも取得できましたが、その下にある.itemsdata-aaaを取得しようとすると、はじめの行<div class="items" data-aaa="10">あああ</div>のみ取得できる形となってしまいます。

そうではなく、すべての.itemsdata-aaaを取得したい場合、どのようにすれば良いでしょうか。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    退会済みユーザー

    2017/10/13 17:38

    はい、そのような認識で合っております。お手数をお掛けいたします。

    キャンセル

  • masaya_ohashi

    2017/10/13 17:41

    .itemsが動的に増減したり、data-aaaが動的に書き換わることはありますか?

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2017/10/13 17:45

    .itemsは頻繁ではありませんが、増減する可能性があるものです。data-aaaは基本的に固定の値となります。

    キャンセル

回答 1

checkベストアンサー

+2

なぜ「stopの度に.each()が動作するのはあまり良くないのではないか」と思ったのかわかりませんが、やろうとしていることを実現するには.each()しか方法はないです。なんとなく良くない、という非合理な理由であればそれは無視してよいです。

    stop: function () {
      var list = [];
      var $elm =  $('#hoge > .items');
      $elm.each(function () {
        var $test =  $(this).data('aaa');
        list.push($test);
      });
      console.log(list); // data-aaaの配列[10, 20, 30, 40, 50]
    },

#hogeの内容が動的に変わらないのであれば、createの際に取ってしまって、stopで再利用する形もよいかもしれません。

    create: function () {
      var list = [];
      var $elm =  $('#hoge > .items');
      $elm.each(function () {
        var $test =  $(this).data('aaa');
        list.push($test);
      });
      $(this).data('list', list);
    },
    stop: function () {
      var list = $(this).data('list');
      console.log(list); // data-aaaの配列[10, 20, 30, 40, 50]
    },

 以下の回答は質問文編集前の内容です

コードとHTMLを見る限り、現状eachを使う理由がありません。これでいいです。

    stop: function () {
      var $elm =  $('#hoge > .items');
      console.log($elm);
    },

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/10/13 17:34

    ご回答ありがとうございます。なるほど、この場合は特にeachを使う必要なく取得できました(勉強になります)。
    ただ度々説明不足で申し訳ないのですが、本来の目的としては.itemsにあるdata属性を取得したかったのです。しかし、うまく取得が出来ません。
    再度、一部内容を修正致しましたので、そちらを拝見いただければと存じます。
    何卒、よろしくお願い致します。

    キャンセル

  • 2017/10/13 18:00

    ありがとうございます!ご案内いただいた内容で取得できました。.itemsの個数が多い場合、stopの度にeachが動くのは処理的に良くないのかと思ったからです。今回は(createとstopとを分けた方が理解しやすいという点で)後者の方法を採用させていただきましたが、その点に関して特に問題のないようでしたら、今後前者の方法も検討していきたいと思います。ありがとうございました。

    キャンセル

  • 2017/10/13 18:23

    itemsの個数が数十個程度ならほとんど気にならない速度だと思うので、そこまで神経質にならなくていいと思いますよ。

    キャンセル

  • 2017/10/13 18:30

    なるほど、あまり気にしすぎなくても良いということで安心いたしました。ご丁寧に説明頂きありがとうございます。

    キャンセル

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

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

関連した質問

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

  • JavaScript

    13797questions

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

  • jQuery

    5810questions

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

  • jQuery UI

    141questions

    jQuery UI はjQuery公式のインターフェースライブラリであり、対話型のウェブアプリケーションを作る際に役立ちます。