例外の共通部分を一緒に処理する

Question

私は現在リファクタリングしようとしているいくつかのコードを持っています。大きなセットの例外には、すべての例外の共通コードと、特定の例外タイプごとに個別に処理する必要のある特定のコードがあります。私は各catchブロックの共通部分を取り除く方法を理解しようとしています。 1つのアイデアは、このようにすることです：

try { 
    /* Stuff that may fail */ 
} catch(const std::exception &) { 
    /* do common part here */ 
    try { throw; } 
    catch(const exception1 &) { 
    /* do stuff for exception1 here */ 
    } 
    catch(const exception2 &) { 
    /* do stuff for exception2 here */ 
    } 
} 

しかし、これは私に多少醜い印象を与えます。

この一般的なロジックを除外するには良い方法がありますか、実際にはこの試みを完全に避ける理由はありますか？

Answer 1

、try/catchはコードだけでまばらに表示されます。問題は、多くの場合、catch句で行われたアクションは、早期返却の場合にも行われるべきであるということです。

イディオムC++では、ほとんどの作業を取り除くcatch節でクリーンアップが必要な状況を避けるためにRAIIを広範囲に使用しています。

あなたのパターンはそれほど悪くはありませんそれ自体はです。しかし、この共通のものはおそらく自動的に処理される可能性があります。

私が遭遇したすべてのコードベースで、ほんの数回だけ私はcatch句の本物の使用を見ましたが、クラッチとして使用しないでください。

Answer 2

例外の大きなセットは、関数またはメソッドに共通のコードを移動

いくつかの共通コード

を有しています。各 catchからメソッドを呼び出します。他の複製を削除するのと同じように。 try .. catchの存在は違いはありません。

---

編集

しかし、あなたは「例外の大集合」について本当に懸念している場合は、本当の問題は、例外の大規模なセットを持っていることかもしれません。異なる例外はなぜ異なった取り扱いを必要とするのですか？例外は実際には例外のイベントにのみ使用されていますか？

Answer 3

私は実際には共通コードを関数に移動するだけです。

ここ

は別の考えである。一般的には

try { 
    /* Stuff that may fail */ 
} catch(const std::exception & e) { 
    /* do common part here */ 
    if (const exception1 * e1 = dynamic_cast<const exception1*>(&e) { 
    /* do stuff for exception1 here */ 
    } else if (const exception2 * e2 = dynamic_cast<const exception2*>(&e) { 
    /* do stuff for exception2 here */ 
    } else 
    throw; 
}