Kurs PHP odc.2 zła ścieżka w ifie

pytanie zadane 23 października 2016 w PHP przez Maciejjy Nowicjusz (120 p.)

Dzień dobry.

Właśnie robię kurs PHP (BARDZO fajnie zrobiony!) i niestety utknąłem w jednym miejscu. Pomimo przeglądania wiele razu kodu i nawet próby skopiowania konkretnej linii (w której chyba jest błąd) z pliku źródłowego ze strony, nie potrafię powiedzieć co jest nie tak.

Problem dotyczy drugiego odcinka kursu i pliku "zaloguj.php".

Robimy tam ifa, który sprawdza czy zmienna $regulamin to poprawna kwerenda SQL i zaczyna walidacje użytkownika. Wygląda to tak:

......
else
	{
		$login = $_POST["login"];
		$password = $_POST["password"];
		
		$sql = "SELECT * FROM uzytkownicy WHERE user='$login' AND pass='$password'";
		
		if ($rezultat = @$polaczenie->query($sql))
		{
			$ilu_userow = $rezultat->num_rows;
			if ($ilu_userow == 1)
			{
				$wiersz = $rezultat->fetch_assoc();
				$user = $wiersz['user'];
				
				
				$rezultat->free();
				echo $user;
			}
			else
			{
				echo "brak uzytkownika w bazie";
			}
		}
		else
		{
			echo "złe zapytanie";
		}
		
		$polaczenie->close();
	}

Kłopot w tym, że mi zawsze wykonuje się linia "echo "złe zapytanie"", czyli jakby warunek w ifie: ($rezultat = @$polaczenie->query($sql)) nie spełnia się.

Próbowałem zmieniać kwerende na taką, która na pewno jest poprawna, sprawdzałem ją w myphpadmin itd. Zmieniałem też warunek ifa na 100% dobry, czyli np. 1 ==1 i wtedy pętla idzie właściwą ścieżką.

Proszę o pomoc. :)

Pokaż 12 poprzednich komentarzy

komentarz 23 października 2016 przez Maciejjy Nowicjusz (120 p.)

@Ehlert a ja właśnie zawsze czytałem, że programista powinien być ciekawy i rozumieć co się dzieje. Jak mi coś nie wychodzi, to chcę wiedzieć czemu, a nie po prostu przerywać i robić inaczej "bo tak jest źle". Co to w ogóle za podejście? Wiem, że jest źle, ale chce wiedzieć co konkretnie. Potem mogę iść dalej.

@efik SQL jest poprawny. Już nawet zmieniałem kwerende po prostu na "SELECT * FROM uzytkownicy" i jest tak samo. Wpisywałem też go to phpmyadmin i normalnie wywala jeden rekord jak wpiszę login i hasło. Wychodzi na to, że warunek:

if ($result = $mysqli->query($sql))

jest falsem i dlatego przechodzi do else. Dlaczego jest falsem? Tego nie wiem.

komentarz 23 października 2016 przez Ehlert Ekspert (212,670 p.)

Jest źle, bo tak się w php nie pisze. Nie wychodzi Ci, bo zrzynasz kod z kiepskiego kursu.

Zgadzam się że forma przekazu MZ. jest wspaniała. Ale czym innym jest to, co przekazuje.

komentarz 23 października 2016 przez Maciejjy Nowicjusz (120 p.)

Ok, jest źle. Wiem już o tym.

Czy możesz mi tylko odpowiedzieć na zadawane już chyba 10 razy pytanie dlaczego u mnie nie działa, skoro na filmie działa?

Tylko tyle. I będę mógł dziś zasnąć spokojnie.

komentarz 23 października 2016 przez efiku Szeryf (75,160 p.)

POKAŻ CAŁY KOD ZALOGUJ.PHP PO TWOICH ZMIANACH.

komentarz 30 lipca 2018 przez Olek Szymański Użytkownik (920 p.)

@Ehlert, Z tym się zgodzę. Uważam, że nie warto oglądać zelenta. Myślę, że dobre polecone książki oraz dokumentacje, fora itp.

3 odpowiedzi

odpowiedź 26 października 2016 przez Karol Gasienica Użytkownik (640 p.)

odpowiedź 26 października 2016 przez tinwepl Użytkownik (990 p.)

if($rezultat = @$polaczenie->query($sql))

Może palnę głupotę ale od kiedy w ifie można przypisać zmienną, jeśli ma sprawdzić true / false ?

1	komentarz 26 października 2016 przez Comandeer Guru (601,550 p.) Od zawsze…

komentarz 26 października 2016 przez Arkadiusz Waluk Ekspert (287,950 p.)

A dlaczego miałoby nie być można?

Przykład działania https://3v4l.org/tTDrc

komentarz 26 października 2016 przez tinwepl Użytkownik (990 p.)

Faktycznie : D No to nie wiem jakoś mi w C++ wywalało błędy :)

1	komentarz 26 października 2016 przez Comandeer Guru (601,550 p.) Ale PHP != C++… To są zupełnie różne języki.

odpowiedź 26 października 2016 przez tirith Mądrala (5,940 p.)

powinno być

 if ($rezultat == @$polaczenie->query($sql))

komentarz 26 października 2016 przez Arkadiusz Waluk Ekspert (287,950 p.)

No raczej nie...

Tak powinno być jeśli byśmy chcieli dostać błąd o niezadeklarowanej zmiennej, bo jak widzę w kodzie pytającego wcześniej deklaracji nie było. A to co robisz to porównanie, porównać do nieistniejącej zmiennej się nie da.

https://3v4l.org/EAR69