Neue epi

[Der Kompaniechef]

Ui toll, ein Verkäufer. Davon haben wir ja erst so wenig.

ehh tribble ich hab damals mich gefreut das ich geschaft hab meinen ersten char zu machen
deswegen will ich das er verwendung findet

[Bòógieman]

Ui toll, ein Verkäufer. Davon haben wir ja erst so wenig.

ehh tribble ich hab damals mich gefreut das ich geschaft hab meinen ersten char zu machen
deswegen will ich das er verwendung findet

kann ich verstehen
aber ehrlich gesagt sieht diese syds gesicht und daves frisur mischung nicht gerade originell aus
und mit gewalt einen neuen char in eine epi einzubauen, nur damit ein neuer char darin vorkommt finde ich auch nicht so toll 

[Der Kompaniechef]

ok verstehe ich
hab anderes problem vielleicht kann wer helfen ??
also das problem is obwohl ich eingestellt habdas man zB ein auto mit dem schlüssel anmacht
funkt das mit jeden inventory
dieses problem is bei mir überall zu finden

[NsMn]

Wie wär's mal mit einem Script, wenn Grammatik schon nicht geht? Meine Güte, langsam versteh' ich Khris 

[Der Kompaniechef]

Wie wär's mal mit einem Script, wenn Grammatik schon nicht geht? Meine Güte, langsam versteh' ich Khris 

hallo wenn ich schnell schreibe is mir das shit egal wie die grammatik und so ist haubtsache
man erkennt was da steht

[Siel]

haubtsache man erkennt was da steht

Das Problem ist aber, dass gerade das nur noch schwer möglich ist.

Logisch, dein Problem ist uns ja auch klar, aber wie sollen wir wissen wo der Fehler liegt, wenn wir nicht wissen, wie das Script/der Code/nenn es wie du willst aussieht. Oder die Fehlermeldung. Aber ohne irgendetwas in dieser Art wird sich wenig machen lassen fürchte ich.
Das ist so wie wenn du jemanden einen Aufsatz auf Rechtschreibfehler prüfen lassen willst, ohne den Aufsatz herzuzeigen.

Davon abgesehen: Gehört sowas nicht viel eher in den Technik-Bereich?

[Cone Arex]

In einem Forum wird durch Schrift kommuniziert, daher wäre ein gewisser Grad an Richtigkeit schon angebracht. Es steht auch so in den Forenregeln. Niemand verlangt von dir, das du den kompletten Duden im Kopf hast. Aber wenn man dann einen Post komplett ohne Großschreibung und Satzzeichen hinlegt, den man dreimal lesen muss um ihn zu verstehen...

