Spieler Protokoll ~ 10 Einträge pro Dialog Seite

**Kasakow** · 17. August 2019

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;
}

Kaliber · 18. August 2019

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.

**Kasakow** · 18. August 2019

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.

**GangstaSunny** · 18. August 2019

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

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;
}

Code

case PLAYER_QUERY_PPROTOCOL:
{
	if(!rows)return SCM(playerid, COLOR_LESSRED, "Es wurde kein Eintrag gefunden.");
	SetPVarInt(playerid, "ProtocolStep", 1);//Statt "rows" machen wir hier jetzt 1, Also die Seiten (1, 2, 3 4 etc).
	query = "Datum\tUhrzeut\tInhalt\n";//Ich sehe bei D_PP kein "new query" also denke ich mal das du query global definiert hast, warum also nen neuen 1024er string?
	for(new i;i<rows;i++)
	{
 		cache_get_value_name(i, "Datum", ProtocolRead[playerid][i][0], 12),//600 Zeichen dürfte das Datum wohl nicht lang sein
   		cache_get_value_name(i, "Uhrzeit", ProtocolRead[playerid][i][1], 12),//genau so wie bei der Uhrzeit - ich weis nicht wie du abspeicherst, deswegen mal 12
	    cache_get_value_name(i, "Inhalt", ProtocolRead[playerid][i][2], 600);//hier solltest du auch mal schauen ob gespeicherte Nachrichten wirklich 600 Zeichen lang sein werden
 		format(query, sizeof query, "%s%s\t%s\t%s\n", query, ProtocolRead[playerid][i][0], ProtocolRead[playerid][i][1], ProtocolRead[playerid][i][2]);
		//Da du eh nur 10 lädst, brauchst du keine Abfrage mehr
	}
	ShowPlayerDialog(playerid, D_PP, DIALOG_STYLE_TABLIST_HEADERS, "Spielerprotokoll", str, "Schließen", "");
}

Alles anzeigen

Code

case D_PP:
{
	if(!response || listitem)
	{
 		SCM(playerid, COLOR_GREY, "Dialog geschlossen.");//Wir sagen es ihm/ihr gleich, wenn es keine Einträge mehr gibt :)
   		return 1;
	}
	query = "Datum\tUhrzeit\tInhalt\n";
	SetPVarInt(playerid, "ProtocolStep", GetPVarInt(playerid, "ProtocolStep") + 1);//Wir zählen die zeiten. 1, 2, 3 usw.
	new newsite = GetPVarInt(playerid, "ProtocolStep") * 10); //Wäre es z.b 2, dann würde die var jetzt 20 sein, bei drei 30 etc.
	mysql_format(handle, query, sizeof query, "SELECT * FROM playerprotocol WHERE Name='%e' LIMIT 10, 20", query),
    mysql_pquery(handle, query, "PlayerQuery", "d", PLAYER_QUERY_PPROTOCOL);
    if(cache_num_rows() < 1)
    {
        SCM(playerid, COLOR_GREY, "Es sind keine weiteren Einträge verfügbar.");
   		return 1;
    }
    for(new i;i<10;i++)
    {
        cache_get_value_name(i, "Datum", ProtocolRead[playerid][i+newsite][0], 12),
   		cache_get_value_name(i, "Uhrzeit", ProtocolRead[playerid][i+newsite][1], 12),
	    cache_get_value_name(i, "Inhalt", ProtocolRead[playerid][i+newsite][2], 600);
 		format(query, sizeof query, "%s%s\t%s\t%s\n", query, ProtocolRead[playerid][i][0], ProtocolRead[playerid][i][1], ProtocolRead[playerid][i][2]);
    }
    ShowPlayerDialog(playerid, D_PP, DIALOG_STYLE_TABLIST_HEADERS, "Spielerprotokoll", query, "Schließen", "");
}

Alles anzeigen

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

**Kasakow** · 18. August 2019

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.

Kaliber · 18. August 2019

@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.

Zitat von Kasakow

ProtocolRead[P_MAX][600][3]

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

Versuche es mal so:

C

