Junit-Test (Java)

Wirtschaftsinformatiker

Bekanntes Mitglied
Ist mein Code und meine Test-Methode richtig? Wie kann ich das verbessern? Was ist falsch? Was ist der einfache Weg? Wie kann ich noch vereinfachen?

Java:
class Produzieren
{
    int leerZeichen=0;
    String umgedreht = new String();
    /**
     * Liefert die Fakultaet von n, also das Produkt aller natuerlichen Zahlen bis n.
     * Die Fakultaet von 0 ist per mathematischer Definition 1.
     *
     * Beispielauswertung:
     *
     *   fak(4)
     * -> 4 * fak(3)
     * -> 4 * (3 * fak(2))
     * -> 4 * (3 * (2 * fak(1)))
     * -> 4 * (3 * (2 * (1 * fak(0))))
     * -> 4 * (3 * (2 * (1 * 1)))
     * -> 4 * (3 * (2 * 1))
     * -> 4 * (3 * 2)
     * -> 4 * 6
     * -> 24
     */
    public int fak(int n)
    {
        int res=1;
        if (n>0)
        {
            res = n*fak(n-1);
        }
        return res;
    }

    /**
     * Berechnet n modulo m, also den Rest der ganzzahligen Division n/m.
     *
     * modulo(14,3) -> modulo(11,3) -> modulo(8,3) -> modulo(5,3) -> modulo(2,3) -> 2
     *
     * @param n ein beliebiger Wert >= 0
     * @param m ein beliebiger Wert > 0
     */
    public int modulo(int n, int m)
    {
        int res=0;
        if (m>0)
        {
            if (n>=m)
            {
                res = modulo (n-m, m);
            }
            else
            {
                res = n;
            }
        }
        else
        {
            throw new IllegalArgumentException("m darf nicht negativ sein!");
        }
        return res;
    }

    /**
     * Gibt an, ob die Zeichenkette einen Vokal enthaelt. Auswertungen:
     *
     * enthaeltVokal("brei") -> enthaeltVokal("rei") -> enthaeltVokal("ei") -> true
     * enthaeltVokal("xyz") -> enthaeltVokal("yz") -> enthaeltVokal("z") -> enthaeltVokal("") -> false
     * @param s ein beliebiger String ungleich null
     */
    public boolean enthaeltVokal(String s)
    {
        boolean res = false;
        if (s.length()==0 && s==null)
        {
            throw new IllegalArgumentException("Sting darf nicht null sein!");
        }
        else
        {
            s = s.toLowerCase();
            if (s.length()!=0)
            {
                if (s.charAt(0) == 'a' || s.charAt(0) == 'e' || s.charAt(0) == 'i' || s.charAt(0) == 'o' || s.charAt(0) == 'u')
                {
                    res = true;
                }
                else
                {
                    return enthaeltVokal(s.substring(1));
                }
            }
        }
        return res;
    }

    /**
     * Gibt an, ob die Zeichenkette ein Palindrom ist. Auswertungen:
     *
     * istPalindrom("anna") -> istPalindrom("nn") -> istPalindrom("") -> true
     * istPalindrom("asta") -> istPalindrom("st") -> false
     * istPalindrom("axa") -> istPalindrom("x") -> true
     * istPalindrom("xyz") -> false
     * @param s ein beliebiger String ungleich null
     */
    public boolean istPalindrom(String s)
    {
        if (s.length()==0 && s==null)
        {
            throw new IllegalArgumentException("Sting darf nicht null sein!");
        }
        else
        {
            s = s.toLowerCase();
            if (s.length() <= 1)
            {
                return true;
            }
            else if (s.charAt(0) != s.charAt(s.length() - 1))
            {
                return false;
            }
        }
        return istPalindrom(s.substring(1, s.length() - 1));
    }

