SendClientMessage wird mehrfach abgerufen

Bei mir war es bis jetzt immer so, dass ein return 1; fehlte. Dann war das Problem immer weg.

Zitat von Canna

Bei mir war es bis jetzt immer so, dass ein return 1; fehlte. Dann war das Problem immer weg.

Dann wird die Schleife aber nicht fortgeführt.

ocmd:hausverkaufen(playerid,params[])
{
if(GetPVarInt(playerid,"Haus")==0)return SendClientMessage(playerid,Color_Red,"Du hast kein Haus!");
for(new haus=1;haus<MAX_HAUS;haus++)
{
new name[MAX_PLAYER_NAME];
GetPlayerName(playerid,name,sizeof(name));
new pfad[50];
format(pfad,50,"/Haeuser/%d.txt",haus);
SetPVarInt(playerid,"Haus",0);
dini_Set(pfad,"besitzer","Keiner");
GivePlayerMoney(playerid,HausInfo[haus][haus_preis]/2);
SendClientMessage(playerid,Color_Green,"Du hast erfolgreich dein Haus verkauft und hast die Hälfte des Kaufpreises zurückbekommen!");
KillHaus(haus);
LoadHaus(haus);
SavePlayer(playerid);
}
return 1;
}

MfG

ocmd:hausverkaufen(playerid,params[])
{
if(GetPVarInt(playerid,"Haus")==0)return SendClientMessage(playerid,Color_Red,"Du hast kein Haus!");
for(new haus=1;haus<MAX_HAUS;haus++)
{
if(GetPVarInt(playerid,"Haus")==haus)
{
new pfad[50];
format(pfad,50,"/Haeuser/%d.txt",haus);
SetPVarInt(playerid,"Haus",0);
dini_Set(pfad,"besitzer","Keiner");
GivePlayerMoney(playerid,HausInfo[haus][haus_preis]/2);
KillHaus(haus);
LoadHaus(haus);
SavePlayer(playerid);
SendClientMessage(playerid,Color_Green,"Du hast erfolgreich dein Haus verkauft und hast die Hälfte des Kaufpreises zurückbekommen!");
break;
}
}
return 1;
}

ocmd:hausverkaufen(playerid,params[])
{
if(GetPVarInt(playerid,"Haus")==0)return SendClientMessage(playerid,Color_Red,"Du hast kein Haus!");
for(new haus=1;haus<MAX_HAUS;haus++)
{
if (haus == GetPVarInt(playerid, "Haus") {
new name[MAX_PLAYER_NAME];
GetPlayerName(playerid,name,sizeof(name));
new pfad[50];
format(pfad,50,"/Haeuser/%d.txt",haus);
SetPVarInt(playerid,"Haus",0);
dini_Set(pfad,"besitzer","Keiner");
GivePlayerMoney(playerid,HausInfo[haus][haus_preis]/2);
SendClientMessage(playerid,Color_Green,"Du hast erfolgreich dein Haus verkauft und hast die Hälfte des Kaufpreises zurückbekommen!");
KillHaus(haus);
LoadHaus(haus);
SavePlayer(playerid);
break;
}
}
return 1;
}

Mache dir klar, was dein Script dort tut. Finde heraus, wie die Schleife und die if-Abfrage fungiert und du wirst erkennen, woran das Problem liegt. Falls du das nicht tust, wirst du erneut und erneut auf Probleme stoßen. Falls dir irgendetwas am Code nur etwas unklar ist, frage hier nach.

break;
unterbricht die Schleife.

Wozu überhaupt die Schleife?

ocmd:hausverkaufen(playerid,params[])
{
new pHaus = GetPVarInt(playerid,"Haus");
if(pHaus != 0)
{
new pfad[50];
format(pfad,50,"/Haeuser/%d.txt",pHaus);
dini_Set(pfad,"besitzer","Keiner");
SetPVarInt(playerid,"Haus",0);
GivePlayerMoney(playerid,HausInfo[pHaus][haus_preis]/2);
SendClientMessage(playerid,Color_Green,"Du hast erfolgreich dein Haus verkauft und hast die Hälfte des Kaufpreises zurückbekommen!");
KillHaus(pHaus);
LoadHaus(pHaus);
SavePlayer(playerid);
}
else if(pHaus == 0)
{
SendClientMessage(playerid,Color_Red,"Du hast kein Haus!");
}
return 1;
}

Die Schleife fing bei 1 an und zählt je Schritt um 1 hoch, bei jedem Schritt wird der Code innerhalb ausgeführt, dort wird eigentlich nur verglichen ob die Schleifenvariable gleich der PVariable ist. Das ist insoweit unnötig weil du auch direkt die PVar nehmen kannst, dazu brauchst du keine Schleife (Siehe meinen Code)

Erklärung: while-, do...while-, for-Schleifen (Mit Struktogramm etc.)