Spieler Protokoll ~ 10 Einträge pro Dialog Seite

Wichtiger Hinweis: Bitte ändert nicht manuell die Schriftfarbe auf schwarz sondern belasst es bei der Standardeinstellung. Somit tragt ihr dazu bei dass euer Text auch bei Verwendung unseren dunklen Forenstils noch lesbar ist!

Tipp: Ihr wollt längere Codeausschnitte oder Logfiles bereitstellen? Benutzt unseren eigenen PasteBin-Dienst Link
  • Leider führt er den Code nicht ordnungsgemäß aus.
    Auch wenn ich einen fehlenden Datensatz suchen möchte, gibt er mir die Fehlermeldung nicht aus.
    Ich möchte dass das System mir pro Seite im Dialog, 10-20 Einträge Zeigt vom Spieler.
    Ich habe auch gemerkt, das der Code ziemlich Recourcenfressend ist, die Dateigröße wurde auf einen Schwung viel größer.


    Quellcode:

    ocmd:pp(playerid, params[])
    {
    if(pInfo[playerid][pAdmin] < 6)return 1;
    new query[256];
    if(sscanf(params, "s", query))return SCM(playerid, COLOR_GREY, "Befehl: /pp [Name]");
    mysql_format(handle, query, sizeof query, "SELECT * FROM playerprotocol WHERE Name='%e'", query),
    mysql_pquery(handle, query, "PlayerQuery", "d", PLAYER_QUERY_PPROTOCOL);
    return 1;
    }


    case PLAYER_QUERY_PPROTOCOL:
    {
    if(!rows)return SCM(playerid, COLOR_LESSRED, "Es wurde kein Eintrag gefunden.");
    SetPVarInt(playerid, "ProtocolStep", rows);
    new str[1024];
    str = "Datum\tUhrzeut\tInhalt\n";
    for(new i;i<rows;i++)
    {
    cache_get_value_name(i, "Datum", ProtocolRead[playerid][i][0], 600),
    cache_get_value_name(i, "Uhrzeit", ProtocolRead[playerid][i][1], 600),
    cache_get_value_name(i, "Inhalt", ProtocolRead[playerid][i][2], 600);
    if(i <= 9)
    {
    if(strlen(ProtocolRead[playerid][i][2]) > 0)
    {
    format(str, sizeof str, "%s%s\t%s\t%s\n", str, ProtocolRead[playerid][i][0], ProtocolRead[playerid][i][1], ProtocolRead[playerid][i][2]);
    }
    }
    }
    ShowPlayerDialog(playerid, D_PP, DIALOG_STYLE_TABLIST_HEADERS, "Spielerprotokoll", str, "Schließen", "");
    }


    case D_PP:
    {
    if(!response || listitem || GetPVarInt(playerid, "ProtocolStep") >= countProtoStep(playerid))
    {
    SCM(playerid, COLOR_GREY, "Du hast entweder das Fenster geschlossen oder es sind keine weiteren Einträge vorhanden!");
    return 1;
    }
    query = "Datum\tUhrzeit\tInhalt\n";
    for(new k = GetPVarInt(playerid, "ProtocolStep"), i; i<k+10;i++)
    {
    format(query, sizeof query, "%s%s\t%s\t%s", ProtocolRead[playerid][i][0], ProtocolRead[playerid][i][1], ProtocolRead[playerid][i][2]),
    SetPVarInt(playerid, "ProtocolStep", GetPVarInt(playerid, "ProtocolStep") + 1);
    }
    ShowPlayerDialog(playerid, D_PP, DIALOG_STYLE_TABLIST_HEADERS, "Spielerprotokoll", query, "Schließen", "");
    }


    stock countProtoStep(playerid)
    {
    new counter;
    for(new i;i<sizeof ProtocolRead;i++)
    {
    if(strlen(ProtocolRead[playerid][i][2]) > 0)counter ++;
    }
    return counter;
    }

  • Mal 2 ganz blöde Fragen.


    • Nutzt du ProtocolRead nur in diesen Codes?
    • Wie hast du das deklariert?
    • Mal ein paar Hinweise generell wegen der Performanz.
      • Nutze LIMIT in deinen SQL Abfragen. Das ist hier auch der Schlüssel, wie du das elegant hier alles lösen kannst. Dann brauchst du keine komischen if-Abfragen wie in Zeile 22. (Wenn du schon sowas nutzt, dann auch ein break, sonst läuft die Schleife ja sinnlos weiter xD)
      • Deine String-größen sind nicht wirklich optimal gewählt...allein bei deinem CMD kannst du 64, statt 256 verwenden.

    ast2ufdyxkb1.png


    Leute, lernt scripten und versucht mal lieber etwas selber zu schreiben, als es aus einem GF zu kopieren. :S

  • Ja, nutze ProtocolRead nur in diesen Codes.
    Ich habe noch nicht mit LIMIT gearbeitet und daher ist es mir noch nicht im Begriff, wie ich es umsetzen soll.
    Ja ich habe es beim Abschicken auch gemerkt, ich optimiere meine Array Größen dann immer mal wieder im Laufe der Woche an eine ordentliche ausreichende Größe an.


    ProtocolRead[P_MAX][600][3] | So habe ich es deklariert.

  • Also ich bin lange raus, aber ja, sowas tut in meinen Augen weh :D


    Code
    ocmd:pp(playerid, params[])
    {
        if(pInfo[playerid][pAdmin] < 6)return 1;
        new query[MAX_PLAYERS_NAME+1];//Warum 256 statt der maximal zulässigen Länge eines Namen
        if(sscanf(params, "s", query))return SCM(playerid, COLOR_GREY, "Befehl: /pp [Name]");
        mysql_format(handle, query, sizeof query, "SELECT * FROM playerprotocol WHERE Name='%e' LIMIT 10", query),//mit LIMIT kannst du schon im MySQL-Query angeben wie viel Einträge geladen werden sollen
        mysql_pquery(handle, query, "PlayerQuery", "d", PLAYER_QUERY_PPROTOCOL);
    	return 1;
    }


    Du scheinst aber ein wenig Ahnung zu haben, deswegen bin ich mir sicher das du darauf aufbauen kannst. Wenn, du möchtest. :)


    Aus dem Kopf - Ungetestet - Keine Garantie

  • new query[MAX_PLAYERS_NAME+1];
    Also allein da, würde es scheitern. Schicke doch noch damit einen Query weg..
    Ich schaue mir deinen Code morgen mal genauer an, bin jetzt schlafen.. sollte auch nicht um die Uhrzeit am Skript sitzen, das geht immer Schief.

  • @Talykos Jetzt übertreiben wir mal nicht, zumal new query[MAX_PLAYERS_NAME+1]; das auch nicht geht, da der Rest des Querys ja noch drum rum muss.



    ProtocolRead[P_MAX][600][3]

    Das frisst deine Ressourcen.
    Lösche es, du kannst das alles schöner schreiben.


    Versuche es mal so:



    Das scheint mir ein eleganter Weg zu sein! :)


    Schlaf schön ;)

    ast2ufdyxkb1.png


    Leute, lernt scripten und versucht mal lieber etwas selber zu schreiben, als es aus einem GF zu kopieren. :S

  • @Kaliber
    Ich konnte mich Relativ okay in deinen Code hineinversetzen.
    Aktuell funktioniert er, so aber auch nicht, also es passiert nach Eingabe des Commands immer noch nichts.
    Ich bekomme in meiner LogCore eine Meldung.


    SQL
    [11:52:47] [ERROR] signal 2 (SIGINT) catched; shutting log-core down (errno: 0, signal code: 128, exit status: 141512)
  • Pack mal crashdetect rein und zeig mal deine Tabellen Struktur :)


    Und hast du mal Beispieldaten? Also mich interessiert die Länge der einzelnen Felder...


    //Edit:


    PS: Zeile 21 mal so schreiben:


    ShowPlayerDialog(playerid, D_PP, DIALOG_STYLE_TABLIST_HEADERS, "Spielerprotokoll", str, "Schließen", ((rows%10 == 0)?("Weiter"):("")));

    ast2ufdyxkb1.png


    Leute, lernt scripten und versucht mal lieber etwas selber zu schreiben, als es aus einem GF zu kopieren. :S

  • Ahhh du speicherst die Sekunden :D


    Daran habe ich nicht gedacht, schreibe mal den Mittleren Teil so:


    ast2ufdyxkb1.png


    Leute, lernt scripten und versucht mal lieber etwas selber zu schreiben, als es aus einem GF zu kopieren. :S

  • Hmmm, dann schreib es mal so:



    Ich bin mir ziemlich sicher, dass es ein Größen Problem ist.

    ast2ufdyxkb1.png


    Leute, lernt scripten und versucht mal lieber etwas selber zu schreiben, als es aus einem GF zu kopieren. :S

  • Dann setze mal prints und schau, wo er sich aufhängt :o
    Und aktivier evtl mal den Mysql Log


    //Edit:
    Habe es mal bei mir lokal getestet und es funzt einwandfrei.
    Bekomme keine Fehler oder dergleichen und habe exakt deine Datenstruktur gewählt.


    Das war der Code, den ich verwendet habe:



    Aber mir ist schleierhaft wie du die playerid an das PlayerQuery übergibst (Zeile 36)


    und wie fragst du die rows ab?


    Ich kenne halt nicht all deinen Code.


    An dem Stück Code liegt der Fehler zumindest nicht :)

    ast2ufdyxkb1.png


    Leute, lernt scripten und versucht mal lieber etwas selber zu schreiben, als es aus einem GF zu kopieren. :S

    Einmal editiert, zuletzt von Kaliber ()

  • @Kaliber
    So ein leichtsinnsfehler.. bei dir hat es gleich funktioniert .. weil du ID 0 bist und dein Enumfeld auch das erste ist.
    Ich hab halt einfach die playerid nicht mit angegeben.
    Aktuell funktioniert es, aber rückwärts. Sprich, wenn ich auf weiter klicke & es sind noch Einträge vorhanden, dann sagt er ist nichts mehr da.
    Wenn aber noch was zu lesen da ist & ich auf Schließen/Zurück gehe.. dann zeigt er mir die nächsten Einträge.