[SAMMELTHREAD] Kleine Scripting Fragen

Allrounder18 · 14. November 2019

Hab das mal gemacht, Login genauso ähnlich?

C

public OnPasswordChecked(playerid)
{
    new bool:match = bcrypt_is_equal(),query[256];
    mysql_format(handle, query, sizeof(query), "SELECT * FROM accounts WHERE Name = '%e' AND Password = '%e'", AccountInfo[playerid][account_Name], hash);
    mysql_pquery(handle, query, "AccountLogin", "d", playerid);
    return 1;
}

C

  public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    switch(dialogid)
    {
        case DIALOG_REGISTRATION:
        {
            bcrypt_hash(inputtext, BCRYPT_COST, "OnPasswordHashed", "d", playerid);
        }

        case DIALOG_LOGIN:
        {
            // Variable hash is expected to contain the hash loaded from the database
            bcrypt_check(inputtext, hash, "OnPasswordChecked", "d", playerid);
        }
    }

    return 1;
}

Alles anzeigen

Soweit ist es mir auch noch klar, aber wie mache ich den Dialog am besten?

Kaliber · 14. November 2019

Da drehst du das um. Du lädst zuerst aus der MySQL Datenbank und prüfst das dann gegen.

So zum Beispiel:

C

case DIALOG_LOGIN:
{
    new query[256];
    mysql_format(handle, query, sizeof(query), "SELECT * FROM `accounts` WHERE `Name` = '%e' LIMIT 1", AccountInfo[playerid][account_Name]);
    mysql_pquery(handle, query, "AccountLogin", "ds", playerid,inputtext);
}
case DIALOG_PW_CHECK:
{
    if(!response) return Kick(playerid);
    if(!inputtext[0]) return SPD(playerid,DIALOG_PW_CHECK,DIALOG_STYLE_INPUT,"Falsches Passwort","Dein eingegebenes Passwort ist falsch!","Bestätigen","Abbrechen");
    bcrypt_check(inputtext, AccountInfo[playerid][account_Password], "OnPasswordChecked", "d", playerid);
}

//Bei AccountLogin
public AccountLogin(playerid,inputtext[])
{
    //Halt account laden und AccountInfo[playerid][account_Password] den hash eben setzen
    
    //Ganz unten dann:
    bcrypt_check(inputtext, AccountInfo[playerid][account_Password], "OnPasswordChecked", "d", playerid);
    return 1;
}

public OnPasswordChecked(playerid)
{
    new bool:match = bcrypt_is_equal();
    if(!match) return SPD(playerid,DIALOG_PW_CHECK,DIALOG_STYLE_INPUT,"Falsches Passwort","Dein eingegebenes Passwort ist falsch!","Bestätigen","Abbrechen");
    //Hier kannst den Spieler dann spawnen und einloggen :)
    return 1;
}

Alles anzeigen

Allrounder18 · 14. November 2019

C

FUNCTION AccountLogin(playerid)
{
    new rows,string[128];
    cache_get_row_count(rows);
    if(rows)
    {
        cache_get_value_name_int(0, "ID", AccountInfo[playerid][account_ID]);
        cache_get_value_name_int(0, "Slot1", AccountInfo[playerid][account_Slot][0]);
        cache_get_value_name_int(0, "Slot2", AccountInfo[playerid][account_Slot][1]);
        cache_get_value_name_int(0, "Slot3", AccountInfo[playerid][account_Slot][2]);
        cache_get_value_name_int(0, "Slot4", AccountInfo[playerid][account_Slot][3]);
        cache_get_value_name_int(0, "Admin", AccountInfo[playerid][account_Admin]);
        cache_get_value_name_int(0, "Mapper", AccountInfo[playerid][account_Mapper]);
        cache_get_value_name_int(0, "Scripter", AccountInfo[playerid][account_Scripter]);
        cache_get_value_name_int(0, "Maintenance", AccountInfo[playerid][account_Maintenance]);
        cache_get_value_name_int(0, "Interact", AccountInfo[playerid][account_Interact]);
        new inputtext;
        bcrypt_check(inputtext, AccountInfo[playerid][account_Password], "OnPasswordChecked", "d", playerid);
        mysql_format(handle, string, sizeof(string), "SELECT * FROM `character` WHERE `AccountID` = '%d'", AccountInfo[playerid][account_ID]);
        mysql_tquery(handle, string, "ShowCharacter", "d", playerid);
    }
    return 1;
}

Alles anzeigen

AccountLogin wilil irgendiwe nicht ganz, das s sagt das inputtext bei bcrypt_check passt nicht.

Wie mache ich am besten den Reister Dialog mit bcrypt fertig? Die Zeile wie im Tutorial hab ich schon.

Kaliber · 14. November 2019

