[php]Problem z where clause Opcje

[aga323]

Witam. Otóż mam problem. Mam zrobione logowanie i gdy wpisze dobre haslo i login to mnie normalnie loguje, a kiedy wpiszę złe zamiast wyświetlić sie po prostu "Zły login" wyświetla się:

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in /var/www/sites/yoyo.pl/g/2/g2test/dziennik/log.php on line 18
Unknown column 'jakistamlogin' in 'where clause'

Może podam kawałek kodu, gdzie wywala błąd

[PHP] pobierz, plaintext 
<?php
$query = "SELECT * FROM user  WHERE user = $login_f";
$result = mysql_query ($query);
//-------------------------
//Sprawdzenie czy baza zwróciła przynajmniej 1 rekord.
$ilosc = mysql_num_rows($result) or die (mysql_error()); // to jest ta nieszczęsna linia 18 
if($ilosc==0)
{
	echo 'Zly login';
}
else
{ // tu tam dalszy ciąg ale już nie istotny
?>
[PHP] pobierz, plaintext 

Proszę o pomoc

[Darti]

Zastosuj odpowiednie apostrofy:

[SQL] pobierz, plaintext 
SELECT * FROM `user` WHERE `user` = '$login_f'
[SQL] pobierz, plaintext 

[aga323]

Dzięki, pomogło. Ale teraz się z kolei nic nie wyświetla A powinno sie wyświetlać "Zły login"

[Darti]

Sprawdź co robi die" title="Zobacz w manualu PHP" target="_manual lub exit" title="Zobacz w manualu PHP" target="_manual

[aga323]

Hmmm... sprawdziłam ale cos mi nie wychodzi, zapewne robie coś źle. Próbuję tak:

[PHP] pobierz, plaintext 
<?php
$query = "SELECT * FROM `user` WHERE `user` = '$login_f' ";
$result = mysql_query ($query);
//-------------------------
//Sprawdzenie czy baza zwróciła przynajmniej 1 rekord.
$ilosc = mysql_num_rows($result) or die (mysql_error());
if($ilosc==0)
{
	exit ( "Zly login");
 
}
else
{
	//Baza zwróciła jakieś pozycje, tak więc dzielimy je na tablice.
	$wynik=mysql_fetch_array($result);
// itd.....
?>
[PHP] pobierz, plaintext 

[MiFlo]

a nie lepiej tak ?

[PHP] pobierz, plaintext 
<?php
$q = mysql_query("SELECT * FROM users WHERE name='".$name."' AND pass='".$pass."';");
$r = @mysql_fetch_array($q);
if(!empty($r['name']))
{
//logowanie
}
else
{
//zly login
}
?>
[PHP] pobierz, plaintext 

Może być jakiś błąd bo nie sprawdzałem

[aga323]

MiFlo Twój pomysl pomógł, ale tylko w pewnym stopniu... Bo teraz coś reszta nie chce działac Wyskkuje błąd:
Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /var/www/sites/yoyo.pl/g/2/g2test/dziennik/log.php on line 23
podam kod:

[PHP] pobierz, plaintext 
<?php
//Nadajemy zmiennej $login_f  wartość jaką użytkownik wpisał w pole Login
$login_f = $_POST['user'];
//Nadajemy zmiennej $pass_f  wartość jaką użytkownik wpisał w pole Haslo
$pass_f = $_POST['pass'];
//Laczenie z baza
include ("connect.php");
//-------------------------
//   Zapytanie
//-------------------------
//pobieramy użytkownika o takim loginie z bazy.
$query = mysql_query("SELECT * FROM `users` WHERE 'login'='".$login_f."' AND 'haslo'='".$pass_f."';");
$result = mysql_fetch_array($query);
if(!empty($result['login']))
{
 echo "Zły login";
}
else
{
	//Baza zwróciła jakieś pozycje, tak więc dzielimy je na tablice.
	$wynik=mysql_fetch_array($result); //tu pokazuje błąd
	$login_b = $wynik['login'];
	$pass_b = $wynik['haslo'];
	if($login_f==$login_b && $pass_f==$pass_b)
	{ //tu tam dalszy ciąg
?>
[PHP] pobierz, plaintext 

W czym tkwi problem?

[Kicok]

Nazw kolumn nie umieszcza się w apostrofach

Ten post edytował Kicok 19.05.2007, 14:07:04

[Darti]

@aga323 przyjrzyj się w którym miejscu @MiFlo dał logowanie a w którym miejscu złe hasło - masz na odwrót zdaje mi się ...

[Noddi]

Przyjrzyj się także lini 13 i 21, po co w lini 21 znowu dzielisz tablicę wyników na kolejną tablice?

[JaRoPHP]

Funkcja mysql_fetch_array" title="Zobacz w manualu PHP" target="_manual spodziewa się argumentu, będącego odwołaniem do wykonanego zapytania MySQL, a nie do wyniku działania funkcji mysql_fetch_array.

[PHP] pobierz, plaintext 
<?php
$result = mysql_fetch_array($query);
if(!empty($result['login'])) {
 echo "Zły login";
} else {
	$login_b = $result['login']; // już masz te dane w tablicy
	$pass_b = $result['haslo'];
// ...
}
?>
[PHP] pobierz, plaintext 

[aga323]

Dziękuję wszystkim za trafne odpowiedzi jdnak teraz znowu jest problem... Po zalogowaniu pojawia się białe tło i nic... pustka. Podaję kod:

[PHP] pobierz, plaintext 
<?
session_start();
ob_start();
//Nadajemy zmiennej $login_f  wartość jaką użytkownik wpisał w pole Login
$login_f = $_POST['user'];
//Nadajemy zmiennej $pass_f  wartość jaką użytkownik wpisał w pole Haslo
$pass_f = $_POST['pass'];
//Laczenie z baza
include ("connect.php");
//-------------------------
//   Zapytanie
//-------------------------
//pobieramy użytkownika o takim loginie z bazy.
$query = mysql_query("SELECT * FROM `users` WHERE login='".$login_f."' AND haslo='".$pass_f."';");
$result = @mysql_fetch_array($query);
if(!empty($result['login'])){
 
   $login_b = $wynik['login'];
   $pass_b = $wynik['haslo'];
	if($login_f==$login_b && $pass_f==$pass_b)
	{
		//Tworzymy zmienną sesji $_SESSION['Login']  i nadajemy jej
		//wartość jaka jest w bazie danych odpowiadająca zalogowanemu użytkownikowi.
		$_SESSION['User']= $login_b;
		$_SESSION['User_id']= $wynik['id'];
		$_SESSION['lewel']= $wynik['lewel'];
		//Zalogowanego użytkownika przenosimy na strone index.php
		header( 'Location: ses.php' );
	}
}
 
 
 
else
{
echo "Zły login";   
   
}
?>
[PHP] pobierz, plaintext 

[JaRoPHP]

No tak, ale nazwy tablic masz różne:

[PHP] pobierz, plaintext 
<?php
$result = @mysql_fetch_array($query); // tutaj nazwa: $result
if(!empty($result['login'])){ 
   $login_b = $wynik['login']; // a tutaj nazwa: $wynik, a powinno być $result
   $pass_b = $wynik['haslo']; // jw
?>
[PHP] pobierz, plaintext 

Kolejna sprawa, nie do końca rozumiem tę linijkę:

[PHP] pobierz, plaintext 
<?php
if($login_f==$login_b && $pass_f==$pass_b)
?>
[PHP] pobierz, plaintext 

Przecież w zapytaniu SQL wybierasz wiersz, który ma pole login równe odpowiedniej wartości (to samo dotyczy pola hasło), więc owe if jest zbyteczne.

[aga323]

Racja, sorki mój błąd. Poprawiłam, ale nie wiem czemu całe czas jets tak samo (chociaz teraz wyświetla cały czas "Zły login" nawt przy podaniu dobrych danych... Podaję kod

[PHP] pobierz, plaintext 
<?
session_start();
ob_start();
//Nadajemy zmiennej $luogin_f  wartość jaką użytkownik wpisał w pole Login
$login_f = $_POST['user'];
//Nadajemy zmiennej $pass_f  wartość jaką użytkownik wpisał w pole Haslo
$pass_f = $_POST['pass'];
//Laczenie z baza
include ("connect.php");
//-------------------------
//   Zapytanie
//-------------------------
//pobieramy użytkownika o takim loginie z bazy.
$query = mysql_query("SELECT * FROM `users` WHERE 'login'='".$login_f."' AND 'haslo'='".$pass_f."';");
$result = @mysql_fetch_array($query);
if(!empty($result['login'])){
 
   $login_b = $result['login'];
   $pass_b = $result['haslo'];
 
		//Tworzymy zmienną sesji $_SESSION['Login']  i nadajemy jej
		//wartość jaka jest w bazie danych odpowiadająca zalogowanemu użytkownikowi.
		$_SESSION['User']= $login_b;
		$_SESSION['User_id']= $result['id'];
		$_SESSION['lewel']= $result['lewel'];
		//Zalogowanego użytkownika przenosimy na strone index.php
		header( 'Location: ses.php' );
 
}
 
 
 
else
{
echo "Zły login";   
   
}
?>
[PHP] pobierz, plaintext 

[+] edit
Poprawiam BB Code : )
---
~strife

[strife]

1. Usuń @ z 15 linijki.
2. Zobacz co zwraca mysql_error" title="Zobacz w manualu PHP" target="_manual

[aga323]

@ usunięta a mysql_error nic nie zwraca.... :| i Nadal "Zły login"

Ten post edytował aga323 20.05.2007, 11:47:40

[mike]

Usuń apostrofy z nazw kolumn. Może to zadziała.
Czyli z:

[PHP] pobierz, plaintext 
<?php
// ...
$query = mysql_query("SELECT * FROM `users` WHERE 'login'='".$login_f."' AND 'haslo'='".$pass_f."';");
// ...
?>
[PHP] pobierz, plaintext 

... na:

[PHP] pobierz, plaintext 
<?php
// ...
$query = mysql_query("SELECT * FROM `users` WHERE login='".$login_f."' AND haslo='".$pass_f."';");
// ...
?>
[PHP] pobierz, plaintext 

[JaRoPHP]

mysql_error nic nie zwraca

A wstawiłaś gdzieś tę funkcję?

Wyświetl zapytanie, skopiuj je do np. phpMyAdmina i tam wykonaj:

[PHP] pobierz, plaintext 
<?php
// ...
$query = "SELECT * FROM `users` WHERE 'login'='".$login_f."' AND 'haslo'='".$pass_f."';";
echo $query;
$query = mysql_query($query) or die(mysql_error());
// ...
?>
[PHP] pobierz, plaintext 

I pokaż rezultat wykonanego skryptu.

Ten post edytował JaRoPHP 20.05.2007, 12:01:55

[aga323]

Lol Mike masz racje, podziałało Dziabeł tkwi w szczegółach... Teraz mi sie zalogowało i wyskoczył błąd w innym pliku, coś podobnego rodzaju jak ten poprzedni, ale myślę, że dam rade. Teraz sie tylko zastanawiam nad strukturą dziennika tak aby danemu wychowawcy pokazywali się tylko jego podopieczni