Testejä vaille vaihe 5

build.xml

@@ -17,7 +17,7 @@
             <zipfileset excludes="META-INF/*.SF" src="C:/devel/jar/Ali.jar"/>
             <zipfileset excludes="META-INF/*.SF" src="C:/devel/jar/fxgui.jar"/>
             <zipfileset excludes="META-INF/*.SF" src="C:/devel/jar/Graphics.jar"/>
-            <zipfileset excludes="META-INF/*.SF" src="C:/devel/jar/Music.jar"/>
+            <zipfileset excludes="META-INF/*.SF" src="C:/Users/omistaja/Ohjelmointi/Java-projektit/Jarreja/Tietorakenne.jar"/>
         </jar>
     </target>
 </project>

src/Kirjalista/Genre.java

 package Kirjalista;
 
+import java.io.PrintStream;
+import java.util.Random;
 
 /**
  * Genre-luokka
@@ -17,7 +19,7 @@ public class Genre {
     /**
      * Ottaa merkkijonon ja muuttaa sen genren tiedoiksi, poistaa turhat völilyönnit ja muuttaa
      * syötteen pieniksi kirjaimiksi
-     * @param mjono Merkkijono, josta kirjan tiedot saadaan
+     * @param mjono Merkkijono, josta genren tiedot saadaan
      * <pre name="test">
      * Genre g = new Genre();
      * g.parse("fantasia");
@@ -58,6 +60,25 @@ public class Genre {
     }
     
     
+    
+    private String[] esim = {"Fantasia", "Sci-fi", "Kauhu"};
+    /**
+     * Asettaa genreksi jonkun ylläolevista
+     * TODO poista
+     */
+    public void esimerkkiGenre() {
+        Random rand = new Random();
+        this.nimi = esim[rand.nextInt(3)];
+    }
+    
+    
+    /**
+     * @return Genren nimi
+     */
+    public String getNimi() {
+        return this.nimi;
+    }
+    
     /**
      * palauttaa genren id:n
      * @return Genren id
@@ -65,4 +86,25 @@ public class Genre {
     public int getId() {
         return this.id;
     }
+    
+    
+    /**
+     * Tulostaa genren tiedot
+     * @param out Tietovirta johon tulostetaan
+     */
+    public void tulosta(PrintStream out) {
+        out.println(this.id + " " + this.nimi);
+    }
+
+
+    /**
+     * Asettaa id:lle annetun arvon
+     * @param id arvo, joka asetetaan id:ksi
+     * Tämä olemassa, koska tietyllä genrellä on aina tietty id ja ilman tätä aliohjelmaa tulisi
+     * ongelmia sellaisten lisättyjen kirjojen kanssa, joiden genre on jo "tietokannassa." Ilman tätä
+     * kyseisten genrejen id olisi 0 ja ohjelma ei osaisi hakea niitä
+     */
+    public void setId(int id) {
+        this.id = id;
+    }
 }

src/Kirjalista/GenreRelaatio.java

