Tjek om data er der i forvejen

Hejsa,

Først og fremmest vil jeg personligt anbefale OOP versionen af MySQLi. Det er dog helt personligt en favorit.

Anyway - dit problemet ligger i, at mysqli_num_rows kun kan kaldes, såfremt der ligger information for hele dit statement i "bufferen". Af samme årsag virker Jespers kode, da han beder MySQLi om, at gemme resultatet af dit statement i denne buffer ($stmt -> store_result()). Du ville, såfremt du tjekkede op på dokumentationen for num_rows, også selv have fundet samme løsning, her står:

Returns the number of rows in the result set. The use of mysqli_stmt_num_rows() depends on whether or not you used mysqli_stmt_store_result() to buffer the entire result set in the statement handle.

If you use mysqli_stmt_store_result(), mysqli_stmt_num_rows() may be called immediately.

Det betyder altså, at den information som dit statement returnerer, gemmes i hukommelsen. Alternativt kunne du vælge at bruge et loop. I dette loop tæller du plus 1 til en given variabel. Herfra kan du så bruge den variabel som reference punkt.

----

Jeg anbefaler dig at gøre brug af MySQLi metoden "prepare". Den er sikkere fremfor direkte brug af variabler direkte i dit kald.
Se nedenfor (objekt-orienteret!)

 <?php
 # Opret en ny instance af mysqli-klassen.
 $dbh = new mysqli('localhost', 'root', '', 'usermanagement');
 # Check om vi har adgang til den givne database.
 if (! $dbh -> connect_errno)) 
 {
     # Ingen connection fejl.. Lad os lave nogle statements.
     if ($sth = $dbh -> prepare("SELECT `email` FROM `users` WHERE `email` = ?")) 
     {
         # Vores variable
         $email = $_GET['email'];
         # Bind vores variable som en parameter i dit database-kald.
         $sth -> bind_param('s', $email);
         # Fuldfør vores database-kald.
         $sth -> execute();
         # GEM RESULTATET I HUKOMMELSEN!
         $sth -> store_result();
         # Check herefter num_rows.
         if($sth -> num_rows > 0) 
         {
             echo 'Issues - occupied.';
         }
         else 
         {
             echo 'No issues - not occupied';
         }
     }
 }
 # Luk vores database up-link.
 $dbh -> close();
  ?>

vh,

Nogle der kan hjælpe, hvordan jeg får den til at tjekke om brugernavn og email er optaget? Jeg har selv arbejdet lidt på det ud fra mysql, men det virker desværre bare ikke når jeg har lavet det om til mysqli.
En der kan forklare hvorfor og hjælpe mig videre?

PHP kode 

 $sql = mysqli_query($con,"SELECT * FROM `members` WHERE `username`='".$username."'");
     $data = mysqli_num_rows($sql);
       if($data > 0){
       $error .="Brugernavnet ".$username.", er optaget.<br />";
       }
       $sql2 = mysqli_query($con,"SELECT * FROM `members` WHERE `email`='".$email."'");
       $data2 = mysqli_num_rows($sql2);
       if($data2 > 0){
       $error .="E-mail adressen ".$email." er allerede oprettet.<br />";
       }

Prøve at kig her :

 <?php
 if ($stmt = $this->mysqli->prepare("SELECT brugernavn FROM bruger WHERE brugernavn = ?"))
 {
       $stmt->bind_param('s', $brugernavn);
       $brugernavn = $_POST["email"];
 
 
       $stmt->execute();
       $stmt->store_result();
       $stmt->bind_result($email);
       $stmt->fetch();
       $count = $stmt->num_rows;
       $stmt->close();
 
     if($count > 0)
     {
         //optaget
     }
     else
     {
         //ligger ind i databasen
     }
 }
 ?>

jeg bruger den selv tit og den virker fint hvis du kan finde ud af sæt den op selv.

Jeg kan desværre ikke rigtig få det til at virke.
Jeg modtager hele tiden "Warning: mysqli_stmt::bind_param(): Number of variables doesn't match number of parameters in prepared statement" - og har nu prøvet at google det, men synes ikke helt jeg får svar på hvad den betyder, samt hvad fejlen kan være.?

