2011-11-29 73 views
0

我有一个Rails应用程序,它具有生成报告的功能。传递参数的优雅方式

之前产生所述报告中,向用户呈现与指定报告的日期范围内的形式,

<%= form_tag({:action => :generate_report}, :multipart => true) do %> 

    <h3>Date range is inclusive</h3> 

    <b>Start Date</b><br /> 
    <%= date_select :range, :startDate, options = {:order => [:day, :month, :year]} % 

    <b>End Date</b><br /> 
    <%= date_select :range, :endDate, options = {:order => [:day, :month, :year]} %> 

    <%= submit_tag "Generate Report", :disable_with => 'Reporting...' %> 

<% end %> 

支撑代码然后生成报告,

# GET /sales_activities/generate_report 
def generate_report 

    @start_date = Date.civil(params[:range][:"startDate(1i)"].to_i,params[:range][:"startDate(2i)"].to_i,params[:range][:"startDate(3i)"].to_i) 
    @end_date = Date.civil(params[:range][:"endDate(1i)"].to_i,params[:range][:"endDate(2i)"].to_i,params[:range][:"endDate(3i)"].to_i) 

    @end_date_inclusive = @end_date + 1.day 

    @sales = SalesActivity.find(:all, 
       :select => [:salesperson_id], 
       :conditions => {:created_at => @[email protected]_date_inclusive, :code => [5]}, 
       :group => :salesperson_id) 

    # <Skip the boring code> 

    respond_to do |format| 
     format.html # show.html.erb 
     format.xml { render :xml => @report } 
     format.json { render :json => @report } 
    end 
    end 

然后是视图以通常的方式呈现。如果用户喜欢他们看到的内容,则他们可以选择将报告导出为csv。

我目前有以下代码将指定的范围传递给生成相同报表以供导出的方法,但它看起来不是很差,错误,很好,并且会导致长时间的URL。

<h2>Regulatory Report</h2> 

<h4>Reporting Period from <%= @start_date.strftime('%d/%m/%y') %> to <%= @end_date.strftime('%d/%m/%y') %></h4> 

    <%= link_to image_tag("Buttons/ExportReport.png", :border => 0), report_export_sales_activities_path(:start_date_str => @start_date, :end_date_str => @end_date) %> 

我的问题是,有没有更好的方法将这些参数传递给导出方法?谢谢。

+0

使用POST请求将让你的参数了网址,并缩短了位产生的PARAMS格式。 http://api.rubyonrails.org/classes/ActionView/Helpers/UrlHelper.html这就是你想要的吗? –

回答

1

AR将让你使用由date_select助手直接

@sales = SalesActivity.where(['(created_at BETWEEN ? AND ?', params[:range][:start_date], params[:range][:end_date]])