Beste Ausnahme für ein ungültiges generisches Typargument

Ich schreibe gerade Code für UnconstrainedMelody, der generische Methoden für Aufzählungen enthält.

Jetzt habe ich eine statische Klasse mit einer Reihe von Methoden, die nur für "Flags" -Aufzählungen verwendet werden sollen. Ich kann dies nicht als Einschränkung hinzufügen ... daher ist es möglich, dass sie auch mit anderen Aufzählungstypen aufgerufen werden. In diesem Fall würde ich gerne eine Ausnahme auslösen, bin mir aber nicht sicher, welche ich auslösen soll.

Nur um dies konkret zu machen, wenn ich so etwas habe:

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

Was ist die beste Ausnahme zu werfen? ArgumentExceptionklingt logisch, aber es ist eine Art eher Argument als ein normales Argument, das könnte leicht verwirren Dinge. Soll ich meine eigene TypeArgumentExceptionKlasse vorstellen ? Verwenden InvalidOperationException? NotSupportedException? Noch etwas?

Ich würde lieber keine eigene Ausnahme dafür erstellen, es sei denn, es ist eindeutig das Richtige.

NotSupportedException klingt so, als würde es eindeutig passen, aber die Dokumentation besagt eindeutig, dass es für einen anderen Zweck verwendet werden sollte. Aus der MSDN-Klasse Anmerkungen:

Es gibt Methoden, die in der Basisklasse nicht unterstützt werden, mit der Erwartung, dass diese Methoden stattdessen in den abgeleiteten Klassen implementiert werden. Die abgeleitete Klasse implementiert möglicherweise nur eine Teilmenge der Methoden aus der Basisklasse und löst NotSupportedException für die nicht unterstützten Methoden aus.

Natürlich gibt es eine Art und Weise, die NotSupportedExceptionoffensichtlich gut genug ist, insbesondere angesichts ihrer Bedeutung für den gesunden Menschenverstand. Trotzdem bin ich mir nicht sicher, ob es genau richtig ist.

Angesichts des Zwecks der unbeschränkten Melodie ...

Es gibt verschiedene nützliche Dinge, die mit generischen Methoden / Klassen ausgeführt werden können, bei denen die Typbeschränkung "T: enum" oder "T: delegate" besteht - diese sind jedoch in C # leider verboten.

Diese Dienstprogrammbibliothek umgeht die Verbote mit ildasm / ilasm ...

... es scheint, als wäre ein neues Exceptionin Ordnung, trotz der hohen Beweislast, die wir zu Recht erfüllen müssen, bevor wir einen Brauch erstellen Exceptions. So etwas InvalidTypeParameterExceptionkönnte in der gesamten Bibliothek nützlich sein (oder auch nicht - dies ist sicherlich ein Randfall, oder?).

Müssen Kunden in der Lage sein, dies von BCL-Ausnahmen zu unterscheiden? Wann könnte ein Kunde dies versehentlich mit einer Vanille aufrufen enum? Wie würden Sie die Fragen beantworten, die sich aus der akzeptierten Antwort auf Welche Faktoren sollten beim Schreiben einer benutzerdefinierten Ausnahmeklasse berücksichtigt werden?

Ich würde NotSupportedException vermeiden. Diese Ausnahme wird in dem Framework verwendet, in dem eine Methode nicht implementiert ist und eine Eigenschaft angibt, dass diese Art von Operation nicht unterstützt wird. Es passt hier nicht

• Schamloses Selbst Referenz: http://blogs.msdn.com/jaredpar/archive/2008/12/12/notimplementedexception-vs-notsupportedexception.aspx

Ich denke, InvalidOperationException ist die am besten geeignete Ausnahme, die Sie hier auslösen können.

Generische Programmierung sollte zur Laufzeit keine ungültigen Typparameter auslösen. Es sollte nicht kompiliert werden, Sie sollten eine Durchsetzung der Kompilierungszeit haben. Ich weiß nicht, was es IsFlag<T>()enthält, aber vielleicht können Sie daraus eine Durchsetzung der Kompilierungszeit machen, z. B. den Versuch, einen Typ zu erstellen, der nur mit 'Flags' erstellt werden kann. Vielleicht kann eine traitsKlasse helfen.

Aktualisieren

Wenn Sie werfen müssen , würde ich für InvalidOperationException stimmen. Der Grund dafür ist, dass generische Typen Parameter haben und Fehler in Bezug auf (Methoden-) Parameter um die ArgumentException-Hierarchie zentriert sind. Die Empfehlung zu ArgumentException besagt jedoch, dass

Wenn der Fehler nicht die Argumente selbst betrifft, sollte InvalidOperationException verwendet werden.

Es gibt mindestens einen Vertrauenssprung darin, dass Empfehlungen für Methodenparameter auch auf generische Parameter angewendet werden sollen, aber es gibt nichts Besseres in der SystemException-Hierarchie imho.

