判斷方法的語法

Question

我做了這些方法的類：

def promotion_available? 
    promotion.present? 
end 

def promotion 
    @promotion ||= PromotionUser.user(@user.id).available.first 
end 

然後，同事取出promotion方法，並以這種方式改變了謂詞promotion_available?：

def promotion_available? 
    @promotion ||= PromotionUser.user(@user.id).available.first 
end 

• 我可以直接在謂詞方法上設置一個實例變量？
• 謂詞方法是否可以返回整個對象而不是true/false（我認爲不是，但我的同事說的相反）？

Answer 1

我可以直接在謂詞方法上設置實例變量嗎？

是的，沒有什麼不對。我經常這樣做。

一個判斷方法可以返回一個完整的對象，而不是TRE /假（我認爲沒有，但我的同事說，對面）

是的，這也是常見的。但它只適用於你使用慣用的紅寶石真實性檢查。

if obj.promotion_available? # GOOD 
if obj.promotion_available? == true # BAD! 

記住，只有nil和false紅寶石是falsey值。其他一切都是真實的。這就是爲什麼返回一個對象或零工作有點像返回真或假。

Answer 2

雖然變「作品」，它引入了一個副作用：你必須調用promotion_available?之前，你可以實際使用的promotion（假設有一個promotion方法）：

your_object.promotion    #=> nil 

your_object.promotion_available? 
your_object.promotion    #=> <#PromotionUser ...> 

你原來的代碼沒有按沒有這種依賴性。

IMO它會更直觀地保留promotion方法並刪除promotion_available?。你可以寫：

if obj.promotion 
    # with promotion 
else 
    # without promotion 
end