- *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.
Felix Stupp