Achte auf meine Details: mysql_pquery(handle, query, "AccountLogin", "ds", playerid,inputtext); Ich übergebe hier den Inputtext an diese Funktion. Du musst den Callback entsprechend anpassen.
Keine Ahnung, was dein ShowCaracter da noch macht?
Die Frage mit dem Register verstehe ich nicht ganz. Denn so wie du es bis jetzt hast, wird er ja dann registriert...

Allrounder18 · 14. November 2019

1. Wo genau meinst du? Der Error wegei nputtext kommt immer noch.

2. ShowCharakter zeigt die Charakteruaswahl.

3. Ich meine, wie ich da den Dialog machen soll? Oder ist dieser DIALOG_PW_CHECK für die Registration?

Kaliber · 14. November 2019

Wie wo genau meine ich? Na an der Funktion, du musst den Inputtext übergeben und auch natürlich oben in den Funktions-Header packen: FUNCTION AccountLogin(playerid, inputtext[])
Ja, das solltest du dann an das Ende von OnPasswordChecked packen. Denn das soll ja erst kommen, wenn er erfolgreich eingeloggt ist
Nein, DIALOG_PW_CHECK ist für den Login. Das wird angezeigt, falls er das Passwort falsch eingegeben hat. Für das Registrieren musst doch einfach nur ein ShowPlayerDialog(playerid,DIALOG_REGISTRATION,DIALOG_STYLE_INPUT,"Registration","Bitte gib ein Passwort für deinen Account an:","Bestätigen","Abbrechen"); machen. Oder so ähnlich natürlich...es muss doch vorher eine Prüfung geben, ob der User bereits einen Account hat oder nicht...das müsste doch nur angepasst werden

Dr. Frauenarzt · 14. November 2019

Ist es möglich 2 Mysql Version zu Verwenden wen ich ein Script in eine Akuellerere Version Schreiben möchte um das ganze zu testen? Wen ja Wie?

LeonMrBonnie · 14. November 2019

2 Versionen gleichzeitig? Das ist nicht möglich

Dr. Frauenarzt · 14. November 2019

Das heißt ich muss alle Funktionen die derzeit auf der alten Version sind sofort in die neue Version umwandeln. Huch ok

Allrounder18 · 14. November 2019

Ich hab jetzt alles so gemacht wie du sagtest, jedoch sagt er mir jetzt immer das das PW falsch iost obwohl es richtig ist.

Kaliber · 14. November 2019

Wie sieht denn nun dein Code aus, poste die entsprechenden Felder.

Nächstes mal kannst du btw ruhig einen eigenen Thread erstellen, falls Leute ähnliche Probleme haben, wäre das sehr hilfreich gewesen. (Hier geht das ja unter und ist nur für kleine Fragen gedacht).

(Darfst auch gerne etwas debuggen, wird in der Datenbank alles richtig gespeichert?)

Allrounder18 · 14. November 2019

C

stock logindialog(playerid, dialogid, response, inputtext[])
{
    switch(dialogid)
    {
        case DIALOG_LOGIN:
        {
            new query[256];
            mysql_format(handle, query, sizeof(query), "SELECT * FROM `accounts` WHERE `Name` = '%e' LIMIT 1", AccountInfo[playerid][account_Name]);
            mysql_pquery(handle, query, "AccountLogin", "ds", playerid,inputtext);
        }
        case DIALOG_PW_CHECK:
        {
            if(!response) return Kick(playerid);
            if(!inputtext[0]) return ShowPlayerDialog(playerid,DIALOG_PW_CHECK,DIALOG_STYLE_INPUT,"Falsches Passwort","Dein eingegebenes Passwort ist falsch!","Bestätigen","Abbrechen");
            bcrypt_check(inputtext, AccountInfo[playerid][account_Password], "OnPasswordChecked", "d", playerid);
        }
    }
    return 1;
}

FUNCTION AccountLogin(playerid, inputtext[])
{
    new rows;
    cache_get_row_count(rows);
    if(rows)
    {
        cache_get_value_name_int(0, "ID", AccountInfo[playerid][account_ID]);
        cache_get_value_name_int(0, "Slot1", AccountInfo[playerid][account_Slot][0]);
        cache_get_value_name_int(0, "Slot2", AccountInfo[playerid][account_Slot][1]);
        cache_get_value_name_int(0, "Slot3", AccountInfo[playerid][account_Slot][2]);
        cache_get_value_name_int(0, "Slot4", AccountInfo[playerid][account_Slot][3]);
        cache_get_value_name_int(0, "Admin", AccountInfo[playerid][account_Admin]);
        cache_get_value_name_int(0, "Mapper", AccountInfo[playerid][account_Mapper]);
        cache_get_value_name_int(0, "Scripter", AccountInfo[playerid][account_Scripter]);
        cache_get_value_name_int(0, "Maintenance", AccountInfo[playerid][account_Maintenance]);
        cache_get_value_name_int(0, "Interact", AccountInfo[playerid][account_Interact]);
        bcrypt_check(inputtext, AccountInfo[playerid][account_Password], "OnPasswordChecked", "d", playerid);
    }
    return 1;
}