Und zum Thema: Bei sowas sollte man immer das Script mitonline stellen. Sonst ist es schwer für uns herraus zu finden, wo dein problem ist. Hast du vllt bei der Inventory-Abfrage die geschweifte Klammer ({) vergessen?

[Der Kompaniechef]

function Haengekasten_UseInv()
{
  if (MovePlayer (139,129 ))
if (character [GetPlayerCharacter ()].activeinv == 17)
FaceDirection(GetPlayerCharacter(), DIR_UP);
{
Wait (5);
PlaySound (100);
AddInventory(15);
ObjectOn(2);
ObjectOff(1);
DisplaySpeech(GetPlayerCharacter(),".");
DisplaySpeech(GetPlayerCharacter(),".");
}
}
das zB ist der code der funkt nicht

[Tribble]

Überleg dir mal was die zweite offene Klammer bewirken soll, und an welche Stelle sie dann wohl gehört (Fehler springt bei einmal kurz draufgucken ins Auge)

[Cone Arex]

*schluck*
Ich möchte dich ja in keinster weise angreifen, aber bei dem Code wirds einem echt schlecht. 

Grundsätzlich mal eines: Es wird immer nur die Funktion "Any click on..." verwendet. KEINE ANDEREN!
Die Funktion UseInv, die du verwendet hast, ist für MMM überflüssig.

Und zu den anderen Fehlern:
Bei MovePlayer hast du die geschweiften Klammern ({ und }) vergessen.
Du hast ein FaceDirection zwischen der Item-Afrage und der Klammer gepackt. Zwischen einer if-Abfrage und der geschweiften Klammer darf aber kein Befehl stehen!

Ich würde den Code so schreiben:

function Haengekasten_AnyClick()
{
  if (any_click_move(139,129,DIR_UP))
  {
    if (UsedAction(A_USE_INV))
    {
      Wait(5);
      if (player.ActiveInventory==inventory[17])
      {
       PlaySound(100);
       player.Addinventory(15);
       object[2].Visible=true;
       object[3].Visible=false;
       player.Say(".");
       player.Say(".");
      }
    }
    else Unhandled();
  }
}

Ich hab den Code ausm Kopf geschrieben, daher folgt keine Garantie auf Richtigkeit!

[rulaman]

Nur so nebenbei als Bemerkung.
Ich würde die Objektnamen verwenden, das erhöht die Lesbarkeit.

z.B.

Code: [Auswählen]

if (player.ActiveInventory==iSchluessel)


Code: [Auswählen]

oTanne.Visible=true;

Je nachdem welche Namen das Item oder die Objekte wirklich haben.

Und das AddInventory-Kommando will keine ID, sondern das Item selbst

Code: [Auswählen]

player.AddInventory(iKey);


[Cone Arex]

Ich selbst verwende natürlich auch Namen, aber da JPJF keine Namen im Script hatte, habe ich auch keine eingebaut.

[rulaman]

Hab's mir fast gedacht, wollte es nur so am Rande erwähnen. 

[Bòógieman]

*schluck*
Ich möchte dich ja in keinster weise angreifen, aber bei dem Code wirds einem echt schlecht. 

Ich möchte dich ja in keinster weise angreifen, aber bei dem Post wirds einem echt schlecht.   

spar dir bitte solche kommentare
JPJF hat um hilfe gebeten und nicht um kritik an seinem programmierstil, der so nebenbei fast 1:1 der selbe stil wie in den alten starterpacks ist
ich vermute mal, dass er das so wie ich macht und per copy&paste bestehenden code des starterpacks in neue funktionen kopiert und dann anpasst 
ER muss damit zurecht kommen, und dabei ist es herzlich egal ob da zb player oder character[GetPlayerCharacter()] steht
ob das nun schön aussieht oder nicht, funktionieren tut es so oder so
du darfst ihn natürlich darauf hinweisen dass das auch einfacher gehen würde, aber halt ohne solche kommentare
(und jetzt bitte nicht wieder die alte "neuer stil vs. alter stil" diskussion, die hat in diesem thread nix verloren)

Grundsätzlich mal eines: Es wird immer nur die Funktion "Any click on..." verwendet. KEINE ANDEREN!
Die Funktion UseInv, die du verwendet hast, ist für MMM überflüssig.

stimmt nicht ganz, bei charakteren muss man das leider seit AGS3 genau so machen 
bei hotspots und objekten empfiehlt es sich allerdings wirklich alles in der any_click auszuwerten

Und zu den anderen Fehlern:
Bei MovePlayer hast du die geschweiften Klammern ({ und }) vergessen.

das ist kein fehler
eine if die nur einen befehl enthält kann auch ohne die klammern geschrieben werden 

[1mal_volltanken]

Grundsätzlich nutze ich neben "Any click on..." auch "Use Inventory on..." eben wegen der Übersichtlichkeit. Am Anfang hatte ich es auch stets in "Any click on..." einprogrammiert und musste danach oft minuten- und stundenlang herumdoktorn, bis ich endlich meine {}-Klammern alle in der richtigen Position waren, alle notwendigen vorhanden waren, keine fehlten und auch keine über waren. Das war immer eine ätzende Arbeit.

So muss man natürlich in "Any click on..." noch folgendes Script einsetzen:

Code: [Auswählen]

if (UsedAction(A_USE_INV)) {

}
Dann kann man bei "Use Inventory on..." wirklich übersichtlich sämtliche Interaktionen mit Gegenständen durchscripten und/oder das Unhandled einsetzen.


BG
der Volltanker