Stadtwachstum Release vs Nightly R8788

[makie]

Ich hab die Änderung Revision 8796 gesehen.
Aus meiner Sicht behebt das nicht die Fehler in der Prüfung zur Gebäude Größe.

schlimmer noch das ist selbst ein neuer Bug

- if(thislevel>level) {
+ if(thislevel>level  &&  thislevel<start_level+6) {
if (selections.empty()) {

Der Sinn von "if(thislevel>level) {" ist im wesentlichen per break die Abarbeitung der for Schleife vorzeitig zu beenden. Das bringt Performance mäßig vermutlich einiges. Vor allem beim Start der Karte, da die hohen Level gar nicht bearbeitet werden.

Mit der Änderung wird die Schleife nur noch abgebrochen wenn thislevel<start_level+6 also des öfteren nie. Jetzt läuft er meistens wohl bis zu bitteren Ende der Tabelle.

Ein if (thislevel > start_level+6) break; wäre wohl besser gewesen

------------
Verzeihung das ich aus meinen vorherigen Postings das mit dem Level + 6 nicht wieder raus gelöscht habe, nach dem ich bei meinen weiteren Tests den Fehler in der Prüfung zur Gebäude Größe lokalisiert habe.
Also entweder bin ich zu dumm und kapieren den Trick nicht, oder (desc->get_x() <= maxsize.x  ||  desc->get_y() <= maxsize.x) ist schlicht Unfug.

Wenn die Nachfolgen Routinen damit umgehen können, das Gebäude zu drehen, wenn es normal nicht passt, dann kann man auch gedrehte durchlassen.

dann könnte man die Prüfung auch so machen:
            // size check
         (  (desc->get_x() <= maxsize.x  &&  desc->get_y() <= maxsize.y  &&
             desc->get_x() >= minsize.x  &&  desc->get_y() >= minsize.y  ) ||
            (desc->get_x() <= maxsize.y  &&  desc->get_y() <= maxsize.x  &&
             desc->get_x() >= minsize.y  &&  desc->get_y() >= minsize.x  )  ) {
das mögen ein paar if mehr sein, aber zumindest ich bin mir dann sicher dass das so funktioniert und richtig ist.
Dann wäre aber eine Prüfung ob es die gedrehte Grafik auch gibt, noch nützlich.