If-Abfrage funktioniert nicht?

Hallo

Wo wird duelstate = 1 gesetzt ? Poste bitte den Code mal, danke.

MfG dUDA

public OnPlayerDeath(playerid, killerid, reason)
{
print("Punkt 1");
SpielerInfo[playerid][Death]++;
SpielerInfo[killerid][Kills]++;
if(duelstate[playerid]==1)
{
print("Punkt 2");
SpielerInfo[killerid][duelkills]++;
SpielerInfo[playerid][dueldeaths]++;
SendClientMessage(playerid,ROT,"Sie sind gestorben!");
SendClientMessage(killerid,ROT,"Sie haben getötet!");
}
print("Punkt 3");
return 1;
}

Einmal bitte den Code ausführen und sagen, was geprintet wird.

Hallo

Eine Frage, setzt Du die Gesundheit des Spielers auf 0 ?

dUDA

public OnPlayerDeath(playerid, killerid, reason)
{
printf("Debug #1");
SpielerInfo[playerid][Death]++;
SpielerInfo[killerid][Kills]++;
printf("Debug #2 %d",duelstate[playerid]);
if(duelstate[playerid]==1)
{
printf("Debug #3");
if(killerid != INVALID_PLAYER_ID) {
SpielerInfo[killerid][duelkills]++; //Hier könnte es zu einem Crash kommen
}
SpielerInfo[playerid][dueldeaths]++;
printf("Debug #4");
SendClientMessage(playerid,ROT,"Sie sind gestorben!");
SendClientMessage(killerid,ROT,"Sie haben getötet!");
printf("Debug #5");
}
printf("Debug #6");
return 1;
}
Wenn die meisten mal anfangen würden mit Debug-Notes zu arbeiten, würde sie 1) ihr Fehler selber lösen und 2) das auch viel schneller.

Mach es genau wie Goldkiller sonst hat es kein Sinn!

Dann ist killerid INVALID_PLAYER_ID und es crasht.
Hab es dort völlig übersehen:
public OnPlayerDeath(playerid, killerid, reason)
{
printf("Debug #1");
SpielerInfo[playerid][Death]++;
if(killerid != INVALID_PLAYER_ID) {
SpielerInfo[killerid][Kills]++;
}
printf("Debug #2 %d",duelstate[playerid]);
if(duelstate[playerid]==1)
{
printf("Debug #3");
if(killerid != INVALID_PLAYER_ID) {
SpielerInfo[killerid][duelkills]++; //Hier könnte es zu einem Crash kommen
}
SpielerInfo[playerid][dueldeaths]++;
printf("Debug #4");
SendClientMessage(playerid,ROT,"Sie sind gestorben!");
SendClientMessage(killerid,ROT,"Sie haben getötet!");
printf("Debug #5");
}
printf("Debug #6");
return 1;
}
INVALID_PLAYER_ID ist 2 ^ 16 ( 65536 ). Dein Array hat aber niemals so eine größe.Deswegen ist das "Out of Bounce" und an der Stelle crasht / stoppt dein Script und es geht nicht weiter.

//Edit:
Die Abfrage mit INVALID_PLAYER_ID ist sehr sehr wichtig unter OnPlayerDeath! Ohne die Abfrage kommt es oft zu problemen,zB wenn der Spieler Selsbtmord begangen hat.Die killerid ist dann INVALID_PLAYER_ID.Was dann passiert, siehst ja an deinem Code ;).

public OnPlayerDeath(playerid, killerid, reason)
{
printf("Debug #1");
SpielerInfo[playerid][Death]++;
if(killerid != INVALID_PLAYER_ID) {
SpielerInfo[killerid][Kills]++;
}
printf("Debug #2 %d",duelstate[playerid]);
if(duelstate[playerid]==1)
{
printf("Debug #3");
if(killerid != INVALID_PLAYER_ID) {
SpielerInfo[killerid][duelkills]++; //Hier könnte es zu einem Crash kommen
SendClientMessage(killerid,ROT,"Sie haben getötet!");
}
SpielerInfo[playerid][dueldeaths]++;
printf("Debug #4");
SendClientMessage(playerid,ROT,"Sie sind gestorben!");
printf("Debug #5");
}
printf("Debug #6");
return 1;
}

Das muss noch rein

Kannst alles nachlesen in a_samp.inc
#define MAX_PLAYER_NAME (24)
#define MAX_PLAYERS (500)
#define MAX_VEHICLES (2000)
#define INVALID_PLAYER_ID (0xFFFF)
#define INVALID_VEHICLE_ID (0xFFFF)
#define NO_TEAM (255)
#define MAX_OBJECTS (400)
#define INVALID_OBJECT_ID (0xFFFF)
#define MAX_GANG_ZONES (1024)
#define MAX_TEXT_DRAWS (2048)
#define MAX_MENUS (128)
#define MAX_3DTEXT_GLOBAL (1024)
#define MAX_3DTEXT_PLAYER (1024)
#define MAX_PICKUPS (2048)
#define INVALID_MENU (0xFF)
#define INVALID_TEXT_DRAW (0xFFFF)
#define INVALID_GANG_ZONE (-1)
#define INVALID_3DTEXT_ID (0xFFFF)

Da SendClientMessage sowieso überprüft ob die playerid gültig ist,ist es sowieso egal ob das im Check ist oder nicht.Der Übersicht halber sollte es aber trotzdem in den Check :p