Beiträge von Kaliber

Kaliber · 15. Januar 2019

Okay...interessant.

Hmm schreibs so:

C

forward AdminList(playerid);
public AdminList(playerid)
{
    static const rang_namen[][] = {"Spieler", "Helfer", "Probe Supporter", "Supporter", "Administration", "Developer", "{00a4b3}Managment", "{FF0000}Projektleitung"};
    new rows;
    cache_get_row_count(rows);
    const MAX_ADMINS = 15;
    new table[MAX_ADMINS][2], admins[MAX_ADMINS][MAX_PLAYER_NAME];
    for(new i; i<rows; i++)
    {
        cache_get_value_name_int(i, "Admin", table[i][0]);
        cache_get_value_name(i, "Name", admins[i]);
        table[i][1] = i;
    }
    SortDeepArray(table, 0);
    new string[(MAX_PLAYER_NAME+25)*MAX_ADMINS] = "Rang\tName\tOnlinestatus";
    for(new i=rows-1,idx,r,status[32]; i!=-1; i--)
    {
        r = table[i][0], idx = table[i][1];
        GetPlayerStatus(admins[idx], status);
        format(string,sizeof(string),"%s\n%s\t%s\t%s",string,rang_namen[r],admins[idx],status);
    }
    ShowPlayerDialog(playerid, 8, DIALOG_STYLE_TABLIST_HEADERS, "ARP Team",string,"Okay","");
    return 1;
}

Alles anzeigen

Kaliber · 15. Januar 2019

Zitat von Allrounder18

bzw. was msste ich machen damit die Namen statt den Zahlen da stehn?

Ist das ein Scherz?

Offensichtlich einfach nur die Namen darein schreiben anstatt der Zahlen...das war ja nur ein Beispiel.

Zitat von Allrounder18

Wie kann ichs machen, dass dDie Ränge geördnet da stehn?

Das ist etwas komplizierter, da du die ja sortieren musst, hier mal ein Beispiel:

C

forward AdminList(playerid);
public AdminList(playerid)
{
    static const rang_namen[][] = {"Spieler", "Helfer", "Probe Supporter", "Supporter", "Administration", "Developer", "{00a4b3}Managment", "{FF0000}Projektleitung"};


    new rows;
    cache_get_row_count(rows);


    const MAX_ADMINS = 15;
    new table[MAX_ADMINS][2], admins[MAX_ADMINS][MAX_PLAYER_NAME];
    for(new i; i<rows; i++)
    {
        cache_get_value_name_int(i, "Admin", table[i][0]);
        cache_get_value_name(i, "Name", admins[i]);
        table[i][1] = i;
    }
    SortDeepArray(table, 0, .sort=SORT_DESC);
    new string[(MAX_PLAYER_NAME+25)*MAX_ADMINS] = "Rang\tName\tOnlinestatus";
    for(new i,idx,r,status[32]; i<rows; i++)    
    {
        r = table[i][0], idx = table[i][1];
        GetPlayerStatus(admins[idx], status);
        format(string,sizeof(string),"%s\n%s\t%s\t%s",string,rang_namen[r],admins[idx],status);
    }    
    ShowPlayerDialog(playerid, 8, DIALOG_STYLE_TABLIST_HEADERS, "ARP Team",string,"Okay","");
    return 1;
}

Alles anzeigen

//Edit:
Benötigt wird diese Include: https://raw.github.com/oscar-b…d-sort/master/md-sort.inc

Thread: https://forum.sa-mp.com/showthread.php?t=343172

Kaliber · 14. Januar 2019

Zitat von Allrounder18

assigned value: '7'

Da ist das Problem, du hast zu viele Ränge.

Musst die quasi mal alle eintragen in das Array:

static const rang_namen[][] = {"0", "1", "2", "3", "4", "5", "6", "7"};

Kaliber · 14. Januar 2019

Zitat von Allrounder18

wenn nur das tquery im Befehl steht

Es soll nur das im Befehl stehen.

printe eventuell mal rows oder schau in den mysql Log.

Eigentlich sieht das richtig aus.

Vorrausgesetzt in der Datenbank steht, dass Admin ein Integer ist

Kaliber · 14. Januar 2019

Zitat von Goldkiller

sind da paar Memory leaks. (Z. 82, Z.90, Z. 100).

