Kleines Error Problem

Luigi Baker · 17. Januar 2012, 17:32

Ich bin grad dabei meine Idee (Bankschließfach) mitsamt einem gesammten Banksystem Just 4 Fun (bzw. zur "Übung") zu scripten.
Man gibt in einem Input Dialog die Zahl ein die man in sein Konto einzahlen soll und dann soll überprüft werden ob man genug Geld auf der Hand hat. Habs in nem Public gemacht.

[pwn]public Einzahlen(playerid, Geld, inputtext[])
{
if(Geld < inputtext[])
{
return 1;
}
return 1;
}[/pwn]

Kriege aber immer folgende Fehlermeldung:

D:\Spiele\GTA San Andreas\gamemodes\Bankscript.pwn(52) : error 029: invalid expression, assumed zero

Was ist daran falsch?
Danke schonmal im vorraus

Accipter · 17. Januar 2012, 18:14

[pwn]public Einzahlen(playerid, Geld, inputtext[])
{
if(Geld < inputtext)
{
return 1;
}
return 1;
}[/pwn]

Probiers mal so hier

Luigi Baker · 17. Januar 2012, 18:44

Dann kommt:
D:\Spiele\GTA San Andreas\gamemodes\Bankscript.pwn(52) : error 033: array must be indexed (variable "inputtext")

Accipter · 17. Januar 2012, 18:53

[pwn]public Einzahlen(playerid, Geld, inputtext[])
{
if(Geld < strval(inputtext))
{
return 1;
}
return 1;
}[/pwn]

Jetzt aber :>

Luigi Baker · 17. Januar 2012, 19:44

Jo geht danke man :D
Aber ne Frage, was bewirkt das strval?

Accipter · 17. Januar 2012, 19:53

Wandelt den eingegeben String in ein Integer um, leider ist es mir nicht vorher eingefallen.
Wenn du Strings vergleichen willst musst du strcmp nehmen ;).

Luigi Baker · 17. Januar 2012, 20:49

Aber noch ne Frage:
Welchen String wandelt das denn um?
Schau mal:
[pwn]public Einzahlen(playerid, Geld, inputtext[])
{
if(Geld < strval(inputtext))
{
SendClientMessage(playerid, ROT, "So viel Geld hast du nicht!");
return 1;
}
if(Geld > strval(inputtext))
{
GivePlayerMoney(playerid, strval(inputtext));
SendClientMessage(playerid, GELB, "Du hast das Geld erfolgreich eingezahlt!");
return 1;
}
return 1;
}[/pwn]

Es passiert aber ingame nichts, weder eine Nachricht "Du hast kein Geld" oder "Das Geld wurde eingezahlt" und mir wird Geld abgezogen, woran liegt das?
Sorry wenn ich stör :D

}

Vincent Petritz · 17. Januar 2012, 20:50

Wie ist "Geld" definiert?

Accipter · 17. Januar 2012, 20:53

strval wandelt das um was in den Klammern steht (nennt man Parameter :))
[pwn]
GivePlayerMoney(playerid, GetPlayerMoney(playerid) - strval(inputtext));
[/pwn]

Wraith · 17. Januar 2012, 22:01

Dein Beispiel ist aber schwachsinnig, dass weißt du hoffentlich?

Luigi Baker · 18. Januar 2012, 13:07

Ah danke :D
Ich dachte wenn man "Geld" z.B. in die Parameter eingibt ist es definiert -_-..
Wo muss ich das denn definieren?
(Wäre doch:
[pwn]new Geld;
Geld == GetPlayerMoney(playerid;[/pwn]
oder? :D)

Wraith · 18. Januar 2012, 13:13

Meine Aussage war an Accipter gerichtet :).
strval formatiert einen string in einen Integer.

Dein Callback (an Luigi) ist allgemein etwas komisch..
Du hast insgesamt 3 Parameter, 2 Integer einen string - wofür der string?
Würde ich an meinem Rechner sitzen würde ich das etwas auseinander nehmen, aber ich fass es mal kurz:
Die eckigen Klammern bei inputtext entfernen bzw. direkt inputtext entfernen & du hast den Operator '=' vergessen,sprich
wenn du genauso viel Geld hast wie er abziehen soll passiert nüx.

