Problem bei Report system mit Dialogen

TheTutoFreak · 27. Juli 2024

Guten Tag,

ich habe vor kurzen mal wieder angefangen ein bisschen zu Scripten und bin gerade über einem Report system mit Dialogen. Ich habe es jetzt so gemacht das ich mit dini Dateien speicher mit dem Report.

Code

public Report(playerid,inputtext[])
{
    new Reportdatei[64];
    new name[MAX_PLAYER_NAME];
    GetPlayerName(playerid,name,sizeof(name));
    format(Reportdatei,sizeof(Reportdatei),"/Reports/%s.txt",name);
    dini_Create(Reportdatei);
    dini_Set(Reportdatei,"Report",inputtext);
    return 1;
}

Jetzt möchte ich aber mit /showreport alle Daten von den Spielern auslesen un in einem DIALOG_STYLE_LIST anzeigen lassen aber es zeigt mir immer nur einen an. Ich weis das da in dem folgenden Code noch was fehlt aber ich weis leider nicht was das es mir mehrere Zeilen anzeigt.

Code

ocmd:showreport(playerid,params[])
{
    new string[128],name[MAX_PLAYER_NAME];
    GetPlayerName(playerid,name,sizeof(name));
    if(PlayerInfo[playerid][pEingeloggt] < 1 ) return SendClientMessage(playerid,Rot,"Du kannst diesen Befehl nicht nutzen, weil du nicht eingeloggt bist.");
    if(PlayerInfo[playerid][pAdminlevel] < 1)return SendClientMessage(playerid,Grau,"Du bist nicht befugt, diesen Befehl zu nutzen!");
    format(string,sizeof(string),"%s",name);
    ShowPlayerDialog(playerid,DIALOG_SHOWREPORT,DIALOG_STYLE_LIST,"Reportsystem",string,"Schließen");
    return 1;
}

nun meine frage was fehlt da dass es mir alle nahmen anzeigt.

und was muss ich machen das ich diese "Reportdatei" auslesen kann und wenn ich auf den Namen drück mir der Inhalt von der .txt Datei angezeigt wird.

ich hoffe mir kann jemand helfen

Kaliber · 27. Juli 2024

Ist nicht so leicht, da du das versuchst über das Dateisystem zu machen, anstatt über ein Datenbanksystem, wie MySQL.

Aber trotzdem habe ich noch Fragen, wie du dir das vorstellst.

Sollen dem Admin alle Reports angezeigt werden, die jemals erstellt wurden?
Sollen ihm nur die angezeigt werden von Spielern, die gerade online sind?
Sollen ihm nur offene Reports angezeigt werden? Willst du da einen Ordner mit Opened/Closed Reports haben?

PS: Denk ggfs über ein Anti-Spam System nach (sowas wie man kann nur jede Stunde einen Report machen o.ä.). Denn mit einem Keybinder, könnte man deinen Server binnen Stunden so zuspammen, dass da gar nichts mehr geht xD Und ja, so Trolle gibts leider wirklich 🤡

TheTutoFreak · 28. Juli 2024

Danke erstmal für die Antwort

ich hab mir das so gedacht das wenn Spieler 1 einen Report schreibt das eine 1.txt erstellt wird und das selbe bei Spieler 2 und 3 diese werden dann ingame angezeigt. Wenn Spieler 2 z.B. einen zweiten Report schreibt soll der erste von ihm „überschrieben“ werden. Mit /closereport [playerid/PartOfName] soll diese Datei dann gelöscht werden. Ich hoffe das konnte deine Fragen beantworten.

Wo vllt auch noch ein Problem ist was mir grade einfällt, ich habe aktuell 2 Reports in dem Reports Ordner es wird mir ingame aber immer nur einer angezeigt.

Wie kann man variablen in Dialogen anzeigen lassen?

Kaliber · 28. Juli 2024

Zitat von TheTutoFreak

