Warum funktioniert das nicht?

Guten Tag,
Es wird einfach nicht abgefragt ob ich weniger als 80 HP habe, warum?

if(listitem == 0)
{
new Float:Heal;
if(SpielerInfo[playerid][pBestellung] == 0)
{
if(GetPlayerHealth(playerid,Heal) <= 80)
{
SetPlayerHealth(playerid,Heal+20);
GivePlayerMoney(playerid,-10);
}
else
{
SendClientMessage(playerid,GRAU,"Du hast kein Hunger mehr!");
return 1;
}
}
else if(SpielerInfo[playerid][pBestellung] == 1)
{
if(GetPlayerHealth(playerid,Heal) <= 90)
{
SetPlayerHealth(playerid,Heal+10);
GivePlayerMoney(playerid,-6);
}
else
{
SendClientMessage(playerid,GRAU,"Du hast kein Hunger mehr!");
return 1;
}
}

Bitte um eure Hilfe!

Mit freundlichen Grüßen
Xtreme

Zitat von namespace

Überleg nochmal genau, wie du die GetPlayerHealth Funktion verwendest.
Für was speicherst du das Leben in der Variable Health ab, wenn du sie dann gar nicht verwendest?

Ich verwende es doch indem ich Abfrage ob er weniger als 80 HP hat?

Zitat von namespace

Ja, aber GetPlayerHealth gibt nicht das Leben des Spielers zurück, sondern speichert is in der Variable "Health".
Du musst diese prüfen, nicht den Rückgabewert der Funktion.

if(listitem == 0)
{
new Float:Heal;
if(SpielerInfo[playerid][pBestellung] == 0)
{
if(GetPlayerHealth(playerid,Heal))
{
if(Heal <= 80)
SetPlayerHealth(playerid,Heal+20);
GivePlayerMoney(playerid,-10);
}
else
{
SendClientMessage(playerid,GRAU,"Du hast kein Hunger mehr!");
return 1;
}
}
else if(SpielerInfo[playerid][pBestellung] == 1)
{
if(GetPlayerHealth(playerid,Heal) <= 90)
{
SetPlayerHealth(playerid,Heal+10);
GivePlayerMoney(playerid,-6);
}
else
{
SendClientMessage(playerid,GRAU,"Du hast kein Hunger mehr!");
return 1;
}
}

so?

Zitat von namespace

Was zum.. Nein, natürlich nicht.
Ausnahmsweise mal etwas Code:

Float:f_Health;

GetPlayerHealth(playerid, f_Health);

if (f_Healh <= 80.0) {
// ...

Alles anzeigen

Nope funktioniert nicht!

if(listitem == 0)
{
new Float:Heal;
GetPlayerHealth(playerid,Heal);
if(SpielerInfo[playerid][pBestellung] == 0)
{
if(Heal <= 80.0)
{
SetPlayerHealth(playerid,Heal+20);
GivePlayerMoney(playerid,-10);
}
else
{
SendClientMessage(playerid,GRAU,"Du hast kein Hunger mehr!");
return 1;
}
}
else if(SpielerInfo[playerid][pBestellung] == 1)
{
if(Heal <= 90.0)
{
SetPlayerHealth(playerid,Heal+10);
GivePlayerMoney(playerid,-6);
}
else
{
SendClientMessage(playerid,GRAU,"Du hast kein Hunger mehr!");
return 1;
}
}
}

Zitat von namespace

Dann stimmt etwas Anderes nicht. Daran liegt es jedenfalls in deinem Code nicht (mehr).
Versuch mit Debug-Nachrichten raus zu finden, wo das Problem liegt.

Hey, das hat jetzt zwar funktioniert jedoch wird jetzt kein HP gesetzt!

if(listitem == 0)
{
new Float:Heal;
GetPlayerHealth(playerid,Heal);
if(SpielerInfo[playerid][pBestellung] == 0)
{
SetPlayerHealth(playerid,Heal+20);
GivePlayerMoney(playerid,-10);
}
else if(SpielerInfo[playerid][pBestellung] == 1)
{
SetPlayerHealth(playerid,Heal+10);
GivePlayerMoney(playerid,-6);
}
else if(SpielerInfo[playerid][pBestellung] == 2)
{
SetPlayerHealth(playerid,Heal+8);
GivePlayerMoney(playerid,-3);
}
else if(SpielerInfo[playerid][pBestellung] == 3)
{
SetPlayerHealth(playerid,Heal+25);
GivePlayerMoney(playerid,-20);
}
else if(SpielerInfo[playerid][pBestellung] == 4)
{
SetPlayerHealth(playerid,Heal+15);
GivePlayerMoney(playerid,-15);
}
}