Z. 82 = Create Table
Z.90 = Insert
Z.100 = Offensichtlich Leer

Soweit ich das mal gelesen hatte, muss man da nichts clearen, weil da eh nichts existiert.

Zitat von Goldkiller

Wenn die INSERT-Statements ein Problem sind was die Laufzeit betrifft, dann würde ich mir mal Transactions ansehen

Ja, das gar keine schlechte Idee, schau ich mal, ob das die Performance verbessert, danke

Kaliber · 14. Januar 2019

Achso deinen Query natürlich so schreiben:

mysql_tquery(handle,"SELECT Name, Admin FROM `accounts` WHERE `Admin`>0", "AdminList", "d", playerid);

Und du musst return 1; einfach darunter packen.

Und den ganzen Code, der muss nur im Callback stehen, im Command steht nur das mysql_tquery

cache_get_value_name_int(i, "Admin", r);

Muss das außerdem natürlich sein, sry..

Das sollte eigentlich alle deine Probleme lösen.

Kaliber · 14. Januar 2019

Dann schau doch, wie du die rows ermittelst.

Dann nur:

new rows;
cache_get_row_count(rows);

Kaliber · 14. Januar 2019

Zitat von Allrounder18

Da kommt aber nur ne leere Liste

Ich habe soo viele Fragen...

1. Wie hast du denn in dem Kontext rows deklariert? Du musst das doch erstmal abfragen.

2. Wieso verwendest du _int? Das sind keine ints? und was soll das mit den SpielerDaten da

3. Wieso verwendest du stumpf "Name" und "Admin"...offensichtlich müssen da die Spalten Namen aus DEINER Tabelle rein..

C

forward AdminList(playerid);
public AdminList(playerid)
{
    static const rang_namen[][] = {"Zivilist","{00a4b3}Manager","{FF0000}Projektleiter"};
    new rows, fields;
    cache_get_data(rows, fields);
    //Rang 0 = Zivilist, Rang 1 = Manager, Rang 2 = Projektleiter
    new string[256] = "Rang\tName\tOnlinestatus",status[32],rows;
    for(new i,n[MAX_PLAYER_NAME],r; i<rows; i++)
    {
        cache_get_value_name(i, "Name", n);
        cache_get_value_name(i, "Admin", r);
        GetPlayerStatus(n, status);
        format(string,sizeof(string),"%s\n%s\t%s\t%s",string,rang_namen[r],n,status);
    }
    ShowPlayerDialog(playerid, 8, DIALOG_STYLE_TABLIST_HEADERS, "ARP Team",string,"Okay","");
}

Alles anzeigen

Und du musst dir halt jetzt sicher sein, dass sowohl "Name", als auch "Admin" in deiner Datenbank sich befinden als Spalten..

Kaliber · 14. Januar 2019

Zitat von Allrounder18

Da kommen dise Fehler

Jaa....natürlich musst du das an deine MySQL Version anpassen.

Schau mal welche Funktionen du in DEINEM Skript verwendest und verändere diese... (Schau einfach bei z.B. deinem Login/Register System)

Das solltest du schon hinbekommen

Kaliber · 14. Januar 2019

Zitat von Allrounder18

Das?

...ich bin mir gerade nicht sicher, ob das ein Scherz sein soll oder nicht...

Aber in dem Code gab es nur 2 Kommentare...die klar gekennzeichnet sind...die sich auf die Positionen des Codes beziehen...

..also JA natürlich dieser Code... xD

Was du auch am Code selbst ablesen kannst...da ist von cache Funktionen und rows die Rede...

Kaliber · 14. Januar 2019

Zitat von Kaliber

//und das hier dann in den Callback

Ich mache nicht umsonst Kommentare...

Kaliber · 14. Januar 2019

Du solltest dir dringend nochmal die Grundlagen durchlesen und in Ruhe, erst mal ein den Basics feilen, bevor du direkt so große Sachen machen willst

SELECT Name,Admin FROM `accounts` So müsstest du das machen, da du ja auch noch den Namen von dem Spieler benötigst, wie du in meinem Skript siehst.

Und anstatt OnQueryLoaded brauchst einen eigenen Callback.

C

mysql_tquery(handle,"SELECT Name, Admin FROM `accounts`", "AdminList", "d", playerid);


forward AdminList(playerid);
public AdminList(playerid)
{
    //Das hier ist dann der Callback, hier dann den anderen Code rein
}

