3. Pachisi-Abgabe (Programm), 4. Abgabe (Hilfe)

Gruppe A (146 006, 845 927, 546 873, 246 016)

• Toolbarbutton "Hilfe" -> Spielfeld auswählen -> keine Verknüpfung zum aktuellen Hilfethema. Bei F1 funktioniert es.
• Wenn zwei Figuren gezogen werden können und eine gewählt wurde muss man diese nochmal klicken ("entwählen") bevor ich die andere Figur auswählen kann (unkomfortabel)
• Rundenablauf: es wäre schön wenn es deutlicher erkennbar wäre wann ich was machen darf. Es benötigt eine ziemlich gründlichen zweiten Blick um zu erkennen wer gerade dran ist und ob der Spieler nochmal würfeln darf.
• Wenn ich keine Zugmöglichkeit habe (im konkreten Fall: 3 Figuren im Startfeld, eine Figur 2 Felder vor dem Ziel), dann erscheint beim Würfeln nur die Meldung "Es besteht keine Zugmöglichkeit...", aber ich sehe nicht was ich überhaupt gewürfelt habe.

Gruppe B (143 376, 843 365)

• Compilewarnungen:

  	PachisiView.cpp(323) : warning C4244: 'argument' : conversion from '__w64 int' to 'int', possible loss of data
  	PachisiView.cpp(333) : warning C4244: 'argument' : conversion from '__w64 int' to 'int', possible loss of data
  	PachisiView.cpp(343) : warning C4244: 'argument' : conversion from '__w64 int' to 'int', possible loss of data
  	PachisiView.cpp(353) : warning C4244: 'argument' : conversion from '__w64 int' to 'int', possible loss of data
  	PachisiView.cpp(681) : warning C4244: 'initializing' : conversion from 'INT_PTR' to 'int', possible loss of data
  	PachisiView.cpp(716) : warning C4244: 'initializing' : conversion from 'INT_PTR' to 'int', possible loss of data
  	spielregeln.cpp(293) : warning C4715: 'CSpielregeln::getZiel' : not all control paths return a value
  	Feld.cpp(30) : warning C4154: deletion of an array expression; conversion to pointer supplied 

• Massenhaft Speicherlöcher schon wenn ein Spiel gestartet und direkt das Programm beendet wird.
• Spielstarten, im Dialog der Spielernameneingabe bei leeren Namen auf "OK" klicken " -> Fehlermeldung "nichts eingegeben" -> Es ist nicht mehr möglich über "Neu" ein neues Spiel zu starten.
• Auch wenn ich nur ein Zweispielerspiel starten will lande ich im Vierspielermodus
• Bedienung: wenn ich eine andere Figur anwählen will muss ich erst die aktuelle Figur "entwählen". Unkomfortabel.
• Bug: bei euch gibt es keinen Grace ?!?!, ich kann jederzeit weitere Figuren ins Spiel bringen.
• Bug: folgender Screenshot zeigt neben dem regulären Zielfeld ein etwas seltsames Zielfeld. Ich kann sogar auf die blaue Zielgerade gehen ;-). Aber danach kommt die Figur nicht mehr weg und das Spiel ist tot.
  
• Wenn ich euer Savegame "Ende.yap" lade und gewinne, dann kann ich anschließend kein anderes Savegame laden, die Meldung "Team 1 hat gewonnen" bleibt erhalten und ich kann nichts machen.
• Kommentierung: eure Doxygen-Doku enthält leider fast nur unkommentiertes. Auch innerhalb der Methoden ist die Kommentierung ungenügend.
• Keine Contextsensitive Hilfe je nach Spielzustand.

• Die Regelerklärungen könnten detaillierter sein. Beispiel: wann kann ich Spielfiguren überspringen (eigene immer, gegnerische nur wenn keine Blockade, auf einer Burg kann ich aber auch 2 oder mehr Figuren überspringen)
• In dem Kapitel "Spielregeln" hätten ein paar mehr Grafiken gutgetan (das Unterkapitel "Spielelemente" hätte im Prinzip hier reingehört).
• Das Stichwort "Grace" taucht in der Hilfe nicht auf.

Gruppe C (546 881, 944 689, 946 665)

