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

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

ただいまの
回答率

90.12%

[jQuery]処理をメソッドにまとめたい。リファクタリング

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,177

k499778

score 521

現在HTML,Javascript(jQuery)を使ってアプリを作っています。

jQueryでボタンタップ時に画面遷移する処理を作っているのですが、うまくメソッドにまとめることができません。
そのリファクタリング方法を教えていただきたいです。

処理は以下です。

$("#aBtn").on({
    'touchstart': function(e) {
      // A処理
    },
    'touchmove': function(e) {
      // B処理
    },
    'touchend': function(e) {
      // C処理
      toPage("/aaa");
    }
  });
$("#bBtn").on({
    'touchstart': function(e) {
      // A処理
    },
    'touchmove': function(e) {
      // B処理
    },
    'touchend': function(e) {
      // C処理
      toPage("/bbb");
    }
  });
$("#cBtn").on({
    'touchstart': function(e) {
      // A処理
    },
    'touchmove': function(e) {
      // B処理
    },
    'touchend': function(e) {
      // C処理
      toPage("/ccc");
    }
  });

function toPage(url){
 // D処理
  location.href = "https:xxx" + url
}

タップ処理の対応をするとすごくひとつの処理が長くなってしまします。
行っている内容はtouchmoveが発生した時にタップをキャンセルするといった対応です。
もし分かる方がいればよろしくお願いいたします。


追記
元々の投稿本文を修正しました。
どのボタンタップ時も
toPage("/ccc");
になっていたので、
それぞれの遷移先へいくように修正しました。
aボタンタップ時 toPage("/ccc"); → toPage("/aaa");
bボタンタップ時 toPage("/ccc"); → toPage("/bbb");


追記2
アドバイスでいただいたswitch文を使う方法はイメージできたので記載してみます。

$("#aBtn,#bBtn,#cBtn").on({
    'touchstart': function(e) {
      // A処理
    },
    'touchmove': function(e) {
      // B処理
    },
    'touchend': function(e) {
      // C処理
      var id = var id = $(this).attr("id");
      switch(id){
        case "aBtn":
          toPage("/aaa");
          break;
        case "bBtn":
          toPage("/bbb");
          break;
        case "cBtn":
          toPage("/ccc");
          break;
      }
    }
  });

function toPage(url){
 // D処理
  location.href = "https:xxx" + url
}

この書き方はどうでしょうか?
あまりよくないなどの意見があればおっしゃってください。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • kei344

    2017/11/18 13:01

    回答が付いた質問の編集は慎重に行ってください。質問文のコードについて回答にて指摘があった場合は「追記」し、元のコードを編集する場合も「直したこと」がわかるようにしてください。

    キャンセル

  • k499778

    2017/11/18 13:08

    修正しました

    キャンセル

回答 3

checkベストアンサー

+2

aの要素をクリックしたらAのページへ画面遷移。
bの要素をクリックしたらBのページへ画面遷移。

「Aの要素をクリックしたら、Bのページへ画面遷移」そのものの一連の流れは共通ですね。
この程度の差なら配列やオブジェクトを使えば共通化できますね。
オブジェクトの配列を作って解決しましょうか。

var link_settings = [
  {selector: "#aBtn", url: "/aaa"},
  {selector: "#bBtn", url: "/bbb"},
  {selector: "#cBtn", url: "/ccc"},
]
link_settings.forEach(function(it) {
  $(it.selector).on({
    'touchstart': function(e) {
      // A処理
    },
    'touchmove': function(e) {
      // B処理
    },
    'touchend': function(e) {
      // C処理
      toPage(it.url);
    }
  });
});

おまけ: 実行したい内容そのものを分岐させる

JavaScriptは引数やオブジェクトに関数を持たせられるので、
やりたいことをオブジェクトなんかに持たせておく解決法もあります。

今回はtoUrlを関数を返す関数に作り変えて実装してみました。
仮引数としてpathを記憶させておいて、次の実行で画面遷移するように仕込んであります。

また、cの処理が画面遷移さえすれば良いと解釈してこんな風にしてみました。
過ぎた抽象化は読みづらくなる原因ですが、使いこなせば良いソースコードを書けるようになりますので、色々試してみてください。

