[PHP][MySQL] Logowanie

[master of disaster]

Witam!

Mam problem z logowaniem, otóż jak odświerze strone to już nie jestem zalogowany.
kod:

Kod: Zaznacz cały


<?php
session_start();

mysql_connect('localhost','root','haslo') or die(mysql_error());
mysql_select_db('nk') or die(mysql_error());

$user = strip_tags($_POST['user']);
$pass = strip_tags(md5($_POST['pass']));
$login=$_POST['login'];
if($login) {
   
   $rezultat=mysql_query("select id FROM tabela WHERE user='$user' AND pass='$pass'");
   
   if(@mysql_num_rows($rezultat)) { 
      $row = mysql_fetch_row($rezultat);
      $_SESSION['uzytkownik'] = $row[0];
      include("includes/logged.php");
      exit;
   }
   else {
      echo "Podales zle dane. Sproboj ponownie.";
   }
   
 }
else {
        include("includes/not_logged.php");
        exit;
}

if (isset($_SESSION['uzytkownik'])){
    include("includes/logged.php");
    exit;
}
else {
    include("includes/not_logged.php");
    exit;
}
?>


[kostek135]

Bo jest to źle napisane po przeładowaniu strony zmienna post zostaje wyzerowana

Kod: Zaznacz cały

if($login) {
   
   $rezultat=mysql_query("select id FROM tabela WHERE user='$user' AND pass='$pass'");
   
   if(@mysql_num_rows($rezultat)) {
      $row = mysql_fetch_row($rezultat);
      $_SESSION['uzytkownik'] = $row[0];
      include("includes/logged.php");
      exit;
   }
   else {
      echo "Podales zle dane. Sproboj ponownie.";
   }
   
}
else {
        include("includes/not_logged.php");
        exit;
}

Wykonuje się więc ostatni else.

[master of disaster]

To co zrobić, usunąć to else ? możesz podać kod ?

[kostek135]

Kod: Zaznacz cały

<?php
session_start();

mysql_connect('localhost','root','haslo') or die(mysql_error());
mysql_select_db('nk') or die(mysql_error());

$user = strip_tags($_POST['user']);
$pass = strip_tags(md5($_POST['pass']));
$login=$_POST['login'];
if($login)
{
   $rezultat=mysql_query("select id FROM tabela WHERE user='$user' AND pass='$pass'");
   if(@mysql_num_rows($rezultat))
   {
      $row = mysql_fetch_row($rezultat);
      $_SESSION['uzytkownik'] = $row[0];
   }
   elseif(!isset($_SESSION['uzytkownik']))
   {
      echo 'Podales zle dane. Sproboj ponownie.';
   }
}
if(isset($_SESSION['uzytkownik']))
{
   include('includes/logged.php');
}
else
{
   include('includes/not_logged.php');
}
?>

Powinno być dobrze, niemniej nie sprawdzałem.

EDIT uaktualniłem kod, zapomniałem o warunku co będzie, jeśli sesja już istnieje, a zmienna login nie będzie istnieć.

[master of disaster]

Dzięki wielkie

[Szarex]

A ja mam pytanie dotyczące tej linijki:

Kod: Zaznacz cały

$pass = strip_tags(md5($_POST['pass'])); 

Wynik funkcji md5 nie zawiera już tagów. Po co stosować więc strip_tags dla ciągu md5? Jakiś sens miało by za to:

Kod: Zaznacz cały

$pass = md5(strip_tags($_POST['pass']));