Rails 2.3.8 named_scope鏈接

Question

我有以下嵌套的if語句hairball，我想知道是否有更有效的方式來編寫此代碼（不需要這麼多條件的代碼行）

每種方法都named_scopes模型中..

box = (params[:b] ? params[:b] : "inbox") 
show = (params[:s] ? params[:s] : "all") 

if box == "inbox" 
    if show == "all" 
    @messages = @current_user.received_messages.all 
    elsif show == "unread" 
    @messages = @current_user.received_messages.unread.all 
    elsif show == "read" 
    @messages = @current_user.received_messages.read.all 
    elsif show == "starred" 
    @messages = @current_user.received_messages.starred.all 
    else 
    @messages = [] 
    end 
elsif box = "sent" 
    @messages = @current_user.sent_messages.all 
else 
    @messages = [] 
end 

我的想法是，我可以用一個「呼叫」型方法的框，顯示像

@current_user.received_messages.call(:box).call(:show).all 

也許..？

哎，應該花一點更多的時間玩了..其正當我以爲，我只是使用了錯誤的方法SOLUTION

@current_user.send(box).send(show) 

Answer 1

您可以使用軌道2.3.8的scoped()方法鏈範圍：

main_method = case (params[:s]||"inbox") 
when "inbox" 
    :received_messages 
when "sent" 
    :sent_messages 
end 
# create a scope. Don't execute the scope yet. 
scope = @current_user.send(main_method).scoped({}) if main_method 

# now chain the scope if needed 
scope = begin 
    if ["unread", "read", "starred"].include?(params[:s]) 
    scope.send(params[:s]) 
    elsif((params[:s] || "all") == "all") 
    scope 
    end 
end if main_method == :received_messages 

@messages = scope.nil? ? [] : scope.all 

參考：

Scoped method source in Rails 2.3.x

Rails casts on Anonymous scopes in 2.3.x

Answer 2

這是我想通了，我不會標記我自己的答案是正確的，除非大家同意這是最好的 - 10其他想法？

if params[:b].present? && ["received_messages", "sent_messages"].include?(params[:b]) 
    box = params[:b] 

    if params[:s].present? && ["all", "unread", "starred"].include?(params[:s]) 
    show = params[:s] 
    else 
    show = "all" 
    end 

    @messages = @current_user.send(box).send(show) 
else 
    @messages = [] 
end 

Answer 3

你的答案是非常接近，但它無法轉換box值。

box = params[:b] || 'inbox' 
show = params[:s] || 'all' 

box_scope = case box 
    when 'inbox' then 'received_messages' 
    when 'sent' then 'sent_messages' 
end 
show_scope = show # no convertion needed at this point, maybe in the future 

# If box_scope is nil, don't query 
@messages = box_scope.nil? ? [] : @current_user.send(box_scope).send(show_scope) 

這是假設你弄死了.all您正在使用的原代碼所有選項，消失在你的答案。