Rails 3 - 如何處理複雜的開關語句/如果語句

Question

我正在構建一種接收傳入電子郵件並處理電子郵件的方法。一路上有很多事情可能會阻止電子郵件成功處理。錯誤的回覆地址，錯誤的地址，空的郵件正文等。

該代碼充滿了Switch語句（case/when/end）和If語句。我想學習一個更智能，更乾淨的方式來做到這一點。另外，一種可以跟蹤錯誤的方法，並且在最後有一個位置，它通過錯誤向用戶發送電子郵件。像這樣的事情可能與鐵軌？

@error = [] 

Case XXX 
when xxxx 
    if XXXXX 
    else 
    @error = 'You don't have permission to reply to the xxxxx' 
    end 
else 
    @error = 'Unfamilar XXXX' 
end 

然後東西在最後像...

If @errors.count > 0 
    Send the user an email letting them know what went wrong 
else 
do nothing 
end 

感謝這裏的幫助。如果你知道任何其他教程會教我如何編寫像上面這樣聰明的邏輯，那會很棒。現在我有案子/如果陳述要3層深入，很難保持直線。

謝謝

Answer 1

我建議使用例外。從this tutorial開始，然後使用Google，試用和錯誤從那裏開始。

編輯：在更復雜的情況下，例外可能不是正確的工具。您可能需要使用驗證器功能來代替，例如（見其他答案），或者你可以只返回嵌套IFS的早期，而不是，例如：

unless sender_valid? 
    @error = "Sender invalid" 
    return 
end 
unless subject_valid? 
    @error = "Invalid command" 
    return 
end 
# normal no-errors flow continues here... 

Answer 2

當事情是不對的，你可能會引發錯誤。然後在你的方法結束時抓住它。

http://phrogz.net/programmingruby/tut_exceptions.html

爲了使您的代碼的可讀性，並沒有很多的開關，如果/ then語句，你可以創建一個驗證某些方面和從你的主要錯誤檢查方法調用它們單獨的方法。

Answer 3

是否可以將消息映射到模型？那麼所有的if/switch邏輯將是驗證並由rails自動處理。良好的出發點是active record validations guide

另外值得一讀的是action mailer guide

Answer 4

首先，我只是分配一個符號給每個錯誤消息作爲一個簡單的哈希：

ErrorsDescription = { 
    :first => "First error", 
    :second => "Second error", 
    ... 
} 

而且使用的符號，而不是字符串。

然後，你的if和switch語句。基本我不能真正幫助你，因爲我沒有看到你有什麼樣的條件陳述。你在檢查什麼？爲什麼你有3個深度的條件？大概你可以使用if和switch來簡化它 - 所以這是我對這個問題的第一個答案。另一種解決方案可能是寫一個簡單的方法來提高可讀性，所以你可以這樣寫：

if @email.has_wrong_reply_to_address? 
    @errors << :wrong_reply_to_address 
else 
    ... 
end 

而且，@mpapis建議，您可以使用Rails的建立驗證系統，而不是作爲ActiveRecord而是作爲ActiveModel。 Here你有一些例子如何做到這一點，以及它是如何工作的（也看看here）。當然，您可能需要編寫自定義驗證，但它們只是簡單的方法。一旦你做到以上所有的工作，你可以使用：

@email.valid? 

如果不是的話，你把所有的錯誤在哈希：

@email.errors 

就像普通ActiveRecord對象。

然後，您可以用send_error_email方法擴展您的Emial類，該方法在發生錯誤時發送電子郵件。

編輯：

這是關於您在評論中附加的新信息。

您不必使用嵌套的ifs並在此切換。你可以把它看起來像這樣：

def is_this_email_valid? 
    if !email_from_user_in_system? 
    @errors << :user_not_in_system 
    return false 
    end 
    if comment_not_exists? 
    @errors << :comment_not_exists 
    return false 
    end 
    if user_cannot_comment_here? 
    @errors << :permision_error 
    return false 
    end 
    ... 
    true 
end 

然後你可以使用它：

if !@email.is_this_email_valid? 
    @email.send_error_mail 
end