Rubyでもっときれいなコード

Question

このコードのスニペットには、よりきれいなバージョンがありますか？

@available_option_types.delete_if {|ot| 
    result = true 
    result = current_user.retailer.id != ot.retailer.id if ot.retailer.present? 
    result 
} unless current_user.has_role? 'admin' 

ありがとうございます！

Answer 1

@available_option_types.delete_if { |ot| 
    ot.retailer.present? ? (current_user.retailer.id != ot.retailer.id) : true 
} unless current_user.has_role? 'admin' 

それとも、モデルにいくつかのロジックを置けばそれもきれいになります：has_manyの

@available_option_types = current_user.retailer.options_types unless current_user.has_role? 'admin' 

Answer 2

unless current_user.has_role? 'admin' 
    @available_option_types.delete_if do |ot| 
    !ot.retailer.present? || 
     (ot.retailer.present? && 
     current_user.retailer.id != ot.retailer.id) 
    end 
end 

Answer 3

これはうまくいくかもしれませんまたは

@available_option_types.select do |ot| 
    ot.retailer.present? && current_user.retailer.id == ot.retailer.id 
end unless current_user.has_role? 'admin' 

Answer 4

@available_option_types.delete_if do |ot| 
    !ot.retailer.present? || current_user.retailer.id != ot.retailer.id 
end unless current_user.has_role? 'admin' 

をoption_types：小売業者があれば

class User 
    def same_retailer_with?(option_type) 
    option_type.retailer.present? ? (self.retailer.id != option_type.retailer.id) : true 
    end 
end 

@available_option_types.delete_if { |ot| current_user.same_retailer_with?(ot) } unless current_user.has_role? 'admin'