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.
Die Debug Klassen "Debug3", "Debug4" und "Debug5" stellen eine Map_World dar, welcher direkt die Namen und Farben einer entsprechenden Anzahl an Spielern (3,4,5) übergeben wird. Damit sollte das Debuggen wesentlich schneller umsetzbar sein.
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[]*
- 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.
Zocker1999NET