Hallo Bald_Eagle

ich mach das hier mal so Öffentlich, damit andere Tool Creator auch den Punkt hier einsehen können - es ist aber zu spezifisch, als das ich das in die Ankündigungen so detailliert schreiben will.

Für ein kommendes Update brauchte ich die Möglichkeit einem einzelnen Planeten andere Multiproduktionswerte zu geben. Dieser Planet soll getrennt von allen Multiproduktionen einen Basis-Wert wieviel er produzieren soll erhalten (beispielsweise 100fach). Dieser Basiswert (also 100fach) soll aber auf alle Universenspeed-Konfigurationen angepasst werden.

Hier ein Beispiel, bei 100fach Produktion bedeutet dies:

RETRO Nexus:

erhalten eine 99x Multiproduktion auf dem Planeten zugschalten

Ein anderes Universum, als Beispiel Sirius (1,5x Universenspeed) wird nach folgender Rechnung aufgelöst:

100/Speed - 1

Also

65,67x (es ist ok, dass wir minimale Verschiebungen zwischen den Universen haben; das kann man hier leider nicht umgehen)

Bei der Entwicklung dieser Funktion ist folgendes Problem aufgetaucht:

Die Grundproduktion und die Minentech ergaben bei einer Multiproduktion durch Rundungsprobleme (mehr Nachkommastellen) unterschiedliche Werte.

Ich habe also die Grund- als auch die Minentech Berechnung so abgeändert, dass diese auf Grundlage von 1x Universenspeed errechnen, dann runden und dann *UniversenSpeed multiplizieren.

Ich weiss an dieser Stelle nicht, welche Formeln du wo genau nutzt. Mein Java ist jetzt auch nicht so gut, dass ich dies schnell nachschlagen kann.

Hier meine Änderungen der 3 Formeln:

Mienentech:

$production[$i] = floor(
                                            $production[$i] / $this->config["game.speed"]
                                            * ($energyTec/50)
                                        ) * $this->config["game.speed"];

Für die Minen-Produktion

$production = (
                                 ($this->config['resource_' . $i . '.production'] / $this->config['game.speed'])
                                    * $stufe * pow(1.15, $stufe)
                                ) * $factors['resource_'.$i.'_factor'] * 0.1 * $productionsFactor;
                $p = round(round($production) * $this->config['game.speed']);

Wie du siehst, habe ich für die Minentech die Nachkommastellen mit floor weggetrimmt, bei der Produktion hab ich mich für ein runden entschieden.

Reichen dir diese Informationen bereits? Ansonsten kann ich versuchen das Problem / die Lösung) genauer zu erklären

Sobald das Update raus kommt, können wir ansonsten auch nochmal über die genaue Herleitung / Änderung sprechen. Ich wollte dich nur (ist glaube auch das erste mal) vorwarnen.

Viele Grüße

Dschibait

Quote from Bald_Eagle

Gibt es einen bestimmten Grund, warum deine 'Basisproduktion' bereits mit dem Speedfaktor behaftet ist? Du könntest doch auch erst die gesamte Produktion komplett mit 1x Speed berechnen und erst am Schluss (ggf. runden und) mit dem Universumsspeed ($this->config['game.speed']) multiplizieren.

Dann würdest du zwischendrin jeweils die Division einsparen.

das war einfach so als ich es übernommen habe

Grundproblem dabei wird sein, dass diese Werte alle schön aufgelistet sein sollten (in der Wirtschaft) und der Programmierer (ich will keinen Flamen) dann einfach noch nicht so weit war, dass im nachhinein zu machen.

Ich kann dir nicht sagen warum das tatsächlich gemacht wurde. Im SI2 wird alles mit 1fach berechnet und dann (durch den %-Einstellung des Gebäudes) hochgerechnet.

Quote from Bald_Eagle

$energyTec/50 für Minentechnik scheint mir auch keine besonders glückliche Namenswahl.

da gibts noch schlimmere Bezeichnungen die noch verwirrender sind. Kommt aber auch nicht von mir.

Quote from Bald_Eagle

Da sind nur 2 Formeln

ja ich hab dir die Spice-Formel (oben wäre es nur für normale Minen Prodd) nicht mit zugeschrieben. Da wäre die Temperatur noch mit drinne; das war recht umständlich eh hier noch aufgebaut (weil wir den Temperaturbonus auch mit dem Unispeed multiplizieren) - ich glaube aber du weisst das für diese 3. Stelle auch umzusetzen.

Quote from Bald_Eagle

Wird nach der Änderung 24x Stundenproduktion = 1 Tagesproduktion sein?

war es das bisher nicht?

Ich hab in alten Universen geschaut, da passte es - mit der neuen Berechnungen auch.

sync für den Fix ist raus; Ankündigung für den Rest kommt die Tage.

Die obere Formel ist die Berechnung der Minen-Grundproduktion.

Darauf kommen ja aber nochmal Gebäudeuppgrades, VIP Bonus und co, die nicht Ganzzahlen (int) sind..

Deswegen kommt da am Ende was ungerades raus, was wir vorher floor nehmen müssten um dann die Tagesproduktion auszurechnen.

SpaceInvasion ist ein sehr altes Spiel - gerade diese Produktionsberechnungen bin ich jedes mal Froh wenn wir uns da auf etwas geeinigt haben, was auch in den Tools so läuft. Das sind viele viele Zeilen Code, die wirklich wirr an Runden oder eben nicht Runden oder sonst was sind.

Eine Umstellung auf eine übersichtliche / sinnvolle Rechenweise wäre tatsächlich recht aufwendig (Fehler auch nicht ausgeschlossen) und würden für den Nutzen, nach meiner Meinung nach, zu viel Zeit kosten.

Ihr dürft an dieser Stelle nicht vergessen, dass hier 3 Rechnungen aufeinander Treffen von Bald_eagle, xsonu und eben mir - wobei meine Berechnung durch software architekturische Entscheidungen unnötig noch mehr aufgebläht wurde (Stichwort Gameserver und Webseite sind unterschiedliche Ermittlungsverfahren auch wenn hier gleiche Formeln grob genutzt werden)

Das macht an einigen Stellen tatsächlich Sinn, an einigen anderen überhaupt nicht; einiges habe ich schon neugeschrieben um Fehler (2 Fenster Bug, Teleporationsbug und plötzliches Ressverschwinden) zu beheben.