OnPlayerDeath ( Tode werden nicht richtig erkannt )?

VAM · 24. August 2016

Hey,

also ich habe im Script die Funktion dass man beim töten eines Spielers 2 Wanteds bekommt und die Cops eine Nachricht davon sowie auch die Medics.

Die Wantedvergabe sieht wie folgt aus :

Code

if(PlayerOnline(killerid))
	{
		if(pInfo[killerid][fraktion]!=2 && pInfo[killerid][fraktion]!=3)return SetPlayerWantedLevel(killerid, GetPlayerWantedLevel(killerid)+2);
		if(pInfo[killerid][fraktion]!=2 && pInfo[killerid][fraktion]!=3)return SendClientMessage(killerid, DROT, "Du hast ein Verbrechen begangen ( Mord ), Zeuge : Niemand");
		new infostr[128];
		format(infostr,sizeof(infostr),"Dein Wantedlevel: %d", wanteds[killerid]);
		if(pInfo[killerid][fraktion]!=2 && pInfo[killerid][fraktion]!=3)return SendClientMessage(killerid, GELB, infostr);
		new meldung[128], zne[64];
		GetPlayer3DZone(killerid, zne, sizeof(zne));
		format(meldung,sizeof(meldung),"* Meldung: Verdächtiger %s, Verbrechen: Mord in %s.",PlayerName(killerid), zne);
		for(new i=0; i<GetMaxPlayers(); i++)
		{
			if(!PlayerOnline(i))continue;
			if(pInfo[i][fraktion]!=2 && pInfo[i][fraktion]!=3)continue;
			SendClientMessage(i, 0x8080FFFF, meldung);
		}
	}

Alles anzeigen

Davor war die Abfrage wenn killerid FraktionsID 2 + 3 ist nicht drinnen ( LSPD + FBI ).

Das heißt, wenn ich einen Zivi als Zivi killen würde, landet der playerid nicht im Krankenhaus und killerid kriegt keine Wanteds.

Aber wenn ich als Cop jemanden getötet habe, bekam ich Wanteds ( NUR ALS COP ) und playerid landete normal im Krankenhaus.
Als ich die Abfrage reingemacht habe, habe ich auch als Cop keine Wanteds und playerid landete auch nicht im Krankenhaus.

Also man bekommt einfach keine Wanteds wenn man Leute tötet.

Bitte helft mir

@Jeffry

Jeffry · 24. August 2016

Entferne nach den drei if-Abfragen jeweils das "return".

VAM · 24. August 2016

Danke für deine Rückmeldung, hab das gemacht und die neue .amx hochgeladen,

jedoch kriege ich immernoch keine Wanteds / playerid kommt nicht ins Krankenhaus.

Jeffry · 24. August 2016

Setze bitte mal einen print direkt vor
if(PlayerOnline(killerid))

Wird der angezeigt?
printf("Aufgerufen für killerid: %d", killerid);

VAM · 24. August 2016

Hey,

habe das gemacht also so :

Code

