2012-01-14 206 views
0

您可以在这里查看页面:http://jsfiddle.net/7JhjN/jQuery函数重复。我怎样才能缩短这个时间?

基本上,有一个地狱重复正在进行,我不喜欢这个。我确信有错误,因为它不是一个正确的编程逻辑。

如何缩短此功能?

该系统的工作原理如下: 1.有6个选择框。 (选项值相同) 2.有6个div用于输出。 3.每当选择框值发生变化时,目标div中的图像都会发生变化。

回答

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); 
}); 
+0

很好的答案,显然是正确的方法。也许'name'应该是'id'? – Hogan 2012-01-14 21:18:33

+0

@Hogan谢谢,对HTML的一点改变总是会最终简化我的javascript – 2012-01-14 21:55:01

+0

@BassamMehanni;谢谢。非常感激。 – Aristona 2012-01-14 22:21:19

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'); 
+0

制作功能是我必须在我的脑海里,但我必须是ca分开使用每个功能。我最感兴趣的是一个简单的函数,它可以相应地检索所有的数据和输出,而不必每次都编辑函数调用。 ShankarSangoli发布的帖子是我正在寻找的,但是,我感谢您的评论,并感谢您的指导。 :) – Aristona 2012-01-14 22:18:36

+0

没问题,祝你好运。 – Hogan 2012-01-14 22:30:03

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); 

}); 

工作Demo

+0

嗯......这是否真的简化了代码?对我来说似乎更加复杂 - 这是一个不知道如何制作功能的用户。 – Hogan 2012-01-14 21:09:41

+0

@Hogan - 这是针对这种场景的一种简单且最佳的解决方案。 OP正在缩短代码。如果OP有任何疑问,我可以回答OP的问题。 – ShankarSangoli 2012-01-14 21:11:43

+0

不知道它是最好的 - 我期望我的速度更快(但不是太多 - 只是'select [name^='item_early']'和6 id查找之间的差异)。 – Hogan 2012-01-14 21:17:08