Jeg har sporet mig ind på at det må være $data -> bind_param('s', $username); der driller. Skal det forresten ikke være POST i stedet for bare $username?

 if ($data = $con -> prepare("SELECT `username` FROM `members` WHERE `username` = '".$username."'"))
 {
 $data -> bind_param('s', $username);
 $data -> execute();
 $data -> store_result();
 if($data -> num_rows > 0)
       $error .="Brugernavnet ".$username." er optaget.<br />";
       }

Skal det forresten ikke være POST i stedet for bare $username?

du skal selvføgelig tildele din variabel $username en værdi, før du tester

men ud fra den code du viser har du tildelt en værdi allerede via post/get/request, eller sat en fast værdi

så før
if ($data = $con -> prepare("SELECT `use
skal du have noget ligne dette $username = $_POST['brugernavn'];

Først og fremmest vil jeg personligt anbefale OOP versionen af MySQLi. Det er dog helt personligt en favorit.

helt enig, OOP vertionen er også marginalt hurtigere da du ikke skal kalde wrapper functionerne mysqli_xxxxx
men for hurtigt at komme igang med mysqli er der intet ivejen for at starte med procedural style som man er vandt til fra det gamle mysql, procedural style blev netop lavet som en let overgang til mysqli
man kan godt blande OOP style og procedural style, men det er pænest at holde stilen rent

denne linje
if ($data = $con -> prepare("SELECT `username` FROM `members` WHERE `username` = '".$username."'"))
skal da se sådan ud
if ($data = $con -> prepare("SELECT `username` FROM `members` WHERE `username` = ?"))
? er en placeholder som du binder i linjen neden under.

hvis du ikke brugere placeholder har du ikke nogle gavn af bind

Beklager det sene svar, men har lige været igennem en akut operation.

Jeg har dog stadig ikke fundet ud af min fejl og er desværre blevet lidt forvirret af det sidste svar Nu kan jeg lige prøve at vedhæfte det hele (uden rettelser fra sidste svar) og håber en er frisk på at hjælpe mig med problemet og forståelse af det

 <?php
     $username = $_POST['username'];
     $password = $_POST['password'];
     $email = $_POST['email'];
     $name = $_POST['name'];
     $gender = $_POST['gender'];
     $date = $_POST['birth_year'] . "-" . $_POST['birth_month'] . "-" . $_POST['birth_day'];
     $ip = getenv('REMOTE_ADDR');
     
     if(isset($_POST['submit']))
     {
     if($username == ""){
       $error .="Angiv et brugernavn<br />";
       }
       if($password == ""){
       $error .="Angiv et password<br />";
       }
       if($email == ""){
       $error .="Angiv din email<br />";
       }
        if (substr_count($email,'@')==0) {
       $error .="Ugyldig e-mail adresse!<br />";
        }
        if($name == ""){
       $error .="Angiv dit fornavn<br />";
       }
       if($gender == ""){
       $error .="Angiv dit køn<br />";
       }
     { 
     if ($data = $con -> prepare("SELECT `username` FROM `members` WHERE `username` = '".$username."'"))
 {
 $data -> bind_param('s', $username);
 $data -> execute();
 $data -> store_result();
 if($data -> num_rows > 0)
       $error .="Brugernavnet ".$username.", er optaget.<br />";
       }
     }
     if($error){
       echo $error;
       }
       else 
       {
     $date = date('Y-m-d', strtotime($date));
     $query = "INSERT INTO members (username, password, email, name, gender, date, ip, signup) VALUES ('{$username}', '{$password}', '{$email}', '{$name}', '{$gender}', '{$date}', '{$ip}', now())" or die (mysqli_error($myConnection));
     $result = mysqli_query($con, $query);
         if ($result) {
             $message = "Brugeren er oprettet.";
         } 
         else {
             $message = "Brugeren er ikke oprettet.";
             $message .= "<br />" . mysqli_error($con);
         }
 }
 
     }
 
 if (!empty($message)) {echo "<p>" . $message . "</p>";}
 ?>

Jeg bliver lidt forvirret når I taler om OOP versionen af MySQLi? Er der nogle gode artikler, hvor jeg kan læse mere om det her.

Du kan benytte UNIQUE på username og email, så kan de kun fremkomme engang i den table i din database.

du kan kigge på http://www.w3schools.com/sql/sql_unique.asp

HJÆLP SØGES ENDNU ...


Kenneth: Det kunne man, men "desværre" er jeg stædig nok til at ville lære dette

Andre der kan hjælpe lidt, hvad det er jeg gør forkert?