Walidacja formularza

Question

Robię walidację w formularzu. Problem polega na tym ze gdy nacisne przycisk dodaj, to powinno wyswietlac błedy albo dodać do bazy danych, a pokazuje się tylko biała strona.

<?php

class FormRegisterValidation
{
    
	public function __construct(
	    private $data,
        private  $database,
    ){}
    
	public function LoginValidation(): void
	{
		 $val = htmlspecialchars($this->data['login']);

        try {

            if (empty($val)) {
                throw new Exception("Puste pole");
            }
        }
        catch (FormRegisterValidation $e){
            echo $e->errorMessage();
        }
	}
}
?>

 

Comments

Metoda errorMessage() wyjątku na pewno coś zwraca? Nie wiem jaki jest dalszy kod, ale jeśli tu się kończy to jest tylko jedno echo i jeśli by nic nie zwróciła to może dlatego jest pusto.

Masz włączone wyświetlanie błędów PHP? Można to sprawdzić/zmienić w pliku php.ini, ewentualnie można też jednorazowo na początku kodu dodać

ini_set('display_errors', 'on');

Składniowo to co pokazałeś wygląda ok, więc ciężko stwierdzić coś więcej.

Answer 1

Pokazałeś przepis na klasę a nie pokazałeś jak to w ogóle wywołujesz. I masz w tej klasie co najmniej kilka błędów:

1. Nie przypisujesz zmiennej data w konstruktorze do pól w klasie strzelam że powinno być:
 

private   $data;

private   $database;

public function __construct(

        private $data,

        private  $database,

    ){

$this->data = $data;

$this->database = $database,

}

2. Na ten moment leci ci wyjątek o nie istniejącym kluczy w tej tablicy (ale nie widzisz go bo tak jak ci zasugerował Arek nie masz pewnie wyłącznych błędów):

$val = htmlspecialchars($this->data['login']);

3. Zmienna $database wygląda jako coś nie potrzebnego bo nie używasz tego nigdzie (chyba że jest to zarysem jakiegoś interfejsu )

Comments

Co do punktu 1 - ma to zrobione dobrze o ile pisze w PHP 8

https://www.php.net/manual/en/language.oop5.decon.php#language.oop5.decon.constructor.promotion