[DynRPG-Plugin][RM2k3] MessageExtender | BETA

[Morden]

HeyHey!

Aufgrund dieses Threads kam zum Thema, dass ein Plugin gewünscht wäre dass Item- und weitere Pointer in Messages ermöglicht. Nun hab ich mich mal rangesetzt und mal mein »Glück« versucht und habe das Teil zwar soweit ausprobiert und bei mir hat auch alles funktioniert, aber ich kann ja nicht alle Situationen nachstellen - deswegen gebe ich das Plugin hier erstmal als »BETA« frei und hoffe, dass ihr das Teil mal fleißig testet^^

Vielen Dank an:

- goldenroy, der indirekt den Wunsch nach einem solchen Plugin geäußert hat ^^
- Cherry, der mir bei der Lösung des Problemes geholfen hat, genau die richtige ScriptLine anzusteuern.

Das »?« entspricht der jeweiligen ID in der Database, bzw der Map bei den Events.
Ihr könnt unendlich verschachteln. Es sind also auch Konstrukte, wie z.B.

Code:

\D[\v[\v[20]]]

bauen. Das bedeutet in diesem Fall, dass der Rang des Helden, welcher die ID des Wertes hat, die in der Variable steht, welche die ID des Inhaltes der Variable mit der ID 20 hat. Ich weiß, zu kompliziert erklärt, aber mit fiel auf die Schnelle nichts besseres ein xD

Es funktioniert in Schleifen und mit PP's. Ich hab's mit 6 PP's getestet und es hat funktioniert.

Code:

\B[?] oder \b[?]
    - Switchwert ( 0 oder 1 )
    
\D[?] oder \d[?]
    - Heldenrang (Degree)
    
\i[?]
    - Itemname
    
\I[?]
    - Itembeschreibung
    
\F[?] oder \f[?]
    - Zustandsname
    
\a[?]
    - Skillname
    
\A[?]
    - Skillbeschreibung
    
\E[?] oder \e[?]
    - Eventname

!ACHTUNG!
Es kann zu Fehlern kommen, wenn mehr als zwei PP's gleichzeitig versuchen eine Message anzuzeigen, ohne dass ein EventCommand dazwischen folgt!

download MessageExtender.zip
download MessageExtenderSRC.zip

PeAcE
MorDen

[MagicMaker]

Code:

\s[?]
    - Skillname
    
\S[?]
    - Skillbeschreibung

...

Ähm... S wird doch aber eigentlich schon für das Tempo verwendet. òo

Code:

\B[?] oder \B[?]

...

Ich glaube eines der [B]s sollte klein sein.

Code:

\e[?] oder \e[?]

...

Ich glaube eines der [e]s sollte groß sein.

[goldenroy]