• Die Klasse "DialogHirnlos" aus meinem Hilfe-Beispiel fliegt bei euch noch rum.
• Keine Eingabe der Spielernamen
• Euer Spielfeld flackert massiv (nicht optimiert, siehe mein Beispiel "Zeichnen ohne Flackern"
• Was bedeutet die Meldung "Figur geht ins Spiel" nachdem eine Figur ins Spiel gebracht wurde ? Wenn ich z.B. beim Spielstart eine 2 würfele ist die Figur dadurch schon im Spiel ! Und wieso wird das Auswählen von "Nein" ignoriert ? Wenn ich mich auf der Zielgerade befinde erscheint diese Meldung ebenfalls.
• Bedienungskomfort: dass eine Figur beim Anklicken schon gezogen wird ist nicht komfortabel. Dadurch dass ihr die möglichen Zielfelder nicht anzeigt muss man sich selbst ausrechnen wohin die Figur kommt.
• Es ist nicht gut erkennbar wenn mehr als eine Figur übereinander stehen (bei 2 geht es noch, bei drei wird es schwierig)
• Bug: wenn bei einem Grace auf "Zug übergehen" geklickt wird, dann ist man nochmal mit Würfeln dran. Eigentlich müßte die Runde für diesen Spieler aber vorbei sein (nicht nur das aktuelle Würfeln übergehen)
• Es ist nicht möglich eine zweite Runde zu drehen
• Wenn keine der Figuren nach dem Würfeln ziehen kann dann erscheint zwar eine Meldung ich sehe aber nicht was gewürfelt wurde. Einmal kam es vor dass ich nochmal würfeln konnte, scheinbar ist da intern ein Grace gefallen.
• Woher kommt die Ausgabe "The thread 'Win32 Thread' (0xae4) has exited with code 0 (0x0)." bei jedem Klick ?
• Laden/Speichern nicht implementiert.
• Kommentierung ist weitgehend OK
• Ich finde es nicht allzu gelungen dass das Spielbrett bei euch als Singleton-Instanz existiert (und außerdem im Document als Instanzvariable, nur dort gehört es auch in).
• "pachisiRightView::OnBnClickedButton1": Diese Methode kapselt zuviel Spiellogik !

Gruppe D (243 408, 340 037, 137 236, 835 794)

• Zwei Compilewarnungen:

    stdafx.h(45) : warning C4005: 'WM_CONTEXTMENU' : macro redefinition
    pachisiview.cpp(1184) : warning C4715: 'CPachisiView::GetHelpID' : not all control paths return a value. 

• Euer Spiel startet auch wenn man im Startdialog auf "Abbrechen" klickt.
• Im Startdialog ist es möglich den Spielernamen komplett zu löschen. Die leere Anzeige ist ziemlich verwirrend.
• Das schon "angemeckerte" 1024-Problem: bei 1024x800 Pixel paßt euer Fenster nicht ganz auf den Bildschirm.
• Komfort: die Teams sind nicht aus dem Spielbrett erkennbar.
• (vielleicht auch mehr ein "Bug" in der Hilfe): Wenn ich gerade einen Grace gewürfelt habe und einen Stein auf das Startfeld gestellt habe darf ich bei einem zweiten Grace keinen weiteren Stein ins Spiel bringen. Wieso nicht ? Ist das ein Bug oder eine Einschränkung der Regeln ?
• Einige sinnlose Debug-Ausgaben beim Contextmenü
• Warum stellt ihr nicht auch einen einzelnen Stein auf einer Burg mit Stern dar ? Dann wäre es einfacher erkennbar was nicht stimmt.
• Wenn alle Spielsteine im Spiel sind ist nicht mehr erkennbar welcher Spieler welche Farbe hat
• Bug: bei euch ist es möglich beliebig oft um das Spielfeld zu laufen (erlaubt ist eine Zusatz-Runde)
• Der "Todeszug" auf das Feld direkt vor dem Ziel ist möglich. Damit ist das Spiel gelaufen. Siehe Screenshot (mögliche Zielfelder sind markiert, ich habe mich aber nicht getraut auch wirklich zu ziehen)
  
• Ob es sinnvoll ist ein gewonnenes Spiel zu speichern sei mal dahingestellt ;-)
• Ich fand die Übersicht verbesserungswürdig: ich mußte auch nach einer halben Stunde testen noch regelmäßig auf den Spielernamen und dann auf die Startposition schauen um herauszukriegen wer dran ist. Hier wäre eine bessere optische Darstellung sinnvoll gewesen.

• Der Todeszug-Bug ist behoben.
• Eine mehrfache Umrandung des Spielfelds ist nirgends verboten, also erlaubt.
• Es ist jetzt durch das in der Spielerfarbe markierte "@" vor dem Spielernamen deutlich erkennbar welcher Spieler dran ist. Einfache Lösung, große Wirkung !
• Leere Spielernamen werden jetzt mit Default belegt.