Geld Begrenzung

Joe_Westbrook · 30. Juli 2009

Momentan habe ich zwei Menüs in meinem Script. Einmal einen um Autos zu kaufen und einen um Waffen zu kaufen.
Das Problem ist das man sich ins unendliche verschulden kann.

Beispiel: Max connectet auf den Server und tippt /buygun ein. Er wählt die M4 aus obwohl er nur 0$ hat. Die M4 kostet 5000$.
Nun ist er für 5000$ im minus, was aber für ihn total egal ist da er sich unendlich weit verschulden kann.

Gibt es da schon etwas vorgefertigtes? Ich habe nichts gefunden.

Docean · 30. Juli 2009

Mach es mit einer ganz einfachen Abfrage

if(GetPlayerMoney(playerid) > Kosten)

So müsste es doch eig. funktionieren

Halo · 30. Juli 2009

case 0:
{
{
if(GetPlayerMoney(playerid) <5000)
{
return 1;
}
GivePlayerMoney(playerid, -5000);
GivePlayerWeapon(playerid,31,500);
}
}

OnlinePsycho · 30. Juli 2009

<: Unter
>: Über
<= Untergleich
>=: Übergleich
==: Gleich
!=: Entspricht nicht

Joe_Westbrook · 30. Juli 2009

So ungefähr?

case 0:
{
GivePlayerWeapon(playerid, 22, 50); //USP
GivePlayerMoney(playerid, -500);
HideMenuForPlayer(pistolmenu, playerid);
TogglePlayerControllable(playerid, 1);
if(GetPlayerMoney(playerid) >0);
}

Da bringt er mir einen Error in genau die Zeile wo das "if[GetPlayerMoney(playerid) >0);" steht:

Code

: error 036: empty statement

BlackFoX · 30. Juli 2009

Wieso fragst du was ab und danach passiert nix, die Aktion muss du innerhalb der abfrage machen wie hier

if(<deine Bedingung>)
{
// Do Something

}

Joe_Westbrook · 30. Juli 2009

Etwa so?

case 0:
{
if(GetPlayerMoney(playerid) >0);
GivePlayerWeapon(playerid, 22, 50); //USP
GivePlayerMoney(playerid, -500);
HideMenuForPlayer(pistolmenu, playerid);
TogglePlayerControllable(playerid, 1);
}

Entschuldige aber ich bin ein totaler Scriptnoob ..

BlackFoX · 30. Juli 2009

nein , ein Statement muss nicht beendet werden mit Similikons, entweder es ist eine einzelaktion wie hier

if(GetPlayerMoney(playerid) >0)SendClientMessage(playerid,farbe,"Text");

oder es sind mehrere Aktionen wie hier

if(GetPlayerMoney(playerid) >0)
{
GivePlayerWeapon(playerid, 22, 50); //USP
GivePlayerMoney(playerid, -500);
HideMenuForPlayer(pistolmenu, playerid);
TogglePlayerControllable(playerid, 1);
}

Du musst deinen Code mit Brackets einbinden , Offen { und zu },
immer beim Prinzip bleiben { Öffnen und schließen }
wie bei den Funktionen Öffnen ( und schließen )

Ich würde dir empfehlen Pawn zu lernen , nicht nur von Seiten wie Sa-mp her sondern auch die Basis

Micktor · 30. Juli 2009

kann man das eigentlich i-wie mahcne das man sich nur bis 10000$ verschulden kann ?

Joe_Westbrook · 30. Juli 2009

Es klappt nun, jedenfalls mehr oder weniger:

Spieler X hat 500$.
Spieler X will sich eine Waffe für 1000$ kaufen.
Spieler X erhält die Waffe und hat -500$.
Spieler X möchte nun nochmals eine Waffe kaufen.
Spieler X kann sich die Waffe nicht kaufen, das Menü schließt sich und der Spieler ist gefreezt.

BlackFoX · 30. Juli 2009

Die Abfrage muss so heißen

if(GetPlayerMoney(playerid) >499)

KeksOfDeath · 30. Juli 2009

