[PHP]/[MySQL] - problem z zapytanie:/ Opcje

[Szypa]

Mam problem z zapytanie do SQL`a. Tak wyglądała rejestracja:

[PHP] pobierz, plaintext 
<?php
$zapytanie = 'INSERT INTO uzytkownicy (user_id, uzytkownik, haslo, email, data) VALUES (0, "'.$_POST['uzytkownik'].'", "'.crypt($_POST['haslo1']).'", "'.$_POST['email'].'", now())';
?>
[PHP] pobierz, plaintext 

A tak wygląda cały skrypt logowania:

[PHP] pobierz, plaintext 
<?php
ini_set ('display_errors', 1);
error_reporting (E_ALL & ~E_NOTICE);
$tytul = "Logowanie";
require ('szablony/menu.html');
?>
 
 
<div id="content">
	<div id="posts">
		<div class="post">
			  <h2 class="title">Tutaj możesz się zalogować</h2> 
			<div class="story">
			<?php
				if (isset ($_POST['login']))
				{
					$problem = FALSE;
 
					if (empty ($_POST['uzytkownik']))
					{
						$problem = TRUE;
						print '<p>Proszę podac nazwę użytkownika!</p>';
					}
 
					if (empty ($_POST['haslo']))
					{
						$problem = TRUE;
						print '<p>Proszę podac hasło!</p>';
					}
 
					if (empty ($_POST['email']))
					{
						$problem = TRUE;
						print '<p>Proszę podac adres email!</p>';
					}
 
 
					if (!$problem)
					{
 
						// Sprawdzanie działania Bazy Danych
						if ($dbc = @mysql_connect ('localhost', 'uzytkownik', 'XxX'))
						{
							if (!@mysql_select_db ('XxX'))
							{
								die ('<p>Nie można wybrać bazy danych, ponieważ: <b>' . mysql_error() . '</b></p>');
							}
						}
						else
						{
							die ('<p>Nie można połączyć się z MySQL, ponieważ: <b>' . mysql_error() . '</b></p>');
						}
 
							$zapytanie = 'SELECT haslo, email, grupa FROM uzytkownicy WHERE (uzytkownik=' . $_POST['uzytkownik'] . ')';
							if ($r = mysql_query ($zapytanie))
							{
								$wiersz = mysql_fetch_array ($r);
								if ( ($_POST['email'] == $wiersz['email']) AND (crypt($_POST['haslo']) == $wiersz['haslo']))
								{
									print '<p><b>Zostałeś zalogowany na stronie!</b><br />
									Możesz teraz skorzystać z funkcji zapewnionych Twojej grupie</p>';
									session_start();
									$_SESSION['uzytkownik'] = $_POST['uzytkownik'];
									$_SESSION['grupa'] = $wiersz['grupa'];
									$_SESSION['zalogowany'] = time();
								}
								else
								{
 
									if ($_POST['email'] != $wiersz['email'])
										{
											print '<p>Podałeś zły adres Email!</p>';
										}
									if (crypt($_POST['haslo']) != $wiersz['haslo'])
										{
											print '<p>Podałeś złe hasło!</p>';
										}
								}
 
							}
 
 
						mysql_close();
					}
					else
					{
						print '<p>Uzupełnij brakujące dane i spróbuj jeszcze raz!</p>';
					}
				} 
				else
				{
				?>
				<p><strong>Wpisz odpowiednie dane w przeznaczone do tego miesjca w formularzu</strong></p>
				<form action ="logowanie.php" method="post">
				<p>Nazwa użytkownika: <input style="position:relative; left:13px" type="text" name="uzytkownik" size="20" /><br />
				Hasło: <input style="position:relative; left:80px" type="password" name="haslo" size="20" /><br />
				Adres Email: <input style="position:relative; left:52px"type="text" name="email" size="20" /><br />
				<input type="submit" name="login" value="Zaloguj" /></p>
				</form>
				<?php
				}
				?>
 
 
			</div>
		</div>
	</div>
	<!-- end #posts -->
<?php
require ('szablony/linki.html');
require ('szablony/stopka.html');
?>
[PHP] pobierz, plaintext 

No i gdzieś tutaj coś popieprzyłem. Po podaniu nawet nieprawidłowego hasła, nazwy i adresu email pokazuje mi puste okno bez informacji o zalogowaniu lub jego błędzie:/ Jedynie reaguje w razie nieuzupełnienia jakiegoś pola. Wydaje mi się że to po prostu problem z odczytywaniem danych z MySQL`a ale nie jestem pewien. Mógłby ktoś to sprawdzić?

[Grzyw]

Czemu do pola user_id wsadzasz wartość zero?

[Szypa]

Szczerze? Nie mam pojęcia:) To pole jest w bazie zapisane jako auto_increment. W książce z której uczę się PHP tak to właśnie było rozwiązane. Zresztą sprawdzałem i kolejni użytkownicy mają kolejne numery ID, wiec to nie tutaj jest błąd, tym bardziej że nawet tego nie wykorzystuje w skrypcie logowania.

[Grzyw]

Hmm... Zatem kombinujemy dalej. Nie musisz przypadkiem wsadzić apostrofów/cudzysłowów w zapytaniu? (bez backslashów możesz):

[PHP] pobierz, plaintext 
<?php
$zapytanie = 'SELECT haslo, email, grupa FROM uzytkownicy WHERE uzytkownik="' . $_POST['uzytkownik'] . '")';
?>
[PHP] pobierz, plaintext 

