remove redundant local variable #92
Conversation
ruderphilipp
force-pushed
the
refactoring/redundant_local_variable
branch
from
3cba074
to
e39e737
Compare
| @@ -116,8 +116,7 @@ public void format(TableItem item) { | |||
| item.setForeground(Color.COMMENT.getSWTColor()); | |||
|
|
|||
| // Checken, ob der Auftrag einen Reminder hat oder ob es ein geclonter Auftrag ist | |||
| HibiscusDBObject o = (HibiscusDBObject) l; | |||
There was a problem hiding this comment.
Durch Entfernen der lokalen Variable o, wird die Semantik der Identität bei ihrer Verwendung gebrochen. Der Code sollte daher an dieser Stelle bleiben, wie er war.
There was a problem hiding this comment.
Hallo @meigelb,
ehrlich gesagt, verstehe ich nicht, was Du mit "die Semantik der Identität bei ihrer Verwendung" ausdrücken möchtest.
Die Aussage ist "checke ob l ein Klon ist", wozu in Zeile 119 die UUID und in Zeile 133 das Template verwendet wird.
Damit der Code compiliert, ist der Downcast nicht notwendig, da die Variable l vom Typ SepaSammelTransfer ist, welches wiederum ein HibiscusDBObject ist, und somit automatisch beim Funktionsaufruf entsprechend gecastet wird:
public interface SepaSammelTransfer<T extends SepaSammelTransferBuchung> extends HibiscusDBObject, Terminable, Duplicatable, SepaPaymentThere was a problem hiding this comment.
Hm, ich weiß auch nicht, was ich damals dort gesehen habe. Aus heutiger Sicht passt es. Vielleicht habe ich nicht erkannt, dass l ein Objekt vom Typ SepaSammelTransfer ist, sondern das als eine 1 gelesen. Danke fürs kritische Hinterfragen.
| @@ -95,8 +95,7 @@ else if (l.ausgefuehrt()) | |||
| item.setForeground(Color.COMMENT.getSWTColor()); | |||
|
|
|||
| // Checken, ob der Auftrag einen Reminder hat oder ob es ein geclonter Auftrag ist | |||
| HibiscusDBObject o = (HibiscusDBObject) l; | |||
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
|
Ehrlichweise würde ich diesen PR ungemerged schliessen. Die Änderungen bringen jetzt nun nicht den riesen Vorteil, erzeugen aber an zwei potentiellen Stellen Probleme und erzeugt derzeit einen Merge-Konflikt. Das ist doch schade um unser aller Zeit. |
ruderphilipp
force-pushed
the
refactoring/redundant_local_variable
branch
from
e39e737
to
6e4e0f2
Compare
Entfernt doppelt erzeugte Referenzen auf ein identisches Objekt.
ruderphilipp
force-pushed
the
refactoring/redundant_local_variable
branch
from
6e4e0f2
to
8013389
Compare
Änderungen passen aus meiner Sicht. (Insgesamt stimme ich aber zu, dass die Änderungen kaum etwas bewirken werden, zumal die meisten Stellen bisher bestimmt sogar vom Compiler rausoptimiert wurden.) |
Entfernt doppelt erzeugte Referenzen auf ein identisches Objekt. Diese stellt nur ein Alias dar und hat keinen Mehrwert, wenn direkt darunter bereits schon wieder eine Zuweisung/ ein
returnerfolgt.