Wie gehe ich mit geprüften Ausnahmen um, die niemals ausgelöst werden können?

Beispiel:

foobar = new InputStreamReader(p.getInputStream(), "ISO-8859-1");

Da die Codierung fest codiert und korrekt ist, wird der Konstruktor niemals die in der Spezifikation deklarierte UnsupportedEncodingException auslösen (es sei denn, die Java-Implementierung ist fehlerhaft, in diesem Fall bin ich trotzdem verloren). Java zwingt mich sowieso, mit dieser Ausnahme umzugehen.

Derzeit sieht es so aus

try {
    foobar = new InputStreamReader(p.getInputStream(), "ISO-8859-1");
}
catch(UnsupportedEncodingException e) { /* won't ever happen */ }

Irgendwelche Ideen, wie man es besser macht?

Ich habe die Gewohnheit, nur um auf der sicheren Seite zu sein, einen assertin den Fangblock zu stecken. Jemand könnte den Inhalt des tryBlocks später ändern , und Sie möchten wissen, ob der Code fehlschlägt, oder nicht?

Wenn ich einen Cent für jedes Mal bekommen hätte, wenn ich ein Protokoll / einen Fehler "Das sollte niemals passieren" gesehen hätte, hätte ich ... nun, zwei Cent. Aber dennoch...

Leere Fangklötze lassen meine Spinnengefühle kribbeln und die meisten guten Code-Analysetools beklagen sich. Ich würde um jeden Preis vermeiden, sie leer zu lassen. Sicher, jetzt wissen Sie, dass der Fehler niemals auftreten kann, aber in einem Jahr führt jemand ein globales Suchen-Ersetzen von "ISO-8859-1" durch, und plötzlich kann es sein, dass Sie einen äußerst schwer zu findenden Fehler haben.

Der assert falseVorschlag ist gut, aber da Zusicherungen zur Laufzeit deaktiviert werden können, sind sie keine Garantie. Ich würde RuntimeExceptionstattdessen eine verwenden. Diese müssen nicht von aufrufenden Klassen abgefangen werden, und wenn sie jemals auftreten, wird ein Stack-Trace erstellt, der vollständige Informationen liefert.

Ich habe es immer so gemacht:

try {
    foobar = new InputStreamReader(p.getInputStream(), "ISO-8859-1");
} catch(UnsupportedEncodingException e) {
    throw new AssertionError(e);
}

Mag etwas ausführlich sein (Java ist ...), aber zumindest wird ein Assertionsfehler angezeigt, wenn das Unmögliche möglich ist.

Wenn die Java-Implementierung fehlerhaft ist, sollten Sie so schnell wie möglich eine möglichst gute Fehlermeldung erhalten, anstatt nur das Unmögliche zu ignorieren. Und selbst wenn die Java-Implementierung nicht kaputt ist, könnte jemand Ihren Code in geändert haben "UTF8"(hoppla - sollte es gewesen sein "UTF-8"?).

Dies sollte in erster Linie eine Laufzeitausnahme gewesen sein. JDK steckt voller falscher Entscheidungen.

Wenn Sie der einzige Entwickler sind, der diesen Code jemals sehen wird, würde ich sagen, dass es in Ordnung ist. Wenn Sie dies nicht tun, würde ich dies als echte Möglichkeit betrachten oder zumindest den Kommentar "Wird nie passieren" ändern etwas nützlicheres.

Der Teil dieser Ausnahmen, der mich am meisten stört, ist, dass es meine Codeabdeckung verletzt.

Wenn ich in Bezug auf die Berichterstattung zwanghaft werde, rolle ich das Try / Catch auf, das "niemals passieren kann" (... oder nur, wenn ich eine mutierte JVM verwende, die irgendwie vergessen hat, "US-ASCII" aufzunehmen) Eine Klasse und Methode, die diesen Versuch / Fang kapselt und die aktivierte Ausnahme auf eine der hier genannten Arten ersetzt (normalerweise wird eine nicht aktivierte Ausnahme durch eine Snide-Nachricht ausgelöst).

Dann hat meine Codeabdeckung einen Treffer in der Utility-Klasse, aber nicht in allen Verweisen auf diese Operation, die über meinen Code verstreut sind.

Manchmal nehme ich mir die Zeit, um wie Operationen in einer Klasse mit tatsächlich kohärenter Semantik aufzurollen. Aber da es für meine Teamkollegen ziemlich offensichtlich ist, was los ist, halte ich es normalerweise so einfach wie möglich und mache mir nicht so viele Gedanken über das bestmögliche Design.

Wie bereits erwähnt, haben Guava und andere Bibliotheken Möglichkeiten, diesen Schmerz zu lindern - aber das ist im Grunde die gleiche Strategie. Bewegen Sie den Ärger von der Bühne, damit Ihr Hauptcode den Treffer in der Berichterstattung nicht mitmacht.