ruby - この配列の包含/デフォルトのフォールバックコードをより簡潔に書く方法は?

Question

Railsコントローラーでこれをかなりの回数行ってきたので、（可能であれば）より良い書き方を見つけたいと思っています。基本的に、いくつかのオプションへの入力を検証し、入力がどのオプションにも一致しない場合はデフォルト値にフォールバックしています。

valid_options = %w(most_active most_recent most_popular)
@my_param = valid_options.include?(params[:my_param]) ? params[:my_param] : 'most_recent'

Answer 1

配列の代わりにハッシュを使用すると、より高速でクリーンになります。また、デフォルトはであるため"most_recent"、"most_recent"invalid_optionsは冗長です。取り除いたほうがいいです。

filter_options =
Hash.new("most_recent")
.merge("most_popular" => "most_popular", "most_active" => "most_active")

@my_param = filter_options[params[:my_param]]

Answer 2

私もハッシュルートに行きます。

これは想像できるでしょう：

Hash[valid_options.zip valid_options].fetch(params[:my_param], "most_recent")

Answer 3

少しこじつけ。

valid_options = %w(most_active most_recent most_popular)
(valid_options & [params[:my_param]]).first || 'most_recent'

Answer 4

以下はいかがですか。

valid_options = %w(most_active most_recent most_popular)
valid_options.detect(proc{'default_value'}){|i| i == params[:my_param] }

別のもの：

valid_options = %w(most_active most_recent most_popular)
valid_options.dup.delete(params[:my_param]) { "default" }