Problem z password_verify PHP OOP

Question

Witam mam taki kawałek kodu z mojej klasy. Wszystko wykonuje się poprawnie, połączenie z bazą danych. Zmienna $count wskazuje wartość 1 gdy istnieje taki user w bazie. Problem jest z password Verify. $password to zmienna z formularza, $result['password'] hash hasła z bazy danych oto fragment kodu.

public function login($emailLogin=null, $password=null)
	{
		$this->emailLogin = $emailLogin;
		$emailLogin = htmlentities($emailLogin, ENT_QUOTES, "UTF-8");
		$this->password = $password;
		$sql = "SELECT * FROM users WHERE email='$this->emailLogin'";
		$question = $this->connect()->query($sql);
		$count = $question->RowCount();
		if ($count > 0) {
			$result = $question->fetch();
	      	if(password_verify($password, $result['password']))
	      	{
	         	$_SESSION['logged'] = true;
	         	

	      	} else {
				echo "NO";
				$_SESSION['error'] = ["Nie znaleziono użytkownika o takim loginie i haśle!"];
			}
		} else {
			echo "NO1";
			$_SESSION['error'] = ["Nie znaleziono użytkownika o takim loginie i haśle!"];
		}
	}

 

Comments

Jakaś kosmiczna ta Twoja klasa. Łączy z bazą danych, grzebie w niej, modyfikuje sesję i wyświetla. To wszystko w jednej metodzie. Hańba dla OOP.

---

nie chcę być chamski ani nie miły ale to nie jest moim problem na tą chwilę, jeśli nie umiesz go rozwiązać nie musisz tego komentować. Albo przynajmniej napisać to na końcu.

---

ThePatrykOOO to tak samo, jakbys poszedł do dentysty z bolącym zębem, on by Ci powiedział, że do leczenia jest jeszcze 10 innych, a ty byś powiedział, że nie jest to Twoim problemem na ten moment bo nie boli. A kiedyś w końcu zaboli, a wtedy może być za późno...

Answer 1

Używasz password_hash? rowCount małą literą. Używaj var_dump.

Comments

czyli jak to powinno wyglądać za pomocą var_dump. rowCount mam po prostu przyzwyczajenie z pisania camel case. Dlaczego niby mam nie używać password_hash?

---

1. Masz używać password_hash
2. za pomocą var_dumpa sprawdzasz co siedzi w zmiennych i obiektach i dlaczego nie działa to co napisałeś. Najlepiej zainstaluj Xdebug i wszystko będzie jasne.

---

w $result['password'] siedzi string 60

w $password siedzi string 5

---

Nie zrozumieliśmy się.

To Ciebie ma interesować co siedzi w zmiennych, a nie mnie. Nie każdy problem z programowania ogranicza się do wklejenia kodu na forum z dopiskiem HELP. Jeśli sam nie potrafisz, lub nie chcesz kombinować gdzie zaistniały błędy, które sam stworzyłeś to widocznie programowanie nie jest dla Ciebie.

---

Wszystko działa. LINK

---

Po prostu denerwują mnie takie komentarze lub porady tego typu które nie rozwiązują mojego problemu. To znaczy że każdą linijkę kodu mam robić w oddzielnej funkcji lub klasie. Po prostu przestawiam się z programowania strukturalnego na obiektowy. Jeszcze umieszcze wpis w sprawie oceny mojego systemu logowania i rejestracji i wtedy będę coś zmieniał i będziesz mógł skomentować co ci się nie podoba.

---

Też chciałem kiedyś pisać wszystko po swojemu twierdząc, że tak będzie najlepiej, bo to moje i po mojemu. Otóż tak nie jest. Pisząc od zera nie bazując na książkach typu Cookbook, artykułach typu Best practices i frameworkach (PHP szczególnie), marnujesz swój czas i uczysz się złych nawyków. Im szybciej to zrozumiesz tym lepiej.