Pengecualian terbaik untuk argumen tipe umum yang tidak valid

Saat ini saya sedang menulis beberapa kode untuk UnconstrainedMelody yang memiliki metode umum untuk dilakukan dengan enum.

Sekarang, saya memiliki kelas statis dengan banyak metode yang hanya dimaksudkan untuk digunakan dengan enum "flags". Saya tidak dapat menambahkan ini sebagai pembatas ... jadi mungkin saja mereka akan dipanggil dengan jenis enum lain juga. Dalam hal ini, saya ingin memberikan pengecualian, tetapi saya tidak yakin mana yang harus dibuang.

Hanya untuk membuat ini menjadi konkret, jika saya memiliki sesuatu seperti ini:

// Returns a value with all bits set by any values
public static T GetBitMask<T>() where T : struct, IEnumConstraint
{
    if (!IsFlags<T>()) // This method doesn't throw
    {
        throw new ???
    }
    // Normal work here
}

Apa pengecualian terbaik untuk dilempar? ArgumentExceptionterdengar logis, tapi ini adalah tipe argumen daripada argumen normal, yang dapat dengan mudah membingungkan banyak hal. Haruskah saya memperkenalkan TypeArgumentExceptionkelas saya sendiri ? Gunakan InvalidOperationException? NotSupportedException? Ada yang lain?

Saya lebih suka tidak membuat pengecualian saya sendiri untuk ini kecuali hal ini jelas merupakan hal yang benar untuk dilakukan.

NotSupportedException kedengarannya cocok, tetapi dokumentasinya dengan jelas menyatakan bahwa itu harus digunakan untuk tujuan yang berbeda. Dari catatan kelas MSDN:

Ada metode yang tidak didukung di kelas dasar, dengan harapan bahwa metode ini akan diterapkan di kelas turunan. Kelas turunan mungkin hanya mengimplementasikan subset metode dari kelas dasar, dan melontarkan NotSupportedException untuk metode yang tidak didukung.

Tentu saja, ada cara yang NotSupportedExceptionjelas-jelas cukup bagus, terutama mengingat arti yang masuk akal. Karena itu, saya tidak yakin apakah itu tepat.

Mengingat tujuan Melodi Tanpa Batas ...

Ada berbagai hal berguna yang bisa dilakukan dengan metode / kelas umum di mana ada batasan tipe "T: enum" atau "T: delegate" - tapi sayangnya, hal itu dilarang di C #.

Pustaka utilitas ini bekerja di sekitar larangan menggunakan ildasm / ilasm ...

... sepertinya yang baru Exceptionmungkin beres meskipun beban pembuktiannya tinggi yang harus kita penuhi sebelum membuat kebiasaan Exceptions. Sesuatu seperti InvalidTypeParameterExceptionmungkin berguna di seluruh perpustakaan (atau mungkin tidak - ini pasti kasus tepi, kan?).

Apakah klien harus dapat membedakan ini dari Pengecualian BCL? Kapan klien tidak sengaja memanggil ini menggunakan vanilla enum? Bagaimana Anda akan menjawab pertanyaan yang diajukan oleh jawaban yang diterima untuk Faktor apa yang harus dipertimbangkan saat menulis kelas pengecualian khusus?

Saya akan menghindari NotSupportedException. Pengecualian ini digunakan dalam kerangka kerja di mana metode tidak diimplementasikan dan terdapat properti yang menunjukkan bahwa jenis operasi ini tidak didukung. Tidak muat di sini

• Referensi Diri Tak Tahu Malu: http://blogs.msdn.com/jaredpar/archive/2008/12/12/notimplementedexception-vs-notsupportedexception.aspx

Menurut saya, InvalidOperationException adalah pengecualian paling tepat yang dapat Anda berikan di sini.

Pemrograman generik tidak boleh melempar pada waktu proses untuk parameter tipe yang tidak valid. Seharusnya tidak dikompilasi, Anda harus memiliki penegakan waktu kompilasi. Saya tidak tahu IsFlag<T>()isinya, tapi mungkin Anda bisa mengubahnya menjadi penegakan waktu kompilasi, seperti mencoba membuat tipe yang hanya mungkin dibuat dengan 'flags'. Mungkin traitskelas bisa membantu.

Memperbarui

Jika Anda harus melempar, saya akan memilih InvalidOperationException. Alasannya adalah bahwa tipe generik memiliki parameter dan kesalahan yang terkait dengan parameter (metode) yang berpusat di sekitar hierarki ArgumentException. Namun, rekomendasi pada ArgumentException menyatakan itu

jika kegagalan tidak melibatkan argumen itu sendiri, maka InvalidOperationException harus digunakan.

