Vielen Dank für die Hilfe. Hab es jetzt mal so eingebaut und es funktioniert wunderbar. =)Zitat von MagicMaker
Vielen Dank für die Hilfe. Hab es jetzt mal so eingebaut und es funktioniert wunderbar. =)
--
Wenn sich deine Abfragen gegenseitig ausschließen, brauchst du gar keine Verschachtelung.
Würde das, theoretisch gesehen, keine (wenn auch übertrieben minimale) Verschlechterung der Performance bedeuten? Hab mich damit noch nicht so beschäftigt, deshalb frage ich nach (:
Würde einfach jetzt erwarten, dass wenn ich bspw. 200 Abfragen habe und ich diese in bspw. 4 "Abschnitte" unterteile, es "schneller" abgefragt wird, als wenn alle 200 durchlaufen werden.
Bei 200 Abfragen, ohne Unterteilung, prüft das Programm 200 mal, ob XYZ == ABC ist.
Bei 200 Abfragen aufgeteilt in 4 Blöcke ( XYZ < 50, XYZ < 100, XYZ < 150, ELSE) werden maximal 51 Abfragen getätigt.
Zuallerersteinmal musst du dir vergewissern, dass du, wenn du mit Event-Script arbeitest, sowieso eine sehr viel schlechtere Performance hast als wenn du die gesamten Abfragen direkt mit Ruby implementieren würdest.
Und obwohl du zwar recht hast, dass du Performance verlierst wenn du die If-Abfragen hintereinander setzt anstatt sie zu schachteln, so musst du dir im klaren sein, dass diese Abfragen allesamt sehr minimal und einfach sind.
Es sind allesamt lediglich Vergleiche auf Integern oder Abfragen von Booleans. Beides bedeutet minimalen Aufwand.
Wenn du diesen Abschnitt wirklich sehr oft laufen lässt und dir Sorgen um Performance machst so kann ich dir wirklich nur nocheinmal dazu raten es einfach in Ruby zu schreiben und im Event ein Custom-Script aufzurufen.
Wenn du uns einmal zeigen könntest was genau du eigentlich in den Event-Commands stehen hast könnten wir dir sicher dabei helfen falls du dir unsicher bist.
Na ging ja nur um's Prinzip. Ich selbst verwende solche Verschachtelungen/Events nicht ...
Hab mich nur nie tief mit der Abarbeitung vom Eventcode des Makers beschäftigt (VX schon einmal gar nicht).
Habe ja auch gleich gesagt, dass eine simple Abfrage im Normalfall eh schon minmale Auswirkungen hat, aber wie gesagt keine Ahnung was dort im Maker alles passiert.
Die vom TE dargestellte Verschachtelung sieht ganz nebenbei fast so aus,wie du auch schon angedeutet hast, als könnte man diese auch durch 3-4 Zeilen Ruby ersetzen.
Wenn sich die Abfragen gegenseitig ausschließen, kann er ans Ende jedes true-blocks einen "Jump to label" befehl setzen, durch den das script von da aus hinter die letzte Abfrage springt. Dann werden - genau wie bei der verschachtelung auch - die Conditionals nur so lange durchlaufen, bis das script auf die erste bedingung stößt, die "true" ist.
Vom Grundsatz her hast du aber natürlich recht., ich wqürde vermutlich bei 200 Abfragen auch erstmal "Bereiche" abfragen.
Geändert von caesa_andy (15.08.2012 um 22:20 Uhr)
Das ganze sollte dafür gedacht sein, den Inhalt einer Booster-Packung für mein Spiel zu bestimmen. Dafür habe ich am Anfang des Events ein Random 1-346 erstellt.
Die Erweiterung hat eigentlich nur 70 Karten, die 346 kommt zustande, weil ich nicht will das jede Karte die selbe Chance hat in der Packung zu sein. So hat eine Stern-Karte nun 4, eine Karo 5 und Kreis 6 Zahlen zugeordnet bekommen.
Mit den Blöcken klappt es ganz gut. Kann eigentlich keine Performance-Einbrüche wahrnehmen, wenn das Event läuft.
@Cornix: Dachte ich mir schon, aber ich habe leider keine Ahnung vom Scripten. Von daher muss ich mir erstmal mit Events weiterhelfen. Sollte ich aber evtl. mal lernen.
--
Geändert von Sanghelios (16.08.2012 um 23:27 Uhr)
In dem Fall bietet sich aber tatsächlich, wie Vorposter geschrieben haben, es nicht in Elsifs zu packen, sondern mit einem Label ans Ende zu Springen
so kommst du nie tiefer als Verschachtelungstiefe 1 innerhalb dieser Funktion
Nein, das ändert auch nichts. Das Jump mag zwar das ganze verkürzen, aber das kannst du zusätzlich auch benutzen, wenn du Bereiche verwendest.
Nehmen wir mal den Fall, dass die Variable 200 ist. Dann würde er ohne Unterteilung trotzdem 200 Abfragen durchrattern. Bei einer Unterteilung in vier Bereiche wie gesagt nur 50.
Prinzipiell magst du ja recht haben, aber der Event-Interpreter ist sowieso nicht "so effizient". Letztlich bedeutet jede Zeile Eventcode mehr Aufwand für den Interpreter. Zuviele zusätzliche Branches und Einrückungen können das Ganze genauso gut auch langsamer machen.
Aber kommen wir mal auf das Problem zurück: Es geht um Boosterpackungen. Da der Spieler wohl kaum im Millisekundentakt Boosterpackungen erhält, ist es vollkommen Wurscht wie schnell das Programm durchläuft. Mach dir also keinen Stress und setz es so um wie du es am einfachsten und übersichtlichsten findest.
Äh ... ich bin mir jetzt echt nicht sicherf, ob das an mir und mangelndem verständniss deinem Script gegenüber liegt, aber so wie ich das sehe, KANN das Event, das du da oben gepostet hast gar nicht funktionieren, Confodere.
Wenn du mit Else-ifs arbeitest, muss der größte Bedigungswert der erste in der Reihe sein. Denn wenn die Bedingung "Variable 0011 <= 12" wahr ist, ist auch die Bedingung "Variable 0011 <= 4" wahr, womit das Event niemals weiter als bis zur ersten abfrage durchlaufen wird.
Das Event ist meines Erachtens nach korrekt.
Für Werte zwischen 1 und 4 gibt es Shiva, zwischen 5 und 8 Diablo, zwischen 9 und 12 Volt, etc etc.
Immerhin ist 5 nicht kleiner gleich 4 aber kleiner gleich 8.
Ah ... vergiss was ich gesagt hatte, ich war mit den Vorzeichen durcheinander gekommen.
Passt schon so.
Richtig, mein Lösungsvorschlag ist nich effizienter, aber du kannst den Code hinterher noch lesen, weil er nicht ewig weit eingerückt wird. Ich hab nicht mitbekommen dass es um Effizient geht, falls dem so ist, und ansonsten geb ich KD recht, ist vermutlich irrelevant.
Berechtigungen
Zitieren