Was ist hier Falsch???

grad mal so auf die schnelle:

ocmd:bl(playerid,params[])
{
SendClientMessage(playerid,Hellblau,"Blacklist User Online:");
for(new pID = 0; pID<MAX_PLAYERS; pID)
{
new GroveStreetBl[124];
switch(GetPVarInt(playerid,"GroveStreetBl"))
{
case 0:{GroveStreetBl="";}
case 1:{GroveStreetBl=SpielerName(pID);}
}
new BallasBl[124];
switch(GetPVarInt(playerid,"BallasBl"))
{
case 0:{BallasBl="";}
case 1:{BallasBl=SpielerName(pID);}
}
new LosVagosBl[124];
switch(GetPVarInt(playerid,"LosVagosBl"))
{
case 0:{LosVagosBl="";}
case 1:{LosVagosBl=SpielerName(pID);}
}
new ScafoBl[124];
switch(GetPVarInt(playerid,"ScafoBl"))
{
case 0:{ScafoBl="";}
case 1:{ScafoBl=SpielerName(pID);}
}
new RussischeMafiaBl[124];
switch(GetPVarInt(playerid,"RussischeMafiaBl"))
{
case 0:{RussischeMafiaBl="";}
case 1:{RussischeMafiaBl=SpielerName(pID);}
}
new LaCosaNostraBl[124];
switch(GetPVarInt(playerid,"LaCosaNostraBl"))
{
case 0:{LaCosaNostraBl="";}
case 1:{LaCosaNostraBl=SpielerName(pID);}
}
new YakuzZaBl[124];
switch(GetPVarInt(playerid,"YakuzZaBl"))
{
case 0:{YakuzZaBl="";}
case 1:{YakuzZaBl=SpielerName(pID);}
}
new TriadenBl[124];
switch(GetPVarInt(playerid,"TriadenBl"))
{
case 0:{TriadenBl="";}
case 1:{TriadenBl=SpielerName(pID);}
}
if(isPlayerInFrakt(playerid,10))
{
if(GetPVarInt(pID,"GroveStreetBl")==1)
{
SendClientMessage(playerid,Gelb,GroveStreetBl);
}
}
else if(isPlayerInFrakt(playerid,11))
{
if(GetPVarInt(pID,"BallasBl")==1)
{
SendClientMessage(playerid,Gelb,BallasBl);
}
}
else if(isPlayerInFrakt(playerid,12))
{
if(GetPVarInt(pID,"LosVagosBl")==1)
{
SendClientMessage(playerid,Gelb,LosVagosBl);
}
}
else if(isPlayerInFrakt(playerid,13))
{
if(GetPVarInt(pID,"ScafoBl")==1)
{
SendClientMessage(playerid,Gelb,ScafoBl);
}
}
else if(isPlayerInFrakt(playerid,15))
{
if(GetPVarInt(pID,"RussischeMafiaBl")==1)
{
SendClientMessage(playerid,Gelb,RussischeMafiaBl);
}
}
else if(isPlayerInFrakt(playerid,16))
{
if(GetPVarInt(pID,"LaCosaNostraBl")==1)
{
SendClientMessage(playerid,Gelb,LaCosaNostraBl);
}
}
else if(isPlayerInFrakt(playerid,17))
{
if(GetPVarInt(pID,"YakuzZaBl")==1)
{
SendClientMessage(playerid,Gelb,YakuzZaBl);
}
}
else if(isPlayerInFrakt(playerid,18))
{
if(GetPVarInt(pID,"TriadenBl")==1)
{
SendClientMessage(playerid,Gelb,TriadenBl);
}
}
}
return 1;
}

Musste schauen ob es funktioniert.

Ich würde es ja eher so machen:

ocmd:bl(playerid){
SendClientMessage(playerid, -1, "Blacklist");
for(new i=0;i<MAX_PLAYERS;i++){
if(GetPVarInt(playerid, "Fraktion") == 1){
if(GetPVarInt(i, "GroveStreetBl") == 1){
format(string, sizeof(string), "ID: %d - Name: %s", i, SpielerName(i));
SendClientMessage(playerid, -1, string);
}
}
}
return 1;
}

ocmd:blacklist(playerid){
return ocmd_bl(playerid);
}

Sollte theoretisch funktionieren!