Das altbekannte Gib-Problem

[Endres]

Frage besonders an die Skriptkünstler (rulaman, Kruttan,...) der Community:

Ist es nicht Sinnvoll, den Programmierer entscheiden zu lassen, ob der Inventargegenstand bei Geben ins Inventar von dem jeweiligen anderen Spieler übertragen werden soll oder nicht?

Ich glaube wir hatten da schonmal so ein Problem. Und zwar scheinen mir die damit verbundenen Zeilen 1753-1771 nutzlos, da man die ja eh nicht gebrauchen kann, sobald man einen Charakter einbaut, dem nicht alles gegeben werden darf.

Die Zeilen im Überblick: (Mist, immernoch kein AGS-Sourcecode-Highlighting)
[ags]    else if ((GSagsusedmode == 4) && (GetLocationType (mouse.x, mouse.y) == 2) && isAction (eActGiveTo) )
    {
      gAktionText.TextColor = ActionLabelColorHighlighted;
         InventoryItem *invItem = player.ActiveInventory;
         
      if (GoToCharacter (charAt, 0, 1, 2))
      {
            if ( null != invItem )
               ItemGiven = invItem.ID;

        if (IsInteractionAvailable (mrx - GetViewportX (), mry - GetViewportY (), eModeUseinv) == 1)
        {
               charAt.RunInteraction(eModeUseinv);
        }
        GiveInv (ItemGiven, charAt);
        invItem = null;
      }
      SetAction (eActDefault);
    }[/ags]

Das Skript überprüft zwar, ob eine UseInv-Interaktion verfügbar ist und lässt diese auch laufen - Nichtsdestotrotz wird der Inventargegenstand zum Schluss trotzdem abgegeben. Wenn man das abgeben weglässt, könnte man gleich den ganzen else-if Block weglassen, da der folgende else-Block sowieso auf die Naheligende Aktion "UseInv" des Charakters verweist.

Deshalb plädiere ich dafür, dass diese Zeilen endgültig aus allen neuen Starterpacks verbannt werden, da sonst massiger Inventarschwund auftreten könnte.

Was meint ihr dazu?

[Der Kompaniechef]

nunja das problem ist wirklich bekannt und nervt.(hatte es ja selber)
neuanfänger stehen völlig auf dem schlauch und wissen nicht wie das problem beheben werden kann.

[Endres]

Mich wundert auch nur, dass es früher noch nicht passiert ist, obwohl es scheinbar ja auch im alten Starterpack drin ist.

[Bissiger Witzbold]

Der Bug tauchte früher nicht so häufig auf, weil er in den alten Starterpacks (2.62) beizeiten behoben wurde.
In den 2.72er Starterpacks war der Bug dann wieder weit verbreitet, vermutlich wurde da ein veraltetes Starterpack als Grundlage genommen.

[Endres]

Dann ist es doch wohl mal an der Zeit ihn endlich zu beheben? 

[rulaman]

Dann warte ich mit der Umarbeitung bis die Sache feststeht.

[Endres]

Lass dir Zeit, ich werde dein Vorhaben im Tutorial erwähnen (und wenn es vollbracht ist natürlich auch wieder entfernen )

[mukk]

-hat sich erledigt bzw. geht hier weiter: http://www.maniac-mansion-mania.de/forum/index.php?topic=333.msg40615#msg40615 -

[rulaman]

Habe mich gerade nochmal mit dem Gib-Problem auseinandergesetzt
und ein bischen damit rumgespielt.

Bin mittlerweile Endres' Auffassung, den Block zu entfernen.
Ich hatte ihn Testweise mal drinnengelassen und den Charakter hingehen lassen, aber das passt irgendwie nicht.
Das sollte im entsprechenden Charakter-Event haben.

Die anderen gemeldeten Fehler habe ich nachgebessert und bereits hochgeladen (Ebenso Police Molest; wird noch veröffentlicht, wenn das Gib-Problem übernommen wurde).

Grüße
Rulaman

[mukk]

Ich hab wahrscheinlich n Kollateralschaden. Ich habe den oben genannten Block ausgeschaltet und kriege jetzt in unregelmäßigen Abständen einen nicht reproduzierbaren crash im Script
[ags]    
else if (button == LEFTINV)
{
...

else
    {
      used_action = global_action;
      if (GetCursorMode () == 2)
      {
        if (isAction (A_USE) && IsInventoryInteractionAvailable (GSlocid, MODE_USE) == 1)
        {
          SetLabelColor (ACTION, 0, ActionLabelColorHighlighted);
          RunInventoryInteraction (GSlocid, MODE_USE);
          SetAction (A_DEFAULT);
        }
        else SetActiveInventory (GSlocid);   //// <----In dieser Zeile meldet AGS den Fehler
      }
      else
      {
        GSagsusedmode = GetCursorMode ();
        SetLabelColor (ACTION, 0, ActionLabelColorHighlighted);
        RunInventoryInteraction (GSlocid, GetCursorMode ());
        SetAction (A_DEFAULT);
      }
    }
...
}
[/ags]

Siehe Script für die Zeile, in der AGS den Fehler meldet. Passieren tut es anscheinend zufällig, beim Anschauen von Objekten sowie beim Benutzen von Inventargegenständen.

An error has occurred. Please contact the game author for support, as this is likely to be a scripting error and not a bug in AGS.
(ACI version 3.12.1074)

in "GlobalScript.asc", line 1470

Error: SetActiveInventory: character doesn't have any of that inventory

Der zitierte Block steht im selben Abschnitt wie der von endres genannte in der
[ags]function on_mouse_click (int button)[/ags]

Hier nochmal was ich per /*...*/ abgeschaltet habe:

[ags]else if ((GSagsusedmode == 4) && (GetLocationType (mouse.x, mouse.y) == 2) && isAction (A_GIVE_TO) && (GetCharacterAt (mouse.x, mouse.y) < 7))
    {
      SetLabelColor (ACTION, 0, ActionLabelColorHighlighted);
      if (GoToCharacter (GSlocid, 0, 1, 2))
      {
        ItemGiven = character [GetPlayerCharacter ()].activeinv;
        if (IsInteractionAvailable (mrx - GetViewportX (), mry - GetViewportY (), MODE_USEINV) == 1)
        {
          RunCharacterInteraction (GSlocid, MODE_USEINV);
        }        
        GiveInv (ItemGiven, GSlocid);
        character [GetPlayerCharacter ()].activeinv = -1;      
      }
      SetAction (A_DEFAULT);
    }
[/ags]

...daher geh ich ma davon aus, dass das irgendwie zusammen hängt, aber wie?

Bitte um Hilfe!


Edit: Wenn ich das richtig verstehe kommt der Fehler, weil das Script versucht, ein Objekt/Hotspot/Charakter, wasauchimmer, als Inventargegenstand zu benutzen....kann das sein?

Also könnte ich das ganze stoppen, wenn ich die besagte Zeile so bedinge:

else if (GetInvAt (mouse.x, mouse.y) >= 0) SetActiveInventory (GSlocid);

Problem is halt, der Crash ist nicht reproduzierbar...d.h. ich brauche hier nen Fachman der mir sagt ob das wirklich klappt