diff --git a/src/main/java/com/ugsbo/VokableKartenSchreiber/Vokabelkarte.java b/src/main/java/com/ugsbo/VokableKartenSchreiber/Vokabelkarte.java index cc9b314..6e874aa 100644 --- a/src/main/java/com/ugsbo/VokableKartenSchreiber/Vokabelkarte.java +++ b/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; } diff --git a/src/test/java/com/ugsbo/VokabelKartenSchreiber/NichtLeeralsohinzufügenTest.java b/src/test/java/com/ugsbo/VokabelKartenSchreiber/NichtLeeralsohinzufügenTest.java index b604a98..747fb80 100644 --- a/src/test/java/com/ugsbo/VokabelKartenSchreiber/NichtLeeralsohinzufügenTest.java +++ b/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); } } diff --git a/src/test/java/com/ugsbo/VokabelKartenSchreiber/makeStringTest.java b/src/test/java/com/ugsbo/VokabelKartenSchreiber/makeStringTest.java index aaebf49..ef77f66 100644 --- a/src/test/java/com/ugsbo/VokabelKartenSchreiber/makeStringTest.java +++ b/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); }*/ }