Warnings entfernt (#177)

- Unnötige import entfernt
- Raw Type warnings unterdrückt
- Zuweisung  von DBIterator an Liste nicht prüfen
- an SelectInput nur Listen übergeben
- einige new durch .valueOf ersetzt
- Endlosschleife 1==1 durch true ersetzt

src/com/schlevoigt/JVerein/gui/control/BuchungsTextKorrekturControl.java

@@ -17,7 +17,6 @@
 package com.schlevoigt.JVerein.gui.control;
 
 import java.rmi.RemoteException;
-import java.util.ArrayList;
 
 import com.schlevoigt.JVerein.Queries.BuchungsKorrekturQuery;
 import com.schlevoigt.JVerein.util.Misc;

src/de/jost_net/JVerein/gui/action/BackupRestoreAction.java

@@ -128,9 +128,11 @@ public void run(ProgressMonitor monitor) throws ApplicationException
           {
 
             @Override
+            @SuppressWarnings("rawtypes")
             public GenericObject create(String type, String id, Map values)
                 throws Exception
             {
+              @SuppressWarnings("unchecked")
               AbstractDBObject object = (AbstractDBObject) Einstellungen
                   .getDBService().createObject(
                       (Class<AbstractDBObject>) loader.loadClass(type), null);

src/de/jost_net/JVerein/gui/action/BuchungKontoauszugZuordnungAction.java

@@ -17,14 +17,12 @@
 
 package de.jost_net.JVerein.gui.action;
 
-import de.jost_net.JVerein.Messaging.BuchungMessage;
 import de.jost_net.JVerein.gui.control.BuchungsControl;
 import de.jost_net.JVerein.gui.dialogs.BuchungsartZuordnungDialog;
 import de.jost_net.JVerein.gui.dialogs.KontoauszugZuordnungDialog;
 import de.jost_net.JVerein.rmi.Buchung;
 import de.willuhn.jameica.gui.Action;
 import de.willuhn.jameica.gui.GUI;
-import de.willuhn.jameica.system.Application;
 import de.willuhn.logging.Logger;
 import de.willuhn.util.ApplicationException;
 import de.willuhn.jameica.system.OperationCanceledException;

src/de/jost_net/JVerein/gui/control/ArbeitseinsatzControl.java

@@ -487,7 +487,7 @@ private GenericIterator<ArbeitseinsatzZeile> getIterator()
   {
     ArrayList<ArbeitseinsatzZeile> zeile = arbeitseinsatzueberpruefungList
         .getInfo();
-
+    @SuppressWarnings("unchecked")
     GenericIterator<ArbeitseinsatzZeile> gi = PseudoIterator
         .fromArray(zeile.toArray(new GenericObject[zeile.size()]));
     return gi;

src/de/jost_net/JVerein/gui/control/BuchungsartControl.java

@@ -221,6 +221,7 @@ public void handleEvent(Event event)
           try
           {
             DBIterator<Buchungsart> it = getFilteredBuchungsart();
+            @SuppressWarnings("unchecked")
             List<Buchungsart> buchungsartenListe = it != null ? PseudoIterator.asList(it) : null;
             steuer_buchungsart.setPleaseChoose("Bitte wählen");
             steuer_buchungsart.setAttribute(getBuchungartAttribute());
@@ -251,6 +252,7 @@ public SelectInput getSteuerBuchungsart() throws RemoteException
     steuer_buchungsart = new SelectInput(PseudoIterator.asList(it), null);
     if (it != null)
     {
+      @SuppressWarnings("unchecked")
       List<Buchungsart> buchungsartenListe = it != null ? PseudoIterator.asList(it) : null;
       steuer_buchungsart.setAttribute(getBuchungartAttribute());
       steuer_buchungsart.setPleaseChoose("Bitte wählen");
@@ -347,7 +349,7 @@ public Input getBuchungsklasse() throws RemoteException
     DBIterator<Buchungsklasse> list = Einstellungen.getDBService()
         .createList(Buchungsklasse.class);
     list.setOrder(getBuchungartSortOrder());
-    buchungsklasse = new SelectInput(list,
+    buchungsklasse = new SelectInput(PseudoIterator.asList(list),
         getBuchungsart().getBuchungsklasse());
     buchungsklasse.setValue(getBuchungsart().getBuchungsklasse());
     buchungsklasse.setAttribute(getBuchungartAttribute());

src/de/jost_net/JVerein/gui/control/EigenschaftControl.java

@@ -25,6 +25,7 @@
 import de.jost_net.JVerein.rmi.Eigenschaft;
 import de.jost_net.JVerein.rmi.EigenschaftGruppe;
 import de.willuhn.datasource.GenericObject;
+import de.willuhn.datasource.pseudo.PseudoIterator;
 import de.willuhn.datasource.rmi.DBIterator;
 import de.willuhn.datasource.rmi.DBService;
 import de.willuhn.jameica.gui.AbstractControl;
@@ -93,7 +94,7 @@ public Input getEigenschaftGruppe() throws RemoteException
     DBIterator<EigenschaftGruppe> list = Einstellungen.getDBService()
         .createList(EigenschaftGruppe.class);
     list.setOrder("ORDER BY bezeichnung");
-    eigenschaftgruppe = new SelectInput(list,
+    eigenschaftgruppe = new SelectInput(PseudoIterator.asList(list),
         getEigenschaft().getEigenschaftGruppe());
     eigenschaftgruppe.setValue(getEigenschaft().getEigenschaftGruppe());
     eigenschaftgruppe.setAttribute("bezeichnung");

src/de/jost_net/JVerein/gui/control/FamilienbeitragNode.java

@@ -177,12 +177,14 @@ public boolean hasChild(GenericObjectNode object)
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getPossibleParents()
   {
     return null;
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getPath()
   {
     return null;
@@ -195,6 +197,7 @@ public GenericObjectNode getParent()
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getChildren() throws RemoteException
   {
     if (children != null)

src/de/jost_net/JVerein/gui/control/MitgliedNextBGruppeControl.java

@@ -25,6 +25,7 @@
 import de.jost_net.JVerein.rmi.Mitglied;
 import de.jost_net.JVerein.rmi.MitgliedNextBGruppe;
 import de.jost_net.JVerein.util.JVDateFormatTTMMJJJJ;
+import de.willuhn.datasource.pseudo.PseudoIterator;
 import de.willuhn.datasource.rmi.DBIterator;
 import de.willuhn.jameica.gui.AbstractControl;
 import de.willuhn.jameica.gui.AbstractView;
@@ -183,7 +184,7 @@ public Input getBeitragsgruppeInput() throws RemoteException
           ArtBeitragsart.FAMILIE_ZAHLER.getKey(),
           ArtBeitragsart.FAMILIE_ANGEHOERIGER.getKey());
     }
-    beitragsgruppe = new SelectInput(list,
+    beitragsgruppe = new SelectInput(PseudoIterator.asList(list),
         getMitgliedNextBGruppe().getBeitragsgruppe());
     beitragsgruppe.setName("Beitragsgruppe");
     beitragsgruppe.setMandatory(true);

src/de/jost_net/JVerein/gui/control/MitgliedskontoControl.java

@@ -602,6 +602,7 @@ public void paint(Composite composite) throws RemoteException
         List<MitgliedskontoNode> items = mitgliedskontoTree.getItems();
         for (MitgliedskontoNode mkn : items)
         {
+          @SuppressWarnings("rawtypes")
           GenericIterator items2 = mkn.getChildren();
           while (items2.hasNext())
           {
@@ -637,6 +638,7 @@ public TablePart getMitgliedskontoList(Action action, ContextMenu menu)
       throws RemoteException
   {
     this.action = action;
+    @SuppressWarnings("rawtypes")
     GenericIterator mitgliedskonten = getMitgliedskontoIterator();
     settings.setAttribute("differenz", getDifferenz().getValue().toString());
     if (mitgliedskontoList == null)
@@ -745,6 +747,7 @@ private GenericIterator<Mitglied> getMitgliedIterator() throws RemoteException
     return mitglieder;
   }
 
+  @SuppressWarnings("rawtypes")
   public GenericIterator getMitgliedskontoIterator() throws RemoteException
   {
     DBService service = Einstellungen.getDBService();

src/de/jost_net/JVerein/gui/control/MitgliedskontoNode.java

@@ -240,12 +240,14 @@ public boolean hasChild(GenericObjectNode object)
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getPossibleParents()
   {
     return null;
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getPath()
   {
     return null;
@@ -258,6 +260,7 @@ public GenericObjectNode getParent()
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getChildren() throws RemoteException
   {
     if (children != null)

src/de/jost_net/JVerein/gui/control/QIFBuchungsImportControl.java

@@ -321,6 +321,7 @@ private void aktuallisierePosListe(QIFImportHead head) throws RemoteException
     endSaldo.setValue(Double.valueOf(saldo));
   }
 
+  @SuppressWarnings("rawtypes")
   private GenericIterator getIterator() throws RemoteException
   {
     ArrayList<QIFImportPos> zeile = new ArrayList<>();

src/de/jost_net/JVerein/gui/control/QIFBuchungsartZuordnenControl.java

@@ -338,6 +338,7 @@ private void refreshSelectedQIFImportPos() throws RemoteException
    * 
    * @throws RemoteException
    */
+  @SuppressWarnings("rawtypes")
   private GenericIterator getDistinctQIFBuchartList() throws RemoteException
   {
     DBIterator<QIFImportPos> it = Einstellungen.getDBService()

src/de/jost_net/JVerein/gui/control/QIFMitgliedZuordnenControl.java

@@ -277,7 +277,7 @@ public SelectInput getMitgliederInput() throws RemoteException
     {
       return mitgliederInput;
     }
-    mitgliederInput = new SelectInput((GenericIterator) null, null);
+    mitgliederInput = new SelectInput((Object[]) null, null);
     mitgliederInput.setAttribute("namevorname");
     mitgliederInput.setPleaseChoose("Bitte auswählen");
     mitgliederInput.setComment(" ");
@@ -356,6 +356,7 @@ private void refreshSelectedQIFImportPos() throws RemoteException
    * 
    * @throws RemoteException
    */
+  @SuppressWarnings("rawtypes")
   private GenericIterator getDistinctQIFNameList() throws RemoteException
   {
     DBIterator<QIFImportPos> it = Einstellungen.getDBService()

src/de/jost_net/JVerein/gui/control/SpendenbescheinigungAutoNeuControl.java

@@ -160,6 +160,7 @@ public void handleAction(Object context) throws ApplicationException
           SpendenbescheinigungNode spn = (SpendenbescheinigungNode) items
               .get(0);
           // Loop über die Mitglieder
+          @SuppressWarnings("rawtypes")
           GenericIterator it1 = spn.getChildren();
           while (it1.hasNext())
           {
@@ -182,6 +183,7 @@ public void handleAction(Object context) throws ApplicationException
             spbescheinigung.setBetrag(0.01);
             spbescheinigung.setAutocreate(Boolean.TRUE);
             // Loop über die Buchungen eines Mitglieds
+            @SuppressWarnings("rawtypes")
             GenericIterator it2 = sp1.getChildren();
             while (it2.hasNext())
             {

src/de/jost_net/JVerein/gui/dialogs/BuchungsjournalSortDialog.java

@@ -17,7 +17,6 @@
 
 package de.jost_net.JVerein.gui.dialogs;
 
-import org.eclipse.swt.SWT;
 import org.eclipse.swt.widgets.Composite;
 import org.eclipse.swt.widgets.Event;
 import org.eclipse.swt.widgets.Listener;

src/de/jost_net/JVerein/gui/dialogs/ExportDialog.java

@@ -32,7 +32,6 @@
 import de.jost_net.JVerein.io.IORegistry;
 import de.jost_net.JVerein.util.Dateiname;
 import de.willuhn.datasource.GenericObject;
-import de.willuhn.datasource.pseudo.PseudoIterator;
 import de.willuhn.jameica.gui.Action;
 import de.willuhn.jameica.gui.GUI;
 import de.willuhn.jameica.gui.dialogs.AbstractDialog;

src/de/jost_net/JVerein/gui/formatter/JaNeinFormatter.java

@@ -30,7 +30,7 @@ public String format(Object o)
     if (o instanceof Byte)
     {
       Byte b = (Byte) o;
-      if (b.equals(new Byte("1")))
+      if (b.equals(Byte.valueOf("1")))
       {
         return "Ja";
       }

src/de/jost_net/JVerein/gui/parts/SplitbuchungListTablePart.java

@@ -35,6 +35,7 @@ public SplitbuchungListTablePart(Action action)
     super(action);
   }
 
+  @SuppressWarnings("rawtypes")
   public SplitbuchungListTablePart(GenericIterator list, Action action)
   {
     super(list, action);

src/de/jost_net/JVerein/gui/view/KontoauszugView.java

@@ -16,8 +16,6 @@
  **********************************************************************/
 package de.jost_net.JVerein.gui.view;
 
-import java.util.Date;
-
 import de.jost_net.JVerein.gui.action.DokumentationAction;
 import de.jost_net.JVerein.gui.control.MitgliedskontoControl;
 import de.willuhn.jameica.gui.AbstractView;

src/de/jost_net/JVerein/io/CSVConnection.java

@@ -207,7 +207,7 @@ public int getNumberOfRows() throws SQLException
   public void openCsvDB() throws SQLException
   {
     Properties props = new java.util.Properties();
-    props.put("separator", (new Character(seperator)).toString()); // separator
+    props.put("separator", (Character.valueOf(seperator)).toString()); // separator
                                                                    // is a bar
     props.put("suppressHeaders", "false"); // first line contains data
     props.put("charset", "ISO-8859-1");

src/de/jost_net/JVerein/io/FormularAufbereitung.java

@@ -33,7 +33,6 @@
 
 import de.jost_net.JVerein.Einstellungen;
 import de.jost_net.JVerein.Variable.AllgemeineVar;
-import de.jost_net.JVerein.Variable.MitgliedskontoVar;
 import de.jost_net.JVerein.rmi.Einstellung;
 import de.jost_net.JVerein.rmi.Formular;
 import de.jost_net.JVerein.rmi.Formularfeld;
@@ -44,8 +43,6 @@
 import de.willuhn.jameica.gui.internal.action.Program;
 import de.willuhn.jameica.messaging.StatusBarMessage;
 import de.willuhn.jameica.system.Application;
-import de.willuhn.logging.Level;
-import de.willuhn.logging.Logger;
 import de.willuhn.util.ApplicationException;
 
 public class FormularAufbereitung

src/de/jost_net/JVerein/io/Kontoauszug.java

@@ -126,11 +126,13 @@ private void generiereMitglied(Mitglied m, Date von, Date bis)
 
     MitgliedskontoNode node = new MitgliedskontoNode(m, von, bis);
     generiereZeile(node);
+    @SuppressWarnings("rawtypes")
     GenericIterator gi1 = node.getChildren();
     while (gi1.hasNext())
     {
       MitgliedskontoNode n1 = (MitgliedskontoNode) gi1.next();
       generiereZeile(n1);
+      @SuppressWarnings("rawtypes")
       GenericIterator gi2 = n1.getChildren();
       while (gi2.hasNext())
       {

src/de/jost_net/JVerein/io/Mahnungsausgabe.java

@@ -50,6 +50,7 @@ public Mahnungsausgabe(MitgliedskontoControl control) throws IOException
     }
     else
     {
+      @SuppressWarnings("rawtypes")
       GenericIterator it = control.getMitgliedskontoIterator();
       Mitgliedskonto[] mk = new Mitgliedskonto[it.size()];
       int i = 0;

src/de/jost_net/JVerein/server/EigenschaftenNode.java

@@ -147,6 +147,7 @@ private EigenschaftenNode(EigenschaftenNode parent, Mitglied mitglied,
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getChildren() throws RemoteException
   {
     if (childrens == null)
@@ -169,12 +170,14 @@ public EigenschaftenNode getParent()
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getPath()
   {
     return null;
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getPossibleParents()
   {
     return null;

src/de/jost_net/JVerein/server/SpendenbescheinigungNode.java

@@ -180,6 +180,7 @@ private SpendenbescheinigungNode(Mitglied mitglied, Buchung buchung)
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getChildren() throws RemoteException
   {
     if (childrens == null)
@@ -202,12 +203,14 @@ public SpendenbescheinigungNode getParent()
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getPath()
   {
     return null;
   }
 
   @Override
+  @SuppressWarnings("rawtypes")
   public GenericIterator getPossibleParents()
   {
     return null;
@@ -236,6 +239,7 @@ public Object getAttribute(String name) throws RemoteException
       }
       case MITGLIED:
       {
+        @SuppressWarnings("rawtypes")
         GenericIterator it1 = getChildren();
         double betrag = 0.0;
         while (it1.hasNext())

src/de/jost_net/JVerein/util/MemoryAnalyzer.java

@@ -10,7 +10,7 @@ public class MemoryAnalyzer extends Thread
   @Override
   public void run()
   {
-    while (1 == 1)
+    while (true)
     {
       // System.out.println(Runtime.getRuntime().freeMemory());
       MemoryMXBean memBean = ManagementFactory.getMemoryMXBean();