    /**
     * Berechnet die Anzahl Leerzeichen in der Zeichenketten. Auswertung:
     *
     * anzahlLeerzeichen("a bc d")
     * -> 0 + anzahlLeerzeichen(" bc d")
     * -> 0 + (1 + anzahlLeerzeichen("bc d"))
     * -> 0 + (1 + (0 + anzahlLeerzeichen("c d")))
     * -> 0 + (1 + (0 + (0 + anzahlLeerzeichen(" d"))))
     * -> 0 + (1 + (0 + (0 + (1 + anzahlLeerzeichen("d")))))
     * -> 0 + (1 + (0 + (0 + (1 + (0 + anzahlLeerzeichen(""))))))
     * -> 0 + (1 + (0 + (0 + (1 + (0 + 0)))))
     * -> 0 + (1 + (0 + (0 + (1 + 0))))
     * -> 0 + (1 + (0 + (0 + 1)))
     * -> 0 + (1 + (0 + 1))
     * -> 0 + (1 + 1)
     * -> 0 + 2
     * -> 2
     * @param s ein beliebiger String ungleich null
     */
    public int anzahlLeerzeichen(String s)
    {
        if (s.length()==0 && s==null)
        {
            throw new IllegalArgumentException("Sting darf nicht null sein!");
        }
        else
        {
            if (s.length()!=0)
            {
                if (s.charAt(0)==' ')
                {
                    leerZeichen++;
                }
                anzahlLeerzeichen(s.substring(1));
            }
        }
        return leerZeichen;
    

    /**
     * Liefert die umgedrehte Zeichenkette. Auswertung:
     *
     *   umgedreht("regal")
     * -> umgedreht("egal") + 'r'
     * -> (umgedreht("gal") + 'e') + 'r'
     * -> ((umgedreht("al") + 'g') + 'e') + 'r'
     * -> (((umgedreht("l") + 'a') + 'g') + 'e') + 'r'
     * -> (((          "l"  + 'a') + 'g') + 'e') + 'r'
     * -> ((           "la"        + 'g') + 'e') + 'r'
     * -> (            "lag"              + 'e') + 'r'
     * ->              "lage"                    + 'r'
     * ->              "lager"
     * @param s ein beliebiger String ungleich null
     */
    public String umgedreht(String s)
    {
        char charat;
        if (s.length()==0 && s==null)
        {
            throw new IllegalArgumentException("Sting darf nicht null sein!");
        }
        else
        {
            if (s.length()!=0)
            {
                charat = s.charAt(s.length()-1);
                umgedreht = umgedreht + charat;
                umgedreht(s.substring(0, s.length()-1));
            }
        }
        return umgedreht;
    }
}


[B]Das ist meine Test-Methoden:[/B]

[CODE]public class ProduzierenTest
{
    @Test
    public void testfak3()
    {
        Produzieren testee = new Produzieren();
        assertEquals(6, testee.fak(3));
    }

    @Test
    public void testfak4()
    {
        Produzieren testee = new Produzieren();
        assertEquals(24, testee.fak(4));
    }

    @Test
    public void testfak6()
    {
        Produzieren testee = new Produzieren();
        assertEquals(720, testee.fak(6));
    }

    @Test
    public void testmodulo14_3()
    {
        Produzieren testee = new Produzieren();
        assertEquals(2, testee.modulo(14, 3));
    }

    @Test
    public void testmodulo20_10()
    {
        Produzieren testee = new Produzieren();
        assertEquals(0, testee.modulo(20, 10));
    }

    @Test
    public void testmodulo21_4()
    {
        Produzieren testee = new Produzieren();
        assertEquals(1, testee.modulo(21, 4));
    }

    @Test
    public void testmodulom0()
    {
        Produzieren testee = new Produzieren();
        try
        {
            testee.modulo(21, 0);
        }
        catch (Exception ex)
        {
            return;
        }
        fail ("Es wurde keine Exception geforfen!");
    }

    @Test
    public void testEnthaeltVokalBrei()
    {
        Produzieren testee = new Produzieren();
        assertEquals(true, testee.enthaeltVokal("brei"));
    }

    @Test
    public void testEnthaeltVokalXyz()
    {
        Produzieren testee = new Produzieren();
        assertEquals(false, testee.enthaeltVokal("xyz"));
    }

    @Test
    public void testEnthaeltVokalVokal()
    {
        Produzieren testee = new Produzieren();
        assertEquals(true, testee.enthaeltVokal("Vokal"));
    }

    @Test
    public void testEnthaeltVokalNull()
    {
        Produzieren testee = new Produzieren();
        try
        {
            testee.istPalindrom(null);
        }
        catch (Exception ex)
        {
            return;
        }
        fail ("Es wurde keine Exception geforfen!");
    }

    @Test
    public void testIstPalindromAnna()
    {
        Produzieren testee = new Produzieren();
        assertEquals(true, testee.istPalindrom("anna"));
    }

    @Test
    public void testIstPalindromAsta()
    {
        Produzieren testee = new Produzieren();
        assertEquals(false, testee.istPalindrom("asta"));
    }

    @Test
    public void testIstPalindromAxa()
    {
        Produzieren testee = new Produzieren();
        assertEquals(true, testee.istPalindrom("axa"));
    }

    @Test
    public void testIstPalindromXyz()
    {
        Produzieren testee = new Produzieren();
        assertEquals(false, testee.istPalindrom("xyz"));
    }

    @Test
    public void testIstPalindromNull()
    {
        Produzieren testee = new Produzieren();
        try
        {
            testee.istPalindrom(null);
        }
        catch (Exception ex)
        {
            return;
        }
        fail ("Es wurde keine Exception geforfen!");
    }

    @Test
    public void testanzahlLeerzeichenA_BC_D()
    {
        Produzieren testee = new Produzieren();
        assertEquals(2, testee.anzahlLeerzeichen("a bc d"));
    }

    @Test
    public void testanzahlLeerzeichenABCD()
    {
        Produzieren testee = new Produzieren();
        assertEquals(0, testee.anzahlLeerzeichen("abcd"));
    }

    @Test
    public void testanzahlLeerzeichenA_B_C_D()
    {
        Produzieren testee = new Produzieren();
        assertEquals(3, testee.anzahlLeerzeichen("a b c d"));
    }

    @Test
    public void testanzahlLeerzeichenNull()
    {
        Produzieren testee = new Produzieren();
        try
        {
            testee.anzahlLeerzeichen(null);
        }
        catch (Exception ex)
        {
            return;
        }
        fail ("Es wurde keine Exception geforfen!");
    }

    @Test
    public void testUmgedrehtRegal()
    {
        Produzieren testee = new Produzieren();
        assertEquals("lager", testee.umgedreht("regal"));
    }

    @Test
    public void testUmgedrehtLager()
    {
        Produzieren testee = new Produzieren();
        assertEquals("regal", testee.umgedreht("lager"));
    }

    @Test
    public void testUmgedrehtAbcd()
    {
        Produzieren testee = new Produzieren();
        assertEquals("dcba", testee.umgedreht("abcd"));
    }

    @Test
    public void testUmgedrehtNull()
    {
        Produzieren testee = new Produzieren();
        try
        {
            testee.umgedreht(null);
        }
        catch (Exception ex)
        {
            return;
        }
        fail ("Es wurde keine Exception geforfen!");
    }
}
 
Zuletzt bearbeitet:

KonradN

Super-Moderator
Mitarbeiter
Hab schon jetzt korrigiert
Ok, danke. Ein paar Dinge dazu:

Du hast mehrere Tests der Fakultät - dabei werden aber nur einzelne Zahlen getestet. Es werden dabei keine Randsituationen betrachtet. Klingt hochtrabend, aber schauen wir uns den Code an: Du hast da eine if Abfrage. Du testest in allen Aufrufen (so ich nichts übersehen habe), den Fall: Die Bedingung ist (im ersten Aufruf) wahr. Klar, im Verlauf wird immer auch der Fall mitgetestet, dass die Schleife nicht betreten wird, aber der direkte Aufruf mit 0 ist nicht getestet. Das sind so Bedingungen, die ich immer testen würde.

An der Stelle die Frage: Parametrized Tests sagt Dir etwas? ==> Das kann man vereinfachen z.B. mit einer CsvSource, so dass Du nur noch einen Test hast.

Und dann das, was mir direkt aufgefallen war:

In testIstPalindromNull() testest Du, dass eine Exception geworfen wird. Irgend eine. Du testest aber nicht, ob es Deine Exception ist. Ebenso bei testUmgedrehtNull().

Und ich habe jetzt keinen Test gesehen für einen leeren String. Du testest nur null.

==> Wenn Du das verbessern würdest, dann würdest Du Fehler finden, denn der Test würfe fehlschlagen.

Dazu nur ein Tipp: Kann es sein, dass Du Deine Geldbörse verloren hast UND du in Deine Geldbörse schauen kanmnst und siehst, dass sie leer ist? Also zum einen ist die Geldbörse weg UND gleichzeitig siehst Du in Deine Geldbörse und sie ist leer?
 

KonradN

Super-Moderator
Mitarbeiter
Und Du testest jede Methode genau einmal - Du hast einen Status (int leerZeichen=0;) und die Veränderung dieses Status testest Du nicht.

Das wäre ein Test a.la.
Java:
    @Test
    public void testanzahlLeerzeichenA_B_C_D()
    {
        Produzieren testee = new Produzieren();
        assertAll(
            () -> assertEquals(0, testee.anzahlLeerzeichen("ab")),
            () -> assertEquals(1, testee.anzahlLeerzeichen("a b")),
            () -> assertEquals(3, testee.anzahlLeerzeichen("a b c d"))
        );
    }

(Das nutzt JUnit 5 - falls Du noch mit 4 arbeiten solltest, dann lass das assertAll weg und mache nur die 3 assertEquals Aufrufe.)
 
Ähnliche Java Themen
  Titel Forum Antworten Datum
W Testfälle bei Java ( Junit-Test) Java Basics - Anfänger-Themen 3
S Hilfe zu Java-Programm und JUnit Test!! Java Basics - Anfänger-Themen 5
W junit.Test not accessible? Java Basics - Anfänger-Themen 4
U jUnit 5 Test für eine addMethode Java Basics - Anfänger-Themen 18
A Junit Test für MysqlDataSource JDBC Java Basics - Anfänger-Themen 3
A Test Junit Java Basics - Anfänger-Themen 1
H Junit test Java Basics - Anfänger-Themen 12
P Methoden JUnit 4 - Test Java Basics - Anfänger-Themen 6
N Fehler bei JUnit Test Java Basics - Anfänger-Themen 5
B JUnit Test erstellen Java Basics - Anfänger-Themen 6
W Problem bei JUnit Test Aufgabe Java Basics - Anfänger-Themen 15
W JUnit Test und HashCode Java Basics - Anfänger-Themen 14
A Kfz - Händler Klasse. JUnit-Test gibt noch Fehler an, aber finde Ursache nicht Java Basics - Anfänger-Themen 7
B Palindrom Test mit Junit Java Basics - Anfänger-Themen 23
S Junit Test Java Basics - Anfänger-Themen 2
shiroX Methoden JUnit-Test einer void-Methode Java Basics - Anfänger-Themen 4
B Binäre Suche - Junit Test Java Basics - Anfänger-Themen 6
S Separate Funktion für JUnit-Test Java Basics - Anfänger-Themen 3
T JUnit test failed Java Basics - Anfänger-Themen 3
R JUnit Test mit einer Dateistruktur als Testparameter Java Basics - Anfänger-Themen 3
shiroX OOP Array kleinste Zahl mit jUnit test Java Basics - Anfänger-Themen 3
S Code stimmt nicht für vorgegebenen JUnit-Test Java Basics - Anfänger-Themen 2
R JUnit Test mit mehrfach ausgeführt Java Basics - Anfänger-Themen 6
B JUnit - Mini-Test Java Basics - Anfänger-Themen 9
T Unterschied zwischen Integrationstest und JUnit test? Java Basics - Anfänger-Themen 12
Y Junit Test - Testwert ändert sich Java Basics - Anfänger-Themen 12
T JUNit Test IOException Java Basics - Anfänger-Themen 5
Spin JUNIT Test Case - Problem bei testen Java Basics - Anfänger-Themen 2
T brauche HILFE beim Junit test:eek: Java Basics - Anfänger-Themen 11
timbeau JUnit Test Dauer speichern/loggen Java Basics - Anfänger-Themen 16
A junit test wann verwendet man "was"? Java Basics - Anfänger-Themen 4
J JUnit Test Java Basics - Anfänger-Themen 2
Houly JUnit Test Suite anlegen Java Basics - Anfänger-Themen 6
pun Junit Test erkennt Exception nicht.. Java Basics - Anfänger-Themen 14
G BlueJ jUnit Test Java Basics - Anfänger-Themen 6
V Test mit JUnit verbinden Java Basics - Anfänger-Themen 3
B JUnit Test Klasse Rational Java Basics - Anfänger-Themen 12
C Unsupported major.minor bei jUnit Test Java Basics - Anfänger-Themen 2
6 Best Practice Feedback zu Service / JUnit Tests Java Basics - Anfänger-Themen 3
M Anfängerfehler - Tests JUnit IntelliJ Java Basics - Anfänger-Themen 24
D Cannot find JUnit.framework Java Basics - Anfänger-Themen 1
U JUnit testen auf SomeException Java Basics - Anfänger-Themen 5
A JUnit testing is inkonsistent Java Basics - Anfänger-Themen 12
P Probleme mit JUnit-Tests, es kommt was anderes raus als bei manuellen Tests Java Basics - Anfänger-Themen 5
Y Wie kann ich Konsoleneingaben durch den Scanner mit JUnit 4 testen? Java Basics - Anfänger-Themen 1
B JUnit 4: Wie man die eigene Liste testen kann [TDD] Java Basics - Anfänger-Themen 46
W JUnit Tests Java Basics - Anfänger-Themen 4
hello_autumn Klassen Anzahl sowie die Anzahl der Junit Tests ermitteln? Java Basics - Anfänger-Themen 8
B JUnit / Exceptions/ try-catch Java Basics - Anfänger-Themen 6
L JUnit tests in java Java Basics - Anfänger-Themen 5
N JUnit und private Methoden testen. Java Basics - Anfänger-Themen 9
F JUnit - Was ist mit "side effects" gemeint ? Java Basics - Anfänger-Themen 2
H JUnit in Eclipse: java.lang.NoClassDefFoundError: Java Basics - Anfänger-Themen 9
L Junit Testing bei XML? Java Basics - Anfänger-Themen 3
J LocalDateTime testen mit Junit Java Basics - Anfänger-Themen 20
A Objekt in Methode zurückgeben, JUnit zeigt Error Java Basics - Anfänger-Themen 2
O JUnit - Objektreferenzen Java Basics - Anfänger-Themen 3
G Testen mit JUnit Java Basics - Anfänger-Themen 4
C JUnit Tests. How to Java Basics - Anfänger-Themen 5
shiroX Klassen Klasse/Methode private final jUnit-Fehler Java Basics - Anfänger-Themen 5
L Junit Tests Java Basics - Anfänger-Themen 10
A IllegalArgumentException in JUnit testen Java Basics - Anfänger-Themen 3
V ToString-Methode mit JUnit testen(BlueJ) Java Basics - Anfänger-Themen 10
V JUnit Klassen Java Basics - Anfänger-Themen 3
T Junit Mockito: Instanz von inneren erzeugten Objekten Java Basics - Anfänger-Themen 4
S JUnit - Swing- Anwendung wird nicht neu gestartet Java Basics - Anfänger-Themen 0
S Kommt es zu Seiteneffekten wenn man waehrend den laufenden JUnit Tests den Code aendert? Java Basics - Anfänger-Themen 2
M JUnit Testmethoden mit mehreren assert Methoden Java Basics - Anfänger-Themen 1
S Double und Gleitkommazahlen mit JUnit testen Java Basics - Anfänger-Themen 7
K JUnit: Objekte von eigenen Klassen vergleichen...geht nicht Java Basics - Anfänger-Themen 5
Z JUnit Exception Java Basics - Anfänger-Themen 2
M Verständnisfrage zu JUnit Tests und private Methoden Java Basics - Anfänger-Themen 3
M Ist die Hamcrest Bibliothek auch schon in Junit 4.11 verfügbar? Java Basics - Anfänger-Themen 1
S Unterschied .jar Datei ausführen und junit Testfall... Java Basics - Anfänger-Themen 3
M Reihenfolge von Testmethoden in JUnit beeinflussen Java Basics - Anfänger-Themen 2
G JUnit-Tests im Programmdurchlauf starten Java Basics - Anfänger-Themen 4
T Best Practice JUnit: Wie Klassen durch Stubs/Mockups ersetzen Java Basics - Anfänger-Themen 7
M Junit Tests durchführen, die eine Verbindung zu einer Daten erfordern Java Basics - Anfänger-Themen 3
T Junit in Eclipse Java Basics - Anfänger-Themen 1
P JUnit bedeutungen Java Basics - Anfänger-Themen 3
S JUnit assertEquals funktioniert nichgt wie es sollte :( Java Basics - Anfänger-Themen 7
V Frage zu JUnit Tests Java Basics - Anfänger-Themen 3
B JUnit für JFileChooser Java Basics - Anfänger-Themen 6
S File vergleich - Junit Java Basics - Anfänger-Themen 6
T JUnit Java Basics - Anfänger-Themen 18
G Junit Java Basics - Anfänger-Themen 4
X JUnit testing Java Basics - Anfänger-Themen 7
T JUnit Suite frage Java Basics - Anfänger-Themen 6
S InvocationTargetException bei JUnit Testlauf Java Basics - Anfänger-Themen 2
T Junit --Exception testen Java Basics - Anfänger-Themen 15
A JUnit Tests in Jar-Archiv packen Java Basics - Anfänger-Themen 2
G Erste Schritte JUNIT Regressionstests automatisieren Java Basics - Anfänger-Themen 2
M JUnit - nur einzelne Methode testen? Java Basics - Anfänger-Themen 4
M JUnit - nur Failures loggen? Java Basics - Anfänger-Themen 2
R String Replace für JUnit Java Basics - Anfänger-Themen 19
T JUNIT Nullpointerexception Java Basics - Anfänger-Themen 3
M JUNIT tests mit ant script ausführen Java Basics - Anfänger-Themen 4
S [JUnit] eigener Testsuite Runner + Ausgabe Java Basics - Anfänger-Themen 6
S [JUnit] Print Results while running Java Basics - Anfänger-Themen 6
W Mit jUnit Array testen? Java Basics - Anfänger-Themen 5

Ähnliche Java Themen

Neue Themen


Oben