FUNCTION OnPasswordChecked(playerid)
{
    new bool:match = bcrypt_is_equal(), query[256],string[128],inputtext;
    if(!match) return ShowPlayerDialog(playerid,DIALOG_PW_CHECK,DIALOG_STYLE_INPUT,"Falsches Passwort","Dein eingegebenes Passwort ist falsch!","Bestätigen","Abbrechen");
    mysql_pquery(handle, query, "AccountLogin", "ds", playerid,inputtext);
    mysql_format(handle, string, sizeof(string), "SELECT * FROM `character` WHERE `AccountID` = '%d'", AccountInfo[playerid][account_ID]);
    mysql_tquery(handle, string, "ShowCharacter", "d", playerid);
    return 1;
}

Alles anzeigen

Was nutzt eigentlich das LIMIT im SELECT?

Kaliber · 14. November 2019

Zitat von Allrounder18

Was nutzt eigentlich das LIMIT im SELECT?

Das sagt deiner Datenbank, dass es nur 1 Statement gibt, was zurückgegeben werden muss.

Das ist performant das mit anzugeben, da er dann genau weiß, ah ich hab eins gefunden, jetzt kann ich aufhören und muss nicht noch alle anderen nach dem Namen durchsuchen

Wie mir auffällt, lädst du dort gar nicht das Passwort, also unter AccountLogin?

Das musst du dort laden, also das AccountInfo[playerid][account_Password]...sonst geht das natürlich nicht.

Wie mir auffällt lädst du dort generell nix über den User so wirklich?!

Da stimmt doch was nicht xD

Allrounder18 · 14. November 2019

Habs jetzt dort mal reingeschrieben aber geht trotzem nicht. Soll ich dme Passowrt ne Länge dazuschreiben wie zb AccountInfo[playerid][account_Password][255]?

Du meinst dort muss auch AccountInfo[playerid][account_Name] rein? Mehr gibts nicht zu laden.

Kaliber · 14. November 2019

Du hast da stehen:

new bool:match = bcrypt_is_equal(), query[256],string[128],inputtext;

Wieso dieses inputtext? Bitte nirgends so deklarieren....schau mal wie ich das gemacht habe...

Und die String Länge sollte BCRYPT_HASH_LENGTH sein

#Handy...bin nu unterwegs

Allrounder18 · 14. November 2019

Habs jetzt geändert so wie du sagtest, nur leider funktioniert es immer noch nicht.

Kaliber · 14. November 2019

Dann zeige bitte nochmal deinen ganzen Code

Allrounder18 · 14. November 2019

C

stock logindialog(playerid, dialogid, response, inputtext[])
{
    switch(dialogid)
    {
        case DIALOG_LOGIN:
        {
            new query[256];
            mysql_format(handle, query, sizeof(query), "SELECT * FROM `accounts` WHERE `Name` = '%e' LIMIT 1", AccountInfo[playerid][account_Name]);
            mysql_pquery(handle, query, "AccountLogin", "ds", playerid,inputtext);
        }
        case DIALOG_PW_CHECK:
        {
            if(!response) return Kick(playerid);
            if(!inputtext[0]) return ShowPlayerDialog(playerid,DIALOG_PW_CHECK,DIALOG_STYLE_INPUT,"Falsches Passwort","Dein eingegebenes Passwort ist falsch!","Bestätigen","Abbrechen");
            bcrypt_check(inputtext, AccountInfo[playerid][account_Password], "OnPasswordChecked", "d", playerid);
        }
    }
    return 1;
}

FUNCTION AccountLogin(playerid, inputtext[])
{
    new rows;
    cache_get_row_count(rows);
    if(rows)
    {
        cache_get_value_name_int(0, "ID", AccountInfo[playerid][account_ID]);
        cache_get_value_name(0, "Name", AccountInfo[playerid][account_Name]);
        cache_get_value_name(0, "Password", AccountInfo[playerid][account_Password]);
        cache_get_value_name_int(0, "Slot1", AccountInfo[playerid][account_Slot][0]);
        cache_get_value_name_int(0, "Slot2", AccountInfo[playerid][account_Slot][1]);
        cache_get_value_name_int(0, "Slot3", AccountInfo[playerid][account_Slot][2]);
        cache_get_value_name_int(0, "Slot4", AccountInfo[playerid][account_Slot][3]);
        cache_get_value_name_int(0, "Admin", AccountInfo[playerid][account_Admin]);
        cache_get_value_name_int(0, "Mapper", AccountInfo[playerid][account_Mapper]);
        cache_get_value_name_int(0, "Scripter", AccountInfo[playerid][account_Scripter]);
        cache_get_value_name_int(0, "Maintenance", AccountInfo[playerid][account_Maintenance]);
        cache_get_value_name_int(0, "Interact", AccountInfo[playerid][account_Interact]);
        bcrypt_check(inputtext, AccountInfo[playerid][account_Password], "OnPasswordChecked", "d", playerid);
    }
    return 1;
}

