2012-01-27 61 views
0

我想知道是否有人可以将他们的眼光放在这段代码上,并提供反馈,说明我是否使用了正确的逻辑,它的工作原理和它是有效的,但是我怀疑可能有订购问题,因为addScroller功能不重新初始化正确:用JSON创建表格

function DeliveredReport(){ 

var count=0; 
var content=''; 

spinnerOn(); 
$('#Results tbody').empty(); 
$.getJSON("/central/atv/api/json_delivered_rpt.p?fromNoTime=" + $('#fromNoTime').val() + "&toNoTime=" + $('#toNoTime').val(), 
function(data){ 

$.each(data.results, function(i,result){ 
content = '<tr><td>' + result.c + '</td>'; 
content += '<td>' + result.v + '</td>'; 
content += '<td>' + result.t + '</td>'; 
content += '<td>' + result.s + '</td>'; 
content += '<td>' + result.d + '</td>'; 
content += '<td><a class="iconSprite" id="customer_icon_btn" title="View Customer" href="javascript:ShowCustomer(' + result.cust + ')">Cust</a>' + 
      '<a class="iconSprite" id="deal_icon_btn" title="View Deal" href="javascript:ShowDeal(' + result.deal + ')">Deal</a></td></tr>'; 
count++; 

$('#Results tbody:last').append(content); 
}); 

spinnerOff(); 
if (count===0) {alert('No delivered vehicles\n' + $('#fromNoTime').val() + ' - ' + $('#toNoTime').val());} 
else {zebra(); AddScroller();} 

}); 

} 

斑马是典型的条纹效果和AddScroller被调用tableScroll插件。

任何反馈将不胜感激。

回答

0

我会用一个模板框架,而不是在.. Knockoutjs

http://knockoutjs.com/

看看它甚至还支持MVVM模式!

+0

感谢您的快速响应,我从来没有使用模板框架,但本周我确实查看了backbone.js。不改变我执行任务的方式,你会说代码是好的,或者你认为它不好,我需要实现新的框架(项目应该在5天内生效).... – Rachel 2012-01-27 12:00:28