/auftragskiller

Ich verstehe den Sinn des Strings spieler nicht.
Der random reicht doch vollkommen als Auswahl der zufälligen Zielperson.

Die ausgesuchte Person solltest du global für jeden Spieler speichern. Weiter gehen würde es dann zB. mit:

public OnPlayerDeath(playerid, killerid, reason)
{
if(killer[killerid])
{
if(playerid == zielperson[killerid])
{
killer[killerid] = 0;
GivePlayerMoney(killerid, KILLER_REWARD);
SendClientMessage(killerid, 0xFFFFFFFF, "Du hast die Zielperson getötet.");
}
}

return 1;
}

public OnPlayerDisconnect(playerid)
{
for(new i=0; i
{
killer[i] = 0;
SendClientMessage(i, 0xFFFFFFFF, "Deine Zielperson ist vom Server gegangen. Du kannst einen neuen Auftrag annehmen.");
}

return 1;
}

Zitat von The_Godboy

Edit: Meinte etwas buggy^^

???
Ausversehen den "nicht-edit" Part aus'm Post gelöscht?

Einfach typ2 global definieren. Also, außerhalb eines jeden Callbacks schreiben, am besten ganz nach oben ins Script.

Bei OnPlayerDeath muss es playerid bleiben, die Variable ist ja vom Callback selbst vorgegeben.
Außerdem machen manche Änderungen keinen Sinn, zB.:

if(killer[playerid] == killerid)

killer[] speichert doch nur, ob der Spieler ein Auftragskiller ist oder nicht, also wird boolsch genutzt. Btw könntest du sie auch gleich boolean machen, mit:

new bool:killer[MAX_PLAYERS];

Also so ist der OnPlayerDeath Part richtig:

public OnPlayerDeath(playerid, killerid, reason)
{
if(killer[killerid])
{
if(playerid == zielperson[killerid])
{
killer[killerid] = 0;
GivePlayerMoney(killerid, 5000);
SendClientMessage(killerid, GREEN, "Du hast die Zielperson ausgeschaltet und hast 5000$ bekommen.");
}
}

return 1;
}

Zitat von Killasplash

wo isn der unterschied zwischen nem bool und ner normalen variable

Boolsche Variablen können nur den Wert true oder false besitzen. Damit verstehst du halt auch dein Script besser und du weißt, dass das ein Schalter ist.

Zitat von Killasplash

ja dann kann ichs ja genau so gut bei ner normalen variable lassen

War auch nur'n Vorschlag

Und was ist, wenn du Zufallsid nicht vergeben ist?
public AuftragsKiller(playerid)
{
new typ=random(sizeof(spieler));
new typ2=spieler[typ][0];
if(IsPlayerConnected(typ2))
{
new str[126];
new name[MAX_PLAYER_NAME];
GetPlayerName(typ2, name, sizeof(name));
format(str, sizeof(str), " Du musst %s killen!", name);
SendClientMessage(playerid, COLOR_YELLOW, str);
killer[playerid] = 1;
zielperson[playerid] = typ2;
}
else { AuftragsKiller(playerid); }
}

public AuftragsKiller(playerid)
{
newrandom:
new typ=random(sizeof(spieler));
new typ2=spieler[typ][0];
if(IsPlayerConnected(typ2))
{
new str[126];
new name[MAX_PLAYER_NAME];
GetPlayerName(typ2, name, sizeof(name));
format(str, sizeof(str), " Du musst %s killen!", name);
SendClientMessage(playerid, COLOR_YELLOW, str);
killer[playerid] = 1;
zielperson[playerid] = typ2;
}
else goto newrandom;
}

Hab das jetzt mit nem Label gelöst, geht aber auch anders.

ich bin verwirrt kann mir das einer nochmal komplett zusammenstellen pls
übrigens top gemacht

Spart wieder einiges...^^

public AuftragsKiller(playerid)
{
new typ;
new typ2;
while(!IsPlayerConnected(Typ2))
{
typ=random(sizeof(spieler));
typ2=spieler[typ][0];
}
new str[126];
new name[MAX_PLAYER_NAME];
GetPlayerName(typ2, name, sizeof(name));
format(str, sizeof(str), " Du musst %s killen!", name);
SendClientMessage(playerid, COLOR_YELLOW, str);
killer[playerid] = 1;
zielperson[playerid] = typ2;
return 1;
}