SAMP Konsolen Debug Warnings

German_ScripterHD · 31. Dezember 2015

Hallo Brotfische,

Bei OnPlayerDeath bekomme ich immer Warnings (Wird unten angezeigt)... genau wie in "??" leider weiß ich nicht wo der Fehler liegt.

Warning Codes:

C

[23:41:06] [debug] Run time error 4: "Array index out of bounds"
[23:41:06] [debug]  Accessing element at negative index -1
[23:41:06] [debug] AMX backtrace:
[23:41:06] [debug] #0 0007325c in ?? (0, 65535, 54) from script.amx
[23:41:06] [debug] #1 0000f2c0 in public OnPlayerDeath (0, 65535, 54) from script.amx
[23:43:09] [debug] Run time error 4: "Array index out of bounds"
[23:43:09] [debug]  Accessing element at negative index -1
[23:43:09] [debug] AMX backtrace:
[23:43:09] [debug] #0 0007325c in ?? (1, 65535, 54) from script.amx
[23:43:09] [debug] #1 0000f2c0 in public OnPlayerDeath (1, 65535, 54) from script.amx

OnPlayerDeath:

Spoiler anzeigen

C

public OnPlayerDeath(playerid, killerid, reason)
{
	new Float:tot_x,Float:tot_y,Float:tot_z;
	tot[playerid] = 1;
	giveplayermoney(playerid,-GetPlayerScore(playerid)*50);
	new namenstring[200];
	format(namenstring,sizeof(namenstring),"†\n%s",getPlayerName(playerid));
	if(GetPlayerVirtualWorld(playerid)==0)
	{
		if(GetPlayerInterior(playerid)==0)
		{
			for(new l = 0; l < MAX_LEICHEN; l++)
			{
				if(leichenInfo[l][existiert] == 1) continue;
				GetPlayerPos(playerid,tot_x,tot_y,tot_z);
				leichenInfo[l][l_x] = tot_x;
				leichenInfo[l][l_y] = tot_y;
				leichenInfo[l][l_z] = tot_z;
				leichenInfo[l][pickup] = CreatePickup(1254,1,leichenInfo[l][l_x],leichenInfo[l][l_y],leichenInfo[l][l_z],0);
				leichenInfo[l][NamenLabel] = Create3DTextLabel(namenstring,C_Weiß,leichenInfo[l][l_x],leichenInfo[l][l_y],leichenInfo[l][l_z],10.0,0,0);
				leichenInfo[l][existiert] = 1;
				break;
			}
		}
	}
	new Float: KrankenhausDistance[8],Float: tmp_distance = 9999999.0;
	KrankenhausDistance[0] = GetPlayerDistanceFromPoint(playerid, 2034.1149,-1414.9154,16.9922);
	KrankenhausDistance[1] = GetPlayerDistanceFromPoint(playerid, 1182.1077,-1324.0918,13.5815);
	KrankenhausDistance[2] = GetPlayerDistanceFromPoint(playerid, 1244.3423,331.9934,19.5547);
	KrankenhausDistance[3] = GetPlayerDistanceFromPoint(playerid, 1607.6090,1821.0934,10.8280);
	KrankenhausDistance[4] = GetPlayerDistanceFromPoint(playerid, -317.0090,1056.6217,19.7422);
	KrankenhausDistance[5] = GetPlayerDistanceFromPoint(playerid, -1514.7358,2527.0601,55.7455);
	KrankenhausDistance[6] = GetPlayerDistanceFromPoint(playerid, -2661.2441,633.7098,14.4531);
	KrankenhausDistance[7] = GetPlayerDistanceFromPoint(playerid, -2198.7803,-2306.5793,30.6250);


	for (new i; i != sizeof (KrankenhausDistance); ++i)
	{
		if (KrankenhausDistance[i] < tmp_distance)
		{
			tmp_distance = KrankenhausDistance[i];
			SetPVarInt(playerid, "i", i);
		}
	}
    new string1[40],string2[30];
	new gangzone = IsPlayerInGangZone(playerid);
	if(gwInfo[gangzone][running] == 1) {
		if(sInfo[playerid][fraktion] == gwInfo[gangzone][angreifer] && sInfo[killerid][fraktion] == gwInfo[gangzone][besitzer]) {
			gwInfo[gangzone][punkteVerteidiger]++;
			SendClientMessageToAllEx(-1,"gwInfo[gangzone][punkteVerteidiger]= %d",gwInfo[gangzone][punkteVerteidiger]);
			SendClientMessageToAll(-1,"Punkte wurden angezeigt....");
			format(string1,sizeof(string1),"%d",gwInfo[gangzone][punkteVerteidiger]);
			for(new i; i < MAX_PLAYERS; i++)
			{
				if(sInfo[i][fraktion] == gwInfo[gangzone][angreifer] && sInfo[i][fraktion] == gwInfo[gangzone][besitzer])
	  			{
					PlayerTextDrawSetString(i,GangTD[i][PunkteFraktion1],string1);
				}
			}
		}
		else if(sInfo[playerid][fraktion] == gwInfo[gangzone][besitzer] && sInfo[killerid][fraktion] == gwInfo[gangzone][angreifer]) {
			gwInfo[gangzone][punkteAngreifer]++;
			SendClientMessageToAllEx(-1,"gwInfo[gangzone][punkteAngreifer]= %d",gwInfo[gangzone][punkteAngreifer]);
			SendClientMessageToAll(-1,"Punkte wurden angezeigt....");
			format(string2,sizeof(string2),"%d",gwInfo[gangzone][punkteAngreifer]);
			for(new i; i < MAX_PLAYERS; i++)
			{
				if(sInfo[i][fraktion] == gwInfo[gangzone][besitzer] || sInfo[i][fraktion] == gwInfo[gangzone][angreifer])
	  			{
					PlayerTextDrawSetString(i,GangTD[i][PunkteFraktion2],string2);
				}
			}
		}
		return 1;
	}
	return 1;
}