ich hab mir das so gedacht das wenn Spieler 1 einen Report schreibt das eine 1.txt erstellt wird und das selbe bei Spieler 2 und 3 diese werden dann ingame angezeigt. Wenn Spieler 2 z.B. einen zweiten Report schreibt soll der erste von ihm „überschrieben“ werden.

Dann wäre es logischer, die Datei nach dem Spieler zu benennen, so weißt du direkt, von wem sie ist, und ob bereits ein Report von dem Spieler existiert.

Das Dumme ist nur, dass nativ in SAMP keine Schleife durch das Dateisystem möglich ist. Dafür bräuchtest du ein Plugin, wie https://github.com/JaTochNietDan/SA-MP-FileManager

Aber grundsätzlich ist eher davon abzuraten, da das sehr ineffizient ist.

C++

ocmd:showreport(playerid,params[])
{
    if(PlayerInfo[playerid][pEingeloggt] < 1 ) return SendClientMessage(playerid,Rot,"Du kannst diesen Befehl nicht nutzen, weil du nicht eingeloggt bist.");
    if(PlayerInfo[playerid][pAdminlevel] < 1)return SendClientMessage(playerid,Grau,"Du bist nicht befugt, diesen Befehl zu nutzen!");
    
    const MAX_REPORTS = 15;
    
    new string[MAX_REPORTS * MAX_PLAYER_NAME], dir:dHandle = dir_open("./"), item[40], type, i;
        
    while(dir_list(dHandle, item, type))
    {
        if(type != FM_FILE) continue;
        
        strcat(string, item), strcat(string, "\n");
        
        if(++i >= MAX_REPORTS) break;
    }
            
    dir_close(dHandle);
    
    ShowPlayerDialog(playerid,DIALOG_SHOWREPORT,DIALOG_STYLE_LIST,"Reportsystem",string,"Schließen","");
    return 1;
}

Alles anzeigen

Das zeigt dir z.B. die ersten 15 reports. Ein Seitensystem ist hier auch bissl kniffliger. Am Besten wäre es wohl das in eine Funktion auszulagern und n Start Parameter festzulegen.

Wenn jetzt eine Datei ausgewählt wird, dann steht im "inputtext" der Dateiname. Dann kannst du einfach mit fread diese auslesen und anzeigen

TheTutoFreak · 28. Juli 2024

Sry für die doofe frage aber könntest du mir des nochmal erklären da steh ich grad voll aufm Schlauch

- Wie installiere ich dieses Plugin?

- Wie genau meinst du das mit einer Funktion auslagern?

Die Dateien werden ja schon mit den Spielernamen gespeichert.

Hier mal alles was ich zu dem Report System bis jetzt habe:

Code

ocmd:showreport(playerid,params[])
{
    new string[128],name[MAX_PLAYER_NAME];
    GetPlayerName(playerid,name,sizeof(name));
    if(PlayerInfo[playerid][pEingeloggt] < 1 ) return SendClientMessage(playerid,Rot,"Du kannst diesen Befehl nicht nutzen, weil du nicht eingeloggt bist.");
    if(PlayerInfo[playerid][pAdminlevel] < 1)return SendClientMessage(playerid,Grau,"Du bist nicht befugt, diesen Befehl zu nutzen!");
    format(string,sizeof(string),"%s",name);
    ShowPlayerDialog(playerid,DIALOG_SHOWREPORT,DIALOG_STYLE_LIST,"Reportsystem",string,"Auswählen","Schließen");
    return 1;
}
ocmd:closereport(playerid,params[])
{
    new Reportdatei[128],pID,name[MAX_PLAYER_NAME],string[128];
    GetPlayerName(pID,name,sizeof(name));
    if(PlayerInfo[playerid][pEingeloggt] < 1 ) return SendClientMessage(playerid,Rot,"Du kannst diesen Befehl nicht nutzen, weil du nicht eingeloggt bist.");
    if(PlayerInfo[playerid][pAdminlevel] < 1)return SendClientMessage(playerid,Grau,"Du bist nicht befugt, diesen Befehl zu nutzen!");
     if(sscanf(params,"u",pID)) return SendClientMessage(playerid,Grau,"Benutze: /closereport [playerid/PartOfName]");
     format(Reportdatei,sizeof(Reportdatei),"/Reports/%s.txt",name);
    if(dini_Exists(Reportdatei))
     {
        format(string,sizeof(string),"REPORTSYSTEM: Du hast den Report von %s gelöscht.",name);
        SendClientMessage(playerid,Rot,string);
         DeleteReport(pID);
        return 1;
    }
    else
    {
        SendClientMessage(playerid,Rot,"Dieser Spieler hat keinen Report abgeschickt!");
        return 1;
    }
}

