Czemu nie działa zwracanie wartości?

Question

Witam

Tworzę skrypt logowania, mam problem ze zwróceniem wartości, gdy np napisze echo w funkcji VerificationLogin() to się wyświetla a jak dam return to nie wyświetla się dany błąd

class AuthC
{		
	private $login;
	private $password;
	
	public function login($login, $password)
	{
		$this->login = htmlentities($login);
		$this->password = htmlentities($password);
		
		try
		{
			$vlogin = new verificationLogin();
			$vlogin->verification($this->login, $this->password);		
		}
		catch(Exception $e)
		{
			$_SESSION['exceptionLogin'] = $e->getMessage();
			return DATABASE_ERROR;
		}
	}
}

class VerificationLogin
{
	private $login;
	private $password;
				
	public function verification($login, $password)
	{
		$this->login = $login;
		$this->password = $password;
				
		return LOGIN_FAILED;
    }
}

index.php

if(isset($_POST['login']))
{
	if((empty($_POST['login'])) || (empty($_POST['password'])))
	{
		$_SESSION['viewLoginError'] = true;
	}
	else
	{
		switch($AuthC->login($_POST['login'], $_POST['password']))
		{
			case LOGIN_SUCCESS:
				header('Location: templates/informations.php');
				break;
			case LOGIN_FAILED:
				$_SESSION['viewLoginError'] = true;
				break;
			case DATABASE_ERROR:
				$_SESSION['viewExceptionLogin'] = true;
				break;
		}
	}
}

 

Answer 1

Jak do zwykłej rejestracji to nawaliłeś niepotrzebnie tego kodu.
Np klasa VeryficationLogin mogła by być metodą klasy AuthC.
Co do problemu to wyciągnij "$vlogin = new verificationLogin();" przed "try{".
Tworzenie nowego objektu także zwraca true.

Comments

zrobiłem tak jak napisałeś ale nadal nie działa, spróbowałem przenieść VeryficationLogin do klasy AuthC i zrobiłem z tego metodę, odwołałem się w ten sposób $this->verification($this->login, $this->password); i nadal to samo :/

---

@lrror bzdury gadasz, metoda walidacji nie mogła by być w klasie Auth bo jest to nie zgodne z zasadą SRP

---

@Boshi czyli dobrze zrobiłem że rozbiłem to na dwie klasy, podpowiesz co mogę zrobić żeby rozwiązać mój problem?