HS-Flensburg-PLTP / jlint

Other
0 stars 0 forks source link

`SameExecutionsInIf` Regel funktioniert nicht korrekt #296

Closed mario-koehler closed 1 year ago

mario-koehler commented 1 year ago

Die Regel zum Erkennen von dupliziertem Code in if-else-Konstrukten meldet keine Anmerkungen für die entsprechende Testdatei. Deswegen erwartet der Test aktuell auch keine Meldungen!

Beim Probieren ist allerdings aufgefallen, dass stattdessen Falschmeldungen auf anderen Test-Files erzeugt werden.

In diesem Zusammenhang sollte gleich #158 mit gelöst werden. Die entsprechenden Nachrichten aus checkstyle-pltp:

Die jeweils erste Anweisung im `then`- und im `else`-Zweig der `if`-Anweisung sind gleich.
Die Anweisung kann aus beiden Zweigen vor das `if` herausgezogen werden.
Die jeweils letzte Anweisung im `then`- und im `else`-Zweig der `if`-Anweisung sind gleich.
Die Anweisung kann aus beiden Zweigen hinter das `else` herausgezogen werden.

Die Regel aus checkstyle-pltp hatte im Gegensatz zu Regel in jlint jeweils nur die erste und letzte Anweisung in den Blöcken überprüft. Es sollte nochmal evaluiert werden, welches Verhalten genau erwünscht ist.