ocmd:pp(playerid, params[])
{
    if(pInfo[playerid][pAdmin] < 6)return 1;
    if(IsNull(params))return SCM(playerid, COLOR_GREY, "Befehl: /pp [Name]");
    SetPVarString(playerid,"ProtocolName",params);
    ShowPlayerProtocol(playerid,0);
    return 1;
}
case PLAYER_QUERY_PPROTOCOL:
{
    if(!rows) return SCM(playerid, COLOR_LESSRED, "Es wurde kein Eintrag gefunden.");
    SetPVarInt(playerid,"ProtocolRows",rows);
    new str[(128+11+6)*10] = "Datum\tUhrzeit\tInhalt\n";
    for(new i,date[11],time[6],data[128]; i<rows; i++)
    {
        cache_get_value_name(i, "Datum", date, sizeof(date)),
        cache_get_value_name(i, "Uhrzeit", time, sizeof(time)),
        cache_get_value_name(i, "Inhalt", data, sizeof(data));
        if(data[0] != EOS) format(str, sizeof str, "%s%s\t%s\t%s\n", str, date, time, data);
    }
    ShowPlayerDialog(playerid, D_PP, DIALOG_STYLE_TABLIST_HEADERS, "Spielerprotokoll", str, "Schließen", "");
}


case D_PP:
{
    new step = GetPVarInt(playerid, "ProtocolRows"), end = GetPVarInt(playerid,"ProtocolEnd");
    DeletePVar(playerid,"ProtocolRows"),DeletePVar(playerid,"ProtocolEnd");
    if(!response || listitem || step%10 != 0)
    {
        DeletePVar(playerid,"ProtocolName");
        return SCM(playerid, COLOR_GREY, "Du hast entweder das Fenster geschlossen oder es sind keine weiteren Einträge vorhanden!");
    }
    ShowPlayerProtocol(playerid,end+10);
}


stock ShowPlayerProtocol(playerid,start)
{
    new query[64 + MAX_PLAYER_NAME];
    GetPVarString(playerid,"ProtocolName",query,MAX_PLAYER_NAME);
    mysql_format(handle, query, sizeof query, "SELECT * FROM playerprotocol WHERE Name='%e' LIMIT %d,10", query,start),
    mysql_pquery(handle, query, "PlayerQuery", "d", PLAYER_QUERY_PPROTOCOL);
    return SetPVarInt(playerid, "ProtocolEnd", start),1;
}

Alles anzeigen

Das scheint mir ein eleganter Weg zu sein!

Schlaf schön

**GangstaSunny** · 18. August 2019

Entschuldige, das habe ich über sehen. Dennoch sind 256 hier zu viel. @Kaliber ich übertreibe gerne wenn es ums coding geht

**Kasakow** · 18. August 2019

@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)

Kaliber · 18. August 2019

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"):("")));

**Kasakow** · 18. August 2019

Habe es gerade ausprobiert. Es reagiert immer noch nicht.

//EDIT
@Kaliber
Crashdetect gibt mir nicht aus.

Meine Tabellenstruktur:

Es wird auf jeden Fall was in die Log gepostet:

Kaliber · 19. August 2019

Ahhh du speicherst die Sekunden

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

C

case PLAYER_QUERY_PPROTOCOL:
{
    if(!rows) return SCM(playerid, COLOR_LESSRED, "Es wurde kein Eintrag gefunden.");
    SetPVarInt(playerid,"ProtocolRows",rows);
    new str[(128+11+9)*10] = "Datum\tUhrzeit\tInhalt\n";
    for(new i,date[11],time[9],data[128]; i<rows; i++)
    {
        cache_get_value_name(i, "Datum", date, sizeof(date)),
        cache_get_value_name(i, "Uhrzeit", time, sizeof(time)),
        cache_get_value_name(i, "Inhalt", data, sizeof(data));
        if(data[0] != EOS) format(str, sizeof str, "%s%s\t%s\t%s\n", str, date, time, data);
    }
    ShowPlayerDialog(playerid, D_PP, DIALOG_STYLE_TABLIST_HEADERS, "Spielerprotokoll", str, "Schließen", ((rows%10 == 0)?("Weiter"):("")));
}

Alles anzeigen

**Kasakow** · 19. August 2019

Leider tut sich, immer noch nichts.
In der SQL Error Log, sagt er, dass ich eine, ungültige AMX Instanz habe.

