P1 Task2: MathList.isAtom() Test Widerspruch [geklärt]

Max-Maag
Erstie
Erstie
Beiträge: 16
Registriert: 23. Apr 2013 14:39

P1 Task2: MathList.isAtom() Test Widerspruch [geklärt]

Beitrag von Max-Maag »

In TestTask2.testIsAtom() wird verlangt, dass "-18" atomar ist. In der Aufgabenstellung für MathList.assembleNumbers() steht jedoch
[...] You may assume that numbers are not preceded by a sign ’+’ or ’-’. A minus is always an operator and should not be joined.
Sollte man sich im Zweifel eher an die Aufgabenstellung oder die Tests halten?
Zuletzt geändert von Bug am 23. Apr 2013 22:34, insgesamt 1-mal geändert.
Grund: Threadtitel geändert, damit andere schneller erkennen können, auf welchen Task es sich bezieht

Benutzeravatar
JannikV
Nerd
Nerd
Beiträge: 609
Registriert: 24. Apr 2011 12:42

Re: MathList.isAtom() Test Widerspruch

Beitrag von JannikV »

Bei assembleNumbers kommt das - nicht als Vorzeichen im Eingabestring vor.
Nach Berechnungen wie 5 - 10 kann das Ergebnis sehr wohl negativ werden. isAtom muss das erkennen können.

Es ist also alles in Ordnung.

VG

Benutzeravatar
Bug
Endlosschleifenbastler
Endlosschleifenbastler
Beiträge: 161
Registriert: 12. Okt 2011 10:28

Re: MathList.isAtom() Test Widerspruch

Beitrag von Bug »

Genau.

assembleNumbers() muss niemals ein - ein Ziffern zu einer Zahl zusammen ziehen, zu Beginn ist ein - also immer ein Rechen- und nie ein Vorzeichen.

Im Laufe der Rechnungen kann es aber passieren, dass in einem MathElement eine negative Zahl (mit Vorzeichen) gespeichert wird. Dies ist auch völlig OK, würde man nochmal assembleNumbers() darauf anwenden, dürfte dies auch nicht irgendwie auseinandergezogen werden (aber assembleNumbers() zieht ja sowieso niemals Elemente auseinander oder erzeugt neue, sondern fügt immer nur zusammen). Bei der entsprechenden Passage in der Aufgabenstellung geht es nur um Minuszeichen, die alleine in einem Element stehen.

Daher muss isAtom() am Ende dann auch negative Zahlen akzeptieren können.
Ophasen-Leitung Winter 2014/15, Sommer 2015, Sommer 2016 & Winter 2016/17
TGdI Tutor WiSe 14/15, 13/14 und 12/13
GdI II Betreuung Praktika SoSe 13
CMS Tutor SoSe 13

Max-Maag
Erstie
Erstie
Beiträge: 16
Registriert: 23. Apr 2013 14:39

Re: MathList.isAtom() Test Widerspruch

Beitrag von Max-Maag »

Alles klar. Danke.

Antworten

Zurück zu „Archiv“