search

Team--Rocket / divercity

a city simulation game. Forked from micropolis (the simcity classic opensource version) and implemented numerous new improvements eg. more realistic traffic/new pathfinding, new buildings, technology research and more.
http://team--rocket.github.io/divercity
GNU General Public License v3.0
41 stars 12 forks source link

refactoring collectTax() function in Micropolis.java #41

Closed nylki closed 10 years ago

nylki commented 10 years ago

Wenn wir neue Gebäude hinzufügen die Funding benötigen, könnte es helfen den Teil zu refactoren in dem es um die Berechnung geht ob genug Geld für Polizei, Schule, etc. vorhanden ist, und wieviel tatsächlich bezahlt werden kann. Momentan ist das recht unschön gelöst mit einer sehr verschachtelten if-Abfrage in der auch noch eine Menge an Code-Wiederholungen vorkommen.

Siehe Zeilen ~1825 - 1890 (https://github.com/Team--Rocket/micropolis-java/commit/4d8ce4608a6765ff29eb6fdd991b8334e8028792#diff-1d48b4ceb02f1ec6bde857d43977425aR1826)

Ebef commented 10 years ago

Ich denke der code ist für die Verhältnisse noch 'relativ' schön. Natürlich kann man das schöner gestalten, die Entscheidung würde ich aber dem überlassen, der die neuen Fundingoptionen konkret implementiert.

Ebef commented 10 years ago

siehe auch https://github.com/Team--Rocket/micropolis-java/issues/14

Ebef commented 10 years ago

wontfix wegen https://github.com/Team--Rocket/micropolis-java/issues/14