Oder anders:
if(GetPlayerMoney(playerid)>=500) {
So kann man nicht so oft vergessen, 1 abzuziehen^^

Joe_Westbrook · 30. Juli 2009

Danke danke danke!
Ich glaube ich habs kapiert

//EDIT:

Spieler ist, wenn er nicht genug Geld hat, gefreezt ..

xMine · 30. Juli 2009

Dafür brauchst du TogglePlayerControllable
Also zB:

//...
TogglePlayerControllable (playerid, 1);
//...

Joe_Westbrook · 31. Juli 2009

Mh ..
So siehts momentan aus:

new Menu:current; //Pistolen/Sonstiges menu
current = GetPlayerMenu(playerid);
if(current == pistolmenu)
{
switch(row)
{
case 0:
if(GetPlayerMoney(playerid) >499)
{
TogglePlayerControllable (playerid, 1);
GivePlayerWeapon(playerid, 22, 50); //USP
GivePlayerMoney(playerid, -500);
HideMenuForPlayer(pistolmenu, playerid);
TogglePlayerControllable(playerid, 1);
}

Wie kann ich nun einbauen das wenn der Spieler unter 500$ hat, er eine Nachricht bekommt das er zuwenig Geld hätte aber das Menü geöffnet bleibt?
Mit "else if" ?

Also:

Spieler X kauft sich eine USP.
Spieler X möchte sich noch mehr Munition für die USP kaufen und wiederholt den Vorgang.
Spieler X hat nun aber zuwenig Geld und bekommt eine Nachricht.

Momentan geht das Menü zu und der Spieler ist gefreezt sobald er zuwenig Geld für die Waffe hat.

Etwa irgendwie so? (Bitte haut mich nicht wenns totaler Schrott ist^^) :

new Menu:current; //Pistolen/Sonstiges menu
current = GetPlayerMenu(playerid);
if(current == pistolmenu)
{
switch(row)
{
case 0:
if(GetPlayerMoney(playerid) >499)
{
TogglePlayerControllable (playerid, 1);
GivePlayerWeapon(playerid, 22, 50); //USP
GivePlayerMoney(playerid, -500);
HideMenuForPlayer(pistolmenu, playerid);
TogglePlayerControllable(playerid, 1);
{
else if(GetPlayerMoney(playerid) <501)
SendClientMessage(playerid, COLOR_RED, "Du hast nicht genuegend Geld!");
}}}

RFT · 31. Juli 2009

mach es so

new Menu:current; //Pistolen/Sonstiges menu
current = GetPlayerMenu(playerid);
if(current == pistolmenu)
{
switch(row)
{
case 0:
if(4999 <= GetPlayerMoney(playerid))
{
TogglePlayerControllable (playerid, 1);
GivePlayerWeapon(playerid, 22, 50); //USP
GivePlayerMoney(playerid, -500);
HideMenuForPlayer(pistolmenu, playerid);
TogglePlayerControllable(playerid, 1);
}
else
{
SendClientMessage(playerid,COLOR_RED,"Du hast nicht genug Geld");
}
}}

Joe_Westbrook · 31. Juli 2009

4 Errors:

Code

: error 014: invalid statement; not in switch
: error 001: expected token: ";", but found ":"
: error 029: invalid expression, assumed zero
: fatal error 107: too many error messages on one line

Alle Errors kommen aus der selben Zeile, in welcher "case 1" beginnt:

new Menu:current; //Pistolen/Sonstiges menu
current = GetPlayerMenu(playerid);
if(current == pistolmenu)
{
switch(row)
{
case 0:
if(4999 <= GetPlayerMoney(playerid))
{
TogglePlayerControllable (playerid, 1);
GivePlayerWeapon(playerid, 22, 50); //USP
GivePlayerMoney(playerid, -500);
HideMenuForPlayer(pistolmenu, playerid);
TogglePlayerControllable(playerid, 1);
}
else
{
SendClientMessage(playerid,COLOR_RED,"Du hast nicht genug Geld");
}
}}
case 1: // <---------------------------------------------------------------------- Die Errorzeile ;)
if(GetPlayerMoney(playerid) >649)
{
GivePlayerWeapon(playerid, 24, 50); //Desert Eagle
GivePlayerMoney(playerid, -650);
HideMenuForPlayer(pistolmenu, playerid);
TogglePlayerControllable(playerid, 1);
}

RFT · 31. Juli 2009

xD die letzten klammern müssen hiter dem Case

new Menu:current; //Pistolen/Sonstiges menu
current = GetPlayerMenu(playerid);
if(current == pistolmenu)
{
switch(row)
{
case 0:
if(499 <= GetPlayerMoney(playerid))
{
TogglePlayerControllable (playerid, 1);
GivePlayerWeapon(playerid, 22, 50); //USP
GivePlayerMoney(playerid, -500);
HideMenuForPlayer(pistolmenu, playerid);
TogglePlayerControllable(playerid, 1);
}
else
{
SendClientMessage(playerid,COLOR_RED,"Du hast nicht genug Geld");
}
case 1: // <---------------------------------------------------------------------- Die Errorzeile ;)
if(649 <= GetPlayerMoney(playerid))
{
GivePlayerWeapon(playerid, 24, 50); //Desert Eagle
GivePlayerMoney(playerid, -650);
HideMenuForPlayer(pistolmenu, playerid);
TogglePlayerControllable(playerid, 1);
}
else
{
SendClientMessage(playerid,COLOR_RED,"Du hast nicht genug Geld");
}
}}

Joe_Westbrook · 31. Juli 2009

Ah xD
Danke ..
Nu geht bei zuwenig Geld das Menü zu, da steht das er zuwenig Geld hat aber er ist gefreezt

Ist es richtig wenn ich hier sowas einbaue?

new Menu:current; //Pistolen/Sonstiges menu
current = GetPlayerMenu(playerid);
if(current == pistolmenu)
{
switch(row)
{
case 0:
if(499 <= GetPlayerMoney(playerid))
{
TogglePlayerControllable (playerid, 1);
GivePlayerWeapon(playerid, 22, 50); //USP
GivePlayerMoney(playerid, -500);
HideMenuForPlayer(pistolmenu, playerid);
TogglePlayerControllable(playerid, 1);
}
else
{
SendClientMessage(playerid,COLOR_RED,"Du hast nicht genug Geld");
TogglePlayerControllable(playerid, 1); // <-------------------------------------------------------------- Das da .. ;)
}

//EDIT:

Klappt! DANNKKEEE!! =)
Kommst auch in die Credits ..