1

这是我在关联模型中使用的默认代码,用于从 Paperclip 下载图像作为 save_to。

月控制器

def download
  @wallpaper = Wallpaper.find(params[:wallpaper_id])
  @month = @wallpaper.months.find(params[:id])

  send_file @month.wallpaper_picture.path,
              :filename => @month.wallpaper_picture_file_name,
              :type => @month.wallpaper_picture_content_type,
              :disposition => 'attachment'
end

路线

resources :wallpapers do  
  resources :months  
end  
match 'wallpaper/:wallpaper_id/download/:id' => 'months#download', :as => :download

查看/月/索引

- @months.each do |month|  
  = link_to 'default', download_path(month.wallpaper_id, month.id) 

但是我的应用程序在我的模型中声明了近 6 种不同样式的回形针,并且每种样式都必须是可下载的。为此,我这样做了(我将仅显示 6 个代码块中的 2 个):

月控制器

  def download_iphone4
    @wallpaper = Wallpaper.find(params[:wallpaper_id])
    @month = @wallpaper.months.find(params[:id])

    @month = 'public/system/wallpaper_pictures/' + @month.id.to_s + '/iphone4/' + @month.wallpaper_picture_file_name
    send_file @month,
              :disposition => 'attachment'
  end

  def download_iphone5
    @wallpaper = Wallpaper.find(params[:wallpaper_id])
    @month = @wallpaper.months.find(params[:id])

    @month = 'public/system/wallpaper_pictures/' + @month.id.to_s + '/iphone5/' + @month.wallpaper_picture_file_name
    send_file @month,
              :disposition => 'attachment'
  end 

  def download_ipad ...
  def download_1440 ...
  def download_1680 ...
  def download_1920 ...
  etc ...

路线

match 'wallpaper_pictures/:wallpaper_id/iphone4/:id' => 'months#download_iphone4', :as => :download_iphone4  
match 'wallpaper_pictures/:wallpaper_id/iphone5/:id' => 'months#download_iphone5', :as => :download_iphone5  
match 'wallpaper_pictures/:wallpaper_id/ipad4/:id' => 'months#download_ipad', :as => :download_ipad  
match 'wallpaper_pictures/:wallpaper_id/1440/:id' => 'months#download_1440', :as => :download_1440  
match 'wallpaper_pictures/:wallpaper_id/1680/:id' => 'months#download_1680', :as => :download_1680  
match 'wallpaper_pictures/:wallpaper_id/1920/:id' => 'months#download_1920', :as => :download_1920  

观看次数/月/索引

- @months.each do |month|  
     = link_to 'iphone4', download_iphone4_path(month.wallpaper_id, month.id) 
     = link_to 'iphone5', download_iphone5_path(month.wallpaper_id, month.id) 
     = link_to 'ipad', download_ipad_path(month.wallpaper_id, month.id) 
     = link_to '1440', download_1440_path(month.wallpaper_id, month.id) 
     = link_to '1680', download_1680_path(month.wallpaper_id, month.id) 
     = link_to '1920', download_1920_path(month.wallpaper_id, month.id) 

我的问题来了:
1)我可以用更清洁/更好的方式来做吗?
2)我必须将块从我的控制器移动到模型或新控制器吗?
3)在代码的第一个和默认方法中,有一些哈希值,例如:

:filename => @month.wallpaper_picture_file_name,  
:type => @month.wallpaper_picture_content_type  

但在另一种方法中,我意识到我不需要使用它们。这些哈希是必要的吗?
4)我称它们为“哈希器”。这是正确的吗?还有什么更正吗?


PD:如果 send_file 在 Production 中失败,将其更改为 send_data 或
在 config/production.rb 中注释掉这一行

config.action_dispatch.x_sendfile_header = "X-Sendfile"  

send_file 只是发送一个空文件

4

1 回答 1

3

将您的控制器代码重构为:

def download_iphone4
  send_file (myfile params, :iphone4), :disposition => 'attachment'
end

def download_iphone5
  send_file (myfile params, :iphone5), :disposition => 'attachment'
end 

def download_ipad ...
def download_1440 ...
def download_1680 ...
def download_1920 ...


private 

def myfile params, style
  @wallpaper = Wallpaper.find(params[:wallpaper_id])
  @month = @wallpaper.months.find(params[:id])
  'public/system/wallpaper_pictures/' + @month.id.to_s + "/#{style}/" +@month.wallpaper_picture_file_name
end

或者,您可以使用您的路线修改和控制器进一步干燥:

路线.rb

match 'wallpaper/:wallpaper_id/download/:id/:style' => 'months#download', :as => :download

控制器

def download
  send_file (myfile params), :disposition => 'attachment'
end 


private 

def myfile params
  @wallpaper = Wallpaper.find(params[:wallpaper_id])
  @month = @wallpaper.months.find(params[:id])
  'public/system/wallpaper_pictures/' + @month.id.to_s + "/#{params[:style]}/" +@month.wallpaper_picture_file_name
end

并从您的视图中生成路径,例如:

 = link_to 'iphone4', download_path(month.wallpaper_id, month.id,'iphone4') 
 = link_to 'iphone5', download_path(month.wallpaper_id, month.id,'iphone5') 

在上述两种情况下,您都可以将您的私有方法相应地移动到壁纸/月模型。


3)在代码的第一个和默认方法中,有一些哈希值,例如:

:filename => @month.wallpaper_picture_file_name, :type => @month.wallpaper_picture_content_type

但在另一种方法中,我意识到我不需要使用它们。这些哈希是必要的吗?

:filename 让您可以控制作为附件发送的文件的文件名,您可以修改它并提供与实际文件不同的文件名。默认情况下,它将以其实际文件名返回附件。

:type 再次以类似方式工作,您可以将其强制为其他内容类型;但是它不会有太大影响,因为您已经将文件呈现为附件。

它们不是散列,它们是散列的键、值,或者在这里可以简单地视为属性。

于 2013-01-02T18:18:51.793 回答