Symples Tazer Problem [NO-GF]

Du musst dem Script die PlayerID auch mittteilen, sonst bleibt pID 0 und dann wird immer ID 0 getazert.

if(GetPVarInt(playerid,"Leader") == 1 || GetPVarInt(playerid,"Member") == 1)
{
new Float:x, Float:y, Float:z;
GetPlayerPos(playerid,x,y,z);
new string[280], pID;
{ // Für was geht hier ne klammer auf? :O Is doch garkein if oder so.
if(IsPlayerConnected(pID))
{
if(IsPlayerInRangeOfPoint(playerid,6,x,y,z))
{
TogglePlayerControllable(pID,0);
format(string,sizeof string,"~b~Taze\n~r~Du ~r~wurdest ~r~getazert");
GameTextForPlayer(pID,string,4000,3);
SetTimer("Tazertime",10000,0);
}
}
} // und hier wieder zu? :O
}

So sollte es funktionieren (ungetestet):
forward Tazertime(pID);
ocmd:tazer(playerid,params[])
{
if(GetPVarInt(playerid,"Leader") == 1 || GetPVarInt(playerid,"Member") == 1)
{
new Float:x, Float:y, Float:z;
GetPlayerPos(playerid,x,y,z);
new string[280], pID;
{
if(IsPlayerConnected(pID))
{
if(IsPlayerInRangeOfPoint(playerid,6,x,y,z))
{
TogglePlayerControllable(pID,0);
format(string,sizeof string,"~b~Taze\n~r~Du ~r~wurdest ~r~getazert");
GameTextForPlayer(pID,string,4000,3);
SetTimerEx("Tazertime",10000,0,"i",pID);
}
}
}
}
return 1;
}

public Tazertime(pID)
{
TogglePlayerControllable(pID,0);
}

DU musst einfach in deinen Befehl das rein machen:

if(pID== playerid) { return 1; }

mfg.

Wie soll den das funktionieren Darklight?
Der Varibale pID wird doch nirgends ein Wert zugewiesen.
GetClosestPlayer ist die Lösung.
Schau mal ins LoH Script da is sowas drinne.

Zitat von *** PAWNO ***

DU musst einfach in deinen Befehl das rein machen:

if(pID== playerid) { return 1; }

mfg.

Alles anzeigen

Er will aber das er keine ID eingeben muss, sondern nur /tazern braucht.
Wie EvilToastBrot schon sagt. Lade dir am besten dieses Include herunter und verwende die GetClosestPlayer Funktion.
Oder scripte dir eine While-Schleife, damit funktioniert es auch.

Zitat von Darklight

Er will aber das er keine ID eingeben muss, sondern nur /tazern braucht.
Wie EvilToastBrot schon sagt. Lade dir am besten dieses Include herunter und verwende die GetClosestPlayer Funktion.
Oder scripte dir eine While-Schleife, damit funktioniert es auch.

Hmm.. mit ner for-schleife auch.

Das hier is aus LoH, musste noch umschreiben:

forward Float:GetDistanceBetweenPlayers(p1,p2);
forward GetClosestPlayer(p1);
public GetClosestPlayer(p1)
{
new x,Float:dis,Float:dis2,player;
player = -1;
dis = 99999.99;
for (x=0;x<MAX_PLAYERS;x++)
{
if(IsPlayerConnected(x))
{
if(x != p1)
{
dis2 = GetDistanceBetweenPlayers(x,p1);
if(dis2 < dis && dis2 != -1.00)
{
dis = dis2;
player = x;
}
}
}
}
return player;
}
public Float:GetDistanceBetweenPlayers(p1,p2)
{
new Float:x1,Float:y1,Float:z1,Float:x2,Float:y2,Float:z2;
if(!IsPlayerConnected(p1) || !IsPlayerConnected(p2))
{
return -1.00;
}
GetPlayerPos(p1,x1,y1,z1);
GetPlayerPos(p2,x2,y2,z2);
return floatsqroot(floatpower(floatabs(floatsub(x2,x1)),2)+floatpower(floatabs(floatsub(y2,y1)),2)+floatpower(floatabs(floatsub(z2,z1)),2));
}

// Unter OnPlayerCommandText
if(strcmp(cmd, "/tazer", true) ==0)
{
if(IsPlayerConnected(playerid))
{
if(PlayerInfo[playerid][pMember] == 1 || PlayerInfo[playerid][pLeader] == 1 || PlayerInfo[playerid][pMember] == 2 || PlayerInfo[playerid][pLeader] == 2 || PlayerInfo[playerid][pMember] == 3 || PlayerInfo[playerid][pLeader] == 3|| PlayerInfo[playerid][pMember] == 21 || PlayerInfo[playerid][pLeader] == 21)
{
if(OnDuty[playerid]==0)
{
SendClientMessage(playerid, COLOR_GREY, "Du bist nicht im Dienst!");
return 1;
}
if(IsPlayerInAnyVehicle(playerid))
{
SendClientMessage(playerid, COLOR_GREY, "Du kannst das nicht benutzen während du im Fahrzeug sitzt!");
return 1;
}
new suspect = GetClosestPlayer(playerid);
if(IsPlayerConnected(suspect))
{
if(PlayerCuffed[suspect] > 0)
{
SendClientMessage(playerid, COLOR_GREY, "Dieser Spieler ist schon getazed!");
return 1;
}
if(GetDistanceBetweenPlayers(playerid,suspect) < 5)
{
if(IsPlayerInAnyVehicle(suspect))
{
SendClientMessage(playerid, COLOR_GREY, "Dieser Spieler ist in einem Fahrzeug! Bringe in zuerst raus!");
return 1;
}
GetPlayerName(suspect, giveplayer, sizeof(giveplayer));
GetPlayerName(playerid, sendername, sizeof(sendername));
format(string, sizeof(string), "* Du wurdest von %s getazert! Du liegst für 15 Sekunden flach!", sendername);
SendClientMessage(suspect, COLOR_LIGHTBLUE, string);
format(string, sizeof(string), "* Du hast %s niedergestreckt.", giveplayer);
SendClientMessage(playerid, COLOR_LIGHTBLUE, string);
format(string, sizeof(string), "* %s nimmt seinen Elektroschocker raus und streckt %s nieder.", sendername ,giveplayer);
ProxDetector(30.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
GameTextForPlayer(suspect, "~y~Getazert!", 3000, 3);
TogglePlayerControllable(suspect, 0);
PlayerCuffed[suspect] = 1;
PlayerCuffedTime[suspect] = 15;
}
else
{
SendClientMessage(playerid, COLOR_GREY, "Niemand ist in deiner Nähe!");
return 1;
}
}
}
else
{
SendClientMessage(playerid, COLOR_GREY, "Du bist kein Beamter!");
}
}
return 1;
}

Jetzt müsstest du es aber hinbekommen.

Du musst das ja auch anpassen zB. giveplayer an deine Funktion, die bei dir evtl. anders heißt.
Das selbe bei sendername

hzE