desperation time

src/domain/kortit/Erikoiskortti.java

-package domain.kortit;
-
-public interface Erikoiskortti {
-
-}

src/domain/kortit/Jokerikortti.java

 package domain.kortit;
 
-public class Jokerikortti extends Kortti implements Erikoiskortti {
+public class Jokerikortti extends Kortti {
 	
 	public Jokerikortti(String vari) {
 		super(vari);
 	}
 
 	public String toString() {
-		return "jokerikortti";
+		return "[Jokeri, vri]";
 	}
 
 }

src/domain/kortit/Nosta4Jokerikortti.java

 package domain.kortit;
 
-public class Nosta4Jokerikortti extends Kortti implements Erikoiskortti {
+public class Nosta4Jokerikortti extends Kortti {
 
 	public Nosta4Jokerikortti(String vari) {
 		super(vari);
 	}
 
 	public String toString() {
-		return "nosta 4 -jokerikortti";
+		return "[Jokeri, +4]";
 	}
 }

src/domain/kortit/NostaKaksikortti.java

 package domain.kortit;
 
-public class NostaKaksikortti extends Kortti implements Erikoiskortti {
+public class NostaKaksikortti extends Kortti {
 	
 	
 	public NostaKaksikortti(String vari) {
@@ -8,6 +8,6 @@ public class NostaKaksikortti extends Kortti implements Erikoiskortti {
 	}
 
 	public String toString() {
-		return this.vari + " nosta kaksi -kortti";
+		return "[" + this.vari + ", +2]";
 	}
 }

src/domain/kortit/Ohituskortti.java

 package domain.kortit;
 
-public class Ohituskortti extends Kortti implements Erikoiskortti {
+public class Ohituskortti extends Kortti {
 
 	
 	public Ohituskortti(String vari) {
@@ -8,7 +8,7 @@ public class Ohituskortti extends Kortti implements Erikoiskortti {
 	}
 
 	public String toString() {
-		return this.vari + " ohituskortti";
+		return "[" + this.vari + ", ]";
 	}
 	
 }

src/domain/kortit/Peruskortti.java

@@ -14,7 +14,7 @@ public class Peruskortti extends Kortti {
 	}
 	
 	public String toString() {
-		return this.vari + " peruskortti numero " + this.numero;
+		return "[" + this.vari + ", " + this.numero + "]";
 	}
 	
 

src/domain/kortit/Suunnanvaihtokortti.java

 package domain.kortit;
 
-public class Suunnanvaihtokortti extends Kortti implements Erikoiskortti {
+public class Suunnanvaihtokortti extends Kortti {
 	
 	public Suunnanvaihtokortti(String vari) {
 		super(vari);
 	}
 
 	public String toString() {
-		return this.vari + " suunnanvaihtokortti";
+		return "[" + this.vari + ", <->]";
 	}
 }

src/domain/korttipakat/Nostopakka.java

@@ -26,13 +26,4 @@ public class Nostopakka {
 			throw new IndexOutOfBoundsException();
 		}
 	}
-	
-	public String toString() {
-		StringBuilder str = new StringBuilder("");
-		str.append("Nostopakassa on kortit:\n");
-		for (Kortti kortti : this.kortit) {
-			str.append(kortti + "\n");
-		}
-		return str.toString();
-	}
 }

src/domain/korttipakat/Poistopakka.java

@@ -20,13 +20,4 @@ public class Poistopakka {
 	public Kortti annaPaallimmainenKortti() {
 		return this.kortit.get(this.kortit.size()-1);
 	}
-	
-	public String toString() {
-		StringBuilder str = new StringBuilder("");
-		str.append("Poistopakassa olevat kortit:\n");
-		for (Kortti kortti : this.kortit) {
-			str.append(kortti + "\n");
-		}
-		return str.toString();
-	}
 }

src/domain/pelaajat/Pelaaja.java

@@ -72,13 +72,4 @@ public abstract class Pelaaja {
 		}
 		return summa;
 	}
-	
-	public String toString() {
-		StringBuilder str = new StringBuilder("");
-		str.append("Pelaajan " + this.nimi + " kortit:\n");
-		for (Kortti kortti : this.kortit) {
-			str.append(kortti + "\n");
-		}
-		return str.toString();
-	}
 }

src/logiikka/Peli.java

@@ -31,10 +31,10 @@ public class Peli {
 	
 	public Peli() {
 		this.pelaajat = new ArrayList<>();
-		this.pelaajat.add(new Ihmispelaaja("pelaaja")); // pelaaja nro 0
-		this.pelaajat.add(new Tietokonepelaaja("tietokone 1"));
-		this.pelaajat.add(new Tietokonepelaaja("tietokone 2"));		
-		this.pelaajat.add(new Tietokonepelaaja("tietokone 3"));
+		this.pelaajat.add(new Ihmispelaaja("Pelaaja")); // pelaaja nro 0
+		this.pelaajat.add(new Tietokonepelaaja("Tietokone 1"));
+		this.pelaajat.add(new Tietokonepelaaja("Tietokone 2"));		
+		this.pelaajat.add(new Tietokonepelaaja("Tietokone 3"));
 
 		
 		this.nostopakka = new Nostopakka();
@@ -158,10 +158,12 @@ public class Peli {
 		 * ottamaan vastaan Nosta kaksi -korttia tai Nosta 4-jokeri -korttia.
 		 */
 		if (kortti instanceof Jokerikortti) {
-			System.out.println("Valitse vri (sininen, punainen, keltainen, vihre):");
-			System.out.print(">");
-			String vari = lukija.next();
-			this.vari = vari;
+			do{
+				System.out.println("Valitse vri (sininen, punainen, keltainen, vihre):");
+				System.out.print(">");
+				String vari = lukija.next();
+				this.vari = vari;
+			}while (!(this.vari.equals("sininen")||this.vari.equals("punainen")||this.vari.equals("keltainen")||this.vari.equals("vihre")));
 			this.paivitaVuoronumero();			
 			kelvollinenKortti = true;
 		} 
@@ -186,10 +188,12 @@ public class Peli {
 				return false;
 			}
 			this.annaPelaaja(seuraavaksiVuorossa).nostaNeljaKorttia(this.nostopakka);
-			System.out.println("Valitse vri (sininen, punainen, keltainen, vihre):");
-			System.out.print(">");
-			String vari = lukija.next();
-			this.vari = vari;
+			do{
+				System.out.println("Valitse vri (sininen, punainen, keltainen, vihre):");
+				System.out.print(">");
+				String vari = lukija.next();
+				this.vari = vari;
+			}while (!(this.vari.equals("sininen")||this.vari.equals("punainen")||this.vari.equals("keltainen")||this.vari.equals("vihre")));
 			this.paivitaVuoronumero();
 			kelvollinenKortti = true;
 		} 
@@ -202,7 +206,7 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof NostaKaksikortti) {
-			if (!edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
 					|| edellinenKortti instanceof NostaKaksikortti) {
 				System.out.println("Nosta 2 -kortin voi lyd samanvrisen kortin plle tai"
 						+ " toisen nosta 2 -kortin plle.");
@@ -219,7 +223,7 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof Ohituskortti) {
-			if (!edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
 					|| edellinenKortti instanceof Ohituskortti) {
 				System.out.println("Ohituskortin saa lyd samanvrisen kortin plle tai"
 						+ " toisen ohituskortin plle.");
@@ -236,18 +240,20 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof Suunnanvaihtokortti) {
-			if (!edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
 					|| edellinenKortti instanceof Suunnanvaihtokortti) {
 				System.out.println("Suunnanvaihtokortin saa lyd samanvrisen kortin plle tai"
 						+ " toisen suunnanvaihtokortin plle.");
 			}
 			this.suuntaKasvava = !this.suuntaKasvava;
+			paivitaVuoronumero();
+			paivitaVuoronumero();
 			kelvollinenKortti = true;
 		} 
 		
 		else if (kortti instanceof Peruskortti) {
-			if (edellinenKortti.annaVari() != null) {
-				if (!kortti.annaVari().equals(edellinenKortti.annaVari())
+			if (edellinenKortti != null && edellinenKortti.annaVari() != null) {
+				if (!kortti.annaVari().equals(edellinenKortti.annaVari()) 
 						&& edellinenKortti instanceof Peruskortti) {
 					if (((Peruskortti) kortti).annaNumero() != ((Peruskortti) edellinenKortti).annaNumero()) {
 						//Kortti on peruskortti, mutta erivrinen ja erinumeroinen kuin edellinen kortti
@@ -258,10 +264,10 @@ public class Peli {
 				if(!kortti.annaVari().equals(edellinenKortti.annaVari())) {
 					System.out.println("Kortin tulee olla samaa vri tai sama numero.");
 					return false;					
-				} else {
-					kelvollinenKortti = true;				
-				}	
-			} 		
+				}
+			} else {
+				kelvollinenKortti = true;				
+			}			
 		}
 		
 		this.vari = null;
@@ -280,7 +286,6 @@ public class Peli {
 				return false;
 			}
 			this.vari = null;
-			System.out.println(pelaaja.annaNimi() + " pelasi kortin: " + kortti);
 			return true;
 		}
 		
@@ -339,7 +344,6 @@ public class Peli {
 				}
 			} 
 			if (pelaajallaSopivanVarinenKortti) {
-				System.out.println("Et voi kytt Nosta 4 -korttia, jos sinulla on sopivan vrinen kortti.");
 				return false;
 			}
 			this.annaPelaaja(seuraavaksiVuorossa).nostaNeljaKorttia(this.nostopakka);
@@ -360,7 +364,7 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof NostaKaksikortti) {
-			if (!edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
 					|| edellinenKortti instanceof NostaKaksikortti) {
 				return false;
 			}
@@ -376,7 +380,7 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof Ohituskortti) {
-			if (!edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
 					|| edellinenKortti instanceof Ohituskortti) {
 				return false;
 			}
@@ -392,16 +396,18 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof Suunnanvaihtokortti) {
-			if (!edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
 					|| edellinenKortti instanceof Suunnanvaihtokortti) {
 			}
 			this.suuntaKasvava = !this.suuntaKasvava;
+			paivitaVuoronumero();
+			paivitaVuoronumero();
 			kelvollinenKortti = true; 
 			System.out.println(pelaaja.annaNimi() + " pelasi kortin: " + kortti);
 		} 
 		
 		else if (kortti instanceof Peruskortti) {
-			if (edellinenKortti.annaVari() != null) {
+			if (edellinenKortti != null && edellinenKortti.annaVari() != null) {
 				if (!kortti.annaVari().equals(edellinenKortti.annaVari()) 
 						&& edellinenKortti instanceof Peruskortti) {
 					if (((Peruskortti) kortti).annaNumero() != ((Peruskortti) edellinenKortti).annaNumero()) {

src/ui/Tekstikayttoliittyma.java

 package ui;
 
-import java.io.File;
-import java.io.FileNotFoundException;
-import java.io.FileOutputStream;
-import java.io.OutputStreamWriter;
-import java.io.PrintWriter;
-import java.nio.charset.StandardCharsets;
 import java.util.List;
 import java.util.Scanner;
 
-import domain.kortit.Erikoiskortti;
 import domain.kortit.Kortti;
 import domain.korttipakat.Nostopakka;
 import domain.korttipakat.Poistopakka;
@@ -27,7 +20,7 @@ public class Tekstikayttoliittyma {
 	public void start() {
 		
 		System.out.println("Tervetuloa pelaamaan Uunoa. Kirjoita \"uusi\" aloittaaksesi uuden pelin,");
-		System.out.println(" \"lataa\" ladataksesi vanhan pelin tai 'lopeta' lopettaaksesi.");
+		System.out.println(" \"lataa\" ladataksesi vanhan pelin, \"ohjeet\" lukeaksesi peliohjeita tai 'lopeta' lopettaaksesi.");
 		System.out.print("> ");
 		String input = this.lukija.next();
 		
@@ -36,6 +29,13 @@ public class Tekstikayttoliittyma {
 			return;
 		case "uusi":
 			this.uusiPeli();
+			break;
+		case "lataa":
+//			lataaPeli();
+			break;
+		case "ohjeet":
+//			naytaPeliohjeet();
+			break;
 		default:
 			this.uusiPeli();
 		}
@@ -49,27 +49,17 @@ public class Tekstikayttoliittyma {
 			Pelaaja pelaaja = peli.annaPelaaja(0);
 			Poistopakka poistopakka = peli.annaPoistopakka();
 			Nostopakka nostopakka = peli.annaNostopakka();
-			
-			try {
-				PrintWriter tiedosto = new PrintWriter(new File("pelitilanne.txt"));
-				tiedosto.close();
-			} catch (Exception e) {
-				// TODO: handle exception
-			}
-			
+		
 			Kortti ensimmainenKortti = poistopakka.annaPaallimmainenKortti();
 			
-			while (ensimmainenKortti instanceof Erikoiskortti) {
-				ensimmainenKortti = nostopakka.nostaKortti();
-				poistopakka.lisaaKortti(ensimmainenKortti);
-			}
-						
+			peli.pelaaKorttiTietokone(lukija, ensimmainenKortti, null, pelaaja);
+			
 			boolean peliKesken = true;
 			while (peliKesken) {
 			
 			Pelaaja peliVuorossa = peli.annaPelaaja(peli.annaSeuraavaksiVuorossa());
-			System.out.println("Pelivuorossa on: " + peliVuorossa.annaNimi() 
-			+ " (Pisteet: " + peliVuorossa.laskePisteet() + ")");
+//			System.out.println("Pelivuorossa on: " + peliVuorossa.annaNimi() 
+//			+ " (Pisteet: " + peliVuorossa.laskePisteet() + ")");
 			System.out.println();
 			
 			
@@ -91,7 +81,7 @@ public class Tekstikayttoliittyma {
 					pelaaja.tulostaKortit();
 					
 					System.out.println("Kirjoita kortin numero pelataksesi kortin tai \"nosta\" nostaaksesi kortin.");
-					System.out.print(">");
+					System.out.print("> ");
 					String input = this.lukija.next();
 					List<Kortti> kortit = pelaaja.annaKortit();
 					if (input.equals("nosta")) {
@@ -126,6 +116,8 @@ public class Tekstikayttoliittyma {
 					if (!kelvollinenKortti) {
 						if (kortinNumero == peliVuorossa.annaKortit().size()-1) {
 							peliVuorossa.nostaKortti(nostopakka); //ei sopivaa korttia
+//							System.out.println((peli.annaPelaaja(peli.annaSeuraavaksiVuorossa()).annaNimi()) + " nosti yhden kortin"); //Faulty, saved for syntax
+							System.out.println(peliVuorossa.annaNimi() + " nosti yhden kortin"); //Vaikutti rikkovan jotain? En tied varmaksi. Nyt nytt jo toimivan
 							kelvollinenKortti = true;
 						} else {
 							kortinNumero++;
@@ -134,32 +126,9 @@ public class Tekstikayttoliittyma {
 						poistopakka.lisaaKortti(k);
 						kortit.remove(kortinNumero);
 					}
-				}		
-			}
-			//Tulostaminen tiedostoon
-			try {
-				
-				File tiedosto = new File("pelitilanne.txt");
-				OutputStreamWriter kirjoittaja =
-			             new OutputStreamWriter(new FileOutputStream(tiedosto), StandardCharsets.UTF_8);
-				
-				kirjoittaja.append(peli.annaPelaaja(0).toString());
-				kirjoittaja.append("\n");
-				kirjoittaja.append(peli.annaPelaaja(1).toString());
-				kirjoittaja.append("\n");
-				kirjoittaja.append(peli.annaPelaaja(2).toString());
-				kirjoittaja.append("\n");
-				kirjoittaja.append(peli.annaPelaaja(3).toString());
-				kirjoittaja.append("\n");
-				kirjoittaja.append(nostopakka.toString());
-				kirjoittaja.append("\n");
-				kirjoittaja.append(poistopakka.toString());
+				}
 				
-				kirjoittaja.close();
-			} catch (Exception e) {
-				// TODO: handle exception
 			}
-
 		}
 	}	
 }