Admin-Prison: Spieler Variable bei einer IF-Abfrage funktioniert nicht richtig

palwal · 25. Dezember 2019

Es gibt gewisse unlogische Fehler die keinen Sinn ergeben. Allerdings weiss ich nicht mehr weiter.

Zur Schilderung:

Ich habe ein Admin-Prison, wenn ein Spieler ins Prison kommt, erhält die SpielerVariable pPrison den Wert 1 (Und im folgenden entsteht ein Laufprison, für den der Wert 2 vorgesehen ist (Für die Logik)).

Allerdings wenn der Spieler ins Prison kommt, den Wert pPrison 1 hat, wird diese Anweisung nach der If-Abfrage ignoriert und der Spieler wird zu den Koordinaden von SetSpawnInfo (Was am Anfang vom Login definiert wurde) teleportiert.

Code

SpielerInfo[playerid][pPrison] > 0

Das komische hierbei ist, sobald ich die If-Abfrage in

Code

SpielerInfo[playerid][pPrison] == 0

ändere und der Spieler nicht im Prison ist, wird meine Anweisung dahinter ausgeführt. Ich habe das gleiche mit SpielerInfo[playerid][pPrison] == 1 versucht wenn der Spieler ins Prison geschickt wurde, allerdings wird dann die If-Anweisung gänzlich ignoriert.

Vielleicht habe ich einen Fehler, der auch ein Denkfehler sein könnte, aber ich hoffe ihr Brotfische könntet mir helfen

PS: Die Int Variable pPrison ist wirklich auf 1 wenn der Spieler ins Prison gesteckt wurde. Aber trotz allen werden nur meine Debug-Texte Dev: 1 & Dev: 4 angezeigt beim Spawn

Jeffry

Code

#include <YSI\y_hooks>

hook OnPlayerSpawn(playerid) {
    new string[200], year, month, day, hour, minute, second;
    if(!SpielerInfo[playerid][pLogged]) return 1;
    SendClientMessage(playerid, -1, "DEV: 1");
    format(string, 200, "Prison: %i, Sek: %i, T: %i", SpielerInfo[playerid][pPrison], SpielerInfo[playerid][pPrisonS], SpielerInfo[playerid][pPrisonT]);
    SendClientMessage(playerid, -1, string);
    format(string, 200, "Grund: %s, Admin: %s", SpielerInfo[playerid][pPrisonR], SpielerInfo[playerid][pPrisonA]);
    SendClientMessage(playerid, -1, string);
    
    if(SpielerInfo[playerid][pPrison] > 0) {
        TimestampToDate(SpielerInfo[playerid][pPrisonT], year, month, day, hour, minute, second, 1);
          format(string, sizeof(string), "%s", SpielerInfo[playerid][pPrisonA]);
        PlayerTextDrawSetString(playerid, prisonadmin[playerid], string);
        format(string, sizeof(string), "%s", SpielerInfo[playerid][pPrisonR]);
        PlayerTextDrawSetString(playerid, prisonreason[playerid], string);
        format(string, sizeof(string), "%02d.%02d.%04d / %d:%d Uhr", day, month, year, hour, minute);
        PlayerTextDrawSetString(playerid, prisonreason[playerid], string);
SendClientMessage(playerid, -1, "DEV: 2");
        if(SpielerInfo[playerid][pPrison] == 1) {
            SetPlayerPos(playerid,1510.4912,-1795.7566,-94.1283);
            SetPlayerFacingAngle(playerid,267.6534);
            SetCameraBehindPlayer(playerid);
            SendClientMessage(playerid, COLOR_DARKRED, "Du bist wegen einem Regelverstoß im Adminprison!!");

            format(string, sizeof(string), "Du bist noch für %i Sekunden im Prison!", SpielerInfo[playerid][pPrisonS]);
            PlayerTextDrawSetString(playerid, prison0[playerid], string);
        }
        SendClientMessage(playerid, -1, "DEV: 3");

        PlayerTextDrawShow(playerid, prison1[playerid]);
        PlayerTextDrawShow(playerid, prison2[playerid]);
        PlayerTextDrawShow(playerid, prison3[playerid]);
        PlayerTextDrawShow(playerid, prison0[playerid]);
        PlayerTextDrawShow(playerid, prisonadmin[playerid]);
        PlayerTextDrawShow(playerid, prisonreason[playerid]);
        PlayerTextDrawShow(playerid, prisontime[playerid]);
        return 1;
    } else if(SpielerInfo[playerid][pSpawnChange] == 0) {
        SetPlayerPos(playerid, 1716.6198,-1686.0006,20.2172);
        SetPlayerFacingAngle(playerid, 183.6793);
        SetCameraBehindPlayer(playerid);
        SendClientMessage(playerid, -1, "DEV: 4");
        return 1;
    }
    SendClientMessage(playerid, -1, "DEV: 5");
    return 1;
}

