ToString-Methode mit JUnit testen(BlueJ)

[Volthex]

Hallo,
ich habe ein Problem mit meiner aktuellen Aufgabe. Ich soll ein Programm mithilfe von JUnit testen. Bisher geht mir das auch ganz einfach von der Hand. Nur komme ich jetzt bei der ToString-Methode nicht mehr weiter.

public String toString() {
        if (size() == 0) {
            return "{}";
        }
        String result = "{";
        String separator = "";
        for (String elem : elements) {
            result += separator + elem;
            separator = ",";
        }
        result += "}";
        return result;
    }

Das ist der Code zur Methode. Jetzt habe ich folgendes für den Test gemacht, was leider nicht funktioniert.

@Test
    public void testToStringMethode()
    {
        StringSet stringSe1 = new StringSet();
        assertEquals(true, stringSe1.add("Test"));
        assertEquals(true, stringSe1.add("Test2"));
        assertEquals(true, stringSe1.add("Test3"));
        assertSame(elements, stringSe1.toString());
    }

Wie muss ich das ändern, damit der Test funktioniert?

 

[fhoffmann]

"elements" ist doch anscheinend ein Member deiner Klasse StringSet. Darauf kannst du im Test nicht zugreifen (und willst es auch nicht). Die letze Zeile in deinem Test sollte möglicherweise lauten:

assertEquals("{Test,Test2,Test3}", stringSe1.toString());

Allerdings bin ich mir da nicht ganz sicher, da der Name "StringSet" suggeriert, dass die Reihenfolge der Elemente keine Rolle spielt (aber dann wird ein Test natürlich kompliziert).

 

[Volthex]

Danke erstmal für die Antwort. Der Test funktioniert auch soweit gut. Doch ich weiß nicht, wie man eine Intersect-Methode testen soll...

/**
     * Bestimmt die Schnittmenge mit einer anderen Menge. Die aktuelle Menge
     * ändert sich dabei nicht. Sollte die andere Menge den Wert {@code null}
     * haben, so wird als Resultat ebenfalls {@code null} zurückgegeben.
     * 
     * @param other Die andere Menge. Auch sie wird nicht verändert.
     * @return Eine neue Menge, die die Elemente der Schnittmenge der beiden
     * Mengen enthält oder {@code null}, falls die andere Menge den Wert 
     * {@code null} hat.
     */
    public StringSet intersect(final StringSet other) {
        if (other == null) {
            return null;
        }

        final StringSet findings = new StringSet();
        for (String string1 : elements){
            if (other.contains(string1)) {
                findings.add(string1);
            }
        }
        return findings;    
    }

Kann mir da auch jemand helfen?

 

[mrBrown]

Erstell im Test zwei unterschiedliche Tests, bilde die Schnittmenge der beiden mit der Methode, und prüf dann, ob genau die Elemente drin sind, die auch drin sein sollten

 

[Volthex]

Also ich hab das jetzt so gemacht, aber es funktioniert nicht... Weil ich grade nicht weiß, wie du das mit zwei Tests meinst

@Test
    public void testSchnittmenge()
    {
        StringSet stringSe1 = new StringSet();
        assertEquals(true, stringSe1.add("a"));
        assertEquals(true, stringSe1.add("b"));
        assertEquals(true, stringSe1.add("c"));
        StringSet stringSe2 = new StringSet();
        assertEquals(true, stringSe2.add("b"));
        assertEquals(true, stringSe2.add("c"));
        assertEquals(true, stringSe2.add("d"));
        assertSame("{b,c}", stringSe1.intersect(stringSe2));
    }

 

[stg]

Du vergleichst einen String mit einem StringSet...

mrbrown meinte nicht, dass du zwei Tests erstellen sollst, sondern zwei Sets (so, wie du es auch gemacht hast). Das war nur ein Schreibfehler.

 

[Volthex]

Mir ist auch gerade aufgefallen, dass man die StringSets nicht vergleichen kann... Ich muss ja irgendwie an die inhalte der Arrays kommen. Da bietet sich ja die ToString-Methode an. Doch wie kann ich das jetzt machen, dass man Schnittmenge vergleicht? Ich hab das jetzt soweit raus:

    @Test
    public void testSchnittmenge()
    {
        StringSet stringSe1 = new StringSet();
        assertEquals(true, stringSe1.add("a"));
        assertEquals(true, stringSe1.add("b"));
        assertEquals(true, stringSe1.add("c"));
        StringSet stringSe2 = new StringSet();
        assertEquals(true, stringSe2.add("b"));
        assertEquals(true, stringSe2.add("c"));
        assertEquals(true, stringSe2.add("d"));
        assertEquals("{a,b,c}", stringSe1.toString());
        assertEquals("{b,c,d}", stringSe2.toString());
    }

 

[stg]

Mir ist auch gerade aufgefallen, dass man die StringSets nicht vergleichen kann...

Wie kommst du darauf?

Ich muss ja irgendwie an die inhalte der Arrays kommen.

Was für Arrays?

Da bietet sich ja die ToString-Methode an.

Nein. Sinnvoller wäre es eine auf StringSet gescheit implementierte equals-Methode zu verwenden.

 

[Volthex]

Also ich habe mich vorhin mit einem Kumpel von mir unterhalten der auch Informatik studiert und er meinte das... Ich weiß aber leider grade nicht, wie ich eine equals-Methode auf StringSet anwenden soll...

 

[stg]

Du schreibst Unit-Tests, aber weißt nicht, was eine equals-Methode ist?

(wir kennen deine Klasse StringSet übrigens nicht ... poste die doch einfach mal )

 

[mrBrown]

Dein StringSet hat eine contains Methode, du kannst also für alle Elemente prüfen, ob sie auch wirklich drin sind.
Damit keine zusätzlichen drin sind dann noch die Größe überprüfen