- *Colors*: Statt mit **color** wurde mit **color[]** versucht, das
Array als Argument zu übergeben
- *Start_Button*: Die beiden Werte der Variablen **text** und
**textSize** wurden in den Konstruktoraufruf eingegliedert.
Außerdem wurde der Einzug von 4 Leerzeichen auf Tabs umgestellt, was
eine kleinere Dateigröße bedeutet.
- Kleiner Fix in der act-Methode, damit diese später leichter in die GeneralMap-Klasse kopiert werden kann.
(Durch eine eigene Abfrage der Anzahl der Provinzen wird zukünftige Schreibarbeit eingespart.)
- Less Code. More Army. More Schubser.
- Removed Herobrine
Der Pull Request zeigt an, dass diese Datei einen Unterschied zur der Version hat, die im Master Branch liegt.
Da eigentlich kein Unterschied vorhanden ist und ich dies bereits zuvor gemerkt habe, vermute ich das Problem beim Format des Zeilenendes (Wiki: CRLF und LF). Deshalb wird nun diese Datei gleich erneut hochgeladen, um diesen vermuteten Fehler zu beheben/auszuschließen.
- SCALE_VALUE ist nun eine Kommazahl als double, dies hat sich eher angeboten, um Fehler zu vermeiden
- **getX** und **getY** waren hierfür die falschen Methoden zum Abfragen der hinterlegten Position (sie geben nämlich die aktuelle Position des Actors zurück, was bisher noch nicht geht an dieser Stelle, da die Provinzen noch nicht einer Welt hinzugefügt wurden)
- Expliziter Cast für das Umwandeln in die verschiedenen Typen fehlte noch
ArmySchubser-Aktualisierung
Wird nun früher als geplant durchgelassen, um dringende Fehlerverbesserungen bei verschiedenen anderen Klassen durchzuführen.
- *GeneralMap*: Hinzufügen der Konstanten und der Methode **initProvinces**
- *Map_World*: Umstellung auf Verwendung der neuen Methode
- Architektur: Dokumentation der neuen Methode hinzugefügt, dabei wurde die Methode **addProvinceToMap** gestrichen
Die im Code bereits vorhandenen und genutzten Methoden **getXPos** und **getYPos** der *Province*, welche in der Architektur bisher noch fehlten, wurden nun hinzugefügt.
Es wurde ein Fehler in der Province Klasse festgestellt, der mit den
Nachbarprovinzen zu tun hat. Dies wurde nun hier gefixt.
Damit ist der ArmySchubser bis auf ein paar fehlenden Verbesserungen
prinzipiell fertig.
Die hier aufgeführten Klassen sollen mit Erreichen des Beta-Meilensteins aus dem Projekt entfernt werden. Die Begründungen sind in den jeweiligen Dateien direkt am Anfang zu entnehmen.
Teilweise wird Code aus den Klassen in Andere transferiert.
Sollte für Teile des Codes oder für den gesamten Code wieder eine Verwendung bestehen, wird die Entfernung wieder ausgesetzt.
Die *DicesList* Klasse und deren Unterklassen wurden aufgenommen.
Die Erweiterung der *Utils*-Klasse wurde aufgenommen.
Bei der Funktionsliste wurde das Wort "erweitert" durch "extends" ersetzt.
Die Methode **genDices** wurde direkt in den Konstruktor der *DicesList* integriert.
Es wurde ein Kompilierungsfehler bei der *DicesList* verbessert.
Die Sortierung der Augenzahlen wurde nun zu absteigend geändert, die Funktionsweise hierfür wurde in die *Utils* Klasse ausgelagert.
*DicesList* bekam die Methode **removeAll** dazu.
Die Extra Kategorie "GUI Objekte" wurde entfernt, da an der Vererbung von *GUI_Interface* eindeutig erkennbar sein sollte, welche Art von Actoren sich als GUI Objekte eignen.
Diese neue abstrakte Klasse generiert eine Liste von Würfeln, welche hintereinander dargestellt werden. Die Unterklassen DefenderDices & OffenderDices passen jeweils nur noch die Farbe der Würfel an.
Entsprechende Einträge in die Architektur folgen.
JulienRo