"[00:27:24] [plugins/mysql] CCallback::Execute - invalid AMX instance"

Kaliber · 19. August 2019

Hmmm, dann schreib es mal so:

C

case PLAYER_QUERY_PPROTOCOL:
{
    if(!rows) return SCM(playerid, COLOR_LESSRED, "Es wurde kein Eintrag gefunden.");
    SetPVarInt(playerid,"ProtocolRows",rows);
    new str[2000] = "Datum\tUhrzeit\tInhalt\n";
    for(new i,date[11],time[9],data[512]; i<rows; i++)
    {
        cache_get_value_name(i, "Datum", date, sizeof(date)),
        cache_get_value_name(i, "Uhrzeit", time, sizeof(time)),
        cache_get_value_name(i, "Inhalt", data, sizeof(data));
        if(data[0] != EOS) format(str, sizeof str, "%s%s\t%s\t%s\n", str, date, time, data);
    }
    ShowPlayerDialog(playerid, D_PP, DIALOG_STYLE_TABLIST_HEADERS, "Spielerprotokoll", str, "Schließen", ((rows%10 == 0)?("Weiter"):("")));
}

Alles anzeigen

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

**Kasakow** · 19. August 2019

Nop, irgendwie will er nicht .. immer wieder INVALID AMX instance

Kaliber · 19. August 2019

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:

C

ocmd:pp(playerid, params[])
{
    if(pInfo[playerid][pAdmin] < 6)return 1;
    if(IsNull(params))return SCM(playerid, COLOR_GREY, "Befehl: /pp [Name]");
    SetPVarString(playerid,"ProtocolName",params);
    ShowPlayerProtocol(playerid,0);
    return 1;
}
case PLAYER_QUERY_PPROTOCOL:
{
    if(!rows) return SCM(playerid, COLOR_LESSRED, "Es wurde kein Eintrag gefunden.");
    new str[1024] = "Datum\tUhrzeit\tInhalt\n";
    for(new i,date[11],time[9],data[128]; i<rows; i++)
    {
        cache_get_value_name(i, "Datum", date, sizeof(date)),
        cache_get_value_name(i, "Uhrzeit", time, sizeof(time)),
        cache_get_value_name(i, "Inhalt", data, sizeof(data));
        if(data[0] != EOS) format(str, sizeof str, "%s%s\t%s\t%s\n", str, date, time, data);
    }
    ShowPlayerDialog(playerid, D_PP, DIALOG_STYLE_TABLIST_HEADERS, "Spielerprotokoll", str, "Schließen", ((rows%10 == 0)?("Weiter"):("")));
}


case D_PP:
{
    new end = GetPVarInt(playerid,"ProtocolEnd");
    DeletePVar(playerid,"ProtocolEnd");
    if(response) return DeletePVar(playerid,"ProtocolName"), SCM(playerid, COLOR_GREY, "Du hast entweder das Fenster geschlossen oder es sind keine weiteren Einträge vorhanden!");
    ShowPlayerProtocol(playerid,end+10);
}


stock ShowPlayerProtocol(playerid,start)
{
    new query[64 + MAX_PLAYER_NAME];
    GetPVarString(playerid,"ProtocolName",query,MAX_PLAYER_NAME);
    mysql_format(handle, query, sizeof query, "SELECT * FROM playerprotocol WHERE Name='%e' LIMIT %d,10", query,start),
    mysql_pquery(handle, query, "PlayerQuery", "d", PLAYER_QUERY_PPROTOCOL);
    return SetPVarInt(playerid, "ProtocolEnd", start),1;
}

Alles anzeigen

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

**Kasakow** · 19. August 2019

@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.

Kaliber · 19. August 2019

Wie sieht dein Code denn unter OnDialogResponse aus?

**Kasakow** · 19. August 2019

Der letzte Code den du mir bearbeitest hast, so sieht es da aktuell aus.

Kaliber · 19. August 2019

Dann teste mal aus Spaß if(!response) das tauscht die Funktion der Buttons

**Kasakow** · 19. August 2019

Ich habe einfach mal die Buttons vertauscht, also so, wie ich es gewöhnt bin
Das Ding ist halt, wenn ich eine oder mehrere Seiten zurück gehe, schließt er und sagt dass nichts mehr sei.