Bug Fixing

NichtLeeralsohinzufügen Test verbessert
NichtLeeralsohinzufügen Fehler korrigiert

src/main/java/com/ugsbo/VokableKartenSchreiber/Vokabelkarte.java

@@ -36,22 +36,20 @@ public class Vokabelkarte{
     /***
      * 
      * @param Zwischenwert		fügt ein Zwischenstück zwischen den beiden Strings ein
-     * @param prüfen	Der zu prüfende String
-     * @param Rückgabe	Der String an den angehängt werden soll
+     * @param neuesWort	Der zu prüfende String
+     * @param bisherigerString	Der String an den angehängt werden soll
      * @return	Der fertige String
      */
 	
 	
 	//-------------------------------------------------
 	//Private
-    public String NichtLeeralsohinzufügen(String prüfen, String Rückgabe, String Zwischenwert) {
-    	if (prüfen != "") {
-    		if(Rückgabe!="") {
-                prüfen += Zwischenwert;
-    		}
-    		Rückgabe += prüfen;
-    	}
-    	return Rückgabe;
+    public String NichtLeeralsohinzufügen(String neuesWort, String bisherigerString, String Zwischenwert) {    	
+    	if (!bisherigerString.equals("") && !neuesWort.equals("")) {
+			neuesWort += Zwischenwert;
+		}
+    	bisherigerString += neuesWort;
+    	return bisherigerString;
     }
     
 	//-------------------------------------------------
@@ -86,7 +84,6 @@ public class Vokabelkarte{
     	Ergebnis = NichtLeeralsohinzufügen(Ergebnis, Antwort2, ";");
     	Ergebnis = NichtLeeralsohinzufügen(Ergebnis, Antwort3, ";");
     	Ergebnis = NichtLeeralsohinzufügen(Ergebnis, Antwort4, ";");
-    	
     	this.Ergebnis = Ergebnis;
     	return Ergebnis;
     }

src/test/java/com/ugsbo/VokabelKartenSchreiber/NichtLeeralsohinzufügenTest.java

@@ -2,60 +2,74 @@ package com.ugsbo.VokabelKartenSchreiber;
 
 import static org.junit.Assert.assertEquals;
 
+import org.junit.Before;
 import org.junit.Test;
 
 import com.ugsbo.VokableKartenSchreiber.Vokabelkarte;
 
 public class NichtLeeralsohinzufügenTest {
 	
-	@Test
-	public void NichtAlsLeeralsoHinzufügenTest() {
-		Vokabelkarte WorkingObjekt = new Vokabelkarte();
-		
-		String Ergebnis = 	"RückgabeTestprüfenTestZwischenwertTest";
-		String Test = 		WorkingObjekt.NichtLeeralsohinzufügen("prüfenTest","RückgabeTest","ZwischenwertTest");
-		
-		assertEquals("NichtLeerAlsSolcheHinzufügen funktioniert nicht", Ergebnis, Test);
-	}
+	private Vokabelkarte workingObjekt;
+
 	
+	@Before
+	public void setup() {
+		workingObjekt = new Vokabelkarte();
+	}
+
 	@Test
-	public void NichtAlsLeeralsoHinzufügenRückgabeLeer() {
-		Vokabelkarte WorkingObjekt = new Vokabelkarte();
+	public void NichtAlsLeeralsoHinzufügen_TestMitAllenParametern() {
+		Vokabelkarte WorkingObjekt = workingObjekt;
 		
-		String Ergebnis = 	"RückgabeTestprüfenTestZwischenwertTest";
-		String Test = 		WorkingObjekt.NichtLeeralsohinzufügen("prüfenTest","RückgabeTest","ZwischenwertTest");
+		String neuesWort = "prüfenTest";
+		String wortfolgeBisher = "RückgabeTest";
+		String separator = "ZwischenwertTest";
+		String erwartet = 	"RückgabeTestprüfenTestZwischenwertTest";
+
+		String ergebnis = 		WorkingObjekt.NichtLeeralsohinzufügen(neuesWort,wortfolgeBisher,separator);
 		
-		assertEquals("NichtLeerAlsSolcheHinzufügen funktioniert nicht", Ergebnis, Test);
-		assertEquals("NichtLeerAlsSolcheHinzufügen funktioniert nicht", WorkingObjekt.NichtLeeralsohinzufügen("prüfenTest","","ZwischenwertTest"), "prüfenTest");
+		assertEquals("Seperator angehängt + Neues Wort", erwartet, ergebnis);
 	}
 
 	@Test
-	public void NichtAlsLeeralsoHinzufügenprüfenLeer() {
-		Vokabelkarte WorkingObjekt = new Vokabelkarte();
+	public void NichtAlsLeeralsoHinzufügen_neuesWortIstLeer() {
+		Vokabelkarte WorkingObjekt = workingObjekt;
 		
-		String Ergebnis = 	"RückgabeTest";
-		String Test = 		WorkingObjekt.NichtLeeralsohinzufügen("","RückgabeTest","ZwischenwertTest");
+		String neuesWort = "";
+		String wortfolgeBisher = "RückgabeTest";
+		String separator = "ZwischenwertTest";
+		String erwartet = 	"RückgabeTest";
+
+		String ergebnis = WorkingObjekt.NichtLeeralsohinzufügen(neuesWort,wortfolgeBisher,separator);
 		
-		assertEquals("NichtLeerAlsSolcheHinzufügen funktioniert nicht", Ergebnis, Test);
+		assertEquals("WortFolgebisher zurückgegeben", erwartet, ergebnis);
 	}
-	
+
 	@Test
-	public void NichtAlsLeeralsoHinzufügenZwischenwerLeer() {
-		Vokabelkarte WorkingObjekt = new Vokabelkarte();
+	public void NichtAlsLeeralsoHinzufügen_wortFolgeBisher_IstLeer() {
+		Vokabelkarte WorkingObjekt = workingObjekt;
 		
-		String Ergebnis = 	"prüfenTest";
-		String Test = 		WorkingObjekt.NichtLeeralsohinzufügen("prüfenTest","","ZwischenwertTest");
+		String neuesWort = "prüfenTest";
+		String wortfolgeBisher = "";
+		String separator = "ZwischenwertTest";
+		String erwartet = 	"prüfenTest";
+
+		String ergebnis = WorkingObjekt.NichtLeeralsohinzufügen(neuesWort,wortfolgeBisher,separator);
 		
-		assertEquals("NichtLeerAlsSolcheHinzufügen funktioniert nicht", Ergebnis, Test);
+		assertEquals("Nur das neue Wort ohne Seperator", erwartet, ergebnis);
 	}
 	
 	@Test
-	public void NichtAlsLeeralsoHinzufügenAllesLeer() {
-		Vokabelkarte WorkingObjekt = new Vokabelkarte();
+	public void NichtAlsLeeralsoHinzufügen_AllesLeer() {
+		Vokabelkarte WorkingObjekt = workingObjekt;
 		
-		String Ergebnis = 	"";
-		String Test = 		WorkingObjekt.NichtLeeralsohinzufügen("","","");
+		String neuesWort = "";
+		String wortfolgeBisher = "";
+		String separator = "";
+		String erwartet = 	"";
+
+		String ergebnis = 		WorkingObjekt.NichtLeeralsohinzufügen(neuesWort,wortfolgeBisher,separator);
 		
-		assertEquals("NichtLeerAlsSolcheHinzufügen funktioniert nicht", Ergebnis, Test);
+		assertEquals("leer", erwartet, ergebnis);
 	}        
 }

src/test/java/com/ugsbo/VokabelKartenSchreiber/makeStringTest.java

@@ -8,7 +8,7 @@ import com.ugsbo.VokableKartenSchreiber.Vokabelkarte;
 
 public class makeStringTest {
 	
-	//TODO: Fehler korigieren: Der Test setzt alle Sonderzeichen ans Ende
+	//TODO: Fehler korigieren
 	/*
 	@Test
 	public void makeStringReturnsStringKorrekt() {
@@ -64,6 +64,6 @@ public class makeStringTest {
 		String Abfrage = WorkingObjekt.makeString("TestName","","TestAntwort","TestAntwort","TestAntwort","TestAntwort");
 		
 		//assertTrue(Ergebnis.equals(Abfrage));
-		assertEquals("zurückgeworfener String ist nicht korrekt " ,Abfrage, Ergebnis);
+		assertEquals("zurückgeworfener String ist nicht korrekt " , Ergebnis, Abfrage);
 	}*/
 }