Druckbare Version
Hallo ich habe ein Problem.
Mein Script funktioniert nicht so ganz wie Ich das möchte.
Ich habe 5 Auswahlmöglichkeiten bei einen NPC
Reden,Gedanken,Werwolf,Verwandeln,gehen.
Nund klappt es nicht das es sich hin und her schaltet. Am Anfang geht es nur wenn Ich zweimal nach "-->" klicke auf der Tastertur, dann springt er zu Gedanken. Aber dann geht es nicht mehr.
Was mache Ich falsch?
Script:
Verwende für so etwas bitte den EasyEventExporter. Spart Arbeit und zeigt außerdem Eventeinstellungen an, die normal nicht sichtbar sind (z.B. Picturetransparenzen, etc.)
1. das ganze Event kopieren (Rechtsklick->kopieren)
2. EasyEventExporter öffnen
3. Bei der Frage auf "Ja" klicken und die Datei "vbcode_lightbg.eft" im Ordner "ftemplates" auswählen (dadurch wird der Code im Forum gehighlighted)
4. Das Ergebnis dann hier reinkopieren
mfG Cherry
Könntest deinen Quelltext einfacher gestalten, wenn du die Initialisierung von Var001 am Anfang auf 0 belässt und dann die Abfragen einfach nur um eins senkst, also "Wenn Var001=0...5, dann...".
Die Ansonstenanweisungen sind auch unnötig, einfach nur die Abfragen hintereinander, schließlich ändert sich die Variable ja nicht plötzlich.
Edit:
Warum hast du am Anfang drei riesige Kommentare und im Text selbst nur ein paar mickrige?
Unter Umständen kann man Eventteile auch auf andere Eventseiten (bei Map-Events), bzw. auf Call-Events (bei Common-Events) verlagern. Das verbessert die Übersicht ungemein!
Edit2:
Wofür ist eigendlich die Var002? Die ergibt mMn gar keinen Sinn. Selbst wenn sie nötig wäre, könntest du sie auch einfach mit Var003 ersetzen, da die daraufhin eh wieder mit einem Wert versehen wird.
Edit3:
Hast du vielleicht bei der Tastenabfrage nicht eingestellt, dass das Skript warten soll? Das würde nämlich erklären, warum du am Anfang mehrmals eine Taste betätigen musst.
Ich find den Code seltsam. Versuch mal, Zeugs auszulagern mit Call Event, die Struktur einfach und übersichtlich zu gestalten und das dann in etwa so aufzubauen:
Variable Phase = 0
Variable Maximum = 3
Zyklus
__Enter Password, Wait until Key Hit, Variable Knopf
__Fork (Knopf = links)
____Phase = Phase - 1
____Fork (Phase < 0): Phase = Phase + Maximum
____Call Event (Grafiken um 1 nach links verschieben)
__Fork (Knopf = rechts)
____Phase = Phase + 1
____Fork (Phase > Maximum): Phase = Phase - Maximum
____Call Event (Grafiken um 1 nach rechts verschieben)
__Fork (Knopf = enter)
____Fork (Phase = 0): Call Event(Status)
____Fork (Phase = 1): Call Event(Weltkarte)
____Fork (Phase = 2): Call Event(Speichern)
____Fork (Phase = 3): Call Event(Menü verlassen)
____Zyklus brechen
(getreu dem Paradigma: trenne Hintergrundsoperationen von Ausgabe)
Danke Cherry!!!
So hier ist der Code.
Code:
Zitat:
Zitat von EasyEventExporter
Ich drehe fast durch, ich möchte dfas es endlich klappt:(
Ich habe natürlich bei der Tastenabfrage auf warten bis gedrückt. Aber nur nach Rechts, weil Links klappt es nicht.
Äh, das [code][/code] drumrum gehört weg.
Es war jetzt aber nicht notwendig, dass du den Code deswegen nochmal reinstellst - ich meinte eigentlich fürs nächste Mal.
Außerdem ist es sinnvoller, du kopierst das ganze Event und nicht nur den Eventcode, dann sehen wir nämlich auch Startbedingungen und ähnliches. Mag in diesem Fall zwar keine Rolle spielen, aber vielleicht später mal.
Warum frägst du immer nur einen einzigen Knopf mit wait until key pressed ab? Du bist dir doch hoffentlich im Klaren, dass das Skript an der Stelle still steht wenn man nicht GENAU diesen Knopf drückt, was in einem Menü kaum sinnvoll istZitat:
. <> Key Input Processing: Var. [2], Wait until key pressed, Keys: Right
. . <> Key Input Processing: Var. [3], Wait until key pressed, Keys: Decision
Ich blick da nicht durch *heul*
Ich habe keine Ahnung wie ich es bewerkstelligen soll das wenn Ich nach Links klicke das dann auch nach Links geht, also auf dem nächsten Bild, und bei rechts ebenso :(
Ich habe keine Ahnung, habe alles versucht:(
Dann bau dir dein Skript systematisch auf. Manche Skripte können ne ganze Woche dauern, bis sie so laufen, wie sie sollen.
Wenn du einige Sachen, die in dem Skript sinnlos sind, rausnimmst, würde sich das ganze bestimmt um die Hälfte reduzieren.
Warum hast du die Bilder am Anfang erst initialisiert, dann sichtbar gemacht um sie dann nochmal sinnloserweise erneut zu initialisieren?
Mit der Tastenabfrage, brauchst du doch bloß sagen, dass links, rechts, enter und abbruch (was hier nicht mit dabei ist) akzeptiert werden sollen. Danach musst du nurnoch die Abfrage, welche Taste gedrückt wurde starten und dann die gewünschte Aktion ausführen.
Wo bei den bedingungen muss Ich das den einstellen?Zitat:
Zitat von R.F.
Also:
- Du zeigst 5 Bilder an, "verschiebst" sie auf 0% Tranzparenz - warum zeigst du sie danach nochmal an? (wie R.F. schon genannt hat)
- Du setzt eine Variable auf 0 und addierst 1 - kannst auch gleich auf 1 setzen. (ebenfalls nur Schönheitsfehler)
Kritischer Teil:
- Warte bis rechte Pfeiltaste gedrückt wurde: bedenke dass der Code an dieser Stelle wartet bis eine der zugelassenen Tasten gedrückt wurde und keine anderen Aktionen zulässt! (Wie oben schon genannt)
In dem Fall wäre es ratsamer auch die linke Pfeiltaste, Enter (bestätigen) und ESC (abbrechen) zuzulassen... (um hinterher abzufragen welche Taste denn gedrückt wurde)
- nach Drücken der echten Pfeiltaste wird die Variable um 1 erhöht und die entspr. Aktion zugewiesen. Hier hat man Dank "Key Input Processing" wieder keine andere Wahl als Enter zu drücken um diese Aktion zu starten.
Die Aktion sollte nach Möglichkeit erst ausgewählt werden nachdem sich der Spieler via Enter auf eine festgelegt hat, da nach deiner Variante das anschließende wechseln zwischen den möglichen Aktionen etwas komplex werden würde...
Zuerstmal solltest du überlegen, wie dein Skript ablaufen soll. Ich würde das (mit deiner Variante) so machen:
1. Bilder initialisieren und deren Animation (sichtbar machen), Variable, z.B. Var001, für Position des Menüs=1 (btw. bei mir wärs 0)
2. In einer Schleife sollen alle folgenden Aktionen immewieder wiederholt werden (also Zyklus)
3. Was ist das Hauptziel? (Was soll passieren und wann ist das Skript am Ende, bzw. Abbruchbedingungen)
4. Tastenabfrage (rechts/links (suchs dir aus)-> Rechtsdreh, oben/unten-> nicht belegt, enter-> Bestätigung, abbruch-> was wohl? :D)-> macht 3 verschiedene Tastenabfragen, die mit einem Befehl abgehandelt werden können (in Var002 speichern)
5. Da Var002 bis zum nächsten Schleifendurchlauf gleich bleibt, ist es nichtmal nötig mit if/else zu arbeiten, sondern nur mit if (Vorteil bzgl. der Optik)
<>if Var002=6 (Abbruchtaste) break cycle
<>if Var002=5 (Enter) Fokussierte Aktion wählen (Var001 abfragen)
<>if Var002=1,2,3, oder 4 (hab nicht genau im Kopf, welches welche Richtung ist) Bilder bewegen, Var001 anpassen (erhöhen, oder wieder auf 1 setzen)
Wie bezieht den der maker die + und - Variablel?
Ich check das nicht-.-
Wenn du schon an solch simplen Befehlen scheiterst solltest du dir vielleicht nochmal das Ebook anschauen.
Evtl. kann dir auch dieses Video über Variablen helfen.
http://www.youtube.com/watch?v=wLIt2PYBHBY
Ähm, habe ich sowas nicht schon mal in VD2 gesehen?
Sich das ganze mal anzuschauen und in anderer Form "nachzuscripten", so würde ich das machen.
PS: Ich habe auch so was ähnliches geskriptet, allerdings ist es bei mir ein Common Event, dass mein Ansprechen des NPC-Events gecallt wird. (In VD2 ist es alles in den NPC Event drin - so ne Speicherverschwendung und Arbeit!).
Wenn das NPC-Menü geschlossen wird (Variabel NPC Aktion gesetzt), wird beim NPC-Event die Variabel NPC Aktion abgefragt (Fork Condition) und dann kommt z.B. eine Nachricht, was der NPC denkt, oder die Aktion "Verwandeln".
Ich hab es mao so gemacht wie Du R.F. mir erklärt hast, es klappt teilweise.
Wenn Ich habe keine möglichkeit mehr auf Enter zu drück damit es läuft, es geht automatisch. Wenn ich die und die variable habe. so hab ich mir das script nicht vorgestellt. man muss Enter drücken damit was ablaufen kann.
Hier ist nochmal der Code.
Zitat:
Zitat von EasyEventExporter
Tut mir Leid, aber ich verstehe gerade nicht, worauf du hinaus willst.
AM besten schreibst du das komplette Skript nochmal komplett neu und überlegst dir vorher, was du eigendlich genau machen willst. Außerdem solltest du dir, wenn das Skript schon nicht läuft, vielleicht erstmal ein einfacheres schreiben, mit dem du dich mit der Funktionsweise der Befehle auseinandersetzen kannst.
Sorry, aber mehr kann ich auch nicht für dich tun, ansonsten müsste ich es für dich selbst schreiben, was aber mMn nicht der Sinn des ganzen ist.
Oft hilft es den Code systematisch durchzugehen, um festzustellen an welchen Stellen etwas nicht stimmen kann:
Zur Zeit würde dein Code etwa das tun:
------------------------------------------------------
- Bilder 10-14 initialisieren
- setze Variable1 = 1
- Schleife beginnen:
-- Warte bis Taste gedrückt wurde (beispielsweise "rechts" drücken: Variable2 = 3)
-- Fork: Ist Variable2 = 6? (nein, 3)
-- Fork: Ist Variable2 = 5? (nein, 3)
-- Fork: Ist Variable2 = 2? (nein, 3)
-- Fork: Ist Variable2 = 3? (ja)
--- Text: "rechts?"
--- erhöhe Variable1 + 1 (= 2)
--- Fork Ende
-- Fork: Ist Variable1 = 1? (nein, 2)
-- Fork: Ist Variable2 = 2? (ja)
--- Comment: ****GEDANKEN****
--- Spiele Soundeffekt
--- verschiebe Bilder 11-14 nach 100% Transparenz (Bild 10 nach 0%)
--- entferne Bilder 11-14
--- zeige Bild 14: Aktion reden (nun aktive Bilder: 10, 14)
--- Text: "Test2"
--- Fork Ende
-- Fork: Ist Variable1 = 3? (nein, 2)
-- Fork: Ist Variable1 = 4? (nein, 2)
-- Fork: Ist Variable1 = 5? (nein, 2)
-- Fork: Ist Variable1 = 1? (nein, 2)
-- Fork: Ist Variable 1 = 0? (nein, 2)
- Beginne Schleife von vorne (Variable1 = 2, Bilder 11-13 sind entfernt)
...
-------------------------------------------------------------
Vielleicht hilft dir sowas ja bei der Lösungsfindung... :)
Ansonsten kann ich mich auch nur an R.F. halten...
mfG
ETC
Die Grundstruktur müsste so an sich korrekt sein.
Grundidee beim Debuggen: vereinfache alles.
Schau mal, woran es liegt, lass dir durch Show Message \V[nummer der vari] den aktuellen Inhalt der Variablen an zahlreichen Stellen ausgeben und kopier mal den Codeabschnitt mit den ganzen Move Picture in einzelne Events sodass du testen kannst, ob der astrein ist
Wozu sind eigentlich die Forks ganz unten gut?
Die sind verschachtelt und fragen Bedingungen ab, die nicht gleichzeitig erfüllt sein können (Variable [1] == 1 then ... Variable [1] == 2 then ...), sprich über die erste Ebene der Verschachtelung wirds nicht kommen, wolltest du die nicht hintereinander statt ineinander schalten?
Und nochmal: Lagere aus! Die ganzen Move Pictures haben in dem Event nix verloren, sie machen es nur unübersichtlich, pack sie in ein anderes Event das du an der Stelle mit Call Event aufrufst, dann geht der Debug auch leichter
Danke. das problem hat sich erledigt schon vor nen paar Tagen :)
Danke an alle die mir geholfen haben! :)