Ich würde NotSupportedException verwenden, da dies das ist, was Sie sagen. Andere als die spezifischen Aufzählungen werden nicht unterstützt . Dies würde natürlich in der Ausnahmemeldung deutlicher angegeben.

Ich würde mitgehen NotSupportedException. Während ArgumentExceptionsieht gut aus , es ist wirklich erwartet , wenn ein Argument an eine Methode übergeben nicht akzeptabel ist. Ein Typargument ist ein definierendes Merkmal für die tatsächliche Methode, die Sie aufrufen möchten, kein echtes "Argument". InvalidOperationExceptionsollte ausgelöst werden, wenn die von Ihnen ausgeführte Operation in einigen Fällen gültig sein kann, aber für die jeweilige Situation nicht akzeptabel ist.

NotSupportedExceptionwird ausgelöst, wenn eine Operation von Natur aus nicht unterstützt wird. Zum Beispiel bei der Implementierung einer Schnittstelle, bei der ein bestimmtes Mitglied für eine Klasse keinen Sinn ergibt. Dies sieht nach einer ähnlichen Situation aus.

Anscheinend verwendet Microsoft ArgumentExceptiondafür, wie am Beispiel von Expression.Lambda <> , Enum.TryParse <> oder Marshal.GetDelegateForFunctionPointer <> im Abschnitt Ausnahmen gezeigt. Ich konnte auch kein anderes Beispiel finden (obwohl ich in der lokalen Referenzquelle nach TDelegateund gesucht habe TEnum).

Ich denke also, es ist sicher anzunehmen, dass es zumindest in Microsoft-Code eine gängige Praxis ist, ArgumentExceptionungültige generische Typargumente neben einfachen variablen Argumenten zu verwenden. Da die Ausnahmebeschreibung in Dokumenten nicht zwischen diesen unterscheidet, ist sie auch nicht allzu langwierig.

Hoffentlich entscheidet es ein für alle Mal über die Frage.

Ich würde mit NotSupportedExpcetion gehen.

Das Auslösen einer benutzerdefinierten Ausnahme sollte immer dann erfolgen, wenn dies fraglich ist. Eine benutzerdefinierte Ausnahme funktioniert immer, unabhängig von den Anforderungen der API-Benutzer. Der Entwickler könnte beide Ausnahmetypen abfangen, wenn es ihn nicht interessiert, aber wenn der Entwickler eine spezielle Behandlung benötigt, ist er SOL.

Wie wäre es mit dem Erben von NotSupportedException? Obwohl ich @Mehrdad zustimme, dass es am sinnvollsten ist, höre ich Ihren Standpunkt, dass es nicht perfekt zu passen scheint. Erben Sie also von NotSupportedException, und auf diese Weise können Personen, die gegen Ihre API codieren, weiterhin eine NotSupportedException abfangen.

Ich bin immer vorsichtig, wenn ich benutzerdefinierte Ausnahmen schreibe, nur weil sie nicht immer klar dokumentiert sind und Verwirrung stiften, wenn sie nicht richtig benannt werden.

In diesem Fall würde ich eine ArgumentException für den Fehler der Flagsprüfung auslösen. Es hängt alles von den Vorlieben ab. Einige Codierungsstandards, die ich gesehen habe, gehen so weit, zu definieren, welche Arten von Ausnahmen in solchen Szenarien ausgelöst werden sollen.

Wenn der Benutzer versuchen würde, etwas zu übergeben, das keine Aufzählung ist, würde ich eine InvalidOperationException auslösen.

Bearbeiten:

Die anderen sprechen einen interessanten Punkt an, der nicht unterstützt wird. Meine einzige Sorge bei einer NotSupportedException ist, dass dies im Allgemeinen die Ausnahmen sind, die ausgelöst werden, wenn "dunkle Materie" in das System eingeführt wurde, oder anders ausgedrückt: "Diese Methode muss auf dieser Schnittstelle in das System eingegeben werden, aber wir haben gewonnen." erst in Version 2.4 einschalten "

Ich habe auch gesehen, dass NotSupportedExceptions als Lizenzausnahme ausgelöst werden. "Sie führen die kostenlose Version dieser Software aus, diese Funktion wird nicht unterstützt."

Bearbeiten 2:

Ein weiterer möglicher:

System.ComponentModel.InvalidEnumArgumentException  

Die Ausnahme, die ausgelöst wird, wenn ungültige Argumente verwendet werden, die Enumeratoren sind.

Ich würde auch für InvalidOperationException stimmen. Ich habe ein (unvollständiges) Flussdiagramm zu Richtlinien für das Auslösen von .NET-Ausnahmen erstellt, die auf den Framework Design Guidelines 2nd Ed basieren . vor einiger Zeit, wenn jemand interessiert ist.