/ejecte mich selber. :o

dcmd nutzen ist die leichte,schnelle und zukunftsorientierte Antwort...

füge unter pid = strval(tmp);
if(!IsPlayerConnected(pid)){
SendClientMessage(playerid,0xFF004040,"Dieser Spieler ist nicht auf dem Server!");
return 1;
}

noch hinzu:

if(pid == playerid){
SendClientMessage(playerid,0xFF004040,"Du kannst dich nicht selber rauswerfen!!");
return 1;
}

if(strcmp(cmd, "/eject", true) == 0)
{
if(IsPlayerConnected(playerid))
{
new State;
if(IsPlayerInAnyVehicle(playerid))
{
State=GetPlayerState(playerid);
if(State!=PLAYER_STATE_DRIVER)
{
SendClientMessage(playerid,COLOR_GREY," Du kannst nur Spieler als ein Fahrer rauswerfen !");
return 1;
}
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, COLOR_GRAD2, "USAGE: /eject [playerid/PartOfName]");
return 1;
}
new playa;
playa = ReturnUser(tmp);
new test;
test = GetPlayerVehicleID(playerid);
if(IsPlayerConnected(playa))
{
if(playa != INVALID_PLAYER_ID)
{
if(playa == playerid) { SendClientMessage(playerid, COLOR_GREY, "Du kannst dich nicht selbst rauswerfen!"); return 1; }
if(IsPlayerInVehicle(playa,test))
{
new PName[MAX_PLAYER_NAME];
GetPlayerName(playerid,PName,sizeof(PName));
GetPlayerName(playa, giveplayer, sizeof(giveplayer));
format(string, sizeof(string), "* Du hasst %s aus dem Auto geworfen!", giveplayer);
SendClientMessage(playerid, COLOR_LIGHTBLUE, string);
format(string, sizeof(string), "* Du wurdest von %s aus dem Auto geworfen !", PName);
SendClientMessage(playa, COLOR_LIGHTBLUE, string);
RemovePlayerFromVehicle(playa);
}
else
{
SendClientMessage(playerid, COLOR_GREY, " Dieser Spieler ist nicht in deinem Auto !");
return 1;
}
}
}
else
{
SendClientMessage(playerid, COLOR_GREY, " Ungültige ID/Name!");
}
}
else
{
SendClientMessage(playerid, COLOR_GREY, " Du musst in einem Fahrzeug sein um dies zu nützen !");
}
}
return 1;
}

probier mal das

Zitat von playerid

Soll ich das als Spam sehen?

Das was er geschrieben hat ist kein Spam!
Er aht Recht!
Mit Dcmd ist es erstens Kürzer,zweitens schneller und drittens leichter zu verstehen/schreiben!

Hier einmal ein Beispiel:

dcmd_eject(playerid,params[])
{
#pragma unused params
if(IsPlayerConnected(playerid))
{
if(!strlen(params)) return SendClientMessage(playerid, COLOR_WHITE, "Benutzung /eject [Playerid]");
new player = ReturnUser(params);
if(GetPlayerState(playerid) == PLAYER_STATE_DRIVER && IsPlayerInVehicle(giveplayerid, GetPlayerVehicleID(playerid)))
{
if(IsPlayerConnected(giveplayerid) && playerid != player)
{
SendStringMessage(playerid,COLOR_ORANGE,"Du hast %s(%i) aus deinen Auto geschmissen!", GetName(giveplayerid), player);
SendStringMessage(player,COLOR_ORANGE,"Du wurdest von %s(%d) aus dem Auto geworfen!", GetName(playerid), playerid);
RemovePlayerFromVehicle(player);}
}
}
return true;
}
Und gleich alles viel kleiner und kompakter!
Das ganze kann man natürlich auch noch mit der sscanf Funktion schreiben!
Dann muss man z.b ReturnUser nicht verwenden!

Falls du diesen CMD verwenden willst,solltest du Dcmd und SendStringMessage definieren!
Genauso wie den Stock!

stock GetName(playerid)
{
new name[MAX_PLAYER_NAME]; GetPlayerName(playerid,name,sizeof(name));
return name;
}

//Edit:
Im Code was geändert!

tausch mal giveplayerid mit payer aus!
Wegen ReturnUser...ich glaube da musste was bestimmtes includen bin mir aber ne sicher
Und wie gesagt...sowas wie SendStringMessage musste halt noch selber machen

Zitat von [FoR]Hardcore

Das was er geschrieben hat ist kein Spam!
Er aht Recht!
Mit Dcmd ist es erstens Kürzer,zweitens schneller und drittens leichter zu verstehen/schreiben!

Nunja, eigentlich bin ich eher auf der Seite von "playerid", da er Recht hat. Dein Beitrag ist um einiges produktiver als der von Trooper...

@Topic: Siehe Dateianhang

und kopier dir das IsNumeric hier raus:

http://pastebin.com/f764931c1

Zitat von Trooper[Y]

dcmd nutzen ist die leichte,schnelle und zukunftsorientierte Antwort...

Falsch. DCMD ist veraltet und kaum schneller als reine strcmp-Commands.
==> ZCMD ist die leichte, schnelle und zukunftsorientiere Antwort...