Alles anzeigen

Ich hoffe es kann mir jemand helfen

MfG

Exod_Martinez · 31. Dezember 2015

Erstell mal in deinem Pawno Ordner eine "pawn.cfg" datei mit dem Inhalt "-d3", starte Pawno neu, Compile nochmal und poste deine neuen Logs

German_ScripterHD · 31. Dezember 2015

Habe ich gemacht, als ich j

Zitat von Exod_Martinez

Erstell mal in deinem Pawno Ordner eine "pawn.cfg" datei mit dem Inhalt "-d3", starte Pawno neu, Compile nochmal und poste deine neuen Logs

Danke für deine Hilfe Ich konnte Dank deiner Antwort den Fehler selber beheben.

//Edit:

Habe noch ein kleines Problem...
Immer wenn ich /slay eintippe bekomme ich folgendes Fehler in der Konsole angezeigt.

C

[01:50:11] sscanf warning: Strings without a length are deprecated, please add a destination size.

An was liegt das?

Code:

C

ocmd:slay(playerid, params[])
{
	if(!isAdmin(playerid,3)) return SCM(playerid, C_Error, noaccess);
	new pID,grund[30];
	if(sscanf(params,"us",pID,grund)) return SCM(playerid,C_Weiß,"/slay [id] [grund]");
	if(sInfo[playerid][alevel]<sInfo[pID][alevel])return SCM(playerid,C_Error,"Der Spieler hat ein höheres Adminlevel.");
	SetPlayerHealth(pID,0-100);
	SCMToAllEx(COLOR_WHITE,"{F98B05}** {FFFFFF}%s {F98B05}wurde von {FFFFFF}%s %s {F98B05}getötet.",getPlayerName(pID),AdminLevelName(playerid),getPlayerName(playerid));
	SCMToAllEx(COLOR_WHITE,"{F98B05}** Grund: {FFFFFF}%s",grund);
	TogglePlayerControllable(pID,1);
	return 1;
}

Alles anzeigen

Firerfan · 31. Dezember 2015

Du musst bei Strings die Du mit SSCANF splitten möchtest die Länge des Strings angeben. Das machst du folgendermaßen:
sscanf(sparams,"us[LÄNGE],pID,grund)

Mit freundlichen Grüßen

German_ScripterHD · 31. Dezember 2015

Vielen Dank

**XeroX** · 31. Dezember 2015

Bei OnPlayerDeath wird nicht überprüft ob ein Spieler für den Tod verantworlich war. if(killerid != INVALID_PLAYER_ID)

German_ScripterHD · 31. Dezember 2015

Zitat von XeroX

Bei OnPlayerDeath wird nicht überprüft ob ein Spieler für den Tod verantworlich war. if(killerid != INVALID_PLAYER_ID)

Achso.. und wo füge ich das genau ein?

**XeroX** · 31. Dezember 2015

bevor du killerid in einem array verwendest im Beitrag oben von dir also etwa Zeile 43

German_ScripterHD · 31. Dezember 2015

Zitat von XeroX

bevor du killerid in einem array verwendest im Beitrag oben von dir also etwa Zeile 43

Ich Danke dir

Simon · 31. Dezember 2015

Vielleicht hilfreich zukünftig: Die meldung "Array index out of Bounds" bedeutet, dass der Wert eines Indexes eines Arrays ungültig ist.

Wenn du also einen Array hast:

Krankenhausdistance[10];

Dann sind 0-9 gültig.
Alles unter 0 ist grunsätzlich ungültig. Alles ab der größe des Arrays (10 in diesem Fall) ist ebenso ungültig.

Oftmals ist eine bestimmte variable nicht vergeben (INVALID), wie es zumbeispiel bei der killerid der Fall ist wenn man stirbt. Nicht immer muss man gekillt werden, man kann auch einfach von einer explosion oder durch fallen sterben. in solchen Fällen gibt es keine killerid und der Wert von killerid ist = INVALID_PLAYER_ID.
Wenn du nun trotzdem killerid in einem Array benutzt, ist der Index ungültig.

Also als Tipp: Achte immer darauf, wann welche Variablen welchen Wert annehmen können, und wie du sie dann entsprechend verwendest. Wie bereits erwähnt kann man vorher abfragen ob es einen Killer gibt, bevor dann weiters passieren soll.

breadfish.de 12. März 2022