ruby-on-rails - Ruby on Rails 3でこの式を短くする必要があります

Question

コードをよりコンパクトにする必要があります。次のコードがあります。

params[:investor][:profit] = params[:investor][:profit].nil? ? nil : params[:investor][:profit].gsub(/\D/, '')

基本的に何をするか - params からの利益値をフォーマットして数字のみを含むようにし、nil の場合は nil のままにします...短くする方法はありますか?

score 4 · Accepted Answer

次のように少し締めることができます。

params[:investor][:profit].gsub!(/\D/, '') unless params[:investor][:profit].nil?

score 4 · Accepted Answer

params[:investor][:profit].try(:gsub!, /\D/, '')

score 2 · Accepted Answer

2

うーん

p = params[:investor][:profit]
p = p.nil? ? nil : p.gsub(/\D/,'')

于 2012-04-21T06:51:34.947 に答える

score 1 · Accepted Answer

params[:investor][:profit] &&= params[:investor][:profit].gsub(/\D/, '')

の値params[:investor][:profit]が nil の場合、これはと評価されnil && ...ます。nil は false であるため、nil のままになります。それ以外の場合はgsub.

try別の解決策で言及されている解決策に頭を悩ませていると思います。どちらを選択するかは、個人の好みによるものです。&&=Railsの便利なメソッドではなくルビーであり、tryメソッドのパラメーターで本当にやりたいことを「暗号化」する必要がないため、このソリューションが気に入っています。

score 1 · Accepted Answer

params[:investor][:profit].gsub!(/\D/, '') if params[:investor][:profit]

または私がほとんどいつも使っているもの：

params[:investor][:profit].gsub!(/\D/, '') rescue nil

score 0 · Accepted Answer

0

prof = params[:investor][:profit]
prof.gsub!(/\D/,'') if prof

于 2012-04-21T14:18:39.920 に答える

6 に答える 6