Muokkaaminen valmis ja dokumentaatio kunnossa.

src/HT/fx/Pelaajat.java

@@ -20,7 +20,6 @@ import fi.jyu.mit.ohj2.Mjonot;
 public class Pelaajat {
     private static final int Max_Pelaajat   = 9;
     private int lkm           = 0;
-    private String tiedostonNimi = "";
     private Pelaaja alkiot[]      = new Pelaaja[Max_Pelaajat];
     
     /**
@@ -58,7 +57,9 @@ public class Pelaajat {
         alkiot[lkm] = pelaaja;
         lkm++;
     }
-    
+    /**
+     * lisää yhden paikan taulukkoon mikäli se täyttyy
+     */
     private void lisaatilaa() {
         Pelaaja alkiot2[] = new Pelaaja[lkm + 1];
         int i = 0;
@@ -67,7 +68,6 @@ public class Pelaajat {
             i++;
         }
         alkiot = alkiot2;
-        
     }
 
     /**
@@ -77,9 +77,9 @@ public class Pelaajat {
      * @throws IndexOutOfBoundsException jos i ei ole sallitulla alueella  
      */
     public Pelaaja anna(int i) throws IndexOutOfBoundsException {
-        if (i < 0)
+        if (i < 0) {
             throw new IndexOutOfBoundsException("Laiton indeksi: " + i);
-        
+        }
         Pelaaja palautettava = null;
         int m = 0;
         while(m<alkiot.length) {
@@ -91,43 +91,29 @@ public class Pelaajat {
             }
             m++;
             }
-            
-        
         return palautettava;
     }
-    
-    
     /**
      * Lukee pelaajiston tiedostosta.  Ei toimi.
      * @param tiedosto tiedoston hakemisto
      * @throws SailoException jos lukeminen epäonnistuu
      */
     public void lueTiedostosta(String tiedosto) throws SailoException {
-        
-        //tiedostonNimi = tiedosto;
         try (Scanner fi = new Scanner(new FileInputStream(new File(tiedosto)))) { // Jotta UTF8/ISO-8859 toimii
-
-            
             while ( fi.hasNext() ) {
                 String s = fi.nextLine();
                 if(s.length()>1) {
-                    
-                    
                     StringBuffer sb = new StringBuffer(s);
                     Pelaaja pel = new Pelaaja();
                     pel.parse(sb);
                     lisaa(pel);
-                   
-                    
                 }
             }
         } catch (FileNotFoundException ex) {
             System.err.println("Tiedosto ei aukea! " + ex.getMessage());
             ex.printStackTrace();
             return;
-        }
-        
-        
+        } 
     }
     
     /**
@@ -146,14 +132,11 @@ public class Pelaajat {
             int i = 0;
             Pelaaja pelaajat[] = PadelMainGUIController.getKarkelo().getPelaajat();
             while ( i<pelaajat.length) {
-                if(pelaajat[i]!=null) {
-                    
+                if(pelaajat[i]!=null) { 
                     fo.print(pelaajat[i].getTiedot()+ "\n");
                 }
-                
                 i++;
             }
-            //System.out.println("Heureka ! ");
         } catch(FileNotFoundException ex) {
             System.err.println("Tiedosto ei aukea: " + ex.getMessage());
         }
@@ -166,45 +149,13 @@ public class Pelaajat {
     public int getLkm() {
         return lkm;
     }
-    
-    /**
-     * Testiohjelma jäsenistölle
-     * @param args ei käytössä
-     */
-    public static void main(String args[]) {
-        Pelaajat pelaajat = new Pelaajat();
-
-      Pelaaja aku = new Pelaaja(), aku2 = new Pelaaja();
-        aku.rekisteroi();
-        aku.luoTesti();
-        aku2.rekisteroi();
-        aku2.luoTesti();
-
-        try {
-            pelaajat.lisaa(aku);
-            pelaajat.lisaa(aku2);
-
-            System.out.println("============= Jäsenet testi =================");
-
-            for (int i = 0; i < pelaajat.getLkm(); i++) {
-                Pelaaja pelaaja = pelaajat.anna(i);
-                System.out.println("Pelaaja nro: " + i);
-                pelaaja.tulosta(System.out);
-            }
-
-        } catch (SailoException ex) {
-            System.out.println(ex.getMessage());
-        }
-    }
 
     /**
      * Käydään taulukko lä'pi ja luodaan uusi ilman poistettavaa pelaajaa
      * @param pel pelaaja joka poistetaan
      */
     public void poistaPelaaja(Pelaaja pel) {
-        
-        poistaPelaajanOttelut(pel);
-        
+        poistaPelaajanOttelut(pel); 
         Pelaaja alkiot2[] = new Pelaaja[lkm-1];
         int i = 0;
         int n = 0;
@@ -216,43 +167,50 @@ public class Pelaajat {
             i++;
         }
         alkiot = alkiot2;
-        //paivitaId();
         lkm--;
         
-        
-        
-        System.out.println("  " +lkm + Pelaaja.getSeuraavaNro());
-        
     }
-    
+    /**
+     * poistaa yhden pelaajan ottelut
+     * @param pel kenen poistetaan
+     */
     private void poistaPelaajanOttelut(Pelaaja pel) {
-        
         List<Ottelu> pelit = PadelMainGUIController.getKarkelo().annaOttelut(pel);
         int i = 0;
         while(i<pelit.size()) {
             PadelMainGUIController.getKarkelo().poistaOttelu(pelit.get(i));
             i++;
         }
-        
-        
     }
+    
+    /**
+     * Testiohjelma jäsenistölle
+     * @param args ei käytössä
+     */
+    public static void main(String args[]) {
+        Pelaajat pelaajat = new Pelaajat();
 
+      Pelaaja aku = new Pelaaja(), aku2 = new Pelaaja();
+        aku.rekisteroi();
+        aku.luoTesti();
+        aku2.rekisteroi();
+        aku2.luoTesti();
 
-    public void paivitaId() {
-        int i = 0;
-        
-        
-        while(i<alkiot.length) {
-            
-            if(alkiot[i].getTunnusNro() != i+1) {
-                //System.out.println(alkiot[i].getTunnusNro() +"  " +lkm );
-                alkiot[i].setTunnusNro(i+1);
-                //System.out.println(alkiot[i].getTunnusNro() +"  " +lkm );
+        try {
+            pelaajat.lisaa(aku);
+            pelaajat.lisaa(aku2);
+
+            System.out.println("============= Jäsenet testi =================");
+
+            for (int i = 0; i < pelaajat.getLkm(); i++) {
+                Pelaaja pelaaja = pelaajat.anna(i);
+                System.out.println("Pelaaja nro: " + i);
+                pelaaja.tulosta(System.out);
             }
-            i++;
-            Pelaaja.setSeuraavaNro(i+1);
+
+        } catch (SailoException ex) {
+            System.out.println(ex.getMessage());
         }
-        
-        
     }
+    
 }