0

ここでページを確認できます：http://jsfiddle.net/7JhjN/

基本的に、地獄のような複製が行われていて、私はこれが好きではありません。正しいプログラミングロジックではないため、何か問題があると確信しています。

この関数を短縮するにはどうすればよいですか?

システムは次のように機能します 。 1. 6 つの選択ボックスがあります。(オプション値は同じです) 2. 出力用の 6x div があります。3. 選択ボックスの値が変更されるたびに、ターゲット div のイメージが変更されます。

4

3 に答える 3

2

属性セレクターで始まるを使用すると、コードを簡素化できます。

$('select[name^="item_early"]').change( function() {
    var $this = $(this);
    var value = $this.val();
    var $resultDiv = $('#result_' + $this.attr('name'));
    //directly set loading value as html
    $resultDiv.html('<img src="http://www.sobafire.com/themes/default/images/loading.gif">');

    //change image url source from server (expected return value is something like <img src="" />
    $resultDiv.load('http://www.sobafire.com/ajax.php?ajaxType=ITEM&itemID=' + value);

});

ワーキングデモ

于 2012-01-14T21:07:13.513 に答える
2

これはどう：

HTML:

<select class='item_early' name="item_early1" data-id="1"><option value="0"></option></select>
<select class='item_early' name="item_early2" data-id="2"> <option value="0"></option></select>
<select class='item_early' name="item_early3" data-id="3"> <option value="0"></option></select>
<select class='item_early' name="item_early4" data-id="4"> <option value="0"></option></select>

<div id="result_item_early1"></div>
<div id="result_item_early2"></div>
<div id="result_item_early3"></div>
<div id="result_item_early4"></div>

Javascript:

$('.item_early').change( function() {
  var value = $(this).val();
  var id = $(this).data('id');

  $('#result_item_early'+id).html('<img src="http://www.sobafire.com/themes/default/images/loading.gif">');
  $('#result_item_early'+id).load('http://www.sobafire.com/ajax.php?ajaxType=ITEM&itemID=' + value);
});
于 2012-01-14T21:12:04.123 に答える
2

あなたの問題、次のようなものがあります。

$('#item_early1').change( function() {
    var value = $(this).val();
    //directly set loading value as html
    $('#result_item_early1').html('<img src="http://www.sobafire.com/themes/default/images/loading.gif">');

    //change image url source from server (expected return value is something like <img src="" />
    $('#result_item_early1').load('http://www.sobafire.com/ajax.php?ajaxType=ITEM&itemID=' + value);

});

そして、異なる定数について繰り返します。

答えは、定数を次のような変数名に置き換えてから、関数に入れることです。

function doit(itemID,resultID) {
  $(itemID).change( function() {
      var value = $(this).val();
      //directly set loading value as html
      $(resultID).html('<img src="http://www.sobafire.com/themes/default/images/loading.gif">');

      //change image url source from server (expected return value is something like <img src="" />
      $(resultID).load('http://www.sobafire.com/ajax.php?ajaxType=ITEM&itemID=' + value);
  });
}

次に、すべての定数に対してその関数を呼び出します。

doit('#item_early1','#result_item_early1');
doit('#item_early2','#result_item_early2');
doit('#item_early3','#result_item_early3');
doit('#item_early4','#result_item_early4');
doit('#item_early5','#result_item_early5');
doit('#item_early6','#result_item_early6');
于 2012-01-14T21:05:56.470 に答える