未選擇重構的情況下

它是Rails/Ruby。只是想知道是否有一個DRYer版本刪除重複在下面的代碼：未選擇重構的情況下

case params[:order] 
when 'rating_ascend' 
    order = {:order => 'rating_average ASC'} 
when 'rating_descend' 
    order = {:order => 'rating_average DESC'} 
when 'distance' 
    order = {:order => 'distance ASC'} 
else 
    order = {:order => 'distance ASC'} 
end

謝謝。

來源

2012-08-10 Victor

或當然，有。

order = case params[:order] 
    when 'rating_ascend' 
    {:order => 'rating_average ASC'} 
    when 'rating_descend' 
    {:order => 'rating_average DESC'} 
    else 
    {:order => 'distance ASC'} 
end

甚至這個（我不會寫這樣的，但是這是烘乾機）

order = {:order => case params[:order] 
    when 'rating_ascend' then 'rating_average ASC' 
    when 'rating_descend' then 'rating_average DESC' 
    else 'distance ASC' 
end}

當您簡化代碼到這個地步，你發現你有雙「距離ASC」的結果。這是一個錯字還是故意的？

來源

2012-08-10 12:57:05

其實我只是不打算重複'距離ASC'，但你們重構更多......不，它不是一種類型。這個「訂單」是一個可選參數。 – Victor 2012-08-10 13:27:55

好的，進一步簡化:) – 2012-08-10 13:29:23

爲什麼不通常寫第二種方法？ – Victor 2012-08-10 13:30:51

order = {} 
order[:order] = case params[:order] 
when "rating_ascend" then "rating_average ASC" 
when "rating_descend" then "rating_average DESC" 
else "distance ASC" 
end

來源

2012-08-10 12:57:49 sawa

用於記住關於'then'關鍵字的+1。 – 2012-08-10 13:03:20

我比分號更喜歡它。 – sawa 2012-08-10 13:04:09

謝謝。今天學到了一些東西。 – Victor 2012-08-10 13:37:56

你可以像以前的答案，甚至使其更好地只是刪除不需要的情況下

when 'distance' 
    'distance ASC'

因爲else部分將執行相同的。

order = {:order => case params[:order] 
    when 'rating_ascend' 
    'rating_average ASC' 
    when 'rating_descend' 
    'rating_average DESC' 
    else 
    'distance ASC' 
end}

來源

2012-08-10 13:02:27 PradeepKumar

orders = {'rating_ascend' => 'rating_average ASC', 'rating_descend' => 'rating_average DESC', 'distance' => 'distance ASC'} 
Model.order(orders[params[:order]] || 'distance ASC')

來源

2012-08-10 13:23:39 SKR

不錯，謝謝:) – 2012-08-10 13:35:16

謝謝。今天學到了一些東西。 – Victor 2012-08-10 13:38:52

使用 '提取方法' 重構模式：

def determine_order(order) 
    case order 
    when 'rating_ascend' 
    'rating_average ASC' 
    when 'rating_descend' 
    'rating_average DESC' 
    else 
    'distance ASC' 
end 

order = {:order => determine_order(params[:order])}

來源

2012-08-10 13:39:01 Pavling

怎麼樣創建與diferent選項的哈希，並使用取找到的選項，類似：

order = order_hash.fetch(params[:order], {:order => 'distance ASC'}) 

def order_hash 
    { 
    'rating_ascend' => {:order => 'rating_average ASC'}, 
    'rating_descend' => {:order => 'rating_average DESC'}, 
    'distance' => {:order => 'distance ASC'} 
    } 
end

而且由於fetch可以採用默認選項作爲第二參數，因此您可以輕鬆解決e上的其他問題你的情況。

來源

2014-11-07 22:45:07

未選擇重構的情況下

回答

相關問題