Alles anzeigen

**Andosius** · 26. Dezember 2019

Bist du sicher, dass auch alles wirklich korrekt geladen wird?

Lass dir mal die Stellen ausgeben wo du die Value setzt und überprüfe die auch beim beim spawn, ob sie gleich geblieben ist.

Falls nicht musst du sie folglich irgendwo überschreiben, weshalb der Code nicht funktioniert.

Grundsätzlich müsste der Code super funktionieren, habe eben noch paar Kommentare der Übersicht halber rein gemacht.

Noch eine Verständnisfrage: pPrison = 1 wenn der Spieler seine Zeit im Prison verbüßt, ansonsten 0?

Code

#include <YSI\y_hooks>

hook OnPlayerSpawn(playerid) {
    new string[200], year, month, day, hour, minute, second;
    if(!SpielerInfo[playerid][pLogged]) return 1;

    // DEBUG 1 - Spieler ist eingeloggt
    SendClientMessage(playerid, -1, "DEV: 1");

    format(string, 200, "Prison: %i, Sek: %i, T: %i", SpielerInfo[playerid][pPrison], SpielerInfo[playerid][pPrisonS], SpielerInfo[playerid][pPrisonT]);
    SendClientMessage(playerid, -1, string);
    format(string, 200, "Grund: %s, Admin: %s", SpielerInfo[playerid][pPrisonR], SpielerInfo[playerid][pPrisonA]);
    SendClientMessage(playerid, -1, string);
    
    if(SpielerInfo[playerid][pPrison] > 0) {

        TimestampToDate(SpielerInfo[playerid][pPrisonT], year, month, day, hour, minute, second, 1);
        format(string, sizeof(string), "%s", SpielerInfo[playerid][pPrisonA]);
        PlayerTextDrawSetString(playerid, prisonadmin[playerid], string);
        format(string, sizeof(string), "%s", SpielerInfo[playerid][pPrisonR]);
        PlayerTextDrawSetString(playerid, prisonreason[playerid], string);
        format(string, sizeof(string), "%02d.%02d.%04d / %d:%d Uhr", day, month, year, hour, minute);
        PlayerTextDrawSetString(playerid, prisonreason[playerid], string);

        // DEBUG 2 - Spieler sitzt in irgendeiner Art Gefängnis, daher werden Sachen formatiert
        SendClientMessage(playerid, -1, "DEV: 2");

        if(SpielerInfo[playerid][pPrison] == 1) {

            SetPlayerPos(playerid,1510.4912,-1795.7566,-94.1283);
            SetPlayerFacingAngle(playerid,267.6534);
            SetCameraBehindPlayer(playerid);
            SendClientMessage(playerid, COLOR_DARKRED, "Du bist wegen einem Regelverstoß im Adminprison!!");

            format(string, sizeof(string), "Du bist noch für %i Sekunden im Prison!", SpielerInfo[playerid][pPrisonS]);
            PlayerTextDrawSetString(playerid, prison0[playerid], string);

        }

        // DEBUG 3 - Spieler-Abfrage ob pPrison = 1 passed through
        SendClientMessage(playerid, -1, "DEV: 3");

        PlayerTextDrawShow(playerid, prison1[playerid]);
        PlayerTextDrawShow(playerid, prison2[playerid]);
        PlayerTextDrawShow(playerid, prison3[playerid]);
        PlayerTextDrawShow(playerid, prison0[playerid]);
        PlayerTextDrawShow(playerid, prisonadmin[playerid]);
        PlayerTextDrawShow(playerid, prisonreason[playerid]);
        PlayerTextDrawShow(playerid, prisontime[playerid]);
        return 1;
    } else {
        if(SpielerInfo[playerid][pSpawnChange] == 0) {

            SetPlayerPos(playerid, 1716.6198,-1686.0006,20.2172);
            SetPlayerFacingAngle(playerid, 183.6793);
            SetCameraBehindPlayer(playerid);

            // DEBUG 4 - Alles wird übersprungen, Spieler sieht lediglich "DEV: 1"
            SendClientMessage(playerid, -1, "DEV: 4");
            return 1;
        }
    }
    return 1;
}

Alles anzeigen

palwal · 26. Dezember 2019

Das ist ja das komische, da überprüfe ich ja nochmal die Werte der Variablen

Code

