Dit is mijn code tot nu toe:

public function register() {
$correct = false;
try {
$con = new PDO( DB_DSN, DB_USERNAME, DB_PASSWORD );
$con->setAttribute( PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION ); 
$sql = "INSERT INTO users(username, password) VALUES(:username, :password)";


$stmt = $con->prepare( $sql );
$stmt->bindValue( "username", $this->username, PDO: : PARAM_STR );
$stmt->bindValue( "password", hash("sha256", $this->password . $this->salt), PDO: : PARAM_STR );
$stmt->execute();
return "Registration Successful <br/> <a href='index.php'>Login Now</a>";
}catch( PDOException $e ) {
return $e->getMessage();
}
}

Ik hoop dat iemand mij kan helpen, ik zit nu helemaal vast, heb al een hele hoop dingen geprobeerd

Citaat van dees040

PHP

$stmt = $con->prepare( "SELECT username FROM users WHERE username = :user" );
$stmt->bindValue( ":user", $this->username );
$stmt->execute();


if ($stmt->rowCount > 0) {
  echo "Gebruikersnaam is al in gebruik.";
}

Deze code zou dat moeten oplossen!

Waar zou ik deze dan moeten plaatsen?

Citaat van dees040

Waar je wil checken of de gebruikersnaam al bestaat. Dus waarschijnlijk boven het gedeelte waar je de gebruiker aanmaakt.

Ik heb dat geprobeerd en daardoor werkt de code niet meer, zou je kunnen proberen om het te implementeren in de code die ik gaf, of heb je daarvoor de code van de registratie pagina zelf ook nodig?

Ik wilde er net op terug gaan komen, na het toevoegen van de code zoals aangeleverd door dees040, werkt de complete site niet meer
enig idee hoe dit zou kunnen komen?

Citaat van dees040

Misschien regel 5 veranderen naar:

PHP

$stmt->bindValue( "user", $this->username );

Dit doe jij ook in de code die je zelf geschreven hebt. Misschien is dat het probleem. Zelf doe ik altijd met de ":".

Als ik dit probeer dan krijg ik nog steeds dezelfde verandering, namelijk dat heel m'n site niet meer werkt

Op het moment heb ik dit :

public function register() {
    $correct = false;


    $stmt = $con->prepare( "SELECT username FROM users WHERE username = :user" );
    $stmt->bindValue( ":user", $this->username );
    $stmt->execute();


    if ($stmt->rowCount > 0) {
        echo "Gebruikersnaam is al in gebruik.";
    } else {
        try {
            $con = new PDO( DB_DSN, DB_USERNAME, DB_PASSWORD );
            $con->setAttribute( PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION ); 
            $sql = "INSERT INTO users(username, password) VALUES(:username, :password)";
            $stmt = $con->prepare( $sql );
            $stmt->bindValue( "username", $this->username, PDO: : PARAM_STR );
            $stmt->bindValue( "password", hash("sha256", $this->password . $this->salt), PDO: : PARAM_STR );
            $stmt->execute();
            return "Registration Successful <br/> <a href='index.php'>Login Now</a>";
        } catch( PDOException $e ) {
            return $e->getMessage();
        }
    }

Maar het werkt nog steeds nie, nog steeds werkt heel de site niet meer

Citaat van WebMobiel

Code

public function register() {
    
    $correct = false;
    
    try {
        $con = new PDO( DB_DSN, DB_USERNAME, DB_PASSWORD );
        $con->setAttribute( PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION );
        
        $check = $con->prepare('SELECT * FROM users WHERE username = :username');
        $items = array(':username' => $this->username);
        $check->execute($items);
        
        if($check->rowCount() >= 1)
            return 'Account bestaat al';
        
        $sql = "INSERT INTO users(username, password) VALUES(:username, :password)";
        $stmt = $con->prepare( $sql );
        $stmt->bindValue( ":username", $this->username, PDO: : PARAM_STR );
        $stmt->bindValue( ":password", hash("sha256", $this->password . $this->salt), PDO: : PARAM_STR );
        $stmt->execute();
        
        return "Registration Successful 
 Login Now";
    }
    catch( PDOException $e ) {
        return $e->getMessage();
    }
}

Meer tonen

Probeer dit eens?

Hetzelfde resultaat

Citaat van WebMobiel

Edit: Wat ik eerst poste werkt niet... Wat er nu staat wellicht wel?

Dit werkt perfect! hartelijk dank !

Citaat

if($check->rowCount() >= 1) return 'Account bestaat al';

heb je misschien een top hoe ik de return 'Account bestaat al'; laat printen naar de pagina zelf, i.p.v. naar een blank pagina?