viimeistelyä

Miinaharava/ennatykset.s

@@ -1,9 +1,9 @@
-1:DEFAULT1:500000
-2:DEFAULT2:200000
-3:DEFAULT3:150000
-4:DEFAULT4:100000
-5:DEFAULT5:50000
-6:DEFAULT6:20000
-7:DEFAULT7:10000
-8:DEFAULT8:5000
-9:DEFAULT9:1000
+defaultX:0
+defaultX:0
+default0:1000000
+default0:1000000
+default0:1000000
+default0:1000000
+default0:1000000
+default0:1000000
+default0:1000000

Miinaharava/ennatyksetDefault.s

@@ -1,9 +1,9 @@
-1:DEFAULT1:500000
-2:DEFAULT2:200000
-3:DEFAULT3:150000
-4:DEFAULT4:100000
-5:DEFAULT5:50000
-6:DEFAULT6:20000
-7:DEFAULT7:10000
-8:DEFAULT8:5000
-9:DEFAULT9:1000
+default:500000
+default:200000
+default:150000
+default:100000
+default:50000
+default:20000
+default:10000
+default:5000
+default:1000

Miinaharava/pom.xml

@@ -42,9 +42,13 @@
 				<configuration>
 				<targetClasses>
 					<param>miinaharava.logiikka.*</param>
+					<param>miinaharava.ajastin.*</param>
+					<param>miinaharava.ennatykset.*</param>
 				</targetClasses>
 				<targetTests>
 					<param>miinaharava.logiikka.*</param>
+					<param>miinaharava.ajastin.*</param>
+					<param>miinaharava.ennatykset.*</param>
 				</targetTests>
 			</configuration>
 			</plugin>

Miinaharava/src/main/java/miinaharava/ajastin/Ajastin.java

@@ -3,22 +3,22 @@
 import java.util.Timer;
 import miinaharava.kayttoliittyma.Kayttoliittyma;
 
