最佳實踐on Rails的

我有一些控制器方法我想分享重用在Ruby中控制器之間的代碼。在軌道上做紅寶石的最佳做法是什麼？我應該創建一個我的控制器擴展的抽象類，還是應該創建模塊並將其添加到每個控制器中？以下是我想要分享的控制器方法：最佳實踐on Rails的

def driving_directions 
    @address_to = params[:address_to] 
    @address_from = params[:address_from] 
    @map_center = params[:map_center_start] 

    # if we were not given a center point to start our map on 
    # let's create one. 
    if [email protected]_center && @address_to 
    @map_center = GeoKit::Geocoders::MultiGeocoder.geocode(@address_to).ll 
    elsif [email protected]_center && @address_from 
    @map_center = GeoKit::Geocoders::MultiGeocoder.geocode(@address_from).ll 
    end 
end 

def printer_friendly 
    starting_point = params[:starting_point].split(',').collect{|e|e.to_f} 
    ne = params[:ne].split(',').collect{|e|e.to_f} 
    sw = params[:sw].split(',').collect{|e|e.to_f} 
    size = params[:size].split(',').collect{|e|e.to_f} 
    address = params[:address] 

    @markers = retrieve_points(ne,sw,size,false) 
    @map = initialize_map([[sw[0],sw[1]],[ne[0],ne[1]]],[starting_point[0],starting_point[1]],false,@markers,true) 
    @address_string = address 
end

來源

2008-09-24 Kyle Boon

在這種情況下是否有沒有使用application.rb的特殊原因？ – 2008-09-24 19:07:07

只有一些控制器會使用代碼，但不是全部。 – 2008-09-25 02:20:51

110

在我看來，正常的面向對象的設計原則：

如果代碼確實是一組不需要訪問對象狀態的實用工具，我會考慮把它在一個模塊中被調用分別。例如，如果代碼是所有映射實用程序，請創建一個模塊Maps，並訪問諸如：Maps::driving_directions的方法。
如果代碼需要狀態並在每個控制器中使用或可以使用，請將代碼放在ApplicationController中。
如果代碼需要狀態，並且在被緊密地和邏輯地相關的所有控制器的一個子集被使用（即，所有有關映射），則創建一個基類（class MapController < ApplicationController），並把該共享代碼那裏。
如果代碼需要國家，並在沒有很密切的關係的所有控制器的一個子集時，把它放在一個模塊中，並將其納入必要的控制器。

在你的情況下，方法需要狀態（params），所以選擇取決於需要它的控制器之間的邏輯關係。此外：

另外：