fügt automatisch alle geschweiften Klammern zu if / else / for / while usw. in einer Java-Codebasis hinzu

Am einfachsten wäre es, Eclipse zu verwenden und im gesamten Projekt auf Clean-up zu klicken. In Clean-up Profilkonfiguration wählen Sie Code style tab. Dort können Sie Use blocks in if/while/for/do statements als Always auswählen.

Obwohl es ratsam ist, mit Legacy-Code vorsichtig zu sein, ist es auch eine gute Sache, Bugs in Legacy-Code zu entdecken ... oder zumindest die Bugs leichter zu erkennen.

Betrachten wir Brian Agnews schwierige Fälle:

Was den JLS und den Java-Compiler angeht, bedeutet dies

Wenn also der Transformator den Code neu schreibt:

Es ändert nicht die Bedeutung des Codes, aber es korrigiert ein Problem, das dazu führt, dass jemand den Code falsch liest und einen potenziellen Fehler das ist schon im Code ; d. h. wenn der zweite Anruf bedingt sein soll ...

Noch einmal, wenn das neu geschrieben wird, sollten Sie erhalten:

bedeutet dasselbe wie das Original. Darüber hinaus spiegelt dies höchstwahrscheinlich die Absicht des Programmierers wider ... ob der Einrückung zu glauben ist. Aber bedenke das:

Wenn wir das als

Die tatsächliche Bedeutung des Codes ändert sich nicht, und der falsche Einzug wird nicht mehr in die Irre führen. Wenn der Programmierer entscheidet, den ersten Anruf auskommentieren, erkennt er möglicherweise nicht, dass das vorherige Auskommentieren eine unbeabsichtigte Konsequenz hatte. (Der Beweis war in der ursprünglichen Einrückung, die wir "fixiert" haben.) Aber es gibt eine mögliche Lösung; siehe unten.

Wenn wir davon ausgehen, dass das Code-Transformationstool mit einem korrekten Verständnis der Syntax und Semantik von Java arbeitet, dann wird es nichts brechen, was nicht bereits gebrochen wurde, und es wird (bis zu einem gewissen Grad) eine vorhandene Zerbrochenheit mehr machen offensichtlich für jemanden, der den Code liest. Für mich ist das ein Null-Risiko-Gewinn, sogar für Legacy-Code.

Wenn wir jetzt den Transformator schlauer machen, könnte er erkennen einige der Fälle, in denen die ursprüngliche Einrückung auf einen möglichen Fehler hinweist (wie die Fälle # 1 und # 2a oben) und sie für eine genauere Code-Prüfung kennzeichnen .

Das erscheint mir als potentiell ziemlich gefährlich. Verfügen Sie über eine umfassende Testabdeckung?

Ich kann einige sofort schwierige Fälle sehen, z. B.

und

sind zwei, die mir in den Sinn kommen und nicht gehandhabt werden können (Sie sollten sprachbewusst sein, um diese anzugehen). Ich würde nicht versuchen, eine solche Aufgabe zu automatisieren, es sei denn, Sie haben ein sprachbewusstes und zuverlässiges Werkzeug, sondern erzwingen eine Richtlinie, die Sie Code ändern, wenn Sie es aus anderen Gründen ändern, und kann zu diesem Zeitpunkt die Codeabdeckung durch Komponententests bestätigen vor dem Ändern des Codes.

Robert sagte in einem Kommentar: "@ira, ich wäre auch daran interessiert, von diesem wirklich sprachbewussten Werkzeug zu hören!"

Entschuldigung für die Neckerei. Für die Ursache der Neckerei, überprüfen Sie meine Bio.

Untease: Siehe Ссылка als Basis-Engine und Ссылка . Das Paar macht ein vollständig sprachbewusstes Source-to-Source-Programmprogramm-Transformationssystem.

Man würde diese Aufgabe mit DMS mit zwei Quell-zu-Quell-Transformationsregeln erledigen:

Dies funktioniert zuverlässig, weil DMS vollständig von einer formalen Grammatik gesteuert wird (in diesem Fall für Java) und nicht mit Rohtext arbeitet, sondern mit abstrakten Syntaxbäumen; Dies macht auch unabhängig vom Layout. (Es ist wahrscheinlich nützlich, die Regeln zu verstehen, dass "Blockieren", "Ausdruck" und "Aussage" nichtterminale Grammatik sind.

Während es ein schneller Sonar-Gewinn sein kann, machst du einige gefährliche Sachen. Der empfohlene Ansatz besteht darin, nur Legacy-Code zu reparieren, wenn dieser Code erneut aufgerufen werden muss. Diese pauschalen Ansätze werden möglicherweise sehr subtile Fehler verursachen. Da sich die Richtlinien geändert haben, muss Ihr Management-Team verstehen, dass dies ein gewaltiges Unterfangen ist und garantieren sollte, dass der gesamte zukünftige Code dem Standard entspricht.

Intellij 2017 unterstützt dies mit Reformat Code , wenn Sie zuerst den richtigen Codierungsstil in den Editor-Einstellungen festlegen.