Kaliber · 14. Januar 2019

Mir fällt gerade ein, da du das offline machen magst, kannst du das so nicht direkt machen.

Du musst ne query Abfrage machen aus der Datenbank, wo alle Admins gespeichert sind.

Und dann sowas wie:

C

ocmd:admins(playerid)
{
    //Hier die mysql_tquery abfrage rein


    //und das hier dann in den Callback
    static const rang_namen[][] = {"Zivilist","{00a4b3}Manager","{FF0000}Projektleiter"};
    //Rang 0 = Zivilist, Rang 1 = Manager, Rang 2 = Projektleiter
    new string[256] = "Rang\tName\tOnlinestatus",status[32];
    for(new i,n[MAX_PLAYER_NAME],r; i<rows; i++)
    {
        cache_get_field_content(i, "name", n);
        cache_get_field_content(i, "rang", r);
        GetPlayerStatus(n, status);
    	format(string,sizeof(string),"%s\n%s\t%s\t%s",string,rang_namen[r],n,status);
    }
    ShowPlayerDialog(playerid, 8, DIALOG_STYLE_TABLIST_HEADERS, "ARP Team",string,"Okay","");
    return 1;
}

Alles anzeigen

Kaliber · 14. Januar 2019

Zitat von Allrounder18

Wäre es irgendwie möglich die Namen und Räge ingame einzufügen in die Liste?

Ja klar, wenn du das willst.

C

//Machst in dein Spieler Enum einfach
pAdminRang


//Nennen wir das hier mal SpielerInfo

Das kannst du ja dann beliebig setzen.

Und das sähe dann so aus:

C

ocmd:admins(playerid)
{
    static const rang_namen[][] = {"Zivilist","{00a4b3}Manager","{FF0000}Projektleiter"};
    //Rang 0 = Zivilist, Rang 1 = Manager, Rang 2 = Projektleiter


    new string[256] = "Rang\tName\tOnlinestatus",status[32];
    for(new i=GetPlayerPoolSize(); i!=-1; i--)
    {
        if(!IsPlayerConnected(i) || IsPlayerNPC(i) || !SpielerInfo[i][pAdminRang]) continue;
        GetPlayerStatus(projektleiter[i], status);
    	format(string,sizeof(string),"%s\n%s\t%s\t%s",string,rang_namen[SpielerInfo[i][pAdminRang]],projektleiter[i],status);
    }
    ShowPlayerDialog(playerid, 8, DIALOG_STYLE_TABLIST_HEADERS, "ARP Team",string,"Okay","");
    return 1;
}

Alles anzeigen

//Edit:
Schau dir doch das Makro an, es sind 5 Sekunden.

Kaliber · 14. Januar 2019

Okay...dafuq, die Funktion will kein Array returnen...

Dämlicher Pawn Compiler

Dann mach es so:

C

ocmd:admins(playerid)
{
    static const projektleiter[][] = {"Allrounder18"}; //Alle Projektleiter
    static const manager[][] = {"Kolho", "Flo20"}; //Alle Manager


    new string[256] = "Rang\tName\tOnlinestatus",status[32];


    for(new i; i<sizeof(projektleiter); i++)
    {
        GetPlayerStatus(projektleiter[i], status);
    	format(string,sizeof(string),"%s\n{FF0000}Projektleiter\t%s\t%s",string,projektleiter[i],status);
    }
    for(new i; i<sizeof(manager); i++)
    {
        GetPlayerStatus(manager[i], status);
    	format(string,sizeof(string),"%s\n{00a4b3}Manager\t%s\t%s",string,manager[i],status);
    }
    ShowPlayerDialog(playerid, 8, DIALOG_STYLE_TABLIST_HEADERS, "ARP Team",string,"Okay","");
    return 1;
}


stock GetPlayerStatus(const name[],out[])
{
    static const status[][] = {"{FF0000}Offline","{40FF00}Online","{FF0048}AFK"};

    for(new i=GetPlayerPoolSize(),n[MAX_PLAYER_NAME]; i!=-1; i--)
    {
        if(IsPlayerNPC(i) || !GetPlayerName(i,n,MAX_PLAYER_NAME) || strcmp(name,n) != 0) continue;
        return format(out,32,status[IsPlayerAFK(i) ? 2 : 1]);
    }
    return format(out,32,status[0]);
}

