以下代码总是使用超过十秒。我已经升级了服务器,但它没有帮助。我知道我有一些数据库设计问题,但我无法修改它。
我在不同时间范围内显示某类别产品的不同位置的所有价格,因为价格每个地点每15天更改一次。
控制器
def prods_x_cat
# This will load all the products of a category
@products = Product.prods_x_cat(params[:id],:include => :raw_datas)
@cortes = RawData.cortes
respond_to do |format|
format.js { render :layout=>false}
end
end
prods_x_cat.js.erb
var jqxhr1 = $.ajax($("#loading_overlay .loading_message, #loading_overlay").fadeIn());
$('#datos').html("<%= escape_javascript render :partial=>'/shared/prods_x_cat'%>")
查看
<%@cortes.each do |c|%>
<a href="#<%=c.corte%>" class="round_top"><%=c.corte%></a>
<%end%>
<%@cortes.each do |c|%>
<%@fecha = c.corte_real%>
<div id="<%=c.corte%>" class="block no_padding">
<table class="display datatable">
<thead>
<tr>
<th>SKU</th>
<%Company.active.order('table_field ASC').each do |c|%>
<th><%=c.abbr%></th>
<%end%>
<th>Average</th>
<th>Mode</th>
<th>Minimum</th>
<th>Maximum</th>
</tr>
</thead>
<tbody>
<%@products.each do |p|%>
<tr class="gradeA">
<td><%=p.name%></td>
<%p.raw_datas.where("product_id='#{p.id}' and corte_real='#{@fecha}'").each do |prd|%>
<td><%=prd.location1.to_f.round(2)%></td>
<td><%=prd.location2.to_f.round(2)%></td>
<td><%=prd.location3.to_f.round(2)%></td>
<td><%=prd.location4.to_f.round(2)%></td>
<td><%=prd.location5.to_f.round(2)%></td>
<td><%=prd.location6.to_f.round(2)%></td>
<td><%=prd.location7.to_f.round(2)%></td>
<td><%=prd.location8.to_f.round(2)%></td>
<td><%=prd.promedio.to_f.round(2)%></td>
<td><%=prd.moda%></td>
<td><%=prd.minimo.to_f.round(2)%></td>
<td><%=prd.maximo.to_f.round(2)%></td>
<%end%>
</tr>
<%end%>
</tbody>
</table>
</div>
<%end%>
</div>
答案 0 :(得分:0)
如果不看所有涉及的代码,这个问题几乎是不可能回答的。相反,我可以帮助你找出问题所在。
有很好的工具可以找到您的性能问题(例如ruby-prof),但如果您想要一种快速的原始方式来查找问题所在,只需使用Time.now
即可。例如,您可以将控制器操作更改为:
def prods_x_cat
# This will load all the products of a category
a1 = Time.now
@products = Product.prods_x_cat(params[:id],:include => :raw_datas)
b1 = Time.now
p "Time for finding products: #{b1 - a1}"
a2 = Time.now
@cortes = RawData.cortes
b2 = Time.now
p "Time for finding cortes: #{b2 - a2}"
respond_to do |format|
format.js { render :layout=>false}
end
end
如果打印输出显示其他地方占用了时间,请在模板中开始做类似的事情。专注于数据库调用。