Setidaknya ada satu lompatan keyakinan di sana, bahwa rekomendasi parameter metode juga akan diterapkan ke parameter umum , tetapi tidak ada yang lebih baik dalam hierachy imho SystemException.

Saya akan menggunakan NotSupportedException karena itulah yang Anda katakan. Enum selain yang spesifik tidak didukung . Ini tentu saja akan dinyatakan dengan lebih jelas dalam pesan pengecualian.

Saya akan pergi dengan NotSupportedException. Meskipun ArgumentExceptionterlihat bagus, itu benar-benar diharapkan ketika argumen yang diteruskan ke suatu metode tidak dapat diterima. Argumen tipe adalah karakteristik yang menentukan untuk metode sebenarnya yang ingin Anda panggil, bukan "argumen" yang sebenarnya. InvalidOperationExceptionharus dibuang ketika operasi yang Anda lakukan bisa valid dalam beberapa kasus, tetapi untuk situasi tertentu, itu tidak dapat diterima.

NotSupportedExceptiondilempar saat operasi tidak didukung secara inheren. Misalnya, saat mengimplementasikan antarmuka di mana anggota tertentu tidak masuk akal untuk sebuah kelas. Ini terlihat seperti situasi yang serupa.

Rupanya, Microsoft menggunakan ArgumentExceptionuntuk itu, seperti yang ditunjukkan pada contoh Expression.Lambda <> , Enum.TryParse <> atau Marshal.GetDelegateForFunctionPointer <> di bagian Pengecualian. Saya tidak dapat menemukan contoh yang menunjukkan sebaliknya, baik (meskipun mencari sumber referensi lokal untuk TDelegatedan TEnum).

Jadi, saya pikir aman untuk mengasumsikan bahwa setidaknya dalam kode Microsoft itu adalah praktik umum yang digunakan ArgumentExceptionuntuk argumen tipe generik yang tidak valid selain dari yang variabel dasar. Mengingat bahwa deskripsi pengecualian di dokumen tidak membedakan keduanya, itu juga tidak terlalu berlebihan.

Mudah-mudahan ini memutuskan pertanyaan itu sekali dan untuk selamanya.

Saya akan menggunakan NotSupportedExpcetion.

Melempar pengecualian yang dibuat khusus harus selalu dilakukan dalam kasus apa pun yang meragukan. Pengecualian khusus akan selalu berfungsi, apa pun kebutuhan pengguna API. Pengembang dapat menangkap salah satu tipe pengecualian jika dia tidak peduli, tetapi jika pengembang membutuhkan penanganan khusus dia akan menjadi SOL.

Bagaimana jika mewarisi dari NotSupportedException. Meskipun saya setuju dengan @Mehrdad bahwa ini paling masuk akal, saya dengar pendapat Anda bahwa ini sepertinya tidak sesuai dengan sempurna. Jadi, mewarisi dari NotSupportedException, dan dengan begitu orang yang melakukan pengkodean terhadap API Anda masih bisa menangkap NotSupportedException.

Saya selalu berhati-hati dalam menulis pengecualian khusus, semata-mata karena pengecualian tersebut tidak selalu didokumentasikan dengan jelas dan menyebabkan kebingungan jika tidak dinamai dengan benar.

Dalam hal ini saya akan melempar ArgumentException untuk kegagalan pemeriksaan bendera. Semuanya tergantung pada preferensi. Beberapa standar pengkodean yang telah saya lihat sejauh untuk menentukan jenis pengecualian mana yang harus dilemparkan dalam skenario seperti ini.

Jika pengguna mencoba untuk mengirimkan sesuatu yang bukan enum maka saya akan melempar InvalidOperationException.

Edit:

Yang lain mengangkat poin menarik bahwa ini tidak didukung. Satu-satunya perhatian saya dengan NotSupportedException adalah bahwa secara umum itu adalah pengecualian yang dilemparkan ketika "materi gelap" telah dimasukkan ke sistem, atau dengan kata lain, "Metode ini harus masuk ke sistem pada antarmuka ini, tetapi kami menang hidupkan sampai versi 2.4 "

Saya juga melihat NotSupportedExceptions dilemparkan sebagai pengecualian lisensi "Anda menjalankan versi gratis dari perangkat lunak ini, fungsi ini tidak didukung".

Edit 2:

Kemungkinan lain:

System.ComponentModel.InvalidEnumArgumentException  

Pengecualian dilontarkan saat menggunakan argumen tidak valid yang merupakan enumerator.

Saya juga akan memilih InvalidOperationException. Saya melakukan diagram alur (tidak lengkap) pada pedoman melempar pengecualian NET berdasarkan pada Pedoman Desain Kerangka 2nd Ed. beberapa waktu yang lalu jika ada yang tertarik.