-/*
-*   Luokan tehtävänä on luoda sekunnin välein kutsuttava AjastinTehtava ja pysayttaa se tarvittaessa
+/**
+ *   Luokan tehtävänä on luoda sekunnin välein kutsuttava AjastinTehtava ja pysayttaa se tarvittaessa.
 */
 public class Ajastin extends Timer{
 
-    /*
-    *   Luo AjastinTehtavan ja scheduloi sen suoritettavaksi sekunnin välein
-    *   @param kl Käyttöliittymä jota päivitetään ja joka sisältää viitteen ajastintehtävän tarvitsemaan miinaharava-luokan ilmentymän.
-    *   @see miinaharava.ajastin.AjastinTehtava
+    /**
+     *   Luo AjastinTehtavan ja scheduloi sen suoritettavaksi sekunnin välein.
+     *   @param kl Käyttöliittymä jota päivitetään ja joka sisältää viitteen ajastintehtävän tarvitsemaan miinaharava-luokan ilmentymän
+     *   @see miinaharava.ajastin.AjastinTehtava
     */
-    public void KaynnistaAjastin(Kayttoliittyma kl){
+    public void kaynnistaAjastin(Kayttoliittyma kl){
         scheduleAtFixedRate(new AjastinTehtava(kl), 1000, 1000);
     }
 
-    /*
-    *   Poistaa kaikki ajastimelle scheduloidut ajastintehtävät
+    /**
+     *   Poistaa kaikki ajastimelle scheduloidut ajastintehtävät.
     */
     public void pysaytaAjastin(){
         cancel();

Miinaharava/src/main/java/miinaharava/ajastin/AjastinTehtava.java

@@ -3,8 +3,8 @@
 import java.util.TimerTask;
 import miinaharava.kayttoliittyma.Kayttoliittyma;
 
-/*
-*   Ajastimen käyttämä luokka, jonka tehtävä on päivittää miinaharavan aika-muuttujaa ja päivittää käyttöliittymää sekunnin välein.
+/**
+ *   Ajastimen käyttämä luokka, jonka tehtävä on päivittää miinaharavan aika-muuttujaa ja päivittää käyttöliittymää sekunnin välein.
 */
 public class AjastinTehtava extends TimerTask{
     private final Kayttoliittyma kl;
@@ -13,10 +13,10 @@ public AjastinTehtava(Kayttoliittyma kl){
         this.kl = kl;
     }
 
-    /*
-    *   Kasvattaa aikaa yhdellä, ja päivittää käyttöliittymän.
-    *   @see miinaharava.logiikka.Miinaharava#kasvataAikaa()
-    *   @see miinaharava.kayttoliittyma.Kayttoliittyma#paivita()
+    /**
+     *   Kasvattaa aikaa yhdellä, ja päivittää käyttöliittymän.
+     *   @see miinaharava.logiikka.Miinaharava#kasvataAikaa()
+     *   @see miinaharava.kayttoliittyma.Kayttoliittyma#paivita()
     */
     @Override
     public void run() {

Miinaharava/src/main/java/miinaharava/ennatykset/Ennatys.java

@@ -1,21 +1,23 @@
 package miinaharava.ennatykset;
 
+import java.awt.EventQueue;
 import java.io.File;
 import java.io.FileWriter;
 import java.util.*;
+import miinaharava.kayttoliittyma.Virheilmoitus;
 
 
-/*
+/**
 *   Luokan tehtävä on käsitellä tiedostoa joka sisältää ennätykset.
 */
 public class Ennatys {
-    private String[] rivit;
+    private final String[] rivit;
 
     public Ennatys(){
         this.rivit = new String[9];
     }
 
-    /*
+    /**
     *   Tallentaa tiedostoon ennätyksen mikäli pelaajan pisteet ovat korkeammat kuin jollain aikaisemmalla ennätyksellä.
     *   @param nimi Pelaajan nimi
     *   @param pisteet Pelaajan pisteet
@@ -31,7 +33,7 @@ public void tallennaEnnatys(String nimi, int pisteet){
         tallennaTiedostoon(tiedosto);
     }
 
-    /*
+    /**
     *   Lukee tiedoston "ennatykset.s" sisällön rivi kerrallaan rivit[] - taulukkoon käyttäen lueTiedosto-metodia ja palauttaa saadun taulukon.
     *   @return Palauttaa taulukon jossa nykyiset ennätykset ovat
     *   @see miinaharava.ennatykset.Ennatys#lueTiedosto(File)
@@ -43,15 +45,19 @@ public String[] lueEnnatykset(){
         return this.rivit;
     }
 
-    /*
-    *   Lukee tiedoston "ennatykset.s" sisällön rivi kerrallaan rivit[] - taulukkoon.
+    /**
+    *   Lukee tiedoston "ennatykset.s" sisällön rivi kerrallaan rivit[] - taulukkoon. Jos tiedosto oli tyhjä täytetään se tiedostosta "ennatyksetDefault.s"
+    *   Mikäli tiedostoa ei löytyny, kutsuu kaynnistaVirheIlmoitus-metodia.
+    *   @param tiedosto luettava tiedosto
+    *   @see miinaharava.ennatykset.Ennatys#kaynnistaVirheIlmoitus()
     */
     private void lueTiedosto(File tiedosto){
         Scanner s;
 
         try{
             s = new Scanner(tiedosto);
         }catch(Exception e){
+            kaynnistaVirheilmoitus();
             return;
         }
 
@@ -64,9 +70,14 @@ private void lueTiedosto(File tiedosto){
         }
 
        s.close();
+
+       if(this.rivit[8] == null){
+            lueTiedosto(new File("ennatyksetDefault.s"));
+            tallennaTiedostoon(tiedosto);
+       }
     }
 
-    /*
+    /**
     *   Korvaa ennätyksen rivit[] taulukkoon mikäli pelaajan pisteet ovat korkeammat kuin jollain aikaisemmalla ennätyksellä.
     *   @param nimi Pelaajan nimi
     *   @param pisteet Pelaajan pisteet
@@ -76,17 +87,31 @@ private boolean korvaaEnnatys(String nimi, int pisteet){
         for(int i=0; i<this.rivit.length; i++){
             String[] rivi = this.rivit[i].split(":");
 
-            if(pisteet > Integer.parseInt(rivi[2])){
-                this.rivit[i] = i + ":" + nimi + ":" + pisteet;
+            if(pisteet > Integer.parseInt(rivi[1])){
+                this.rivit[i] = nimi + ":" + pisteet;
+
+                String e = rivi[0] + ":" + rivi[1];
+                loputYhdellaEteenpain(i+1, e);
                 return true;
             }
         }
 
         return false;
     }
 
-    /*
-    *   Tallentaa rivit-taulukon sisällön tiedostoon "ennatykset.s"
+    private void loputYhdellaEteenpain(int alkuindeksi, String ensimmainen){
+        String edellinen = ensimmainen;
+        for(int i=alkuindeksi; i<this.rivit.length; i++){
+            String vali = this.rivit[i];
+            this.rivit[i] = edellinen;
+            edellinen = vali;
+        }
+    }
+
+    /**
+    *   Tallentaa rivit-taulukon sisällön tiedostoon "ennatykset.s".
+    *   Mikäli tiedostoa ei löytyny, kutsuu kaynnistaVirheIlmoitus-metodia.
+    *   @see miinaharava.ennatykset.Ennatys#kaynnistaVirheIlmoitus()
     */
     private void tallennaTiedostoon(File tiedosto){
         FileWriter fw;
@@ -97,6 +122,22 @@ private void tallennaTiedostoon(File tiedosto){
                 fw.write(s + "\n");
             }
             fw.close();
-        }catch(Exception e){}
+        }catch(Exception e){
+            kaynnistaVirheilmoitus();
+        }
+    }
+
+    /**
+    *   Käynnistää virheilmoituksen ilmoittamaan tiedoston puuttumisesta.
+    *   @see miinaharava.kayttoliittyma.Virheilmoitus
+    */
+    private void kaynnistaVirheilmoitus(){
+        EventQueue.invokeLater(new Runnable(){
+                @Override
+                public void run(){
+                    Virheilmoitus v = new Virheilmoitus("Tiedostoa \"ennatykset.s\" ei löytynyt.");
+                    v.setVisible(true);
+                }
+            });
     }
 }

Miinaharava/src/main/java/miinaharava/kayttoliittyma/Asetukset.java

@@ -1,13 +1,14 @@
 package miinaharava.kayttoliittyma;
 
 import java.awt.*;
-import java.awt.event.ActionEvent;
-import java.awt.event.ActionListener;
+import java.awt.event.*;
 import javax.swing.*;
-import miinaharava.logiikka.Miinaharava;
 
+/**
+*   Luokka sisältää asetukset-ikkunan käyttöliittymän.
+*/
 public class Asetukset extends JFrame{
-    private Kayttoliittyma kl;
+    private final Kayttoliittyma kl;
 
     public Asetukset(Kayttoliittyma kl){
         this.kl = kl;

Miinaharava/src/main/java/miinaharava/kayttoliittyma/EnnatysIkkuna.java

@@ -4,6 +4,9 @@
 import javax.swing.*;
 import miinaharava.ennatykset.Ennatys;
 
+/**
+*   Luokka sisältää ikkunan, jossa ennäykset näytetään pelaajalle.
+*/
 public class EnnatysIkkuna extends JFrame{
 
     public EnnatysIkkuna(){
@@ -29,10 +32,10 @@ private void luoKomponentit(Container pane){
         for(String s:ennatykset){
             String[] jaettu = s.split(":");
 
-            String tulos = jaettu[0] + ". " + jaettu[1];
+            String tulos = jaettu[0];
 
             JTextField uusi = new JTextField(tulos);
-            JTextField pisteet = new JTextField(jaettu[2]);            
+            JTextField pisteet = new JTextField(jaettu[1]);            
             uusi.setEditable(false);
             pisteet.setEditable(false);
             pane.add(uusi);

Miinaharava/src/main/java/miinaharava/kayttoliittyma/HiiriKuuntelija.java

@@ -3,6 +3,9 @@
 import java.awt.event.MouseEvent;
 import java.awt.event.MouseListener;
 
+/**
+*   Luokka sisältää hiirikuuntelijan, joka ottaa vastaan pelaajan syötteen ja laittaa sen eteenpäin käyttöliittymälle.
+*/
 public class HiiriKuuntelija implements MouseListener{
     private final Kayttoliittyma kl;
 
@@ -12,7 +15,12 @@ public HiiriKuuntelija(Kayttoliittyma kl){
 
     @Override
     public void mouseClicked(MouseEvent e) {}
-
+
+    /**
+    *   Ilmoittaa käyttöliittymälle että ruutua on klikattu joko vasemmalla tai oikealla painikkeella.
+    *   @see miinaharava.kayttoliittyma.Kayttoliittyma#ruutuKlikattuVasen(int, int)
+    *   @see miinaharava.kayttoliittyma.Kayttoliittyma#ruutuKlikattuOikea(int, int)
+    */
     @Override
     public void mousePressed(MouseEvent e) {
         if(e.getX() < 20 || e.getY() < 20){