クラス内で例外を止めてリダイレクト処理までしてしまう変更は不適切だと思います。クラスを利用するコード側でメール送信失敗時の例外処理を記述することができなくなります。向けの変更も不適切だと思います。クラスを利用することが想定されるクラス(例えば、アクションクラス)よりも上位のクラスで例外をキャッチし、フォワード(もしくはリダイレクト)の処理を行うべきではないでしょうか。タスク内からの呼び出し時に関してはエラー画面とは別の考慮が必要ですが、ログにエラーが記録されるのみでタスク内からメール送信処理の失敗が検知できないような変更よりは、仮にうまく共通化ができなかったとしても個別タスク内できちんと例外処理を行うように修正するほうがよいと思います。例外をキャッチしている処理は取消し、適切な場所で例外がキャッチできなくなる問題を改善する。