/pay Problem

Was genau ist denn dein Problem? Also was funktioniert nicht?

Zitat von Panzerkampfwagen Tiger

ocmd:pay(playerid,params[])
{
new pID, pay, string[128], string2[128];
if(sscanf(params,"ui",pID,pay)) return SendClientMessage(playerid,Grey,"Nutze: /pay [Spieler/ID][Betrag]");
if(pay<1||pay>50000) return SendClientMessage(playerid,Grey,"Du kannst nur bis 1 Million $ zahlen!");
{
if(!IsPlayerConnected(pID)) return SendClientMessage(playerid,Grey,"Dieser Spieler ist nicht Online.");
{
if(GetPlayerMoney(playerid)<pay) return="" sendclientmessage(playerid,grey,"du="" hast="" nicht="" genügend="" geld="" vorhanden.");
{
if(pID==playerid) return SendClientMessage(playerid,Grey,"Du kannst dir nicht selber Geld zahlen.");
{
GivePlayerMoney(pID,pay);
GivePlayerMoney(playerid,-pay);
format(string,sizeof(string),"Spieler %s hat dir %i$ bezahlt.",pName(playerid),pay);
SendClientMessage(pID,LightYellow,string);
format(string2,sizeof(string2),"Du hast dem Spieler %s %i$ gezahlt.",pName(pID),pay);
SendClientMessage(playerid,LightYellow,string2);
}
}
}
}
return 1;
}

Wo ist der Fehler? Ich kann trotzdem zahlen wenn ich im Minus bin und an mich selbst.

LG

Alles anzeigen

Du musst auch überprüfen ob du genug Geld hast,
deine Variable die du dort erstellst, wurde ja zu keiner 'Funktion' zugewiesen.

Klar du musst abfrage ob er Genug geld hat..
Wie die den code geschrieben hast einfügen

if(gID < 1 || gID > 99999)
{
SendClientMessage(playerid, COLOR_GRAD1, "Der Betrag muss zwischen $1 und $99999 sein!");
return 1;
}

Kleines Beispiel.. musste halt anpassen

ocmd:pay(playerid,params[])
{
new pID, pay, string[128], string2[128];
if(sscanf(params,"ui",pID,pay)) return SendClientMessage(playerid,Grey,"Nutze: /pay [Spieler/ID][Betrag]");
if(pay<1||pay>50000) return SendClientMessage(playerid,Grey,"Du kannst nur bis 500 Tausend $ zahlen!");
if(!IsPlayerConnected(pID)) return SendClientMessage(playerid,Grey,"Dieser Spieler ist nicht Online.");
if(GetPlayerMoney(playerid)<pay) return SendClientMessage(playerid,grey,"du="" hast="" nicht="" genügend="" geld="" vorhanden.");
if(pID==playerid) return SendClientMessage(playerid,Grey,"Du kannst dir nicht selber Geld zahlen.");
GivePlayerMoney(pID,pay);
GivePlayerMoney(playerid,-pay);
format(string,sizeof(string),"Spieler %s hat dir %i$ bezahlt.",pName(playerid),pay);
SendClientMessage(pID,LightYellow,string);
format(string2,sizeof(string2),"Du hast dem Spieler %s %i$ gezahlt.",pName(pID),pay);
SendClientMessage(playerid,LightYellow,string2);
return 1;
}

Du fragst dauernd ab, wenn er zu wenig Geld hat, bekommt er den Fehler UND der Befehl geht weiter.
Hier hast du die richtige Variante

Zitat von Panzerkampfwagen Tiger

Pinguin Dev, ich sehe kein unterschied, außer das die Klammern weg sind.
Und was ist mit der Überprüfung dass man nicht an sich selber zahlen kann?

Les dir das mal durch, dann findest du deinen Fehler von ganz allein.

Zitat von Panzerkampfwagen Tiger

Es lag am return, danke Leute.
Kann geclosed werden.

LG

Moment, das will ich jetzt genauer wissen.
Für die Nutzer, die das Problem nicht beheben konnten.
Es lag doch wohl kaum am return. Wenn ja, an welchem?

Zitat von Pinguin Dev.

Moment, das will ich jetzt genauer wissen.
Für die Nutzer, die das Problem nicht beheben konnten.
Es lag doch wohl kaum am return. Wenn ja, an welchem?

Jap, dieser Meinung bin ich auch.
Erläutere uns doch bitte, was genau du abgeändert hast
und welches Return du meinst.

Mit freundlichen Grüßen
#else von PC-Rat.