Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> problemy ze skryptem logowania
Terrorizer
post
Post #1





Grupa: Zarejestrowani
Postów: 345
Pomógł: 3
Dołączył: 31.03.2012

Ostrzeżenie: (0%)
-----


Witam,
Od kilku niedługiego czasu zajmuje się nauką php.
Obecnie jestem w trakcie tworzenia mojego pierwszego projektu, którym jest prosta gra przeglądarkowa.
Mam problem ze skryptem logowania, który ciągle wywala mi jakieś błędy, gdy naprawiam jeden, wyskakuje kolejny i tak bez końca haha (IMG:style_emoticons/default/tongue.gif)
Morze będzie ktoś w stanie zerknąć okiem i wytknąć mi kilka rzeczy widocznych gołym okiem?

w tej chwili wyskakuje mi: "Notice: Undefined variable: users in C:\wamp\www\login.php on line 38" (czyli tutaj linijka zawierająca if md5), oraz "login failed" wyświetla się na ekranie
z góry dziękuje
  1. <?php
  2. mysql_connect('localhost','root','');
  3. mysql_select_db('users') or die('cannoct connect with database');
  4.  
  5. if (($_POST['login'] !='') && ($_POST['password'] !=''))
  6. {
  7. $login=$_POST['login'];
  8. $password=$_POST['password'];
  9. echo $login;
  10.  
  11. $loginresult=mysql_query("SELECT * FROM users WHERE login='$login' LIMIT 1");
  12. if (mysql_num_rows($loginresult)===1)
  13. if (md5($password)===$users['password'])
  14. echo '<p>login successful</p>';
  15.  
  16. else
  17. echo '<p>login failed</p>';
  18. }
  19. ?>


Ten post edytował Terrorizer 2.07.2012, 14:34:52
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
!*!
post
Post #2





Grupa: Zarejestrowani
Postów: 4 298
Pomógł: 447
Dołączył: 16.11.2006

Ostrzeżenie: (0%)
-----


Wyłącz komputer i idź się przejść. Jutro popraw kod

1. łącz się z bazą kiedy jest to konieczne, czyli pierw sprawdź czy formularz został wysłany przez isset
2. tą samą funkcją sprawdzaj czy dane istnieją, jeśli tak, nawiąż połączenie.
3. nie sprawdzaj czy sesja jest inna niż '', tylko sprawdź czy w ogóle istnieje oraz jaką ma wartość
4. nazwy plików to nie edytor tekstu, zamiast spacji używaj pokreslnika _ lub nazwaPliku.php
5. jeśli chcesz się czegoś nauczyć, to zastosuj od rpazu PDO do łączenia z bazą, zapomnij o mysql_* ponieważ są wycofywane.
Go to the top of the page
+Quote Post

Posty w temacie


Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 23.08.2025 - 10:03