在ActionMailer模型動作中發送多個郵件方法是好的做法，還是應該將這個邏輯放在Mailer動作之外？

Question

哪一個會是首選？

class Mailer < ActionMailer::Base 

    # this? 
    def bid_end_notify_failed_bidders(job) 
    @job = job 
    bidders = @job.bidders 
    bidders.delete(@job.winner) 
    bidders.each do |t| 
     mail(:to => t.email, ....) 
    end 
    end 

    # or this? 
    def notify_failed_bid(bidder) 
    mail(:to => bidder ...) 
    end 


end 

Answer 1

我建議的方法＃2，這是因爲你要在你發送電子郵件的方式靈活。

使用它的最普通的方法是：

• 聲明實例
• 宣佈你從實例需要
• 通過實例對梅勒:: Base的方法
• 交付任何關係

實施例：

用戶= User.first

bets_won = user.bets_won

Mailer.send_congrats（用戶，bets_won）.deliver！

現在它只會使用它所服務的數據，所以沒有邏輯必須由梅勒類 來完成也這種方法將允許您輕鬆地重用該方法。