Ten post edytował Grzyw 29.07.2007, 11:37:03

[Szypa]

Obydwie opcje dają taki sam rezultat. Nie w tym siedzi psikus. Wydaje mi się że coś mam źle z odzyskiwaniem informacji z bazy danych, ale nie mogę namierzyć błędu:/

Ten post edytował Szypa 29.07.2007, 11:59:10

[Grzyw]

To może w drugim skrypcie wklej tylko funkcje odpowiadające za pogranie danych i wyrzuć tylko dane na ekran. Zobaczysz, czy z bazy otrzymujesz poprawne wartości.

[Szypa]

No zrobiłem coś co tylko wyświetla zawartość. Jak ktoś mi to poprawi, to sam będę w stanie przerobić resztę poprawnie.

[PHP] pobierz, plaintext 
<?php
 
 
				$uzytkownik = 'Administrator';
				mysql_connect ('localhost', 'uzytkownik', 'XxX');
 
				mysql_select_db ('XxX');
 
				$zapytanie = 'SELECT uzytkownik, haslo, email, grupa FROM uzytkownicy WHERE uzytkownik="' . $uzytkownik . '")';
				$wiersz = mysql_query ($zapytanie);
				$row = mysql_fetch_array($wiersz); // To jest 22 linijka
 
				print "<p>Zostałeś zalogowany na stronie<b>" . $_POST['uzytkownik'] . "</b>!<br />
				Możesz teraz skorzystać z funkcji zapewnionych Twojej grupie</p><br />
				Jesteś:" . $row['uzytkownik'] . "<br />Twój email to: " . $row['email'];
 
 
				mysql_close();	
 
?>
[PHP] pobierz, plaintext 

Przeglądarka zwraca taki napis:

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in c:\usr\krasnal\www\wlan.ostrorog\test.php on line 22

Zostales zalogowany na stronie!
Mozesz teraz skorzystac z funkcji zapewnionych Twojej grupie

Jestes:
Tw�j email to:

Ten post edytował Szypa 29.07.2007, 13:50:58

[Roguś]

A spróbuj tak :

[PHP] pobierz, plaintext 
<?php
While ($row = mysql_fetch_array($wiersz))
{
print "<p>Zostałeś zalogowany na stronie<b>" . $_POST['uzytkownik'] . "</b>!<br />
				Możesz teraz skorzystać z funkcji zapewnionych Twojej grupie</p><br />
				Jesteś:" . $row['uzytkownik'] . "<br />Twój email to: " . $row['email'];
 
}
?>
[PHP] pobierz, plaintext 

[Szypa]

Pokierowaliście mnie na dobrą drogę:) Nie mam już błędów składniowych, ale jest problem z cryptowaniem hasła.

Skrypt zapisywania hasłą i jego cryptowania :

[PHP] pobierz, plaintext 
<?php
$zapytanie = 'INSERT INTO uzytkownicy (user_id, uzytkownik, haslo, email, data) VALUES (0, "'.$_POST['uzytkownik'].'", "'.crypt($_POST['haslo1']).'", "'.$_POST['email'].'", now())';
?>
[PHP] pobierz, plaintext 

oraz skrypt odczytując hasło przy logowaniu:

[PHP] pobierz, plaintext 
<?php
$zapytanie = "SELECT uzytkownik, haslo, email, grupa FROM uzytkownicy WHERE uzytkownik = '{$_POST['uzytkownik']}'";
 
$wiersz = mysql_query ($zapytanie);
 
while ($row = mysql_fetch_array($wiersz))
{
	if ( ($_POST['email'] == $row['email']) AND (crypt($_POST['haslo']) == $row['haslo']))
	{
		print '<p><b>Zostałeś zalogowany na stronie!</b><br />
		Możesz teraz skorzystać z funkcji zapewnionych Twojej grupie</b></p>';
		session_start();
		$_SESSION['uzytkownik'] = $_POST['uzytkownik'];
		$_SESSION['grupa'] = $row['grupa'];
		$_SESSION['zalogowany'] = time();
	}
	else
	{
 
		if ($_POST['email'] != $row['email'])
		{
			print '<p>Podałeś zły adres Email!</p>';
		}
		if (crypt($_POST['haslo']) != $row['haslo'])
		{
			print '<p>Podałeś złe hasło!</p>';
		}
	}
}
?>
[PHP] pobierz, plaintext 

Mógłby ktoś podpowiedzieć co jest źle że za każdym razem wyrzuca komunikat o złym haśle?
Przy czym na bank to samo hasło jest podawane

Edit 30 lipca 11:38

Już wiem co robiłem źle. Nie podawałem klucza do deskrypcji przy porównaniach. Wszystko ładnie już śmiga. Dzięki wszystkim zaangażowanym za pomoc!

A tak przecież powinno wyglądać porównanie takich kluczy:

[PHP] pobierz, plaintext 
<?php
(crypt($_POST['haslo'], $row['haslo']) == $row['haslo'])
?>
[PHP] pobierz, plaintext 

Pozdro

Ten post edytował Szypa 30.07.2007, 10:39:55

[loonger]

a jaki ustawiles limit dlugosci hasla zapisanego w bazie? moze podczas szyfrowania staje sie dluzszy i ucina go podczas zapisu do bazy. kiedys mialem taki problem