Inventar - unerklärliches Problem bei simpler Wertübergabe an globale Variable

GSlash · 18. April 2011, 18:15

hi,

ich bastle nun schon seit einigen Tagen an einem Drag and Drop Inventar für mein aktuelles Projekt.
Bisher bin ich auch gut vorangekommen, doch beim letzten Feature (welches eigentlich das einfachste sein sollte) komme ich nicht weiter.

Bei den Inventargegenständen, die auch ausgerüstet werden können, habe ich das Event "Right Mouse pressed" mit folgendem Code eingebaut:

GML-Quellcode

//Wenn Inventargegenstand ausgerüstet ist, soll er bei Rechtsklick in Inventar abgelegt werden (funktioniert)
if(x == view_xview+436 && y == view_yview+573){
for(i=0;i<global.inventory_space;i+=1){
if(global.inventory_item[i] == inv_empty){
global.inventory_item[i] = global.equipment_legs;
global.equipment_legs = inv_empty;
global.player_armor -= var_armor;
scr_inv_open();
exit;}}}
//Wenn kein Ausrüstungsgegenstan der gleichen Kategorie aktiv ist, bei Rechtsklick auswählen und an die
//entsprechende Position schieben (funktioniert)
if(global.equipment_legs == inv_empty){
global.equipment_legs = object_index;
global.player_armor += var_armor;
instance_create(view_xview+436,view_yview+573,object_index);
if(var_inventory_side == 1){
global.inventory_item[var_inventory_position] = inv_empty;}
if(var_inventory_side == 2){
global.inventory_transfer.var_item[var_inventory_position] = inv_empty;}
instance_create(x,y,inv_empty);
instance_destroy();}
//Wenn bereits Gegenstand vom gleichen Typ aktiv ist, Positionen tauschen und Inventar neu zeichnen (funktioniert nicht!)
else{
global.inventory_show = 0;
if(var_inventory_side == 1){
global.inventory_item[var_inventory_position] = global.equipment_legs;}
if(var_inventory_side == 2){
global.inventory_transfer.var_item[var_inventory_position] = global.equipment_legs;}
global.equipment_legs = object_index;
scr_inv_open();}

Alles anzeigen

Der Fehler hat anscheinend irgendetwas mit dem object_indext zu tun aber ich kommen einfach nicht dahinter inwiefern.
Zum Testen habe ich zwei Objekte erstellt inv_legs01 (id=41) & inv_legs02 (id=52). Die Objekte sind (bis auf die Sprites) identisch.
Wenn inv_legs01 ausgerüstet ist und man auf inv_legs02 rechtsklickt werden die Positionen wie gewünscht getauscht und das Inventar aktualisiert.
Umgekehrt funktioniert es aber nicht, die variable global.equipment_legs wird bei Rechtsklick auf inv_pants01 nicht geändert.

Testweise habe ich inv_legs01 kopiert (neue id = 56), wobei inv_legs01 die Position von inv_legs02 einnehmen konnte, aber umgekehrt funktionierte es nun nicht mehr.
Der object_index in der variable global.equipment_legs kann also immer nur erhöht werden.

Ich hoffe einer von euch kann mir hier weiterhelfen.

mfg
G

interceptor · 18. April 2011, 19:13

Auf Anhieb kann ich keinen Fehler erkennen.
Wäre sicher nützlich mehr bzw. den ganzen Quellcode dazuhaben, um den Fehler zu suchen.

GSlash · 18. April 2011, 19:52

ok,
habe damit es übersichtlicher ist das spiel soweit zusammengestaucht sodass fast nur noch das inventar drin ist. Der Fehler tritt nachwievor wie beschrieben auf.
Ich hoffe ihr könnt mit der Datei etwas anfangen.

P.S. Ich hatte ohnehin vor das File mit detailierten Kommentaren zu versehen und als Tutorial hochzuladen sobald es funktioniert und fertig gepolished ist. Daher würde es mich auch interessieren was ihr bisher davon haltet (mal abgesehen von dem Bug) und ob es als Tutorial interessant wäre.

interceptor · 18. April 2011, 22:37

Gerade beim Öffnen der Datei gestaunt:
Du machst es dir ja sehr kompliziert.
Wieso kopierst du Code 1:1 von einem in ein anderes Objekt?
Mit parents und Scripts kann man da noch einiges vereinfachen.
Auch die Fehlersuche. Bin noch nicht dahintergekommen...

GSlash · 18. April 2011, 23:12

Hab den gamemaker erst vor ca. 2-3 Wochen entdeckt. Gibt sicher noch Verbesserungspotential bei meiner Anwendung des Programms, aber daran arbeite ich bereits

Gibt im Code zwar noch einige Drag and Drop Relikte, aber dadurch dürfte der Bug auch nicht entstanden sein. Hab sicher schon um die 6-8h mit der Fehlersuche verbracht und inzwischen sind mir die Ideen ausgegangen was noch sein könnte.

Falls ich dir irgendwie bei der Fehlersuche helfen kann (z.B. aufzählen was ich inzwischen alles probiert und als Ursache ausgeschlossen habe) sag bitte Bescheid.

interceptor · 18. April 2011, 23:14

Ich hab es zwar noch nicht repariert, dein Problem scheint aber in etwa folgendes zu sein:

Bei jedem Mausklick geht der Game Maker alle Objekte durch und schaut, ob sie angeklickt wurden.
Dein Skript führt dazu, dass du mit einem Klick gleich zwei Objekte anklickst, weil der erste Klick, das zweite Objekt an die Stelle des Klicks bewegt.
Falls das nicht klar sein sollte, hier ein Beispiel:

Funktioniert: Der Spieler drückt die rechte Maustaste (auf Hose 2):
Das Spiel checkt Hose 1 mouse right pressed -> nix
Das Spiel checkt Hose 2 mouse right pressed -> Hose 1 mit 2 austauschen

Funktioniert nicht: Der Spieler drückt die rechte Maustaste (auf Hose 1)
Das Spiel checkt Hose 1 mouse right pressed -> Hose 1 mit 2 austauschen
Das Spiel checkt Hose 2 mouse right pressed -> Hose 1 mit 2 austauschen

Du tauscht die Hose also zweimal aus, dadurch scheint natürlich nichts zu passieren. Lösung: Mach um dein Skript eine Abfrage mit einer Variable (etwa: if (clickable == true)) und setz die bei einem gerade ausgetauschtem Objekt auf false. Per step event oder alarm kann man sie dann kurz darauf wieder auf true setzen.

GSlash · 18. April 2011, 23:47

Du hast Recht, das war der Fehler. Mit nem Alarm dazwischen funktionierts.

Hab gedacht da ich "Right Pressed" als Event und nicht "Right Button" genommen habe würde es nur das Event des ersten Items aktivieren. Ist mir garnicht in den Sinn gekommen dass das Event beim zweiten Item auch aktiviert werden könnte

Auf jeden Fall riesen Dank für die schnelle Hilfe