[FRAGE] Was mache ich Falsch? /Carlist

[GER]Raxos · 7. Januar 2012

Eine Frage Leute ,
Mein Freund hat mir mal nen /carlist befehl gezeigt ,
er hat nun ein Problem, undzwar:
wenn er den Befehl eingibt listet der zu viele ID's auf , statt nur die mann ich besitze.
Ich hab das problem nicht erkannt ... und ihr?

Hier der Screen:

Hier der /carlist befehl:
if(!strcmp(cmd, "/carlist", true))
{
if(IsPlayerConnected(playerid))
{
new strr[128], countr = 0;
SendClientMessage(playerid,COLOR_TUTBLAU,"|~~~~~~~~~~| Deine Fahrzeuge |~~~~~~~~~~|");
for(new i = 0; i<MAX_VEHICLES; i++)
{
format(strr, sizeof strr, "/Cars/%d.ini", i);
if(fexist(strr))
{
if(!strcmp(PlayerCar[i][Key], pn(playerid), false))
{
countr++;
format(str, sizeof str, "ID %d", PlayerCar[i][NSID]);
SendClientMessage(playerid, Grey, str);
}
}
}
if(!countr) SendClientMessage(playerid, Grey, "Du besitzt kein Fahrzeug!");
if(PlayerInfo[playerid][pRentCarKey] > 0)
{
format(str, sizeof str, "Leihschlüssel: %s ID %d", PlayerVehicle[GetModelFromID(PlayerInfo[playerid][pRentCarKey])-400], PlayerInfo[playerid][pRentCarKey]);
SendClientMessage(playerid, Grey, str);
}
}
return 1;
}

p.s es ist das LifePlay Script

Deadmau5 · 7. Januar 2012

Ach du...
Niemals SendClientMessage in einer Schleife ausführen

Versuchs mal so

if(!strcmp(cmd, "/carlist", true))
{
if(IsPlayerConnected(playerid))
{
new strr[64][128], countr = 0;
SendClientMessage(playerid,COLOR_TUTBLAU,"|~~~~~~~~~~| Deine Fahrzeuge |~~~~~~~~~~|");
for(new i = 0; i<MAX_VEHICLES; i++)
{
format(strr, sizeof strr, "/Cars/%d.ini", i);
if(fexist(strr))
{
if(!strcmp(PlayerCar[i][Key], pn(playerid), false))
{
format(str[countr], sizeof str[countr], "ID %d", PlayerCar[i][NSID]);
countr++;
}
}
}
for(new x; x<= countr;x++)
{
SendClientMessage(playerid, Grey, str[x]);
}
if(!countr) SendClientMessage(playerid, Grey, "Du besitzt kein Fahrzeug!");
if(PlayerInfo[playerid][pRentCarKey] > 0)
{
format(str, sizeof str, "Leihschlüssel: %s ID %d", PlayerVehicle[GetModelFromID(PlayerInfo[playerid][pRentCarKey])-400], PlayerInfo[playerid][pRentCarKey]);
SendClientMessage(playerid, Grey, str);
}
}
return 1;
}

[GER]Raxos · 7. Januar 2012

Menno
1 error

C:\Users\asd\Desktop\Gta San Andreas Zeug\Script's\GtaLifePlay-Script-Edit-By-Me\gamemodes\Lifeplay.pwn(16806) : error 080: unknown symbol, or not a constant symbol (symbol "countr")
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.
[/pwn]

16806: format(str[countr], sizeof str[countr], "ID %d", PlayerCar[i][NSID]);

Deadmau5 · 7. Januar 2012

Fail
Moment versuchs so
if(!strcmp(cmd, "/carlist", true))
{
if(IsPlayerConnected(playerid))
{
new strr[64][128], countr = 0, count;
SendClientMessage(playerid,COLOR_TUTBLAU,"|~~~~~~~~~~| Deine Fahrzeuge |~~~~~~~~~~|");
for(new i = 0; i<MAX_VEHICLES; i++)
{
format(strr, sizeof strr, "/Cars/%d.ini", i);
if(fexist(strr))
{
if(!strcmp(PlayerCar[i][Key], pn(playerid), false))
{
format(str[count], 128, "ID %d", PlayerCar[i][NSID]);
countr++;
count++;
}
}
}
for(new x; x<= count;x++)
{
SendClientMessage(playerid, Grey, str[x]);
}
if(!countr) SendClientMessage(playerid, Grey, "Du besitzt kein Fahrzeug!");
if(PlayerInfo[playerid][pRentCarKey] > 0)
{
format(str, sizeof str, "Leihschlüssel: %s ID %d", PlayerVehicle[GetModelFromID(PlayerInfo[playerid][pRentCarKey])-400], PlayerInfo[playerid][pRentCarKey]);
SendClientMessage(playerid, Grey, str);
}
}
return 1;
}

[GER]Raxos · 7. Januar 2012

Zitat

new strr[64][128], countr = 0, count;

muss das nicht so sein
new strr][128], countr = 0, count;

Funtz net
ist jetzt schlimmer..

breadfish.de 11. März 2022