php - Loginscript volder problemer efter skift - Udvikleren.dk

Tags: php mysqli

Bruger #17072 @ 12.09.12 17:11

Hej
Efter at være gået igang med at udskifte MySQL-API'et på min side med MySQLi har jeg indtil videre kun haft ganske få problemer. Men nu har jeg altså fundet et som jeg har en smule svært ved at genneskue.
For det første har jeg lavet selve scriptet til at hente brugeroplysningerne her:

PHP kode

 if ($stmt = $mysqli->prepare('SELECT `id`, `username`, `navn`, `virksomhed`, `usergroup` FROM `users` WHERE `username` = ? AND ´password´ = ?')) {
     /* Bind parametre */
     $stmt->bind_param('ss', $username, $password);
     /* Sæt værdier på parametrene */
     $username = $_POST['username'];
     $password = md5($_POST['password']);
     /* Eksekver forespørgslen */
     $stmt->execute();
     /* Bind resultatet */
     $stmt->bind_result($user_id, $brugernavn, $name, $virksomhed, $usergroup);
     $stmt->fetch();
 }

Så har jeg koblet et ekstra if/else-statement på, som skal tjekke om der overhovedet er fundet data:

PHP kode

 if ($user_id > 0)
  {
     //Hvis brugeren findes bliver de logget ind
     $_SESSION['loggedin'] = 1;
     $_SESSION['username'] = $brugernavn;
     $_SESSION['identifikation'] = $user_id;
     
     //Henter brugergruppe, brugerens rigtige navn, samt virksomhedens navn
     $_SESSION['usergroup'] = $usergroup;
     $_SESSION['navn'] = $name;
     $_SESSION['virksomhed'] = $virksomhed;
 
     //redirect til kundeside uden fejl
     header('Location: ../kunde.php');
  }
  else
  {
     //Redirect til index med fejl besked.
     $_SESSION['fail'] = "Du blev desværre ikke fundet i systemet. Kontroller brugernavn og password og prøv igen.";
     header('Location: ../index.php');
  }

I sidst nævnte kode for jeg fejlen fra else, altså værdien af $_SESSION['fail'].
Men forstår bare ikke hvorfor den ikke vil gøre som den skal. Jeg har tjekket at mit password er korrekt indtastet, men hvorfor det ikke virker, kan jeg ikke finde ud af.

9 svar postet i denne tråd vises herunder
0 indlæg har modtaget i alt 0 karma

Sorter efter stemmer Sorter efter dato

Bruger #16025 @ 12.09.12 17:35

442

Bare for en god ordens skyld: har du husket session_start() ?

Bruger #17072 @ 12.09.12 17:46

152

Ja, den har jeg placeret øverst i min fil. Uden den er mine sessions jo ubrugelige.

EDIT: Blot for god ordens skyld, så er her det fulde login-script:

PHP kode

 session_start(); 
  
  include '../inc/connect.php';
 
 //Lav en query der henter den ønskede bruger i stedet for alle sammen
 if ($stmt = $mysqli->prepare('SELECT `id`, `username`, `navn`, `virksomhed`, `usergroup` FROM `users` WHERE `username` = ? AND ´password´ = ?')) {
     /* Bind parametre */
     $stmt->bind_param('ss', $username, $password);
     /* Sæt værdier på parametrene */
     $username = $_POST['username'];
     $password = md5($_POST['password']);
     /* Eksekver forespørgslen */
     $stmt->execute();
     /* Bind resultatet */
     $stmt->bind_result($user_id, $brugernavn, $name, $virksomhed, $usergroup);
     $stmt->fetch();
 }
 else {
     $_SESSION['fail'] = 'Der opstod en fejl i erklæringen: ' . $mysqli->error;
 }
  if ($user_id > 0)
  {
     //Hvis brugeren findes bliver de logget ind
     $_SESSION['loggedin'] = 1;
     $_SESSION['username'] = $brugernavn;
     $_SESSION['identifikation'] = $user_id;
     
     //Henter brugergruppe, brugerens rigtige navn, samt virksomhedens navn
     $_SESSION['usergroup'] = $usergroup;
     $_SESSION['navn'] = $name;
     $_SESSION['virksomhed'] = $virksomhed;
 
     //redirect til kundeside uden fejl
     header('Location: ../kunde.php');
  }
  else
  {
     //Redirect til index med fejl besked.
     $_SESSION['fail'] = "Du blev desværre ikke fundet i systemet. Kontroller brugernavn og password og prøv igen.";
     header('Location: ../index.php');
  }

