DRY: inline redundant local variable

Entfernt doppelt erzeugte Referenzen auf ein identisches Objekt.

Author: ruderphilipp

src/de/willuhn/jameica/hbci/gui/input/AddressInput.java

@@ -348,11 +348,10 @@ public void handleEvent(Event event)
      */
     public void setText(String s)
     {
-      String before = s;
-      String after = HBCIProperties.clean(s,validChars);
       // Alle Zeichen rauswerfen, die nicht zulaessig sind.
+      String after = HBCIProperties.clean(s,validChars);
       super.setText(after);
-      if (before != null && !before.equals(after))
+      if (s != null && !s.equals(after))
         GUI.getView().setErrorText(i18n.tr("Im Namen wurden nicht zulässige Zeichen entfernt"));
     }
 

src/de/willuhn/jameica/hbci/gui/parts/AbstractSammelTransferList.java

@@ -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;
-          String uuid = MetaKey.REMINDER_UUID.get(o);
+          String uuid = MetaKey.REMINDER_UUID.get((HibiscusDBObject) l);
           if (uuid != null)
           {
             try
@@ -110,7 +109,7 @@ else if (l.ausgefuehrt())
               Logger.error("unable to determine reminder",e);
             }
           }
-          else if (MetaKey.REMINDER_TEMPLATE.get(o) != null)
+          else if (MetaKey.REMINDER_TEMPLATE.get((HibiscusDBObject) l) != null)
           {
             item.setImage(4,SWTUtil.getImage("edit-copy.png"));
           }

src/de/willuhn/jameica/hbci/gui/parts/AbstractSepaSammelTransferList.java

@@ -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;
-          String uuid = MetaKey.REMINDER_UUID.get(o);
+          String uuid = MetaKey.REMINDER_UUID.get((HibiscusDBObject) l);
           if (uuid != null)
           {
             try
@@ -131,7 +130,7 @@ public void format(TableItem item) {
               Logger.error("unable to determine reminder",e);
             }
           }
-          else if (MetaKey.REMINDER_TEMPLATE.get(o) != null)
+          else if (MetaKey.REMINDER_TEMPLATE.get((HibiscusDBObject) l) != null)
           {
             item.setImage(4,SWTUtil.getImage("edit-copy.png"));
           }

src/de/willuhn/jameica/hbci/io/AbstractSepaExporter.java

@@ -107,8 +107,7 @@ void setup(Object[] objects, IOFormat format, OutputStream os, ProgressMonitor m
 
     // User nach der SEPA-Version fragen, die verwendet werden soll.
     PainVersionDialog d = new PainVersionDialog(this.getPainType());
-    SepaVersion version = (SepaVersion) d.open();
-    ctx.version = version;
+    ctx.version = (SepaVersion) d.open();
 
     // Header-Infos zuweisen
     ctx.props.setProperty("src.bic",    StringUtils.trimToEmpty(konto.getBic()));

src/de/willuhn/jameica/hbci/passports/rdh/SelectSizEntryDialog.java

@@ -73,8 +73,7 @@ public void handleAction(Object context) throws ApplicationException
       {
         if (context == null || !(context instanceof Entry))
           return;
-        Entry e = (Entry) context;
-        selected = e;
+        selected = (Entry) context;
         close();
       }
     });
@@ -92,8 +91,7 @@ public void handleAction(Object context) throws ApplicationException
         if (o == null || !(o instanceof Entry))
           return;
 
-        Entry e = (Entry) context;
-        selected = e;
+        selected = (Entry) context;
         close();
       }
     },null,false,"ok.png");

src/de/willuhn/jameica/hbci/server/UmsatzTypImpl.java

@@ -497,8 +497,7 @@ public Object getAttribute(String arg0) throws RemoteException
         return null;
       try
       {
-        Integer i = new Integer(n);
-        return i;
+        return new Integer(n);
       }
       catch (Exception e)
       {

src/de/willuhn/jameica/hbci/server/VersionUtil.java

@@ -84,7 +84,6 @@ public static int delete(HBCIDBService service, String name) throws RemoteExcept
     if (name == null || name.length() == 0)
       throw new RemoteException("no name given");
 
-    int i = service.executeUpdate("delete from version where name = ?",name);
-    return i;
+    return service.executeUpdate("delete from version where name = ?",name);
   }
 }