JUnit-Tests aus Ü4?

Horschie
BASIC-Programmierer
BASIC-Programmierer
Beiträge: 132
Registriert: 10. Nov 2005 17:51
Wohnort: Heppenheim
Kontaktdaten:

JUnit-Tests aus Ü4?

Beitrag von Horschie »

Hallo,

hier mal die Musterlösung zu Übung 4. Da sollte der JUnit-Test zur
Berechnung der Fläche geschrieben werden.

Jetzt steht da drüber als Frage, ob das ein guter Test ist.
Schön wär ne definitive Lösung gewesen, aber ok :mrgreen:

Code: Alles auswählen

@Test
public void testFlaeche() throws Exception {
int a = 10; int b = 20; int c = 30; int d = 40;
long actual = BerechneFlaeche.berechne(a, b, c, d);
long expected = (c - a)*(d - b);
assertEquals("Die berechnete Flaeche muss gleich (x2 - x1)*(y2 - y1) sein",
expected, actual);
}
Jetzt diei Frage: Ist das ein guter Test oder nicht? Wenn nein, wie sieht ein guter Test aus?


Danke
Christoph

Horschie
BASIC-Programmierer
BASIC-Programmierer
Beiträge: 132
Registriert: 10. Nov 2005 17:51
Wohnort: Heppenheim
Kontaktdaten:

Re: JUnit-Tests aus Ü4?

Beitrag von Horschie »

weils mir gerade in der Musterlösung zu Ü5 aufgefallen ist:

import static irgend.ein.package.*;

Hier die Erklärung:
http://java.sun.com/j2se/1.5.0/docs/gui ... mport.html

Ach ja, ich hatte aus gewissen Gründen kein GdI, also nicht hauen wenn ihrs scho kennt 8)

yourmaninamsterdam
Nerd
Nerd
Beiträge: 681
Registriert: 26. Okt 2006 14:04
Kontaktdaten:

Re: JUnit-Tests aus Ü4?

Beitrag von yourmaninamsterdam »

Der Plan bei dem expected result ist eigentlich, dass du das selbst ausrechnest. Insofern wäre folgendes besser:

Code: Alles auswählen

long expected = 400;

Mspringer
Nerd
Nerd
Beiträge: 555
Registriert: 19. Okt 2006 14:41
Wohnort: Darmstadt / Alzenau
Kontaktdaten:

Re: JUnit-Tests aus Ü4?

Beitrag von Mspringer »

Wenn du so die tests schreiben würdest, müsste du testen ob deine testfunktion auch immer das richtige ergebnis liefert. Ich weiß nicht, ob du verstehst was ich meine, aber wie your.... schon sagte, solltest du hier per hand ausgerechnete Ergebnisse reinhaun.

Antworten

Zurück zu „Archiv“