這是一個錯誤處理反模式？

Question

一個同事最近告訴我，這是一個反模式（record是ActiveRecord）：

begin 
    record.save! 
    do_something_else 
    rescue => e 
    puts "Unable to save" 
    end 

...我應該做這個：

if record.save 
    do_something_else 
    else 
    puts "Unable to save" 
    end 

他的理由是我m使用流量控制的異常（我同意這很糟糕），但我相信這是一種典型的錯誤處理模式。

想法？

Answer 1

這是一個反模式，因爲有更好的方法來檢查記錄的有效性，而不是運行save!（如果它無效，會產生錯誤）。

這可能是你的同事在越來越：

if record.save 
    do_something_else 
    else 
    puts record.errors.full_messages 
    end 

你看，這裏存在，因爲有較少的迂迴的方式來做到這一點使用誤差控制流的只是沒有好處。

您也可以獨立於保存嘗試運行驗證。當調用record.valid?時，將填充（字符串）errors.full_messages數組。 record.save內部調用record.valid?。

if record.valid? 
    record.save # or save!; by this point you can assume the record is valid 
else 
    puts record.errors.full_messages 
end