@@ -28,15 +28,29 @@ public class GenreRelaatio {
      * esim. palauttaa kaikkien kirjojen id:t, joiden genre on fantasia
      * @param genreId Genren id, jonka perusteella haetaan kirjojen id:itä
      * @return Kaikkien niiden kirjojen id:t, joiden genren id on genreId 
+     * TODO testit
+     * <pre name="test">
+     * GenreRelaatio gr = new GenreRelaatio();
+	 * gr.lisaa(new KirjanGenre(1, 2));
+	 * int[] t = gr.haeKirjat(2);  
+	 * t[0] === 1;
+	 * TODO lisää testejä
+	 * </pre>
      */
     public ArrayList<Integer> haeKirjat(int genreId) {
+        //if (alkiot.length() <= 0) throw new IndexOutOfBoundsException();        
         ArrayList<Integer> kirjojenIdt = new ArrayList<Integer>();
+        
+        alkiot.iter = alkiot.getEka();
+        
         for (int i = 0; i < alkiot.length(); i++) {
-            KirjanGenre alkio = alkiot.getByIndex(i);
+            KirjanGenre alkio = alkiot.iter.getSisalto();
             if (alkio.getGenreId() == genreId) {
-                kirjojenIdt.add(alkio.getKirjaId());
+                kirjojenIdt.add(alkio.getGenreId());
             }
+            alkiot.iter = alkiot.iter.getNext();
         }
+        
         return kirjojenIdt;
     }
     
@@ -46,15 +60,22 @@ public class GenreRelaatio {
      * eli toisin sanoen palauttaa tietyn kirjan kaikkien genrejen id:t
      * @param kirjaId Kirjan id, jonka perusteella haetaan genrejen id:itä
      * @return Kaikkien niiden genrejen id:t, joiden kirjan id on kirjaId 
+     * TODO testit
      */
     public ArrayList<Integer> haeGenret(int kirjaId) {
+        //if (alkiot.length() <= 0) throw new IndexOutOfBoundsException(); ??? tämä ei mahdollista
         ArrayList<Integer> genrejenIdt = new ArrayList<Integer>();
+        
+        alkiot.iter = alkiot.getEka();
+        
         for (int i = 0; i < alkiot.length(); i++) {
-            KirjanGenre alkio = alkiot.getByIndex(i);
+            KirjanGenre alkio = alkiot.iter.getSisalto();
             if (alkio.getKirjaId() == kirjaId) {
                 genrejenIdt.add(alkio.getGenreId());
             }
+            alkiot.iter = alkiot.iter.getNext();
         }
+        
         return genrejenIdt;
     }
     

src/Kirjalista/Genret.java

+package Kirjalista;
+
+import java.util.ArrayList;
+import java.util.NoSuchElementException;
+
+/**
+ * Monikkoluokka genreille
+ * - Sisältää listan genreistä
+ * - Lisää ja poistaa kirjoja
+ * - Lukee ja kirjoittaa tiedostoon
+ * - Etsii ja lajittelee
+ */
+public class Genret {
+    private ArrayList<Genre> alkiot = new ArrayList<Genre>();
+    
+    
+    /**
+     * Lisää genren listaan, jos samannimistä genreä ei ole olemassa
+     * @param genre Lisättävä genre
+     */
+    public void lisaa(Genre genre) {
+        if (onOlemassa(genre.getNimi())) {
+            genre.setId(haeIdNimella(genre.getNimi()));
+            return;
+        }
+        genre.rekisteroi();
+        alkiot.add(genre);
+    }
+    
+    
+    private int haeIdNimella(String nimi) {
+        for (int i = 0; i < alkiot.size(); i++) {
+            if (alkiot.get(i).getNimi() == nimi) return alkiot.get(i).getId();
+        }
+        return -1;
+    }
+
+
+    /**
+     * Poistaa genren annetulla id:llä
+     * @param id Id, jonka perusteella genre poistetaan
+     * TODO testit
+     */
+    public void poista(int id) {
+        for (int i = 0; i < alkiot.size(); i++) {
+            if (alkiot.get(i).getId() == id) {
+                alkiot.remove(i);
+                return;
+            }
+        }
+        throw new NoSuchElementException("Genreä id:llä " + id + " ei löydy.");
+    }
+    
+    
+    /**
+     * Tarkistaa, onko tietyn nimistä genreä olemassa
+     * @param nimi Genren nimi
+     * @return True, jos genre on olemassa
+     */
+    public boolean onOlemassa(String nimi) {
+        for (int i = 0; i < alkiot.size(); i++) {
+            if (alkiot.get(i).getNimi() == nimi) {
+                return true;
+            }
+        }
+        return false;
+    }
+    
+    
+    /**
+     * Hakee genren nimen id:n perusteella
+     * @param id haettavan genren id
+     * @return Viite genreen halutulla id:llä
+     * TODO testit
+     */
+    public Genre hae(int id) {
+        for (int i = 0; i < alkiot.size(); i++) {
+            if (alkiot.get(i).getId() == id) return alkiot.get(i);
+        }
+        
+        throw new NoSuchElementException("Genreä id:llä " + id + " ei löydy!");
+    }
+    
+}

src/Kirjalista/KirjanGenre.java

@@ -11,15 +11,12 @@ public class KirjanGenre {
     
     /**
      * Luo olion
-     * @param kirjaId Kirjan id
-     * @param genreId Genren id
-     * <pre name="test">
-     
-     * </pre>
+     * @param kirja Kirja
+     * @param genre Genre
      */
-    public KirjanGenre(int kirjaId, int genreId) {
-        this.kirjaId = kirjaId;
-        this.genreId = genreId;
+    public KirjanGenre(Kirja kirja, Genre genre) {
+        this.kirjaId = kirja.getId();
+        this.genreId = genre.getId();
     }
     
     

src/Kirjalista/Kirjasto.java

 package Kirjalista;
 
-
+import java.util.ArrayList;
+import java.util.NoSuchElementException;
 
 /**
  * Luokka, joka keskustelee käyttöliittymän kanssa 
@@ -8,8 +9,8 @@ package Kirjalista;
  */
 public class Kirjasto {
     private Kirjat kirjat = new Kirjat();
-    private GenreRelaatio genreRelaatio;
-    //private Genret genret;
+    private GenreRelaatio genreRelaatio = new GenreRelaatio();
+    private Genret genret = new Genret();
     
     
     
@@ -23,25 +24,50 @@ public class Kirjasto {
     
     
     /**
-     * Ottaa taulukollisen genreId:itä ja liima ne kirjan id:n kanssa yhteen. Lisää lopuksi listaan
-     * @param kirjaId Kirjan id
-     * @param genreId Taulukko kaikista kirjan genrejen id:istä
+     * Lisää genret, jos sen nimisiä ei vielä ole olemassa
+     * @param genre Lisättävät genret
      */
-    public void lisaaGenreRelaatio(int kirjaId, int[] genreId) {
-        for (int i = 0; i < genreId.length; i++) {
-            genreRelaatio.lisaa(new KirjanGenre(kirjaId, genreId[i])); 
+    public void lisaaGenre(Genre[] genre) {
+        for (int i = 0; i < genre.length; i++) {
+            genret.lisaa(genre[i]);
         }
     }
     
-    // TODO haku ja poisto genreRelaatioille
+    /**
+     * Ottaa taulukollisen genrejä ja liima ne kirjan id:n kanssa yhteen
+     * @param kirja Kirja
+     * @param kirjanGenret Taulukko kaikista kirjan genreistä
+     */
+    public void lisaaGenreRelaatio(Kirja kirja, Genre[] kirjanGenret) {
+        for (int i = 0; i < kirjanGenret.length; i++) {
+            genreRelaatio.lisaa(new KirjanGenre(kirja, kirjanGenret[i])); 
+        }
+    }
+    
+    // TODO haku ja poisto genreRelaatioille ???
     
     /**
-     * @param indeksi Indeksipaikka, josta kirja haetaan
+     * @param idnro Haettavan kirjan id
      * @return Kirja annetussa indeksipaikassa
-     * @throws IndexOutOfBoundsException Jos indeksi laiton
+     * @throws NoSuchElementException Jos id laiton
+     */
+    public Kirja haeKirja(int idnro) throws NoSuchElementException {
+        return kirjat.hae(idnro);
+    }
+    
+    
+    /**
+     * Hakee taulukon kirjan genreistä kirjan id:n perusteella
+     * @param kirjaId Sen kirjan id, jonka genrejä haetaan
+     * @return Kaikki kirjan genret taulukossa
      */
-    public Kirja haeKirja(int indeksi) throws IndexOutOfBoundsException {
-        return kirjat.hae(indeksi);
+    public Genre[] haeGenret(int kirjaId) {
+        ArrayList<Integer> idt = genreRelaatio.haeGenret(kirjaId);
+        Genre[] palautettava = new Genre[idt.size()];
+        for (int i = 0; i < idt.size(); i++) {
+            palautettava[i] = genret.hae(idt.get(i));
+        }
+        return palautettava;
     }
     
     
@@ -51,9 +77,14 @@ public class Kirjasto {
     public int getKirjaLkm() {
         return this.kirjat.getLkm();
     }
-    /*
-    public Genre haeGenre(Kirja kirja) {
-        GenreRelaatio.haeGenret();
-    }
+
+    
+    /**
+     * Hakee kirjan indeksin perusteella
+     * @param ind Indeksipaikka, josta kirja haetaan
+     * @return Kirja annetussa indeksipaikassa
      */
+    public Kirja haeKirjaIndeksilla(int ind) {
+        return kirjat.haeIndeksilla(ind);
+    }
 }

src/Kirjalista/Kirjat.java

 package Kirjalista;
 
+import java.util.NoSuchElementException;
 
 /**
  * Monikkoluokka kirjoille.
@@ -10,7 +11,7 @@ package Kirjalista;
  */
 public class Kirjat {
     private int maxLkm = 8;
-    private int lkm;
+    private int lkm = 0;
     private Kirja[] alkiot = new Kirja[maxLkm];
     
     
@@ -21,21 +22,56 @@ public class Kirjat {
     public void lisaa(Kirja kirja) throws SailoException {
         if (this.lkm >= this.alkiot.length) throw new SailoException("Liikaa alkioita");
         this.alkiot[this.lkm] = kirja;
+        kirja.rekisteroi();
         this.lkm++;
     }
     
     
+    /**
+     * Poistaa kirjan annetulla id:llä
+     * @param id Poistettavan kirjan id
+     * @throws NoSuchElementException Jos id:tä vastaavaa kirjaa ei löydy
+     * TODO testit
+     */
+    public void poista(int id) throws NoSuchElementException {
+        for (int i = 0; i < alkiot.length; i++) {
+            if (alkiot[i].getId() == id) {
+                for (int j = i; j < alkiot.length - 1; j++) {
+                    alkiot[j] = alkiot[j + 1];
+                }
+                alkiot[alkiot.length] = null;
+                this.lkm--;
+                return;
+            }
+        }
+        throw new NoSuchElementException("Kirjaa id:llä " + id + " ei löydy.");
+    }
+
+    /**
+     * Hakee kirjan id:n perusteella
+     * @param id Halutun kirjan id
+     * @return Viite kirjaan halutulla id:llä
+     * @throws NoSuchElementException
+     * TODO testit
+     */
+    public Kirja hae(int id) throws NoSuchElementException {
+        
+        for (int i = 0; i < alkiot.length; i++) {
+            if (alkiot[i].getId() == id) return alkiot[i];
+        }
+        
+        throw new NoSuchElementException("Kirjaa id:llä " + id + " ei löytynyt.");
+    }
+
     
     /**
-     * Hakee kirjan indeksipaikassa i
-     * @param i Halutun kirjan indeksi
-     * @return viite kirjaan, jonka indeksi on i
-     * @throws IndexOutOfBoundsException jos indeksi liian suuri tai pieni  
+     * Hakee kirjan indeksin perusteella
+     * @param ind Indeksipaikka, josta kirja haetaan
+     * @return Kirja annetussa indeksipaikassa
      */
-    public Kirja hae(int i) throws IndexOutOfBoundsException {
-        if (i < 0 || this.lkm <= i)
-            throw new IndexOutOfBoundsException("Laiton indeksi: " + i);
-        return this.alkiot[i];
+    public Kirja haeIndeksilla(int ind) {
+        if (ind < 0 || ind >= this.lkm) throw new NullPointerException("Laiton indeksi");
+        return alkiot[ind];
     }
     
     

src/Kirjalista/test/GenreRelaatioTest.java

+package Kirjalista.test;
+// Generated by ComTest BEGIN
+import static org.junit.Assert.*;
+import org.junit.*;
+import Kirjalista.*;
+// Generated by ComTest END
+import tietorakenne.LinkattuLista;
+
+/**
+ * Test class made by ComTest
+ * @version 2025.03.05 22:52:50 // Generated by ComTest
+ *
+ */
+@SuppressWarnings({ "all" })
+public class GenreRelaatioTest {
+
+
+
+  // Generated by ComTest BEGIN
+  /** testHaeKirjat33 */
+  @Test
+  public void testHaeKirjat33() {    // GenreRelaatio: 33
+    GenreRelaatio gr = new GenreRelaatio(); 
+    gr.lisaa(new KirjanGenre(1, 2)); 
+    int[] t = gr.haeKirjat(2); 
+    assertEquals("From: GenreRelaatio line: 37", 1, t[0]); 
+  } // Generated by ComTest END
+}
\ No newline at end of file

src/Kirjalista/test/KirjanGenreTest.java

@@ -2,41 +2,16 @@ package Kirjalista.test;
 // Generated by ComTest BEGIN
 import static org.junit.Assert.*;
 import org.junit.*;
-
-import Kirjalista.Genre;
-import Kirjalista.Kirja;
-import Kirjalista.KirjanGenre;
+import Kirjalista.*;
+// Generated by ComTest END
+ * @version 2025.03.10 18:15:06 // Generated by ComTest
+package Kirjalista.test;
 
 /**
  * Test class made by ComTest
- * @version 2025.02.24 13:51:58 // Generated by ComTest
+ * @version 2025.03.10 18:15:06 // Generated by ComTest
  *
  */
 @SuppressWarnings({ "all" })
 public class KirjanGenreTest {
-
-
-
-  // Generated by ComTest BEGIN
-  /** testKirjanGenre16 */
-  @Test
-  public void testKirjanGenre16() {    // KirjanGenre: 16
-    Kirja k = new Kirja(); 
-    k.rekisteroi(); 
-    Kirja k1 = new Kirja(); 
-    k1.rekisteroi(); 
-    Genre g = new Genre(); 
-    g.rekisteroi(); 
-    Genre g1 = new Genre(); 
-    g1.rekisteroi(); 
-    KirjanGenre kg = new KirjanGenre(k, g); 
-    KirjanGenre kg1 = new KirjanGenre(k1, g1); 
-    KirjanGenre kg2 = new KirjanGenre(k, g1); 
-    assertEquals("From: KirjanGenre line: 31", k.getId(), kg.getKirjaId()); 
-    assertEquals("From: KirjanGenre line: 32", k1.getId(), kg1.getKirjaId());
-    assertEquals("From: KirjanGenre line: 33", k.getId(), kg2.getKirjaId()); 
-    assertEquals("From: KirjanGenre line: 35", g.getId(), kg.getGenreId()); 
-    assertEquals("From: KirjanGenre line: 36", g1.getId(), kg1.getGenreId()); 
-    assertEquals("From: KirjanGenre line: 37", g1.getId(), kg2.getGenreId()); 
-  } // Generated by ComTest END
 }
\ No newline at end of file

src/fxKirjalista/KirjalistaGUIController.java

@@ -5,6 +5,7 @@ import java.net.URL;
 import java.util.Optional;
 import java.util.ResourceBundle;
 
+import Kirjalista.Genre;
 import Kirjalista.Kirja;
 import Kirjalista.Kirjasto;
 import fi.jyu.mit.fxgui.Dialogs;
@@ -171,7 +172,7 @@ public class KirjalistaGUIController implements Initializable {
         
         int index = 0;
         for (int i = 0; i < kirjasto.getKirjaLkm(); i++) {
-            Kirja kirja = kirjasto.haeKirja(i);
+            Kirja kirja = kirjasto.haeKirjaIndeksilla(i);
             if (kirja.getId() == idnro) index = i;
             chooserKirjat.add(kirja.getNimi(), kirja);
         }
@@ -191,6 +192,10 @@ public class KirjalistaGUIController implements Initializable {
         areaKirja.setText("");
         try (PrintStream os = TextAreaOutputStream.getTextPrintStream(areaKirja)) {
             valittuKirja.tulosta(os);
+            os.println("=========================================");
+            for (Genre g : kirjasto.haeGenret(valittuKirja.getId())) {
+                g.tulosta(os);
+            }
         }
     }
     

src/fxKirjalista/LisaaController.java

 package fxKirjalista;
 
+import Kirjalista.Genre;
 import Kirjalista.Kirja;
 import Kirjalista.Kirjasto;
 import Kirjalista.SailoException;
@@ -73,15 +74,51 @@ public class LisaaController {
      * taulukkoon ja antaa virheen, jos ei onnistu.
      */
     protected void uusiKirja() {
+        /* Lopullisessa versiossa käyttöliittymästä tulisi 
+         * tekstiä, josta parsetaan kirjan tiedot ja tekstiä, josta
+         * parsetaan genret. 
+         */
+        
+        
         Kirja uusi = new Kirja();
+        Genre genre1 = new Genre();
+        Genre genre2 = new Genre();
+        Genre genre3 = new Genre();
+        // uusi.parse( käyttöliittymästä tullut teksti );
+        // 
+        // String[] genreStr = teksti.split("|");
+        // Genre[] genret = new Genre[genreStr.length];
+        // for (int i = 0; i < genreStr.length; i++) {
+        //     genre g = new Genre();
+        //     genret[i] = g.parse(genreStr[i]);
+        // }
+        // kirjasto.lisaaGenre(genret);
+        // 
+        // kirjasto.lisaaGenreRelaatio(uusi, genret);
+        //
+        // ^^^^ näin sitten toimivassa ohjelmassa ^^^^
+        
+
+        
         uusi.esimerkkiKirja();
-        uusi.rekisteroi();
+        genre1.esimerkkiGenre();
+        genre2.esimerkkiGenre();
+        genre3.esimerkkiGenre();
+        
+        
+        Genre[] g = {genre1, genre2, genre3};
+        
+        kirjasto.lisaaGenre(g);
+        
         try {
             kirjasto.lisaaKirja(uusi);
         } catch (SailoException e) {
             Dialogs.showMessageDialog("Ongelma " + e.getMessage());
             return;
         }
+        
+        kirjasto.lisaaGenreRelaatio(uusi, g);
+        
         isanta.hae(uusi.getId());
     
     }