switchステートメントのデフォルトでNotImplementedExceptionをスローする

Question

にNotImplementedException()を投げるべきでしょうか？すべての可能な列挙型のケースがある場合は？

Answer 1

数値がの場合、が定義されていなければならず、列挙値に相当する値を探していれば、何か他のものを受け取った、それは間違いなく無効な引数です。

しかし、あなたは文脈を考慮する必要があります。

このメソッドはプライベートで、クラスライブラリまたはアプリケーションのメンバーのみがアクセスできますか？そうであれば、最初に発生してはならないコーディングエラーです。断言し、失敗する。

一方、パブリックまたは保護されたメソッドであり、ライブラリを消費するクライアントがアクセスできる場合は、意味のあるメッセージ（よく知られている例外タイプが望ましい）を間違いなく投げてください。

フレームワークで列挙型が範囲チェックされていないことを覚えておくことは重要です。メソッドがEnvironment.SpecialFolder型のパラメータを必要とすることを指定することができます。 はすべての32ビット整数値を受け入れます。

したがって、要するに、あなたの方法が一般消費の場合、はい、是非、投げてください。一般消費用でない場合はAssertです。

Answer 2

これは実際には特定のプロセスに依存しますが、はい、何かが存在しないと思われる場合はデフォルトで応答するのが良いプロセスです。

Answer 3

これは合理的な選択肢のようです。

個人的には、新しいタイプの例外（おそらくInvalidEnumException）を作成し、サポートチームに意味をなさせる別の名前を付けて、それをスローします。

Answer 4

コードがデフォルトに達していて、期待していなかった場合、これはコード内の何かがあなたの思考のように振る舞わないことを意味するので、私はApplicationExceptionを投げます。

Answer 5

少なくとも、あなたはDebug.Fail()を置くべきでしょう。

メソッドを実行できない場合に備えて、例外をスローする必要があります。しかし、列挙型の値を文字列表現に変換する場合は、代わりに警告文字列を返すことができます。明白な間違いで製品がクラッシュすることはなく、回避策があり、誰もが幸せになるでしょう。

Answer 6

NotImplementedExceptionではなく、ArgumentExceptionである可能性があります。それは本当にあなたがそれを使っている場所に依存します。

Answer 7

実際の使用状況によって異なります。統合の初期段階で例外をスローすると役立ちます。あなたのライブラリーのユーザーはすぐにエラーを知ることができます

Answer 8

何が起こるのでしょうか？どの文脈でswitch文が実行されていますか？そのような状況が起こるべきか？実行時に実動コードで実行する必要がありますか？あなたの単体テストはこの状況をカバーしていますか？もしそうなら、おそらくアサーションが良いでしょう。

Answer 9

まず、既知のケース以外の値が得られた場合、つまりスイッチオーバーされる変数は何を表していますか？次に、実際に起こっていることに合った例外タイプを使用するだけです。

Answer 10

本当に依存しています。

• NotImplementedExceptionは、私のためにtodoマークのようなものです。誰かが後でコードを完成させることを意味します。しかし、私はそれが起こらないはずのデフォルトケースのケースだとは思わない。

• オブジェクトの状態を確認するときは、InvalidOperationExceptionと考えるとよいでしょう。あなたの方法は、既存のケースでのみ機能するように設計されています。

• 入力パラメータを区別する場合は、常にArgumentExceptionが適切です。

• 他の場合、私はNotSupportedExceptionを好む。これは、プラットフォームやバージョンに問題があることをわずかに示しています。そして、コードの互換性のないバージョンは、発生してはならないスイッチのデフォルトのケースが起こったときの問題の真のルートです。