Ich achte sehr drauf, dass mein Code gut lesbar ist, aber zu starre Regeln behindern die Lesbarkeit bei mir eher.
Bei dem Switch behandle ich z.B. jeden Fall eines Enums. Da jetzt einen leeren default-case reinzupacken finde ich ein viel größeres WTF als gar kein default.
Das ist so, als würde man bei jedem if einen else-Block zwingend vorschreiben...
Man könnte höchstens eine
Exception drin werfen, für den Fall, dass das Enum erweitert wurde und die Stelle nicht angepasst wurde – das hatte ich in Erwägung gezogen –, aber dann meckert der Checkstyle auch wieder rum, weil die nicht abgefangen wird
, und ich müsste in jedes Glied der Aufrufkette ein throws packen, wo man sich beim Lesen dann erst recht fragen würde: WTF? Warum wirft das denn ausgerechnet
die Exception?