Cześć, uczę się z książek, kursów i tutoriali programowania PHP jak i front-end. Dążę do full stacka, więc pracy przede mną bardzo dużo, ale idzie mi w mojej ocenie całkiem dobrze.
Wkroczyłem wreszcie w etap obiektów w PHP. Na początku nie za bardzo rozumiałem o co z tym chodzi, ale powoli zaczynam chyba łapać. Piszę otóż chcę się upewnić, czy w prawidłowy sposób używam obiektów.

Napisałem prosty programik przyjmujący i wyświetlający dane. Schemat działania na zasadzie (pomijając wszystkie mechanizmy) rejestracji użytkownika, a później wyświetlenia jego danych. Funkcjonalność ograniczyłem do minimum, aby nie skupiać się na tym co nieistotne, dlatego dane wprowadzam z GETa.
Programik na potrzeby ćwiczeń z obiektów, więc też nie musicie zwracać uwag na względy bezpieczeństwa danych z GETa.

Po tym wywodzie jeszcze raz zadam pytanie, aby była pewność o co mi chodzi.. Czy ten schemat odpowiada programowaniu obiektowemu? Czy o to w obiektach chodzi? Albo czy zrobiłem coś do czego bez sensu używać obiekty.

Z góry dziękuję za pomocne wypowiedzi

[PHP] pobierz, plaintext 
<?php
 
class uzytkownik
{
	private $imie; 
	private $nazwisko; 
	private $data_urodzenia; 
	private $miejscowosc;
 
	function __construct($imie, $nazwisko, $data_urodzenia, $miejscowosc)
	{
		$this->imie = $imie;
		$this->nazwisko = $nazwisko;
		$this->data_urodzenia = $data_urodzenia;
		$this->miejscowosc = $miejscowosc;
	}
 
	function getDane()
	{
		if ($this->imie and $this->nazwisko and $this->data_urodzenia and $this->miejscowosc)
		{
			return '<h2>Dane poprawnie dodane do programu!</h2>';
		}
		else
		{
			return '<h2>Brak danych o użytkowniku!</h2>';
		}
	}
 
	function uzytkownik()
	{
		if ($this->imie and $this->nazwisko and $this->data_urodzenia and $this->miejscowosc)
		{
			echo 'Dane użytkownika:<br>Imię: <b>'.$this->imie.'</b><br>
			Nazwisko: <b>'.$this->nazwisko.'</b><br>
			Data urodzenia: <b>'.$this->data_urodzenia.'</b><br>
			Miejscowość urodzenia: <b>'.$this->miejscowosc.'</b>';
		}
		else
		{
			echo '<h2>Brak danych o użytkowniku!</h2>';
		}
	}
}
 
$uzytkownik['imie'] = $_GET['imie'];
$uzytkownik['nazwisko'] = $_GET['nazwisko'];
$uzytkownik['data'] = $_GET['data'];
$uzytkownik['miejscowosc'] = $_GET['miejscowosc'];
 
$rejestracja = new uzytkownik($uzytkownik['imie'], $uzytkownik['nazwisko'], $uzytkownik['data'], $uzytkownik['miejscowosc']);
 
echo $rejestracja->getDane();
 
if ($rejestracja->getDane() == '<h2>Dane poprawnie dodane do programu!</h2>')
{
	$rejestracja->uzytkownik();
}
 
?>
[PHP] pobierz, plaintext 

Alc....

Obiekty z reguly nie zwracaja komunikatow a juz na pewno nie w stylach html i na pewno nie tak dlugie

Nie bardzo rozumiem po co ci funkcja getdane() skoro getuzytkwnik() ja nie jako powiela.
W ogole powinna byc tylko jedna funkcja getAllData() ktora zwroci tablice z danymi usera lub NULL lub rzuci wyjatkiem, gdy dane usera nie sa ustawione.
Zas za wyswietlanie danych to powinien odpowiadac widok/szablon a nie obiekt usera. Obiekty z reguly nie maja nic do wyswietlania.

To co tu przedstawiłeś (czyli klasa użytkownik, która w połączeniu z PSR-4 powinna być raczej Uzytkownik) jest częścią logiki biznesowej i jest tak zwaną encją (entity). Zazwyczaj encje mapują dane ze źródeł danych czyli np tabeli w bazie. Zawiera głównie settery/gettery chociaż niektórzy dają tutaj też walidację danych. Dodatkowo stosuj zawsze modyfikatory widoczności.

[PHP] pobierz, plaintext 
class Uzytkownik 
{
    protected $dataUrodzenia;
    protected $wiek;
 
    public function getDataUrodzenia() {
        return $this->dataUrodzenia;
    }
 
    public function setDataUrodzenia(\DateTime $data) { // można przekazać obiekt DateTime
        $this->dataUrodzenia = $data;
        return $this;
    } 
 
    // z walidacją danych
    public function setWiek($wiek) {
        if ($wiek < 15 || $wiek > 130) {
            throw new InvalidArgumentException('Wiek musi być w zakresie 15 - 130 lat');
        }
        $this->wiek = $wiek;
        return $this;
    }
}
[PHP] pobierz, plaintext