Bewertung der 3. Malefizabgabe (Programm, Handbuch)

Gruppe 1 (A)

• Spielerreihenfolge wird bei euch nicht ausgewürfelt oder ?
• Speicherung nur per MFC-Serialisierung, nicht per XML.
• Crash nach dem Schlagen einer Spielfigur.
  Lösung: in "CSpielerManager::stein_zurueck" wurde die Figur gleichzeitig auf Start- und Zielfeld gesetzt. Das Setzen auf das Zielfeld habe ich rausgeworfen.
• Stapeln von Figuren klappt nicht.
• "L_WaehleFigur::suche": Hier scheinen auch Zielfelder als "CSpielFeld" hereinzugeraten. Das Problem ist dass CZielfeld von CFeld -> CFeldInAktion abgeleitet ist, aber kein CSpielfeld ist, es aber durch einen doofen Typecast in ein "CSpielFeld" gecastet wird und und eine vom Objekt garnicht vorhandene Methode "anz_Nachbarn()" aufgerufen wird.
  Lösungsversuch: in "L_WaehleFigur::suche" dieses eingebaut:

  	if (spf->GibTyp() == ENDFELD )
  	{
  		//Nix zu tun, keine sinnvollen Nachbarn !
  		erreichbar.AddHead (spf);
  		return;
  	} 

  Dadurch kam es nicht mehr zum Crash, allerdings konnte ich trotzdem nicht auf das Zielfeld ziehen.
• Unit-Tests sind alle in einer Testmethode codiert. Es wäre sinnvoller gewesen pro Test eine eigene Methode zu bauen.

Gruppe 2 (B)

• Crash in "CMalefizDoc::IstSpielende" wenn noch kein Spiel gestartet ist und man irgendwo ins Feld klickt (m_pAktuellerSpieler ist NULL)
• Der Spieler-Eingabe-Dialog ist sehr unkomfortabel (viermal starten für vier Spieler), außerdem kann man auf "OK" klicken ohne zu würfeln.
• Wieso werden beim Sperre-versetzen Zielfelder angezeigt (im Screenshot ist die markierte Sperre die zu versetzende) ?
  
• Gekoppelte Figuren sollen gemeinsam bewegt werden, nicht einzeln.
• Speicherlöcher !
• Eure XML-Datei referenziert nicht die DTD auf dem FH-Server sondern eine lokale im Projekt (ist zum Testen aber ganz praktisch ;-) ).
• Probleme beim Speichern: wenn mehr als eine Figur auf einem Feld stehen dann werden bei euch mehrere Elemente "Feld" für angelegt, mit jeweils einer einzigen Figur. Hier wären mehrere Subelemente "Spielfigur" nötig gewesen (siehe meine Beispiele)
• Problem beim Laden in CMalefizDoc Zeile 669: die Variable "k" wird in der Schleife nicht geändert, das heißt keine Figur außer der mit der figurid "0" wird geladen ! Ein "++k" ans Ende der Schleife einbauen und plötzlich tut alles ;-)
• Probleme beim Speichern: Ihr habt x und y in eurer Speicherdatei vertauscht, das heißt es ist nicht möglich eure Spielstände mit denen anderer Gruppen zu tauschen.
• Euer Spiel ist tot wenn nur noch eine Figur im Spiel ist und diese nicht mit einem Zug ins Ziel kommt, siehe mein manipuliertes Savegame MalefizB.xml. Ist es überhaupt möglich auf eure Zielfelder zu ziehen ? Immerhin, wenn ich per Savegame alle Figuren ins Ziel setze dann habe ich gewonnen ;-)

Gruppe 3 (C)

• XML-Laden/Speichern nicht fertig implementiert (zuständig: 949095, individueller Abzug -2)
• Spielbretteditor funktioniert (aber wieso können Start- und Zielfelder nicht frei gesetzt werden ?), das Spielbrett kann aber nicht geladen/gespeichert und deshalb nicht bespielt werden. Zuständig: 145945, 549473, individueller Abzug: -7).

Gruppe 4 (D)

• Datei "res\CMalefiz.rc2" fehlt in Abgabe.
• Unit-Tests fehlen.
• ZWei Compilerwarnungen.
• Cool, Drag&Drop !
• Euer Würfel liefert als allererstes Würfelergebnis immer "6" (deshalb darf Cemil immer als erster ziehen). Hier hätte beim allerersten Aufrufen "srand" aufgerufen werden müssen um für die Zufallszahl eine Basis anzugeben.
  Aus der Doku zu "rand": srand( (unsigned)time( NULL ) );
• Ihr würfelt automatisch beim Rundenstart, also ein Wenig-Klick-Programm ;-).
• Bei euch kann man keine zwei Spielfiguren auf ein Feld stellen (stapeln/koppeln).
• Beim Versetzen einer Sperre ist es ein wenig verwirrend dass die Spielfigur beim Drop nicht gezeichnet wird, sie wird erst sichtbar wenn die Sperre weg ist.
• Nach ein paar Minuten Spielzeit gab es regelmäßig einen Crash in "CMalefizView::InitWerkzeuge": die erste Zeile "tools->pDC = GetDC();" hat NULL zurückgegeben. Wieso ruft ihr da eigentlich "GetDC" auf ? Ihr verwendet die aus dem OnDraw, könnt den CDC also übergeben ! Wahrscheinlich gibt es an dieser Stelle einen Konflikt beim Zugriff auf die Bildschirm-Resource, weil die OnDraw-Routine ja bereits einen Handle offen hat.
• Speicherung leider nur per MFC-Serialisierung, kein XML.
• Es ist bei euch nicht erkennbar wieviele Figuren im Ziel stehen.
• Bei euch läuft das Spiel einfach weiter wenn ein Spieler alle Figuren im Spiel hat. Siehe Savegame MalefizD
• Laden eines Spiels führt zu Speicherlöchern.
• Das geforderte 5-Personen-Gruppe-Feature "Spielbretteditor" fehlt komplett !
• Keine Zustandsklassen !

Gruppe 6 (F)

• "Neues Spiel", dann auf "Abbrechen": Spiel startet trotzdem mit 2 Spielern.
• Die Radiobuttons für die Spielerauswahl funktionieren nicht korrekt ("2 Spieler" und einer der beiden anderen können gleichzeitig gewählt sein).
• Sehr viele Speicherlöcher.
• Laden der XML-Datei funktioniert (noch) nicht.
• So gut wie keine (Doxygen-)Kommentierung.
• Euer Unit-Test-Projekt crashte beim Anwendungsstart, und es gab einen Compilefehler über falsche Version der MSVCRT.dll. Lösung: in den Projekteigenschaften unter "Konfigurationseigenschaften" -> "Linker" -> "Eingabe" muss statt "cppunit.lib" die Datei "cppunitd.lib" (Debug-Version) referenziert werden. Danach fliegt der Test allerdings am Ende von "initTest1" ("CPPUNIT_ASSERT(!fixture->istErreichbar(i));") weg. Hier müsste der Code meiner Meinung nach so aussehen:

  		//Steckt da überhaupt ein Feld ?
  		if (fixture->hatCFeld (i) == TRUE)
  		{
  			CPPUNIT_ASSERT(!fixture->istErreichbar(i));
  		} 

  Danach schlägt der Test wegen nicht erreichbaren Feldern fehl.
• Es sind nur drei Tests codiert. Die sehen allerdings gut aus.