Найкращий виняток для недійсного аргументу загального типу

Я зараз пишу код для UnconstrainedMelody, який має загальні методи, що стосуються перерахунків.

Тепер у мене є статичний клас із купою методів, які призначені лише для перерахувань "прапорів". Я не можу додати це як обмеження ... тому можливо, що вони будуть викликані і з іншими типами перерахунків. У такому випадку я хотів би кинути виняток, але я не впевнений, який саме кинути.

Просто щоб зробити це конкретніше, якщо у мене є щось подібне:

// 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
}

Який найкращий виняток кинути? ArgumentExceptionзвучить логічно, але це аргумент типу, а не звичайний аргумент, який може легко заплутати речі. Чи варто ввести власний TypeArgumentExceptionклас? Використовувати InvalidOperationException? NotSupportedException? Щось іще?

Я б краще не створив для цього власного винятку, якщо явно це не правильно.

NotSupportedException звучить так, як це прямо підходить, але в документації чітко зазначено, що його слід використовувати для інших цілей. З зауважень класу MSDN:

Існують методи, які не підтримуються в базовому класі, сподіваючись, що ці методи будуть реалізовані замість похідних класів. Похідний клас може реалізувати лише підмножину методів із базового класу та викидати NotSupportedException для непідтримуваних методів.

Звичайно, існує спосіб, який NotSupportedException, очевидно, досить хороший, особливо з огляду на його здоровий глузд. Сказавши це, я не впевнений, чи правильно це.

Враховуючи мету Нестриманої мелодії ...

Є різні корисні речі, які можна зробити за допомогою загальних методів / класів, де є обмеження типу "T: enum" або "T: delegate" - але, на жаль, вони заборонені в C #.

Ця утиліта працює над заборонами використання ildasm / ilasm ...

... здається, що нове Exceptionможе бути в порядку, незважаючи на високий тягар доказів, з яким нам справедливо доводиться зустрічатися, перш ніж створювати замовлення Exceptions. Щось подібне InvalidTypeParameterExceptionможе бути корисним у всій бібліотеці (а може й ні - це, безумовно, кращий випадок, правда?).

Чи повинні клієнти мати можливість відрізняти це від виключень BCL? Коли клієнт може випадково зателефонувати цьому за допомогою ванілі enum? Як би ви відповіли на запитання, що поставлені прийнятою відповіддю на те, Які фактори слід враховувати при написанні спеціального класу винятків?

Я б уникнув NotSupportedException. Цей виняток використовується в рамках, коли метод не реалізований і є властивість, що вказує на те, що цей тип операції не підтримується. Тут не підходить

• Безсоромне посилання: http://blogs.msdn.com/jaredpar/archive/2008/12/12/notimplementedexception-vs-notsupportedexception.aspx

Я думаю, що InvalidOperationException - це найбільш підходящий виняток, який можна сюди кинути.

Загальне програмування не повинно запускати час виконання для недійсних параметрів типу. Він не повинен компілювати, у вас має бути час виконання компіляції. Я не знаю, що IsFlag<T>()містить, але, можливо, ви можете перетворити це на виконання часу компіляції, наприклад, намагаючись створити тип, який можна створити лише за допомогою "прапорів". Можливо, traitsклас може допомогти.

Оновлення

Якщо ви повинні кинути, я би проголосував за InvalidOperationException. Аргументація полягає в тому, що загальні типи мають параметри, а помилки, пов'язані з (методом) параметрами, зосереджені навколо ієрархії ArgumentException. Однак у рекомендації щодо ArgumentException зазначено, що

якщо відмова не включає самих аргументів, слід використовувати InvalidOperationException.

У цьому методі є хоча б один стрибок віри рекомендації щодо параметрів також слід застосовувати до загальних параметрів, але нічого кращого в ієрархії системи SystemException немає.

Я б використовував NotSupportedException як саме те, що ви говорите. Інші перерахунки, ніж специфічні, не підтримуються . Це, звичайно, буде викладено чіткіше у повідомленні про виключення.

Я б пішов з NotSupportedException. Хоча це ArgumentExceptionвиглядає чудово, але насправді очікується, коли аргумент, переданий методу, неприйнятний. Аргумент типу - визначальна характеристика для фактичного методу, який ви хочете викликати, а не реальний "аргумент".InvalidOperationExceptionви повинні бути кинуті, коли операція, яку ви виконуєте, може бути дійсною в деяких випадках, але для конкретної ситуації це неприпустимо.

NotSupportedExceptionкидається, коли операція по суті не підтримується. Наприклад, при реалізації інтерфейсу, коли певний член не має сенсу для класу. Це схоже на подібну ситуацію.

Очевидно, Microsoft використовує ArgumentExceptionдля цього, як показано на прикладі Expression.Lambda <> , Enum.TryParse <> або Marshal.GetDelegateForFunctionPointer <> у розділі Винятки. Я не зміг знайти жодного прикладу, який би вказував інакше (незважаючи на пошук локального довідкового джерела для TDelegateта TEnum).

Отже, я вважаю, що можна припустити, що принаймні в коді Microsoft це звичайна практика використання ArgumentExceptionдля недійсних аргументів загального типу, окрім основних змінних. Враховуючи, що опис винятку в документах не розмежовує їх, також це не так вже й велика розтяжність.

Сподіваємось, це вирішує питання питання раз і назавжди.

Я йду з NotSupportedExpcetion.

Викидання на замовлення виключення завжди слід робити в будь-якому випадку, коли це сумнівно. Спеціальний виняток завжди працюватиме, незалежно від потреб користувачів API. Розробник може зловити будь-який тип винятку, якщо він не піклується, але якщо розробнику потрібна спеціальна обробка, він буде SOL.

Як щодо наслідування від NotSupportedException. Хоча я погоджуюся з @Mehrdad, що це має найбільше сенс, я чую вашу думку, що це, здається, не ідеально підходить. Тож успадковуйте від NotSupportedException, і таким чином люди, що кодують ваш API, можуть все-таки зловити NotSupportedException.

Я завжди насторожено пишу власні винятки, виключно на тій підставі, що вони не завжди чітко зафіксовані та створюють плутанину, якщо не названі правильно.

У цьому випадку я б кинув ArgumentException за помилку перевірки прапорів. Все дійсно до уподобань. Деякі стандарти кодування, які я бачив, йдуть далеко до того, щоб визначити, які типи винятків слід кидати в таких сценаріях.

Якщо користувач намагався передати щось, що не було перерахуванням, я б кинув InvalidOperationException.

Редагувати:

Інші піднімають цікавий момент, що це не підтримується. Моє єдине занепокоєння щодо NotSupportedException - це те, що, як правило, це винятки, які викидаються, коли "темна матерія" була введена в систему, або, кажучи іншим способом, "Цей метод повинен зайти в систему на цьому інтерфейсі, але ми виграли не вмикайте його, поки версія 2.4 "

Я також бачив, що NotSupportedExceptions кидається як виняток з ліцензування "Ви використовуєте безкоштовну версію цього програмного забезпечення, ця функція не підтримується".

Редагувати 2:

Інший можливий варіант:

System.ComponentModel.InvalidEnumArgumentException  

Виняток, викинутий при використанні недійсних аргументів, які є перелічниками.

Я також проголосував би за "InvalidOperationException". Я зробив (неповну) блок-схему в .NET виключення керівних принципів, заснованих на Framework Design Guidelines 2nd Ed. деякий час назад, якщо хтось зацікавлений.