Weitere If-Abfrage durch Else ersetzt, da sonst eventuell auch nach einem erfolgreichen Verschieben angezeigt wird, dass man nicht genügend Einheiten habe.
Laut Java sind "08" und "09" keine gültigen int-Werte, aber "01", "02", ... sind gültig.
Die vorangestellte 0, die beim Erstellen des Arrays geholfen hat, wurde daher wieder entfernt.
Die Kurzschreibweise für die Provinz-Initialisierung wurde nun wieder hinzugefügt. Zeitnah wird über einen anderen Branch die alte Schreibweise entfernt und die neue stattdessen eingesetzt.
Diese Vereinfachung fasst die Daten der Provinzen zum Besitzer und zur Einheitenanzahl in ein Array namens dataL zusammen.
Bisher ist der alte Code noch im Einsatz, eine For-Schleife überprüft bisher nur den Inhalt des dataL-Arrays mit den vom alten Code gegeben Daten.
Später, sobald feststeht, dass die Arrays korrekt erzeugt werden, der Prüfcode durch einen Code ersetzt werden, der die Daten des Arrays verwendet und den Provinzen übergibt. Damit werden die langen Zeilen dann ersetzt.
Bisher unvollständige und daher fehlerverursachende Vereinfachungen wurden entfernt. Alle zukünftigen Änderungen für partialle Vereinfachungen werden fehlerfrei sein!
*.redrawProvince() für jede Provinz wurde entfernt zusammen mit der For-Schleife, die dies erledigen sollte, da die Provinz dies selbst beim Festlegen einer Einheitenanzahl macht.
- Verbesserung: Abgeschnittener Kommentar ergänzt
- If-Blöcke zu einem If-Block kombiniert mit "else if"
- Anfang Kürzung der vielen Befehle durch ein Ersetzen mit **ownerL**
und **entitiesL** als *int[]*
- 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
- *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 meisten Methoden in der GeneralMap wurden kommentiert und verbessert, außerdem bekam GeneralMap endlich nun wichtige Variablen dazu.
Bei der Province Klasse wurde ein Import-Fehler nun endgültig behoben.
Die Klassen Spieleranzahl, Spieloptionen und Weltenauswahl wurden nun entfernt, da bei der neuen Aufgabenteilung diese unnötig wurden. Sie enthielten kaum Code, welcher aber für massig Fehler sorgte.
Da die Konstruktoren der hier verwendeten Klassen verändert wurden,
arbeiten diese Code-Zeilen nicht mehr korrekt. Um Fehlern aus dem Weg zu
gehen, wurden diese auskommentiert. Diese Klasse wird später über einen
anderen Branch wieder verbessert. Diese Änderung muss beim Pull Request
nicht übernommen werden.
Zocker1999NET