Indlæg senest redigeret d. 12.09.2012 18:12 af Bruger #17072

Bruger #16025 @ 12.09.12 18:13

442

Nåå læste det nederste i dit spørgsmål forkert. Sorry.

Hvor sætter du variablen $user_id henne? Og hvad får den indhold af. For jeg ville tro at du skulle gøre noget lign.:

PHP kode

 if($stmt->num_rows > 0)
 {
     // ...
 }

Bruger #17072 @ 12.09.12 18:14

152

Jeg får $user_id herfra:
$stmt->bind_result($user_id, $brugernavn, $name, $virksomhed, $usergroup);

Bruger #16025 @ 12.09.12 18:23

442

Ja, men skal du ikke tjekke på om der er nogle resultater? Ligesom mysql_num_rows? For så ville jeg mene at du skulle erstatte

PHP kode

 if ($user_id > 0)

med

PHP kode

 if($stmt->num_rows > 0)

Bruger #17072 @ 12.09.12 18:31

152

I MySQL har jeg gjort således:

PHP kode

 session_start(); 
  
  include '../inc/connect.php';
  
 //Henter oplysninger fra login-side
  $username = mysql_real_escape_string($_POST['username']);
  $password = mysql_real_escape_string($_POST['password']);
  
 //Lav en query der henter den ønskede bruger i stedet for alle sammen
  $data = mysql_query("SELECT * FROM users WHERE username = '" . $username . "' AND password = '" . md5($password) . "'") or die(mysql_error());
 
  if ($info = mysql_fetch_array($data))
  {
     //Hvis brugeren findes bliver de logget ind
     $_SESSION['loggedin'] = 1;
     $_SESSION['username'] = $info['username'];
     $_SESSION['identifikation'] = $info['id'];
     
     //Henter brugergruppe, brugerens rigtige navn, samt virksomhedens navn
     $_SESSION['usergroup'] = $info['usergroup'];
     $_SESSION['navn'] = $info['navn'];
     $_SESSION['virksomhed'] = $info['virksomhed'];
 
     //redirect til index uden fejl
     header('Location: ../kunde.php');
  }
  else
  {
     //Redirect til index med fejl besked.
     $_SESSION['fail'] = "Du blev desværre ikke fundet i systemet. Kontroller brugernavn og password og prøv igen.";
     header('Location: ../index.php');
  }

Fordi i MySQL gav det fejl, hvis min array var tomt eller ikke kunne bygges. Men havde lidt på fornemmelsen at jeg skulle bygge det lidt anderledes med MySQLi. Dog har jeg lige forsøgt med den ændring i koden du foreslog, dog uden resultat.

Bruger #17305 @ 12.09.12 21:34

Nu skal du jo huske at PHP læser oppe fra og ned ikke hist og her derfor skal du huske at sætte $password og $username ovenover

sådan her

PHP kode

 session_start();
     include '../inc/connect.php';
     //Lav en query der henter den ønskede bruger i stedet for alle sammen
     if ($stmt = $mysqli->prepare('SELECT `id`, `username`, `navn`, `virksomhed`, `usergroup` FROM `users` WHERE `username` = ? AND ´password´ = ?')) {
 
     $username = $_POST['username'];
     $password = md5($_POST['password']);
     /* Bind parametre */
     $stmt->bind_param('ss', $username, $password);
     /* Eksekver forespørgslen */
     $stmt->execute();
     /* Bind resultatet */
     $stmt->bind_result($user_id, $brugernavn, $name, $virksomhed, $usergroup);
     $stmt->fetch();
     }
     else {
     $_SESSION['fail'] = 'Der opstod en fejl i erklæringen: ' . $mysqli->error;
     }
     if ($user_id > 0)
     {
     //Hvis brugeren findes bliver de logget ind
     $_SESSION['loggedin'] = 1;
     $_SESSION['username'] = $brugernavn;
     $_SESSION['identifikation'] = $user_id;
     //Henter brugergruppe, brugerens rigtige navn, samt virksomhedens navn
     $_SESSION['usergroup'] = $usergroup;
     $_SESSION['navn'] = $name;
     $_SESSION['virksomhed'] = $virksomhed;
     //redirect til kundeside uden fejl
     header('Location: ../kunde.php');
     }
     else
     {
     //Redirect til index med fejl besked.
     $_SESSION['fail'] = "Du blev desværre ikke fundet i systemet. Kontroller brugernavn og password og prøv igen.";
     header('Location: ../index.php');
     }

