[php+MySQL] Logowanie, Proszę o pomoc... Opcje

[lukas_jg]

Witam
Próbowałem napisać własne logowanie ale co chwile coś mi zawodzi.
-pobiera userów z bazy mysql
-sprawdza czy wpisane haslo i pass jest ok jezeli ok to ustawia zmienna xx= '1' i ustawia cookies loginu i hasła po to aby trzymało logowanie
-wylogowanie po klikniaciu na log out




działało przy jednym userze w bazie

[PHP] pobierz, plaintext 
<?php
session_start();
 
if($page == "logout&")
{ 
session_destroy(); 
 
?>
<meta http-equiv="Refresh" content="0; URL=">
<?
 
exit;
}
 
$xx = '0';
 
$link = mysql_connect(xxxxxxxx.pl', 'xxxxxxxxxxxxxx', 'xxxxxxxxx') or die ("Nie masz uprawnien");
mysql_select_db("xxxxxxx");
$query = "SELECT * FROM users ORDER BY data DESC";
$result = mysql_query ($query);
$ile = mysql_num_rows($result);
 
for($a=0;$a<$ile;$a++)
{
$row=mysql_fetch_array($result);
if(($login == $row['email']) and ($passwd == $row['pass'])){
$xx = '1';
session_register('login');
session_register('passwd');
}
else { if($xx == '0') session_destroy(); }
}
 
 
 
 
?>
[PHP] pobierz, plaintext 

Ten post edytował lukas_jg 22.08.2006, 14:09:34

[Coyote]

uzywaj tagow php.... bo tak to nawet nie chce mi sie czytac tego kodu

[lukas_jg]

Nikt nie wie ?

[rama]

Hmm, spróbuj leciutko zmodyfikować swój skrypt. Zakładam, że dane userów (takie jak email i haslo) przesyłaś poprzez formularz, racja? Wówczas wystarczy zrobić procedure logowania wyłącznie dla jednego użytkownika, czyli możesz zmodyfikować swój sql syntax, aby zawierał dodatkowy składnik, tj.

[SQL] pobierz, plaintext 
... WHERE email = $_POST['email'] ...
[SQL] pobierz, plaintext 

Składnia ta wybierze jedynie jeden wiersz z danymi odpowiedniego użytkownika, więc w efekcie wystarczy porównać hasło znajdujące się w bazie z tym z formularza

PS Zastanawiam sie, po co robisz pętle?

[PHP] pobierz, plaintext 
<?php for($a=0;$a<$ile;$a++) { ... } ?>
[PHP] pobierz, plaintext 

PS1

[HTML] pobierz, plaintext 
<meta http-equiv="Refresh" content="0; URL=">
[HTML] pobierz, plaintext 

możesz zamienić na

[PHP] pobierz, plaintext 
<?php header( 'Location: index.php' ); ?>
[PHP] pobierz, plaintext 

Ten post edytował rama 22.08.2006, 15:59:09

[piwoszeq]

calosc jest zle.
Lepiej napisz od nowa

[Cysiaczek]

Masz tak:

[PHP] pobierz, plaintext 
<?php
$query = "SELECT * FROM users ORDER BY data DESC";
?>
[PHP] pobierz, plaintext 

Czyli wybierze wszystkie rekordy z bazy, a Tobie jest potrzebny jeden konkretny prawda?
Sprawdź za pomocą mysql_num_rows() ile rekordów wybrałeś i zastanów sie, co się dzieje z Twoją zmienną $xx - jakie wartości przybiera i ile razy.

Pozdrawiam.