problemy ze skryptem logowania Opcje

[Terrorizer]

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
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

[PHP] pobierz, plaintext 
<?php
mysql_connect('localhost','root','');
mysql_select_db('users') or die('cannoct connect with database');
 
if (($_POST['login'] !='') && ($_POST['password'] !=''))
{
$login=$_POST['login'];
$password=$_POST['password'];
echo $login;
 
$loginresult=mysql_query("SELECT * FROM users WHERE login='$login' LIMIT 1");
if (mysql_num_rows($loginresult)===1)
if (md5($password)===$users['password'])
echo '<p>login successful</p>';
 
else
echo '<p>login failed</p>';
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Terrorizer 2.07.2012, 14:34:52

[hind]

z tym że nie ma nigdzie $users = mysql_fetch_assoc($loginresult);

[Terrorizer]

błąd występuje niestety w dalszym ciągu
gdy wpisuje poprawne dane, które istnieją w bazie wyskakuje mi tylko "login failed"
gdy wpisuje dane które nie są dobre wywala to samo co poprzednio, czyli "Undefined variable: users" i też "login failed"

[!*!]

[PHP] pobierz, plaintext 
if (($_POST['login'] !='') && ($_POST['password'] !=''))
[PHP] pobierz, plaintext 

Do sprawdzania czy istnieją użyj isset()

[PHP] pobierz, plaintext 
if (md5($password)===$users['password'])
[PHP] pobierz, plaintext 

Komunikat mówi wyraźnie że nigdzie nie zdefiniowałeś zmiennej $users, więc skąd bierzesz $users['password']?

Zapoznaj się http://pl.wikibooks.org/wiki/PHP/Jak_to_się_robiło_kiedyś%3F

Ten post edytował !*! 2.07.2012, 15:14:56

[Terrorizer]

dodałem linijkę, taką jak zasugerował kolega na początku, czy to jeszcze nie jest zdefiniowana zmienna?

przepraszam jeśli piszę jakieś głupoty, ale wiele rzeczy opieram na przykładach z książki lub internetu i nie zawsze rozumiem każdy szczegół swojego kodu...

[!*!]

Ważne jak i gdzie ją dałeś pokaż kod.

[Terrorizer]

mój kod obecnie podzielony jest na kilka plików, nic się tu kupy nie trzyma, nie wiem o co chodzi
BARDZO PROSZĘ o pomoc
wygląda to dokładnie tak:

login.php

[PHP] pobierz, plaintext 
<?php
session_start();
mysql_connect('localhost','root','') or die('cannot connect with server');
mysql_select_db('users') or die('cannoct connect with database');
 
if(isset($_GET['action'])){
switch($_GET['action']){
   case  'login': include 'login script.php';break;
   case 'logout': include 'logout script.php';break;
   default      : include 'default script.php';break;
}}
else{ echo'dupa';}
mysql_close();
?>
[PHP] pobierz, plaintext 

default script.php

[PHP] pobierz, plaintext 
<?php
if ($_SESSION['login'] !=''){
  echo 'welcome'.$_SESSION['$login'];
}
else{
  echo 'you are not aproved to be here';
}
 
echo '<br/>what is it?? -a default site';
?>
[PHP] pobierz, plaintext 

login script.php

[PHP] pobierz, plaintext 
<?php
if (($_POST['login'] !='') && ($_POST['password'] !=''))
{
$login=$_POST['login'];
$password=$_POST['password'];
echo $login;
 
$loginresult=mysql_query("SELECT * FROM users WHERE login='$login' LIMIT 1");
if (mysql_num_rows($loginresult)===1)
$users = mysql_fetch_assoc($loginresult);
if (md5($password)===$users['password']){
echo '<p>login successful</p>';
$_SESSION['id']=$users['id'];
$_SESSION['login']=$users['login'];
$_SESSION['password']=$users['password'];
}
else{
echo '<p>login failed</p>';}
}
?>
[PHP] pobierz, plaintext 

logout script.php

[PHP] pobierz, plaintext 
<?php
unset($_SESSION['id']);
unset($_SESSION['login']);
unset($_SESSION['password']);
unset($_SESSION);
?>
[PHP] pobierz, plaintext 

Ten post edytował Terrorizer 2.07.2012, 20:17:34

[!*!]

Aha i? Całość jest źle napisana, nie znasz podstaw, zapoznaj się z linkiem jaki Ci podałem jak i całym kursem php. Nie poprawiłeś też wskazanych błędów.

[Terrorizer]

Uczę się na bazie kodu innych ludzi, uważam ,że to bardziej procentuje niż czytanie niekończących się artykułów, co zabija moje oczy , które niemal krwawią już od tego.

Może ja poprostu nie zrozumiałem wskazówek, jak sam wspomniałeś, brak mi podstaw i nie całkiem wiedziałem co z tym zrobić.

Prawdopodobnie nie powinienem pisać tu w tak banalnych sprawach, ale skoro chcecie być posiadaczami najlepszego forum w okolicy, to chyba liczba zadanych pytań powinna być jak największa, czyż nie ?

[!*!]

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.

[Terrorizer]

no i taka odpowiedź, to ja rozumiem
dzięki wielkie