Alles anzeigen

Code

ocmd:report(playerid,params[])
{
    ShowPlayerDialog(playerid,DIALOG_REPORT,DIALOG_STYLE_INPUT,"Reportsystem","Bitte geb deinen Report ein \n Dein Report wird umgehend an \n einen Admin weitergeleitet!","Absenden","Abbrechen");
    return 1;
}

Code

if(dialogid == DIALOG_REPORT)
    {
        if(response == 0)
        {
            SendClientMessage(playerid,Rot,"Du hast das Reportsystem geschlossen!");
            return 1;
        }
        if(response == 1)
        {
            if(!strlen(inputtext))
            {
                SendClientMessage(playerid,Rot,"Du hast keinen Report in das Eingabefeld eingegeben!");
                ShowPlayerDialog(playerid,DIALOG_REPORT,DIALOG_STYLE_INPUT,"Reportsystem","Bitte geb deinen Report ein \n Dein Report wird umgehend an \n einen Admin weitergeleitet!","Absenden","Abbrechen");
            }
            else
            {
                new name[MAX_PLAYER_NAME],string[128];
                SendClientMessage(playerid,Rot,"Dein Report wurde an das Adminteam weitergeleitet!");
                GetPlayerName(playerid,name,sizeof(name));
                format(string,sizeof(string),"REPORTSYSTEM: Spieler %s hat einen Report abgeschickt!",name);
                SendClientMessageToAdmin(Rot,string);
                Report(playerid,inputtext);
                return 1;
            }
            return 1;
        }
    }
    if(dialogid == DIALOG_SHOWREPORT)
     {
        if(response == 0)
        {
            SendClientMessage(playerid,Rot,"Du hast das Reportsystem geschlossen!");
            return 1;
        }
        if(response == 1)
        {
            ShowReport(playerid,inputtext);
            return 1;
        }
    }

Alles anzeigen

Code

public Report(playerid,inputtext[])
{
    new Reportdatei[64];
    new name[MAX_PLAYER_NAME];
    GetPlayerName(playerid,name,sizeof(name));
    format(Reportdatei,sizeof(Reportdatei),"/Reports/%s.txt",name);
    dini_Create(Reportdatei);
    dini_Set(Reportdatei,"Report",inputtext);
    return 1;
}
public ShowReport(playerid,inputtext[])
{
    new Reportdatei[128];
    new string[128];
    new name[MAX_PLAYER_NAME];
    GetPlayerName(playerid,name,sizeof(name));
    format(Reportdatei,sizeof(Reportdatei),"/Reports/%s.txt",name);
     format(string,sizeof(string),"Reportsystem: %s: %s",name,Report(playerid,inputtext));
    SendClientMessage(playerid,Rot,string);
    return 1;
}
public DeleteReport(playerid)
{
    new Reportdatei[128];
    new name[MAX_PLAYER_NAME];
    GetPlayerName(playerid,name,sizeof(name));
    format(Reportdatei,sizeof(Reportdatei),"/Reports/%s.txt",name);
    dini_Remove(Reportdatei);
    return 1;
}

Alles anzeigen

Kaliber · 28. Juli 2024

Zitat von Kaliber

Sollen ihm nur die angezeigt werden von Spielern, die gerade online sind?

Nochmal zu dieser Frage, soll das so sein?

Denn dein aktuelles System funktioniert nur so.

TheTutoFreak · 28. Juli 2024

Grundlegend mal wäre das nicht verkehrt ja