var toUrl = function(path) {
  return function(e) {
    // C処理
    location.href = "https:xxx" + url
  }
}
var link_settings = [
  {selector: "#aBtn", touchend: toUrl("/aaa")},
  {selector: "#bBtn", touchend: toUrl("/bbb")},
  {selector: "#cBtn", touchend: toUrl("/ccc")},
]
link_settings.forEach(function(it) {
  $(it.selector).on({
    'touchstart': function(e) {
      // A処理
    },
    'touchmove': function(e) {
      // B処理
    },
    'touchend': it.touchend
  });
});

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/18 16:01

    このような書き方があるのですね!コード量が少なく、シンプルでわかりやすいです。
    参考にさせて頂きます。

    キャンセル

+2

<!DOCTYPE HTML>
<html lang="ja">
    <head>
        <meta charset="UTF-8">
        <title>Sample</title>
    </head>
    <body>
        <button type="button" class="func_btn" data-url="aaa">A button</button>
        <button type="button" class="func_btn" data-url="bbb">B button</button>
        <button type="button" class="func_btn" data-url="ccc">C button</button>

        <script type="text/javascript" src="//code.jquery.com/jquery-3.1.1.min.js"></script>
        <script type="text/javascript">
            // 処理A
            let func_a = function (event) {
                alert('func_a');
            };
            // 処理B
            let func_b = function (event) {
                alert('func_b');
            };
            // 処理C
            let func_c = function (event) {
                let url = this.getAttribute('data-url');
                location.href = "https:www.example.com/" + url;
            };
            $(".func_btn").on({
                'touchstart': func_a,
                'touchmove': func_b,
                'touchend': func_c
            });
        </script>
    </body>
</html>

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/18 15:52 編集

    追記したコードそもそも矛盾している。」
    動作確認くらいしてから書いてください。

    キャンセル

  • 2017/11/18 15:58

    idですよね。失礼しました。
    以下に修正しました。
    case "aaa": → case "aBtn":
    case "bbb": → case "bBtn":
    case "ccc": → case "cBtn":

    言い方きついですね(⌒-⌒; )
    ローカルで動作確認はしました。もっとシンプルなアラートを出す書き方で書いていたので
    投稿本文に合わせたときに誤記しました。修正致しましたのでよろしくお願いします。

    キャンセル

  • 2017/11/18 16:02

    回答ありがとうございます。
    このような書き方もあるのですね!参考にさせていただきます

    キャンセル

  • 2017/11/18 16:02

    動くことは動きますね。コードの保守性ということを考えると、再考すべきです。
    #dBtn を追加する場面になったとき、修正すべきコードが少ない方がバグを生みにくくなる。ごs自身が書いたコードと、私が提示したコードでそれぞれ、#dBtn を追加したコードに修正するときどちらが楽にできますか?

    キャンセル

+1

いまいちどういうことをしたいかがわかりませんが、提示のコードなら下記のようにまとめられます。

$("#aBtn, #bBtn, #cBtn").on({
    'touchstart': function(e) {
      // A処理
    },
    'touchmove': function(e) {
      // B処理
    },
    'touchend': function(e) {
      // C処理
      toPage("/ccc");
    }
  });

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/18 13:03

    回答ありがとうございます。
    失礼しました。url部分に間違いがありました。
    それぞれ違うページ遷移するので,url末尾が変わります。

    ですのでtoPage("/ccc");を3パターンに分岐したいのです。
    セレクタの複数指定は思いついたのですが、この部分をどのようにすればシンプルに書けるかわかりません。
    何かフラグを設定するとかなら、現状のコードの方がまだシンプルなんじゃないかなと。
    いい方法はありますでしょうか?

    キャンセル

  • 2017/11/18 13:04

    投稿本文のurl末尾部分を修正しました。

    キャンセル

  • 2017/11/18 13:30

    touchend の function 内で クリックされた要素の id で switch などすればいいのではないでしょうか。

    キャンセル

  • 2017/11/18 15:46

    回答ありがとうございます。
    その方法を追記に記載してみます

    キャンセル

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

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