FUNCTION OnPasswordChecked(playerid)
{
    new bool:match = bcrypt_is_equal();
    new string[128];
    if(!match) return ShowPlayerDialog(playerid,DIALOG_PW_CHECK,DIALOG_STYLE_INPUT,"Falsches Passwort","Dein eingegebenes Passwort ist falsch!","Bestätigen","Abbrechen");
    //mysql_pquery(handle, query, "AccountLogin", "ds", playerid,inputtext);
    mysql_format(handle, string, sizeof(string), "SELECT * FROM `character` WHERE `AccountID` = '%d'", AccountInfo[playerid][account_ID]);
    mysql_tquery(handle, string, "ShowCharacter", "d", playerid);
    return 1;
}

Alles anzeigen

Das benutze ich bei Login.

Kaliber · 14. November 2019

Okay, sieht gut aus

Dann leg mal einen neuen Account an...und schau, ob alles richtig ein- & ausgelesen wird

(Mit print Statements Werte vergleichen)

Ansonsten probiere ich es die Tage selbst mal aus...

Allrounder18 · 14. November 2019

Ich hab jetzt mal Prints gesetzt, AccountLogin ruft er auf.

Beim Registrieren ruft er anscheinend nichts auf.

Edit: Das sind die Funktionen für die Registrierung.

C

stock registerdialog(playerid, dialogid, inputtext[])
{
    switch(dialogid)
    {
        case DIALOG_REGISTER:
        {
            if(strlen(inputtext) < 6 || strlen(inputtext) > 24)
            {
                ShowPlayerDialog(playerid,DIALOG_REGISTER,DIALOG_STYLE_INPUT,"Registration","Bitte gib ein Passwort für deinen Account an:","Bestätigen","Abbrechen");
                bcrypt_hash(inputtext, BCRYPT_COST, "OnPasswordHashed", "d", playerid);
            }
            return 1;
        }
    }
    return 1;
}

FUNCTION AccountRegister(playerid)
{
    printf("ACR");
    new string [512];
    AccountInfo[playerid][account_ID] = cache_insert_id();
    AccountInfo[playerid][account_Eingeloggt]  = true;
    mysql_format(handle, string, sizeof(string), "SELECT * FROM `character` WHERE `AccountID` = '%d'", AccountInfo[playerid][account_ID]);
    mysql_tquery(handle, string, "ShowCharacter", "d", playerid);
    return 1;
}

FUNCTION AccountCheck(playerid)
{
    printf("ACC");
    new rows,string[255];
    cache_get_row_count(rows);
    if(rows)
    {
        format(string,sizeof(string),"{FFFFFF}Der Account {E8BD1A}%s {FFFFFF}wurde gefunden\nBitte melde dich an",AccountInfo[playerid][account_Name]);
        ShowPlayerDialog(playerid, DIALOG_LOGIN, DIALOG_STYLE_PASSWORD, "{FFFFFF}Anmelden",string, "{FFFFFF}Weiter", "");
        //SetTimerEx("KickPlayer", 100, false, "i", playerid);
    }
    else
    {
        format(string,sizeof(string),"{FFFFFF}Der Account {E8BD1A}%s {FFFFFF}wurde nicht gefunden\nBitte erstelle dir einen Account",AccountInfo[playerid][account_Name]);
        ShowPlayerDialog(playerid, DIALOG_INFO, DIALOG_STYLE_PASSWORD, "{FFFFFF}Fehler",string, "{FFFFFF}Weiter", "");
        //SetTimerEx("KickPlayer", 100, false, "i", playerid);
    }
    return 1;
}

FUNCTION OnPasswordHashed(playerid)
{
    printf("OPWH");
    new hash[BCRYPT_HASH_LENGTH],query[256];
    bcrypt_get_hash(hash);
    mysql_format(handle, query, sizeof(query), "INSERT INTO `accounts` (`Name`, `Password`) VALUES ('%e', '%e')", AccountInfo[playerid][account_Name], hash);
    mysql_pquery(handle, query, "AccountRegister", "d", playerid);
    return 1;
}

Alles anzeigen