/E: Ab zweiten Satz hinzugefügt

Luigi Baker · 18. Januar 2012, 14:50

Wo habe ich einen String? xD
Wenn ich nur die [] (eckigen Klammern) bei inputtext entferne kriege ich 3 Fehlermeldungen:

D:\Spiele\GTA San Andreas\gamemodes\Bankscript.pwn(87) : error 035: argument type mismatch (argument 1)
D:\Spiele\GTA San Andreas\gamemodes\Bankscript.pwn(92) : error 035: argument type mismatch (argument 1)
D:\Spiele\GTA San Andreas\gamemodes\Bankscript.pwn(94) : error 035: argument type mismatch (argument 1)

(Die Zeilen sind die, wo ich inputtext verwendet habe).
Wenn ich es komplett entferne, wie solle ich dann den Text den er dort eingegeben hat abfragen?
Sorry wenn ich mich etwas dumm tue, ist wiegesagt mein erstes mal das ich etwas scripte was sich von einem DM Script entfernt :D

Wraith · 18. Januar 2012, 15:24

Natürlich ist die Verwendung von strval dann auch nicht mehr nötig.
Dann mach's so:
[pwn]public Einzahlen(playerid, Geld)
{
if(GetPlayerMoney(playerid) < Geld) return SendClientMessage(playerid, ROT, "So viel Geld hast du nicht!");
else
{
GivePlayerMoney(playerid, -Geld);
SendClientMessage(playerid, GELB, "Du hast das Geld erfolgreich eingezahlt!");
}
return 1;
}[/pwn]

/E: Pwn-tag hinzugefügt (eventuelle Einrückfehler durch den Editor)

Accipter · 18. Januar 2012, 15:30

Wraith schrieb:

Dein Beispiel ist aber schwachsinnig, dass weißt du hoffentlich?

Kein Beispiel, eine Funktion.

Wraith · 18. Januar 2012, 16:08

Accipter schrieb:

Wraith schrieb:

Dein Beispiel ist aber schwachsinnig, dass weißt du hoffentlich?

Kein Beispiel, eine Funktion.

Sehe ich aber anders:

Accipter schrieb:

strval wandelt das um was in den Klammern steht (nennt man Parameter :))
[pwn]
GivePlayerMoney(playerid, GetPlayerMoney(playerid) - strval(inputtext));
[/pwn]

Totaler Schwachsinn es so zu nutzen, du gibst im folgenden Betrag:
Was er derzeit auf der Hand hat - den Inhalt von inputtext.
Wenn ich nun 2$ in der Hand habe, ich einen 1$ abziehen lasse, kriege ich 1$ gutgeschrieben.
Das würde bestimmt die meisten User glücklich machen ..
(GivePlayerMoney addiert das bereits vorhandene Geld mit dem zweiten Parameter von GivePlayerMoney)

Accipter · 18. Januar 2012, 16:10

Wraith schrieb:

Accipter schrieb:

Wraith schrieb:

Dein Beispiel ist aber schwachsinnig, dass weißt du hoffentlich?

Kein Beispiel, eine Funktion.

Sehe ich aber anders:

Accipter schrieb:

strval wandelt das um was in den Klammern steht (nennt man Parameter :))
[pwn]
GivePlayerMoney(playerid, GetPlayerMoney(playerid) - strval(inputtext));
[/pwn]

Totaler Schwachsinn es so zu nutzen, du gibst im folgenden Betrag:
Was er derzeit auf der Hand hat - den Inhalt von inputtext.
Wenn ich nun 2$ in der Hand habe, ich einen 1$ abziehen lasse, kriege ich 1$ gutgeschrieben.
Das würde bestimmt die meisten User glücklich machen ..
(GivePlayerMoney addiert das bereits vorhandene Geld mit dem zweiten Parameter von GivePlayerMoney)

Du siehst meinen Code als Beispiel an *facepalm*
Nadenn da spricht der Meister, ich halte mich raus.

Luigi Baker · 18. Januar 2012, 16:24

-> Danke für eure Hilfe. Funkt jetzt.

Accipter · 18. Januar 2012, 16:31

Ich schließe hier mal.

Accipter

Teilen