Alles anzeigen

Kaliber · 14. Januar 2019

Hast du denn auch das ShowPlayerDialog eingefügt, so wie ich das gezeigt hatte?

Hab den Beitrag nochmal nacheditiert.

Kannst du ja auch einfach mal kurz davor den String printen

Kaliber · 14. Januar 2019

Zitat von Allrounder18

Ich hab alles geschrieben wie du meintest, edoch wnn ichd e Befehl eingeb kommt nix.

Hast du crashdetect drinnen?

Vielleicht meckert der wegen der String-Größe mach mal:

new string[512] = "Rang\tName\tOnlinestatus";

Kaliber · 14. Januar 2019

Zitat von Allrounder18

Danke, ich bekom folgendeFehler.

Ohh, jaa da war ja was, state ist ja ein Keyword

Dann nehmen wir halt Deutsch

C

stock GetPlayerStatus(const name[])
{
    static const status[][] = {"{FF0000}Offline","{40FF00}Online","{FF0048}AFK"};

    for(new i=GetPlayerPoolSize(),n[MAX_PLAYER_NAME]; i!=-1; i--)
    {
        if(IsPlayerNPC(i) || !GetPlayerName(i,n,MAX_PLAYER_NAME) || strcmp(name,n) != 0) continue;
        return status[IsPlayerAFK(i) ? 2 : 1];
    }
    return status[0];
}

Alles anzeigen

Kaliber · 14. Januar 2019

Zitat von Allrounder18

@Kaliber
Ich hab da och keine Variable.

Ahh jaa okay

C

//Pack oben in dein Skript
#define IsPlayerAFK(%0) ((gettime()-AFK[%0])>5)


//Das auch oben irgendwo hin
new AFK[MAX_PLAYERS];


//Und das unter OnPlayerUpdate
AFK[playerid] = gettime();

So jetzt hast ein AFK System

Und dann den Befehl so:

C

ocmd:admins(playerid)
{
    static const projektleiter[][] = {"Allrounder18"}; //Alle Projektleiter
    static const manager[][] = {"Kolho", "Flo20"}; //Alle Manager

new string[256] = "Rang\tName\tOnlinestatus";
    for(new i; i<sizeof(projektleiter); i++)
    {
    format(string,sizeof(string),"%s\n{FF0000}Projektleiter\t%s\t%s",string,projektleiter[i],GetPlayerStatus(projektleiter[i]));
    }


    for(new i; i<sizeof(manager); i++)
    {
    format(string,sizeof(string),"%s\n{00a4b3}Manager\t%s\t%s",string,manager[i],GetPlayerStatus(manager[i]));
    }
    ShowPlayerDialog(playerid, 8, DIALOG_STYLE_TABLIST_HEADERS, "ARP Team",string,"Okay","");
    return 1;
}


stock GetPlayerStatus(const name[])
{
    static const state[][] = {"{FF0000}Offline","{40FF00}Online","{FF0048}AFK"};


    for(new i=GetPlayerPoolSize(),n[MAX_PLAYER_NAME]; i!=-1; i--)
    {
        if(IsPlayerNPC(i) || !GetPlayerName(i,n,MAX_PLAYER_NAME) || strcmp(name,n) != 0) continue;
        return state[IsPlayerAFK(i) ? 2 : 1];
    }
    return state[0];
}

Alles anzeigen

Also die Tickets hab ich aus dem Befehl weggelassen, weil ich da keine Infos hatte.

Wenn du die noch haben magst, kannst du es gerne selbst probieren oder mir sagen, in welcher Variable du die speicherst, welchen Wert das haben soll, wenn der Spieler offline ist.

Kaliber · 14. Januar 2019

Zitat von Allrounder18

Wie bekomme ich es hin, das mir angezeigt wird ob der Spieler Online, Offline oder AFK ist?

Mit welcher Variable prüfst du denn, ob jemand AFK ist?

Zitat von Sub Royal

Moin.

Wenn man über 100 Zeilen Code + Logik Fragen hat...daaann ist das keine kleine Scripting Frage mehr

Eröffne da mal lieber einen eigenen Thread xD

Beiträge von Kaliber

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[INCLUDE] Unique Random Number

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen

[SAMMELTHREAD] Kleine Scripting Fragen