printf("Aufgerufen für killerid: %d", killerid);
	if(PlayerOnline(killerid))
	{

Jedoch kriege ich in der server-log.txt nur das :

[18:48:03] [kill] TestAcc killed TestAcc2 Desert Eagle

Jeffry · 24. August 2016

Dann wird der Code irgendwo drüber schon beendet.
Setze weitere prints in den Zeilen über diesem Abschnitt und finde so heraus, bis wohin der Code durchlaufen wird.

VAM · 24. August 2016

Danke für die Rückmeldung,

setzt ein return 1; nicht die komplette Funktion aus?

Egal wo?

Wenn ja, habe ich im Code dass wenn man im Anruf ist dass alles abgebrochen wird, und da drinnen sind auch mehrere Returns 1; drinnen.

Code

//////////////////////////////////////////////////////////




    // BIS HIERHIN KLAPPT PRINT     printf("test");

 ///////////////////////////////////////////////////////
	//Anrufen abbrechen




	if(isAmTelen[playerid])
	{
		if(Call_Anrufer[playerid]!=-1)
		{
			if(!PlayerOnline(Call_Anrufer[playerid]))return 1;
			SendClientMessage(Call_Anrufer[playerid],GRAU,"Du hörst nur noch ein Rauschen am anderen Ende der Leitung...");
			isAmTelen[Call_Anrufer[playerid]]=0;
			Call_Anrufer[playerid]=-1;
		}
		else if(Call_Anrufen[playerid]!=-1)
		{
			if(!PlayerOnline(Call_Anrufen[playerid]))return 1;
			SendClientMessage(Call_Anrufen[playerid],GRAU,"Du hörst nur noch ein Rauschen am anderen Ende der Leitung...");
			isAmTelen[Call_Anrufen[playerid]]=0;
			Call_Anrufen[playerid]=-1;
			pInfo[playerid][handykarte]-=gpv(playerid,"MussTeleZahlen");
		}
		isAmTelen[playerid]=0;
	}

Alles anzeigen

Jeffry · 24. August 2016

Wenn es aufgerufen wird (das return), dann ja.
Ändere es zu:
if(isAmTelen[playerid])
{
if(Call_Anrufer[playerid]!=-1)
{
if(PlayerOnline(Call_Anrufer[playerid]))
{
SendClientMessage(Call_Anrufer[playerid],GRAU,"Du hörst nur noch ein Rauschen am anderen Ende der Leitung...");
isAmTelen[Call_Anrufer[playerid]]=0;
Call_Anrufer[playerid]=-1;
}
}
else if(Call_Anrufen[playerid]!=-1)
{
if(PlayerOnline(Call_Anrufen[playerid]))
{
SendClientMessage(Call_Anrufen[playerid],GRAU,"Du hörst nur noch ein Rauschen am anderen Ende der Leitung...");
isAmTelen[Call_Anrufen[playerid]]=0;
Call_Anrufen[playerid]=-1;
pInfo[playerid][handykarte]-=gpv(playerid,"MussTeleZahlen");
}
}
isAmTelen[playerid]=0;
}

VAM · 24. August 2016

Also es ist leider nicht das, sondern der Code ist so ( komplett ) :

Code

if(isAmTelen[playerid])
	{
		if(Call_Anrufer[playerid]!=-1)
		{
			if(!PlayerOnline(Call_Anrufer[playerid]))return 1;
			SendClientMessage(Call_Anrufer[playerid],GRAU,"Du hörst nur noch ein Rauschen am anderen Ende der Leitung...");
			isAmTelen[Call_Anrufer[playerid]]=0;
			Call_Anrufer[playerid]=-1;
		}
		else if(Call_Anrufen[playerid]!=-1)
		{
			if(!PlayerOnline(Call_Anrufen[playerid]))return 1;
			SendClientMessage(Call_Anrufen[playerid],GRAU,"Du hörst nur noch ein Rauschen am anderen Ende der Leitung...");
			isAmTelen[Call_Anrufen[playerid]]=0;
			Call_Anrufen[playerid]=-1;
			pInfo[playerid][handykarte]-=gpv(playerid,"MussTeleZahlen");
		}
		isAmTelen[playerid]=0;
	}


	if(killerid!=INVALID_PLAYER_ID)
	{
		if(!PlayerOnline(killerid))return 1;
		if(pInfo[killerid][fraktion]!=2 && pInfo[killerid][fraktion]!=3)return 1;
		if(Robber[playerid]==1)
		{
			Robber[playerid]=0;
			BankIsRobbed=0;
			BankRobbed=1;
			new str[128];
			format(str,sizeof(str),"* Der Räuber %s wurde vom Beamten %s getötet. Der Bankraub ist gescheitert.",PlayerName(playerid), PlayerName(killerid));
			SendClientMessageToAll(HBLAU,str);
			SetTimer("BankRobReset", 10800*1000, false);
		}
	}

Alles anzeigen

Also soll ich einfach alle "return 1;" entfernen?

Jeffry · 24. August 2016

Ändere es hierzu ab:
if(isAmTelen[playerid])
{
if(Call_Anrufer[playerid]!=-1)
{
if(PlayerOnline(Call_Anrufer[playerid]))
{
SendClientMessage(Call_Anrufer[playerid],GRAU,"Du hörst nur noch ein Rauschen am anderen Ende der Leitung...");
isAmTelen[Call_Anrufer[playerid]]=0;
Call_Anrufer[playerid]=-1;
}
}
else if(Call_Anrufen[playerid]!=-1)
{
if(PlayerOnline(Call_Anrufen[playerid]))
{
SendClientMessage(Call_Anrufen[playerid],GRAU,"Du hörst nur noch ein Rauschen am anderen Ende der Leitung...");
isAmTelen[Call_Anrufen[playerid]]=0;
Call_Anrufen[playerid]=-1;
pInfo[playerid][handykarte]-=gpv(playerid,"MussTeleZahlen");
}
}
isAmTelen[playerid]=0;
}

if(killerid!=INVALID_PLAYER_ID)
{
if(PlayerOnline(killerid))
{
if(pInfo[killerid][fraktion]==2 || pInfo[killerid][fraktion]==3)
{
if(Robber[playerid]==1)
{
Robber[playerid]=0;
BankIsRobbed=0;
BankRobbed=1;
new str[128];
format(str,sizeof(str),"* Der Räuber %s wurde vom Beamten %s getötet. Der Bankraub ist gescheitert.",PlayerName(playerid), PlayerName(killerid));
SendClientMessageToAll(HBLAU,str);
SetTimer("BankRobReset", 10800*1000, false);
}
}
}
}

VAM · 24. August 2016

Danke !! Es klappt! Hatte schonmal ein THread erstellt,aber keiner hat mir geholfen!

breadfish.de 12. März 2022