Clean Code

Nein. Clean Code beschreibt, wie verständlich Programmcode ist, nicht ob er das Richtige berechnet. Die belastbarste dazu veröffentlichte Messung stammt von W. Ward, Hewlett-Packard Journal, April 1989: zwei Firmware-Projekte, eine Korrelation von 0,8 zwischen zyklomatischer Komplexität und Fehlerdichte, wiedergegeben in NIST Special Publication 500-235. Eine Korrelation auf zwei Projekten belegt, dass verzweigter Code häufiger Fehler enthält – nicht, dass lesbarer Code keine enthält. Ein sprechend benannter Rechenweg kann durchgehend die falsche Formel verwenden.

Clean Code schreibt keine nachrechenbare Zeilenzahl vor. Die einzige Obergrenze mit belegbarer Fundstelle steht in NIST Special Publication 500-235, Abschnitt 3.1: 60 Zeilen als verbreitete Grenze, begründet damit, dass ein Modul auf eine gedruckte Seite passen solle. Dieselbe Stelle nennt die Begründung veraltet und hält fest, dass zwischen Zeilenzahl und Verzweigungsgrad kein gleichbleibender Zusammenhang besteht. Messbar ist deshalb die zyklomatische Komplexität, für die Abschnitt 2.5 die Obergrenze 10 nennt – nicht die Länge.

Nur über messbare Merkmale. Beim Werkvertrag macht § 633 Absatz 2 BGB die vereinbarte Beschaffenheit zum Maßstab für Sachmängel; fehlt eine Vereinbarung, zählt die bei Werken gleicher Art übliche Beschaffenheit. Sauberer Code benennt keine prüfbare Beschaffenheit und fällt damit auf diesen zweiten, schwächeren Maßstab zurück. Vereinbaren lässt sich, was ein Werkzeug nachweist: eine benannte Fassung der Formatierregeln, eine Obergrenze für die Verzweigungstiefe je Modul, eine Mindestabdeckung der Tests. Was niemand messen kann, lässt sich später nicht einfordern.

Clean Code betrifft die einzelne Datei, Clean Architecture die Richtung der Abhängigkeiten zwischen den Schichten. Robert C. Martin veröffentlichte dazu am 13. August 2012 die Dependency Rule: Quellcode-Abhängigkeiten dürfen ausschließlich nach innen zeigen, und nichts in einem inneren Kreis darf etwas über einen äußeren wissen. Daraus folgt eine Grenze: Sprechende Namen in jeder Datei ändern nichts daran, wenn die Geschäftslogik das Datenbankpaket importiert. Beide Ebenen sind getrennt prüfbar und getrennt verletzbar.