SendClientMessage(playerid, -1, "DEV: 1");
    format(string, 200, "Prison: %i, Sek: %i, T: %i", SpielerInfo[playerid][pPrison], SpielerInfo[playerid][pPrisonS], SpielerInfo[playerid][pPrisonT]);
    SendClientMessage(playerid, -1, string);
    format(string, 200, "Grund: %s, Admin: %s", SpielerInfo[playerid][pPrisonR], SpielerInfo[playerid][pPrisonA]);
    SendClientMessage(playerid, -1, string);

und pPrison hat den Wert 1, aber führt die If-Anweisung nicht aus

Jeffry · 26. Dezember 2019

Versuche bitte mal folgendes:

Code

if(SpielerInfo[playerid][pPrison] > 0) {
    TimestampToDate(SpielerInfo[playerid][pPrisonT], year, month, day, hour, minute, second, 1);

zu:

Code

printf("Vor Abfrage: pPrison: %d", SpielerInfo[playerid][pPrison]);
if(SpielerInfo[playerid][pPrison] > 0) {
    printf("Nach Abfrage: pPrison: %d", SpielerInfo[playerid][pPrison]);
    TimestampToDate(SpielerInfo[playerid][pPrisonT], year, month, day, hour, minute, second, 1);
    printf("Nach TimestampToDate: pPrison: %d", SpielerInfo[playerid][pPrison]);

Was gibt dir der Server Log dann aus?

palwal · 26. Dezember 2019

Zitat von Jeffry

Versuche bitte mal folgendes:

Code

if(SpielerInfo[playerid][pPrison] > 0) {
    TimestampToDate(SpielerInfo[playerid][pPrisonT], year, month, day, hour, minute, second, 1);

zu:

Code

printf("Vor Abfrage: pPrison: %d", SpielerInfo[playerid][pPrison]);
if(SpielerInfo[playerid][pPrison] > 0) {
    printf("Nach Abfrage: pPrison: %d", SpielerInfo[playerid][pPrison]);
    TimestampToDate(SpielerInfo[playerid][pPrisonT], year, month, day, hour, minute, second, 1);
    printf("Nach TimestampToDate: pPrison: %d", SpielerInfo[playerid][pPrison]);

Was gibt dir der Server Log dann aus?

Server-Log:

Code

[14:57:56] Vor Abfrage: pPrison: 0
[14:57:56] Vor Abfrage: pPrison: 0
[14:58:05] sscanf warning: Strings without a length are deprecated, please add a destination size.
[14:58:05] Vor Abfrage: pPrison: 1
[14:58:05] Nach Abfrage: pPrison: 1
[14:58:05] [debug] Run time error 4: "Array index out of bounds"
[14:58:05] [debug]  Attempted to read/write array element at index 12 in array of size 12
[14:58:05] [debug] AMX backtrace:
[14:58:05] [debug] #0 0003ae3c in ?? (3485, 19835168, 19835164, 19835160, 19835156, 19835152, 19835148, 1, 0) in vortex.amx
[14:58:05] [debug] #1 000504e4 in ?? (0) in vortex.amx
[14:58:05] [debug] #2 00014938 in public OnPlayerSpawn (0) in vortex.amx

Alles anzeigen

Jeffry · 26. Dezember 2019

Das dachte ich mir schon

Das TimestampToDate hat da einen Fehler, diesen habe ich im englischen Forum auch mal angemerkt.

Ändere das in der Include, dann sollte alles klappen.

https://forum.sa-mp.com/showthread.php?p=3807026#post3807026

Spoiler anzeigen

palwal · 26. Dezember 2019

Jeffry es geeeehhhht! Nur warum TimestampToDate so ein Einfluss hatte ist mit schleierhaft^^

Was wäre SAMP ohne unseren Jeffry

Jeffry · 26. Dezember 2019

Gerne

Zitat von palwal

Nur warum TimestampToDate so ein Einfluss hatte ist mit schleierhaft^^

Der Grund ist, dass TimestampToDate an der Stelle das Array sprengt, daher auch die Fehlermeldung vom Crashdetect.

Das Array MonthTimes ist 12 Zellen groß (es sind ja 12 Monate). Das bedeutet, dass man mit den Indizes 0 bis 11 darauf zugreifen kann. Der Fehler in der Include ist aber, dass mit dem Monat darauf zugegriffen wird (sprich 1 bis 12). Somit gibt es immer im Dezember Fehler, die durch die TimestampToDate Include auftreten, da dort das Array gesprengt wird und der Code an der Stelle aufhört weiter zu laufen (daher hat er bei dir nie etwas gemacht).

breadfish.de 12. März 2022