Fehler im Registersystem

Allrounder18 · 1. Februar 2020

Hallo!

Ich bin gerade dabei mein Registersystem zu bearbeiten, jedoch habe ich ein kleines Problem.

Code

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    registerdialog(playerid, dialogid, inputtext);
    return 1;
}

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

FUNCTION AccountCheck(playerid)
{
    printf("ACC");
    new rows,string[255];
    cache_get_row_count(rows);
    if(rows == 0)
    {
        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_REGISTER, DIALOG_STYLE_PASSWORD, "{FFFFFF}Fehler",string, "{FFFFFF}Weiter", "");
        //SetTimerEx("KickPlayer", 100, false, "i", playerid);
    }
    else
    {
        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);
    }
    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`, `Passwort`) VALUES ('%e', '%e')", AccountInfo[playerid][account_Name], hash);
    mysql_pquery(handle, query, "AccountRegister", "d", playerid);
    return 1;
}

FUNCTION AccountRegister(playerid)
{
    printf("ACR");
    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;
}

Alles anzeigen

Aus irgendeinem Grund wird der Register Dialog doppelt angezeigt. Die Funktionen werden alle aufgerufen, jedoch doppelt, sodass ich zwei mal den gleichen Account mit unterschiedlichem PW in der DB habe. Was ist da inkorrekt?

MfG

Allrounder18

Kaliber · 2. Februar 2020

Lösche Zeile 14...

Wieso zeigst du auch den Dialog 2x an...das macht ja keinen Sinn, dort hat er ja bereits sein pw eingegeben xD

Allrounder18 · 2. Februar 2020

Hab ich gemacht, es finktoiert jetzt. Ist es normal dass dann nurmehr accountcheck und registerdialog geprintet bzw aufgerufen werden?

Kaliber · 2. Februar 2020

Also AccountCheck muss immer aufgerufen werden, sonst weißt du ja nicht, ob der Spieler schon registriert ist oder nicht.

Und RegisterDialog wird bei dir immer aufgerufen...

Das könntest du ja so schreiben:

C

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

Alles anzeigen

Allrounder18 · 2. Februar 2020

Jetzt wird alles richtig aufgerufen. Was muss ich machen wenn ich bei AccountCheck nen Abbrechen Button habe damit man gekickt wird?

Edit: Wenn ich aktuell auf Esc drücke und nichts eingebe komme ich trotzdem weiter.

Kaliber · 2. Februar 2020

C

stock registerdialog(playerid, dialogid, inputtext[])
{
    if(!response) return Kick(playerid);
    if(6 <= strlen(inputtext) <= 24)
    {
        bcrypt_hash(inputtext, BCRYPT_COST, "OnPasswordHashed", "d", playerid);
    }
    else ShowPlayerDialog(playerid,DIALOG_REGISTER,DIALOG_STYLE_PASSWORD,"Registration","Bitte gib ein Passwort für deinen Account an:","Bestätigen","Abbrechen");
    return 1;
}

Allrounder18 · 2. Februar 2020

das bringt irgendwie nix, ich werde bei abbrechen immernoch nicht gekickt. muss dan ich t bei Accpuntcheckauch was hin?

Edit: Fail, ich hatte bei accountcheck if(rows == 0)

Kaliber · 2. Februar 2020

Zitat von Allrounder18

das bringt irgendwie nix, ich werde bei abbrechen immernoch nicht gekickt. muss dan ich t bei Accpuntcheckauch was hin?

Nein, da muss nur angegeben werden, dass es einen Abbrechen Button gibt.

Mal doof gefragt, wie rufst denn jetzt registerdialog auf, da ja response gar nicht in der Funktion definiert ist

Übergibst du das denn richtig? xD

Allrounder18 · 2. Februar 2020

Code

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    switch(dialogid)
    {
        case DIALOG_REGISTER: registerdialog(playerid, response, inputtext);
    }
    return 1;
}

stock registerdialog(playerid, response, inputtext[])
{
    if(!response) return Kick(playerid);
    if(6 <= strlen(inputtext) <= 24)
    {
        bcrypt_hash(inputtext, BCRYPT_COST, "OnPasswordHashed", "d", playerid);
    }
    else ShowPlayerDialog(playerid,DIALOG_REGISTER,DIALOG_STYLE_PASSWORD,"{FFFFFF}Registration","{FFFFFF}Bitte gebe ein Passwort für deinen Account an!","{FFFFFF}Bestätigen","{FFFFFF}Abbrechen");
    return 1;
}

Alles anzeigen

Aus irgendeinem Grund sagt mir das Game jetzt is ist ein Account vorhanden obwohl keiner da ist.

Edit: Hab den Fehler gefunden, hab die dialoge vertauscht, darum stahd da der acc wurde gefunden.

Kaliber · 2. Februar 2020

Zitat von Allrounder18

Aus irgendeinem Grund sagt mir das Game jetzt is ist ein Account vorhanden obwohl keiner da ist.

Wie und wo rufst n AccountCheck auf?

Allrounder18 · 2. Februar 2020

Code

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", "{FFFFFF}Abbrechen");
    }
    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_REGISTER, DIALOG_STYLE_PASSWORD, "{FFFFFF}Fehler",string, "{FFFFFF}Weiter", "{FFFFFF}Abbrechen");
    }
    return 1;
}

Alles anzeigen

Kaliber · 2. Februar 2020

Ja, das ist deine Funktion, aber wie rufst du sie auf...

Allrounder18 · 2. Februar 2020

Code

public OnPlayerConnect(playerid)
{
    new string[128];
    GetPlayerName(playerid,AccountInfo[playerid][account_Name],MAX_PLAYER_NAME);
    mysql_format(handle, string, sizeof(string), "SELECT * FROM `accounts` WHERE `Name` = '%e'", AccountInfo[playerid][account_Name]);
    mysql_tquery(handle, string, "AccountCheck", "d", playerid);
    return 1;
}

Edit: Weiterer Fehler:

Code

stock logindialog(playerid, response, inputtext[])
{
    if(!response) return Kick(playerid);
    if(6 <= strlen(inputtext) <= 24)
    {
        bcrypt_check(inputtext, AccountInfo[playerid][account_Passwort], "OnPasswordChecked", "d", playerid);
    }
    //else ShowPlayerDialog(playerid,DIALOG_PW_CHECK,DIALOG_STYLE_INPUT,"Falsches Passwort","Dein eingegebenes Passwort ist falsch!","Bestätigen","Abbrechen");
    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);
    return 1;
}

stock pwcheckdialog(playerid, response, inputtext[])
{
    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_Passwort], "OnPasswordChecked", "d", playerid);
    return 1;
}

FUNCTION AccountLogin(playerid, inputtext[])
{
    printf("ACL");
    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_Passwort]);
        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, "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_Passwort], "OnPasswordChecked", "d", playerid);
    }
    return 1;
}

FUNCTION OnPasswordChecked(playerid)
{
    printf("OPWC");
    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

So sieht meine Loginfunktion aus, somit wird überprüft oft das Passwort richtig ist. Irgendwas ist hier falsch, immer wenn ich das richtige Passwort eingebe komt der Dialog, dass das PW falsch ist.

breadfish.de 12. März 2022