Function Mafiengt(playerid)
{
new Float: x, Float: y, Float: z;
for(new i=0;i<MAX_PLAYERS; i++)
if(IsPlayerConnected(i))
{
GetPlayerPos(playerid, x, y, z);
if(IsPlayerInRangeOfPoint(i, 50.0, x,y,z) && IsAGang(i))
{
GivePlayerMoney(i, 30000);
SendClientMessage(i,copblau,"Bonus für deine Hilfe: 30.000$");
return 1;
}
}
return 1;
}
Versuchs mal so... KEINE GARANTIE 
Unsinn.
erstens, playerid kannste eigendlich rausnehmen. Dann GetPlayerPos(i...); und nicht playerid.
Das return 1;!! rausnehmen, niemals return 1 in einer Schleife. Die Funktion wird dadurch beendet.
Also rausnehmen und dann sollte das klappen.
Neee, lass es so!
Hab die Lösung 
Function Mafiengt(playerid)
{
new Float: x, Float: y, Float: z;
for(new i=0;i<MAX_PLAYERS; i++)
if(IsPlayerConnected(i))
{
GetPlayerPos(playerid, x, y, z);
if(IsPlayerInRangeOfPoint(i, 50.0, x,y,z) && IsAGang(i))
{
GivePlayerMoney(i, 30000);
SendClientMessage(i,copblau,"Bonus für deine Hilfe: 30.000$");
break;
}
}
return 1;
}
Funzt immer noch net 
(Deadmouse Lösung)
//EDIT: Linclon Lösung funzt Danke 
Mal so nebenbei, was ich immer wieder sehe..
for(new i=0;i<MAX_PLAYERS; i++)
Wozu denn bitte i=0;? Das ist richtig sinnlos, weil eine neu erstellte Variable sowieso auf 0 steht..
Ändert jetzt natürlich nicht wirklich was, aber es fällt mir einfach extrem oft auf..
Normalerweise gibt man der Variable einen Wert, um ab diesem Wert die Schleife zu beginnen.
Es stimmt schon, aber es ist halt Angewohnheit von einigen.
Aber vielleicht kommt das auch dadurch das es in einigen Programiersprachen noch den Wer NULL gibt aber NULL ist nicht 0 sonder leer.
Kann man denn von einer Leeren Variable hochzählen?
Nein.
Wenn man es ganz genau nehmen will kann man auch
new i;
for(;i!=MAX_PLAYERS;i++)
machen das ist eigentlich die schnellste Form wenn man 500 Slots bzw 500 Spieler hat.
//Edit _ zu ;
Klugscheißer du 
Hast aber Recht, geht angeblich wirklich schneller.
