Waffen werden nicht an alle Spieler gegeben, trotz der Schleife

Hier ein paar Infos:

1. Nicht dcmd nutzen...das nur ein Makro für strcmp...nutze lieber ocmd/zcmd/ycmd..bla blub

2. SendClientMessageToAll sendet schon die Nachricht an alle, da musst du es nicht in die Schleife packen, dass es...~SpielerN mal aufgerufen wird..

3. Selbiges für GameTextForAll

4. return 1; In der Schleife ist natürlich nicht sinnig

Also so sollte das aussehen:

dcmd_minigunall(playerid,params[])
{
    #pragma unused params
    if(AccInfo[playerid][Level] < 2) return SendClientMessage(playerid,-1,"* No permission!");
    new string[125];
    format(string,sizeof(string),"* Admin %s (ID:%d) gave everyone a Minigun!", pName(playerid),playerid);
    SendClientMessageToAll(yellow,string);
    GameTextForAll("~w~Free~n~~g~Minigun!", 3000, 3);
    for(new i; i < MAX_PLAYERS; i++)
    {
    	if(!IsPlayerConnected(i)) continue;
        PlayerPlaySound(i,1057,0.0,0.0,0.0);
        GivePlayerWeapon(i,38,10000);
    }
    return 1;
}

Zitat von PeRable

Vielen Dank, Kaliber.

dcmd sollte ich also garnicht nutzen? Merkt man arg, dass es langsamer ist oder warum ist davon abzuraten?

Je mehr Befehle du hast, desto langsamer wird das.

Du musst verstehen..bei strcmp steht das alles in einer if-Abfrage..das sieht dann so aus (mal als Abstraktes Beispiel mit Zahlen)

new x = 5;
if(x == 1)
else if(x==2)
else if(x==3)
else if(x==4)

Hier muss nun von 1 bis...z.B. 100 durchgelaufen werden, bis dein Befehl gefunden wurde...und das kann wirklich lange dauern, wenn der Command ganz unten steht...
Die Vorteile bei ocmd/zcmd/ycmd..usw diese springen direkt an den Punkt wo dieser Command startet
Und dadurch erreichst du eine enorme Performanz Verbesserung...egal wie viele Befehle du hast! (Denn bei strcmp (dcmd) wird es immer langsamer..je mehr Befehle du hast).