Rails 3 - 複雑なSwitch文/ If文を処理する方法

Question

私は受信メールを取り込み、そのメールを処理するメソッドを構築しています。途中で、電子メールが正常に処理されないようにする多くのことがあります。

コードにはSwitch Statements（case/when/end）とIf文がいっぱいです。私は、これを行うよりスマートでクリーンな方法を学びたいと思います。さらに、エラーを追跡し、最後にエラーを返す電子メールを送信する場所が1つあります。このようなものはレールで可能ですか？

その後

@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 

感謝。上記のようなロジックを書く方法を教えてくれる他のチュートリアルを知っていれば、それは素晴らしいことでしょう。今、私はcase/if文が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ロジックが検証され、レールによって自動的に処理されます。

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

と使用シンボルの代わりの文字列：良い出発点は読む価値もactive record validations guide

あるaction mailer guide

Answer 4

まずですが、私は単純なハッシュとして各エラーメッセージにシンボルを割り当てます。

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メソッドで拡張すると、エラーがあった場合に電子メールを送信できます。

編集：

これはコメントに添付された新しい情報です。

ネストされたifを使用してスイッチする必要はありません。あなたはそれがこのように見えることができます：

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