Praktikum 1 Task 1 Tests Klammer fehlt

LukasP
Erstie
Erstie
Beiträge: 14
Registriert: 17. Okt 2014 22:35

Praktikum 1 Task 1 Tests Klammer fehlt

Beitrag von LukasP » 3. Mai 2015 19:37

Ich denke in Zeile 144 in TestTask1.java fehlt eine oeffnende Klammer.

String[] match2 = {"(", "14.5", "+", "2", "-", "1765", "*", "13.1", "/", "78", ")", "+", "33.7", ")"};

Es sei denn, die Klammer wurde absichtlich weggelassen, um Fehler zu provozieren.

charfi90
Mausschubser
Mausschubser
Beiträge: 61
Registriert: 8. Sep 2010 15:40

Re: Praktikum 1 Task 1 Tests Klammer fehlt

Beitrag von charfi90 » 3. Mai 2015 20:13

Es fehlt keine öffnende Klammer .

LukasP
Erstie
Erstie
Beiträge: 14
Registriert: 17. Okt 2014 22:35

Re: Praktikum 1 Task 1 Tests Klammer fehlt

Beitrag von LukasP » 3. Mai 2015 20:21

Die schliessende Klammer bei [10] bezieht sich doch auf die oeffnende Klammer bei [0], die schliessende Klammer bei [13] hat dann aber kein Gegenstueck soweit ich sehen kann.

Benutzeravatar
felicis
BASIC-Programmierer
BASIC-Programmierer
Beiträge: 114
Registriert: 14. Apr 2015 20:25

Re: Praktikum 1 Task 1 Tests Klammer fehlt

Beitrag von felicis » 3. Mai 2015 22:01

Deine Verwunderung ist verständlich, aber schau mal genauer in die Testklasse:

Code: Alles auswählen

String[] match = {"(", "14.5", "+", "2", "-", "1765", "*", "13.1", "/", "78", ")"};
		assertMatch(match, ml);
		
		//Add some new elements...
		ml.addElement(new MathElement("+"));
		ml.addElement(new MathElement("3"));
		ml.addElement(new MathElement("3"));
		ml.addElement(new MathElement("."));
		ml.addElement(new MathElement("7"));
		ml.addElement(new MathElement(")"));	
Hier werden in das (noch syntaktisch korrekte) StringArray die Elemente "+", "3", "3", ".", "7" und ")" hinzugefügt (Achtung: Nur EINE Klammer!). Damit wird nur eine Klammer eingefügt, und die Syntax ist verletzt! Das ist hier aber egal, da ja nur auf AssembleNumbers getestet wird und da eine Klammer mehr oder weniger keinen Einfluss auf die Auswertung der Methoden und damit den Test hat.

:D
felicis

Antworten

Zurück zu „Archiv“