如何正確重構代碼：

Question

我創造了我的軌道類似的模型application.Firstly我寫了這樣的事情：

- if BonusLikePolicy.new(current_user, bonus).create? 
     = link_to bonus_likes_path(bonus), method: :delete, 
       data: { remote: true, behavior: "fragments" } do 
     span.dislike 
    - else 
     = link_to bonus_likes_path(bonus), method: :post, 
       data: { remote: true, behavior: "fragments" } do 
     span.like 

它工作正常，但是當你看到這裏的一對夫婦重複。所以我需要使其乾燥，重寫：

= bonus.link_to_like(bonus, current_user) 

，並創建方法獎金：

def link_to_like(bonus, user) 
    options = { class: "like", method: :post } 
    options = { class: "dislike", method: :delete } unless BonusLikePolicy.new(user, bonus).create? 

    h.link_to(
     h.tag(:span, class: options[:class]), h.bonus_likes_path(bonus), 
     method: options[:method], data: { remote: true, behavior: "fragments" } 
    ) 
    end 

我有一個錯誤，未定義的方法破壞了無類。也許我錯過了一些東西，但找不到它。請幫助:)

UPD：

而且我跨度圖標沒有正確顯示（這意味着類，我通過在h.tag也未通過）

NoMethodError (undefined method `destroy' for nil:NilClass): 


app/controllers/likes_controller.rb:12:in `destroy' 

Answer 1

按照邏輯你在你工作的代碼片段

更換unless與if

options = { class: "dislike", method: :delete } if BonusLikePolicy.new(user, bonus).create? 

Answer 2

我不喜歡重寫選項。在我看來更好做某事像

步驟1.

options = (!BonusLikePolicy.new(user, bonus).create? ? { class: "dislike", method: :delete } : { class: "like", method: :post }) 

步驟2. 使私有方法

def can_not_create_like? 
    !BonusLikePolicy.new(user, bonus).create? 
end 

步驟3.

options = (can_not_create_like? ? { class: "dislike", method: :delete } : { class: "like", method: :post })