jqueryのグローバル変数を呼び出す方法

Question

グローバル変数をfunction内に呼び出したいと思っています。

var hoge = $('.hoge').val();
$('#sample').click(function(){
  $('.hoge-hoge').text(hoge);
});
$('#sample-2').click(function(){
  $('.hoge-hoge-hoge').text(hoge);
});

このようにidがsampleかsample-2のボタンを押したらhogeのclassに書いてあるテキストがidがsampleの場合hoge-hogeのクラスに表示されidがsample-2の場合hoge-hoge-hogeに表示されるようにしたいのですがこの書き方だと、うまく表示されません

$('#sample').click(function(){
  var hoge = $('.hoge').val();
  $('.hoge-hoge').text(hoge);
});
$('#sample-2').click(function(){
  var hoge = $('.hoge').val();
  $('.hoge-hoge-hoge').text(hoge);
});

この書き方だとうまく表示されるのですが、var hoge = $(.hoge).val();を二回書かずに一回だけで済ませたいのですがどうすればよろしいでしょうか？
かなり初歩的な質問かもしれませんがよろしくお願いします。

Answer 1

原因は ShunsukeIzui さんの仰る通りだと思います。
一つ気になったのですが、$(.hoge).val() の行で SyntaxError になりますね(セレクタが文字列化されていません)。
その部分を修正してハンドラ関数を一つにまとめたコードが下記になります。

(2016/2/3 16:33追記) jsfiddle で jquery.js を読み込む設定にしていなかったのと修正

• Edit fiddle - JSFiddle

<input class="hoge" value="replaced!" />
<p id="sample-1">#sample-1</p>
<p class="hoge-hoge">Hello, World!</p>
<p class="hoge-hoge">Hello, World!</p>
<p id="sample-2">#sample-2</p>
<p class="hoge-hoge-hoge">Hello, World!</p>
<p class="hoge-hoge-hoge">Hello, World!</p>
<script>
'use strict';
jQuery(function (jQuery) {
  function handleClick (event) {
    var data = event.data;

    data.element.text(data.value);
  }

  function main () {
    var value = jQuery('.hoge').val();

    jQuery('#sample-1').on('click', {element: jQuery('.hoge-hoge'), value: value}, handleClick);
    jQuery('#sample-2').on('click', {element: jQuery('.hoge-hoge-hoge'), value: value}, handleClick);
  }

  main();
});
</script>

(2016/2/3 18:33追記)
別解。data-* 独自属性を使えばスマートに書けます。

<input class="hoge" value="replaced!" />
<p id="sample-1" class="foo" data-target-selector=".hoge-hoge">#sample-1</p>
<p class="hoge-hoge">Hello, World!</p>
<p class="hoge-hoge">Hello, World!</p>
<p id="sample-2" class="foo" data-target-selector=".hoge-hoge-hoge">#sample-2</p>
<p class="hoge-hoge-hoge">Hello, World!</p>
<p class="hoge-hoge-hoge">Hello, World!</p>
<script>
'use strict';
jQuery('.foo').on('click', {value: jQuery('.hoge').val()}, function handleClick (event) {
  jQuery(this.getAttribute('data-target-selector')).text(event.data.value);
  jQuery(this).off(event.type, handleClick);  // 2回目以降は無意味なのでイベントを削除しておく
});
</script>

Re: mhl さん

Comments

[think49]

癖でscript要素を後に書きましたが、この書き方ならば jQuery(function () {}) が不要です。
script要素を</body>手前に置くのも解決手段の一つだと思います。

[mhl]

ありがとうございます！

無事に解決しました！

Answer 2

要件にそうかわかりませんが、以下の方法もありかと思います。

$('#sample, #sample-2').click(function(){
  var hoge = $('.hoge').val();
  $('.hoge-hoge').text(hoge);
});

Comments

[think49]

その方法をとるなら #sample-2 で $('.hoge-hoge-hoge') を参照するように分岐処理が必要ではないでしょうか。
jQuery('#sample-1,#sample-2').on('click', function () {
jQuery({'sample-1': '.hoge-hoge', 'sample-2': '.hoge-hoge-hoge'}[this.id]).text(jQuery('.hoge').val());
});

[icham]

取得するテキストが違うのを見落としてました！
ご指摘ありがとうございます。

Answer 3

button id=hog
button id=pig
label id=label-hog
label id=label-pig
input id=inp
という構成で、

$(function() {
    $("x").on("click", function(e) {
          var v = $("#inp").val();
          $("#"+"label-"+e.target.id).text(v);
    });
});

こんな感じです。ボタンa には、hoge1 が、ボタンb には、hoge2 がそれぞれ対応するなら、ラベルのid は、ラベルの名前ーボタンの名前、っていう風に紐づけるようにしておけば少ない記述で実装できますね。
"x" は、ボタン2つを囲った何かです。
適当に調整してください。コピペじゃ動きません。

Comments

[mhl]

ありがとうございます！

参考にしてみます！

Answer 4

おそらくDOMの準備が整う前に実行されてしまっているためだろうと思います。
次のコードで試してみてください。

$(function(){
    var hoge = $(.hoge).val();
    $('#sample').click(function(){
        $('.hoge-hoge').text(hoge);
    });
    $('#sample-2').click(function(){
        $('.hoge-hoge-hoge').text(hoge);
    });
});

Comments

[mhl]

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

上記のコードで試しましたが、表示されませんでした・・・

[think49]

質問本文の修正前にあった下記コードが原因だと思われます。
var hoge = $(.hoge).val();
その点を修正して管理方法を変えたコードを別回答で挙げてみました。

[orange0190]

ああ、セレクタのミスに気が付きませんでした・・・