Ihan vaan niin kuin muuten vaan palautteesta gitille, 1/5 tähteä, elämäni kehnoin kokemus

(cherry picked from commit f1cff9d2)

src/domain/kortit/Erikoiskortti.java

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

src/domain/kortit/Jokerikortti.java

 package domain.kortit;
 
-public class Jokerikortti extends Kortti {
+public class Jokerikortti extends Kortti implements Erikoiskortti {
 	
 	public Jokerikortti(String vari) {
 		super(vari);

src/domain/kortit/Nosta4Jokerikortti.java

 package domain.kortit;
 
-public class Nosta4Jokerikortti extends Kortti {
+public class Nosta4Jokerikortti extends Kortti implements Erikoiskortti {
 
 	public Nosta4Jokerikortti(String vari) {
 		super(vari);

src/domain/kortit/NostaKaksikortti.java

 package domain.kortit;
 
-public class NostaKaksikortti extends Kortti {
+public class NostaKaksikortti extends Kortti implements Erikoiskortti {
 	
 	
 	public NostaKaksikortti(String vari) {

src/domain/kortit/Ohituskortti.java

 package domain.kortit;
 
-public class Ohituskortti extends Kortti {
+public class Ohituskortti extends Kortti implements Erikoiskortti {
 
 	
 	public Ohituskortti(String vari) {

src/domain/kortit/Suunnanvaihtokortti.java

 package domain.kortit;
 
-public class Suunnanvaihtokortti extends Kortti {
+public class Suunnanvaihtokortti extends Kortti implements Erikoiskortti {
 	
 	public Suunnanvaihtokortti(String vari) {
 		super(vari);

src/domain/korttipakat/Nostopakka.java

@@ -26,4 +26,13 @@ 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,4 +20,13 @@ 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,4 +72,13 @@ 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

@@ -202,7 +202,7 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof NostaKaksikortti) {
-			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (!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 +219,7 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof Ohituskortti) {
-			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (!edellinenKortti.annaVari().equals(kortti.annaVari())
 					|| edellinenKortti instanceof Ohituskortti) {
 				System.out.println("Ohituskortin saa lyd samanvrisen kortin plle tai"
 						+ " toisen ohituskortin plle.");
@@ -236,7 +236,7 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof Suunnanvaihtokortti) {
-			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (!edellinenKortti.annaVari().equals(kortti.annaVari())
 					|| edellinenKortti instanceof Suunnanvaihtokortti) {
 				System.out.println("Suunnanvaihtokortin saa lyd samanvrisen kortin plle tai"
 						+ " toisen suunnanvaihtokortin plle.");
@@ -246,7 +246,7 @@ public class Peli {
 		} 
 		
 		else if (kortti instanceof Peruskortti) {
-			if (edellinenKortti != null && edellinenKortti.annaVari() != null) {
+			if (edellinenKortti.annaVari() != null) {
 				if (!kortti.annaVari().equals(edellinenKortti.annaVari())
 						&& edellinenKortti instanceof Peruskortti) {
 					if (((Peruskortti) kortti).annaNumero() != ((Peruskortti) edellinenKortti).annaNumero()) {
@@ -258,11 +258,11 @@ 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;				
 				}	
 			} 		
+		}
 		
 		this.vari = null;
 		return kelvollinenKortti;
@@ -280,6 +280,7 @@ public class Peli {
 				return false;
 			}
 			this.vari = null;
+			System.out.println(pelaaja.annaNimi() + " pelasi kortin: " + kortti);
 			return true;
 		}
 		
@@ -359,7 +360,7 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof NostaKaksikortti) {
-			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (!edellinenKortti.annaVari().equals(kortti.annaVari())
 					|| edellinenKortti instanceof NostaKaksikortti) {
 				return false;
 			}
@@ -375,7 +376,7 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof Ohituskortti) {
-			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (!edellinenKortti.annaVari().equals(kortti.annaVari())
 					|| edellinenKortti instanceof Ohituskortti) {
 				return false;
 			}
@@ -391,7 +392,7 @@ public class Peli {
 		 */
 		
 		else if (kortti instanceof Suunnanvaihtokortti) {
-			if (edellinenKortti != null && !edellinenKortti.annaVari().equals(kortti.annaVari())
+			if (!edellinenKortti.annaVari().equals(kortti.annaVari())
 					|| edellinenKortti instanceof Suunnanvaihtokortti) {
 			}
 			this.suuntaKasvava = !this.suuntaKasvava;
@@ -400,7 +401,7 @@ public class Peli {
 		} 
 		
 		else if (kortti instanceof Peruskortti) {
-			if (edellinenKortti != null && edellinenKortti.annaVari() != null) {
+			if (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;
@@ -20,7 +27,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, \"ohjeet\" lukeaksesi peliohjeita tai 'lopeta' lopettaaksesi.");
+		System.out.println(" \"lataa\" ladataksesi vanhan pelin tai 'lopeta' lopettaaksesi.");
 		System.out.print("> ");
 		String input = this.lukija.next();
 		
@@ -29,13 +36,6 @@ public class Tekstikayttoliittyma {
 			return;
 		case "uusi":
 			this.uusiPeli();
-			break;
-		case "lataa":
-//			lataaPeli();
-			break;
-		case "ohjeet":
-//			naytaPeliohjeet();
-			break;
 		default:
 			this.uusiPeli();
 		}
@@ -50,9 +50,19 @@ public class Tekstikayttoliittyma {
 			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();
 			
-			peli.pelaaKorttiTietokone(lukija, ensimmainenKortti, null, pelaaja);
+			while (ensimmainenKortti instanceof Erikoiskortti) {
+				ensimmainenKortti = nostopakka.nostaKortti();
+				poistopakka.lisaaKortti(ensimmainenKortti);
+			}
 						
 			boolean peliKesken = true;
 			while (peliKesken) {
@@ -125,8 +135,31 @@ public class Tekstikayttoliittyma {
 						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
+			}
+
 		}
 	}	
 }