Gibt es \v und \n nicht schon vom Maker aus? Ansonsten eine sehr hilfreiches Plugin, wird bestimmt gut Verwendung finden (Falls es denn funktioniert, ist ja schließlich Beta). (:

Oh, und zusätzlich zu MagicMakers Post wird "\c" schon für die Farbe verwendet. 0:

[MagicMaker]

Gibt es \v und \n nicht schon vom Maker aus?

...

Ja, undzwar in exakt der beschriebenen Funktion.

Warum die hier extra erwähnt werden und alle anderen ausbleiben, weiss ich allerdings nicht.
Nur weil's keine Buchstaben sind? Wäre seltsam.

Ach und... \c[?] gibt es auch schon.

[Morden]

Ähm... S wird doch aber eigentlich schon für das Tempo verwendet. òo


Ich glaube eines der [B]s sollte klein sein.


Ich glaube eines der [e]s sollte groß sein.

...

Nicht nur das, "c" im Übrigen auch >.< Ich hätte es vorher nochmal durchschauen sollen, bevor ich's aus meinem ImageText Plugin kopiert habe ^^'' Da gab's kein Speed und keine Farbe xDD

Sorry :/ Hab's korrigiert und ich hoffe, dass ich kein Standard-Kommando vergessen habe. Sollte jetzt funktionieren ^^''

PeAcE
MorDen

[Cherry]

morden, y u no src?

Ich würd mich für deine Lösung des Loopproblems interessieren.

EDIT: Erweiterungsvorschlag: \v4[123] - gibt min. 4 Stellen aus, mit 0-Padding. Sollte natürlich für \v2 bis \v7 gehen.

[Morden]

morden, y u no src?

Ich würd mich für deine Lösung des Loopproblems interessieren.

EDIT: Erweiterungsvorschlag: \v4[123] - gibt min. 4 Stellen aus, mit 0-Padding. Sollte natürlich für \v2 bis \v7 gehen.

...

Weil der Source mehr als schlecht ist, wie ich finde xDD - und mir vor allem gerade aufgefallen ist dass dort noch einiges fehlt bzw. nicht ganz rund laufen wird - wie gesagt hab den Großteil aus meinem TextImage-Plugin kopiert, was dazu führt, dass nicht alle Maker-Standard-Commands ignoriert werden und deshalb Fehler geworfen werden^^ Aber heute habe ich keine Lust mehr darauf xD

Das Loop-Problem habe ich ganz einfach mit nem vector gelöst. Da schiebe ich die jeweilige MessageZeile rein, bevor ich sie parse - das passiert ja alles in einem einzigen Command. Und jedes Command frage ich ab, ob der Vector leer ist, wenn nicht dann springe ich so viele Zeilen zurück, wie der vector groß ist und fange an die Lines wieder mit dem alten Text zu ersetzen. Danach leere ich den vector. Und das sollte eigentlich IMMER funktionieren, weil EndOfEvent ja auch ein EventCommand ist, oder nicht?

Hier der Source:
ENTFERNT

EDIT:
Wahrscheinlich sollte ich aufgrund von PP usw noch die EventId usw speichern und auswerten?

PeAcE
MorDen

[Cherry]

Tja, es kann aber mehr als ein Skript laufen! Ein PP reicht aus und deine Lösung funktioniert nicht mehr sondern corrupted das andere Event indem irgendwelche Stringparameter überschrieben werden oder, wenn du Pech hast, auf Zeile -1 oder so zugegriffen wird und das Spiel crasht.
Darum meine Idee, auch wenn sie hacky ist - aber damit sind die Sachen wirklich konkret an eine ScriptLine gebunden!

EDIT: return false verhindert nur dass die aktuelle Skriptzeile ausgeführt wird, das ist sinnvoll wenn man einen Eventbefehl ersetzen will durch eigenen Code. Sag mal, warum verwendest du am Ende "i = -1" statt "break" um einen Loop zu beenden? Davon abgesehen ist der Source nicht schlecht, da gibts weitaus schlimmeres. Im Falle DynRPG ist es immer erfreulich wenn Source da ist, weil es eh an Beispielen fehlt.

EDIT2: eventId ist aber nicht verlässlich im Moment, genauso wie pageId... Und selbst wenn das schon gefixt wäre: Du vergisst dass dasselbe Event 2x laufen kann, nämlich wenn es durch zwei verschiedene PPs gecallt wird (oder 1x durch ein PP gecallt wird und 1x ohne ein PP läuft). Und Callstack ist noch nicht implementiert.

[Morden]

Tja, es kann aber mehr als ein Skript laufen! Ein PP reicht aus und deine Lösung funktioniert nicht mehr sondern corrupted das andere Event indem irgendwelche Stringparameter überschrieben werden oder, wenn du Pech hast, auf Zeile -1 oder so zugegriffen wird und das Spiel crasht.
Darum meine Idee, auch wenn sie hacky ist - aber damit sind die Sachen wirklich konkret an eine ScriptLine gebunden!

EDIT: return false verhindert nur dass die aktuelle Skriptzeile ausgeführt wird, das ist sinnvoll wenn man einen Eventbefehl ersetzen will durch eigenen Code. Sag mal, warum verwendest du am Ende "i = -1" statt "break" um einen Loop zu beenden? Davon abgesehen ist der Source nicht schlecht, da gibts weitaus schlimmeres. Im Falle DynRPG ist es immer erfreulich wenn Source da ist, weil es eh an Beispielen fehlt.

...

Das ist mir auch gerade aufgefallen xD Und du meintest ja irgendwann mal, dass sich EventId usw nicht zuverlässig abfragen lassen - zumindest hab ich das irgendwann mal gelesen. Wie gesagt, das wäre sonst vielleicht auch noch ne Idee, zumindest für Map-Events - bei CE's gibt's ja keine eindeutige Identifizierung, außer vielleicht die DatabaseID.

Wahrscheinlich wäre deine Lösung aber momentan wirklich das effektivste.

Ich habe ehrlich gesagt keine Ahnung warum ich das gemacht habe xD Wahrscheinlich war ich nicht ganz da oder so

PeAcE
MorDen

[Morden]

So, nachdem Cherry und ich auf 'ne gute Lösung für das Problem mit PP's usw. kamen, hab ich das Ganze jetzt mal soweit fertig gemacht. Ich hab's mit 6 PP's getestet und es funktioniert.

Leider gibt es aber einen Fall, bei dem das Ganze nicht funktioniert, bzw. wo es zu Fehlern kommen kann. Nämlich dann, wenn zwei PP's gleichzeitig eine Message anzeigen wollen und dies unmittelbar nacheinander geschiet - also ohne, dass eine Eventzeile dazwischen ausgeführt wird. Aber ich denke, dass kann man verhindern und das wird auch nicht allzu oft vorkommen ^^''

Hab den ersten Post aktualisiert. Für Interessierte gibt's dort auch den Source - den hab ich mal ein wenig kommentiert.

PeAcE
MorDen

[Cherry]

Code:

                if( id > 0 && id <= RPG::map->events.count() )
                    return RPG::map->events[id]->getName();
                break;

...

Keine gute Idee. Angenommen eine Map hat die Events 1, 2, 5 und 9. RPG::map->events.count() ist dann 4. \e[5] und \e[9] werden also fehlschlagen. Besser: if(RPG::map->events[id]) bzw. RPG::Event *event = RPG::map->events[id] und dann if(event) return event->getName()

Technisch betrachtet trifft das auch auf Actors und andere NamedCatalogPtr-Typen zu (count() sollte da nur für das Iterieren über alle, mit ->list[i], verwendet werden), aber bei denen sind eigentlich immer die IDs mit 1 beginnend ohne Lücken vorhanden.

[Morden]

Keine gute Idee. Angenommen eine Map hat die Events 1, 2, 5 und 9. RPG::map->events.count() ist dann 4. \e[5] und \e[9] werden also fehlschlagen. Besser: if(RPG::map->events[id]) bzw. RPG::Event *event = RPG::map->events[id] und dann if(event) return event->getName()

...

Hab's schon korrigiert, aber noch nicht updated -> Wollte nicht kurz danach schon wieder nen Update schmeißen. Merke aber gerade, dass das wirklich nicht so trivial ist, wie ich dachte. Lade das korrigierte denn heute Abend auf jeden Fall hoch (wegen der Events-Sache).

In der nächsten Version will ich dann noch deine Idee mit dem Padding umsetzen, Cherry. Und meinst du, es wäre performancetechnisch groß problematisch, wenn ich zwei Pointer speichern würde (in einem [2]-Array) und dann jeweils beide überprüfen lasse? So würde die Sache mit den PP's zumindest bei zwei parallelen Events aus der Welt geschafft sein, oder nicht? Denn ich denke nicht, dass jemand mehr als zwei PP's hat, die gleichzeitig eine Message anzeigen wollen.

PeAcE
MorDen

[Cherry]

Sollte keinen großen Unterschied machen.

[DNKpp]

Sofern du nicht ein Array von 10k Größe (oder vector, oder list, oder was weiß ich) mit einem Bubble Sort oder so etwas in der Art bearbeiten möchtest, wird keiner Performance mäßig merken, das du was speicherst und überprüfst. C++ != maker, in C++ kann man viel mehr, viel besser und viel schneller checken Also lieber zu viel prüfen, als zu wenig, weil an die Grenze wirst du so schnell nicht stoßen, und wenn es Probleme gibt, ist meistens eine andere Stelle daran Schuld und nicht der Check an sich.

mfg

[Morden]

So, hab's mal updated.

Funktioniert jetzt auch, wenn zwei Event's gleichzeitig eine Message anzeigen wollen ohne Probleme. Mehr - denke ich - sollten aufeinmal nicht auftreten.
Und Cherry's Eventproblematik ist auch behoben.

Viel Spaß^^ In der Hoffnung, dass das auch jemand gebrauchen kann

PeAcE
MorDen

[PepsiOtaku]

Sorry für die schreckliche google Übersetzung. Ich liebe dieses Plugin, aber ich fand ein riesiges Problem mit ihm. In vielen Orten (meine Steal System zum Beispiel) Ich versuche, dies zu tun:

Code:

"Stole \i[\v[23]]!

um das Element mit der ID 23 angezeigt. vielleicht 30-50% der Zeit funktioniert es gut, aber für die anderen 50-70% der Zeit, zeigt diese:

Code:

Stole [23]!

Ich habe dann entweder zurück zum Hauptmenü und legen Sie eine neue Datei oder das Programm beenden, um vielleicht die Chance bekommen, es funktioniert wieder. Ich schaute auf den Quellcode, aber es ist alles über onComment getan, also bin ich nicht sicher, was das Problem sein könnte. Parsing problem?

[Corti]

Probier mal zwei Sachen aus:

• Teste die Funktion in dem du eine MessageBox auf einer normalen Map aufrufst
• Teste die Funktion in dem du den MessageBox Befehl im Kampfsystem aufrufst ( den aus Liste von Befehlen im Kampfsystem )
• Jetzt schreibe dir ein CommonEvent in dem die MessageBox enthalten ist und rufe dieses CE im Kampfsystem auf.

Wichtig: Erstelle jeweils die MessageBox per Klick, kein Copy&Paste von Sonstwo.

Ansonsten: Schreib die Inhalte der MessageBox nochmal neu. Manchmal gibts so Zeichenkombinationen wie ^` die unsichtbar irgendwo in Zeichenketten drin sind, sieht man nicht, stören aber Compiler und Interpreter.

[goldenroy]

Sorry für die schreckliche google Übersetzung. Ich liebe dieses Plugin, aber ich fand ein riesiges Problem mit ihm. In vielen Orten (meine Steal System zum Beispiel) Ich versuche, dies zu tun:

Code:

"Stole \i[\v[23]]!

um das Element mit der ID 23 angezeigt. vielleicht 30-50% der Zeit funktioniert es gut, aber für die anderen 50-70% der Zeit, zeigt diese:

Code:

Stole [23]!

Ich habe dann entweder zurück zum Hauptmenü und legen Sie eine neue Datei oder das Programm beenden, um vielleicht die Chance bekommen, es funktioniert wieder. Ich schaute auf den Quellcode, aber es ist alles über onComment getan, also bin ich nicht sicher, was das Problem sein könnte. Parsing problem?

...

You don't have to use Google Translator, I do think people understand English quite well here.
That problem sounds interesting, I doubt that it has anything to do with writing "[23]" instead of "[0023]", right? ;0
Btw - I love your plugins. Especially the menu ones, that's some of the best stuff I've seen with DynRPG! Q_Q

[PepsiOtaku]

@Corti: Yes, I tried all of those scenarios. Those steal commands are pulled from common events.

@goldenroy: I'll try using leading zeros and see if that makes a difference, although I'm a little skeptical. Thanks for the compliment! I've actually got a couple more plugins on the way. The biggest one being auto-switches. I wrote a little blurb about them here: http://rpgmaker.net/forums/topics/13...940#post490940

Edit: Still no luck after trying leading zeros. :/

[Morden]

@PepsiOtaku
Did you get the same result, when you output these messages vie an event on map and not via a common event? I will try to reproduce this problem in the next week. I also wanted to extend this plugin so that you can use padding like Cherry mentioned some posts before, but I didn't found time last months. But now I should have more time for delevoping some DynRPG-Stuff.

PeAcE
MorDen