Virker det ?

Bruger #3427 @ 12.09.12 21:58

1.520

Nu bruger jeg selv PDO men et bud:

PHP kode

 if ($stmt = $mysqli->prepare('SELECT `id`, `username`, `navn`, `virksomhed`, `usergroup` FROM `users` WHERE `username` = ? AND ´password´ = ?')) 
 {
     /* Bind parametre */
     $stmt->bind_param('ss', $username, $password);
     /* Sæt værdier på parametrene */
     $username = $_POST['username'];
     $password = md5($_POST['password']);
     /* Eksekver forespørgslen */
     $stmt->execute();
     /* Bind resultatet */
     $stmt->bind_result($user_id, $brugernavn, $name, $virksomhed, $usergroup);
     if($stmt->fetch())
     {
         if ($user_id > 0)
         {
              //Hvis brugeren findes bliver de logget ind
              $_SESSION['loggedin'] = 1;
              $_SESSION['username'] = $brugernavn;
              $_SESSION['identifikation'] = $user_id;
              
              //Henter brugergruppe, brugerens rigtige navn, samt virksomhedens navn
              $_SESSION['usergroup'] = $usergroup;
              $_SESSION['navn'] = $name;
              $_SESSION['virksomhed'] = $virksomhed;
 
              //redirect til kundeside uden fejl
              header('Location: ../kunde.php');
              exit;
         }
     }
  }
  
  if(!isset($_SESSION['identifikation']))
  {
      //Redirect til index med fejl besked.
      $_SESSION['fail'] = "Du blev desværre ikke fundet i systemet. Kontroller brugernavn og password og prøv igen.";
      header('Location: ../index.php');
      exit;
  }

Indlæg senest redigeret d. 12.09.2012 21:59 af Bruger #3427

Bruger #17072 @ 13.09.12 17:24

152

@Rix: Gav det et forsøg, men desværre ingen forskel.
@Kenneth: Forsøgte at rette det til som du havde skrevet det og fik desværre stadig en fejl.

Nogen andre gode forslag?

EDIT: Så fandt jeg ud af det. Her er løsningen:

PHP kode

 session_start(); 
 include '../inc/connect.php';
 
 //Lav en query der henter den ønskede bruger i stedet for alle sammen
 if ($stmt = $mysqli->prepare('SELECT `id`, `username`, `navn`, `virksomhed`, `usergroup` FROM `users` WHERE `username` = ? AND `password` = ?')) {
     
     /* Bind parametre */
     $stmt->bind_param('ss', $username, $password);
     /* Sæt værdier på parametrene */
     $username = $_POST['username'];
     $password = md5($_POST['password']);
     /* Eksekver forespørgslen */
     $stmt->execute();
     /* Bind resultatet */
     $stmt->bind_result($user_id, $brugernavn, $name, $virksomhed, $usergroup);
     $stmt->store_result();
     if ($stmt->num_rows > 0){
     $stmt->fetch();
     
     //Hvis brugeren findes bliver de logget ind
     $_SESSION['loggedin'] = 1;
     $_SESSION['username'] = $brugernavn;
     $_SESSION['identifikation'] = $user_id;
     
     //Henter brugergruppe, brugerens rigtige navn, samt virksomhedens navn
     $_SESSION['usergroup'] = $usergroup;
     $_SESSION['navn'] = $name;
     $_SESSION['virksomhed'] = $virksomhed;
 
     //redirect til kundeside uden fejl
     header('Location: ../kunde.php');
  }
  else{
     //Redirect til index med fejl besked.
     $_SESSION['fail'] = "Du blev desværre ikke fundet i systemet. Kontroller brugernavn og password og prøv igen.";
     header('Location: ../index.php');
  }
 }
 else {
     $_SESSION['fail'] = 'Der opstod en fejl i erklæringen: ' . $mysqli->error;
 }

Indlæg senest redigeret d. 16.09.2012 07:36 af Bruger #17072

Loginscript volder problemer efter skift

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler