Programowanie Obiektowe Opcje

[Oczko21]

Witam,
Napisalem moj pierwszy skrypt z wykorzystaniem programowania obiektowego i chcialbym spytac czy to o cos takiego chodzi w programowaniu obiektowym, gdyz w wielu ksiazkach model projektowania obiektowego jest przedstawiany za pomoca zwierzat, pojazdow itp. Niestety jeszcze nie zdolalem wyksztalcic sobie wyobrazni do takiego stopnia aby model zwierzat przelozyc na model chociazby formularza sluzacego do kontaktu uzytkownika strony z jej wlascicielem.

[PHP] pobierz, plaintext 
include"klasa_db.php"; 
 
class Kontakt
{
	public $formularz = array(email => '', temat => '', tresc => '');
	public $wiadomosc;
	public $ip;
 
	public function Zapisz()
	{
		$this->ip = $_SERVER['REMOTE_ADDR'];
		$db = new BazaDanych;
		$query="INSERT into wiadomosci (Email, Tytul, Tresc, Data, IP) VALUES('".$this->formularz['email']."', '".$this->formularz['temat']."', '".$this->formularz['tresc']."', NOW(), '$this->ip')";
		$db->db($query);
		$this->wiadomosc = 'Wszystko poszlo elegancko.';
	}
	public function SprawdzFormularz()
	{
		if(isset($_POST['email']) && isset($_POST['temat']) && isset($_POST['tresc']))
		{
		$this->formularz['email'] = addslashes($_POST['email']);
		$this->formularz['temat'] = addslashes($_POST['temat']);
		$this->formularz['tresc'] = addslashes($_POST['tresc']);
		$this->Zapisz();
		}else{
		$this->wiadomosc = 'Nie wypełniłeś wszystkich pól formularza.';
		}
	}
}
 
if(isset($_POST['wyslij']))
{
$b = new Kontakt;
$b->SprawdzFormularz();
echo $b->wiadomosc;
}
 
[PHP] pobierz, plaintext 

[HTML] pobierz, plaintext 
<form method="post" action="">
<input type="text" name="email" /><br />
<input type="text" name="temat" /><br />
<textarea name="tresc"></textarea><br />
<input type="submit" name="wyslij" value="Wyslij" />
</form>
[HTML] pobierz, plaintext 

Skrypt dziala poprawnie, prosze jedynie o zwrocenie mi uwagi w kwesti bledow ktore popelnilem. Jak wspomnialem na poczatku dopiero zaglebiam sie w tajniki programowania obiektowego i chcialbym nauczyc sie go poprawnie, dlatego prosze wytykac bledy, nawet najdrobniejsze, nie chce wyrobic sobie zlych nawykow zaraz z poczatku.

[greycoffey]

Ogółem połączyłeś model, widok i kontroler w jedno.

• Metoda SprawdzFormularz
  • Właścicwie walidacją danych powinna się zajmować inna klasa - zasada jednej odpowiedzialności.
  • Dane pobierasz z określonych pół, do tego przez POST. Powinny być przekazywane przez argumenty metody, albo przekazywane w konstruktorze klasy. W OOP ważna jest przenośność. Co z tego, jak nie będziesz mógł zastosować tej klasy dla dwóch różnych formularzy?
• Metoda Zapisz
  • Startujesz nowe połączenie z bazą danych. Jest trochę nieefektywnie startować za każdym razem nowe. Poczytaj o Dependency Injection Container, Singletonie czy Rejestrze.
• Ogół klasy
  • Klasa służy do kontaktu. Pomyśl, że chcesz zrobić klasę do kontaktu od zapisywania do bazy danych wiadomości, po mail, irc czy GG, tak, by móc je stosować zmieniając tylko linijkę w tekście. Niech potem wszystkie klasy służące do poszczególnych form kontaktu - Kontakt_Email, Kontakt_Baza, Kontakt_Irc implementują interfejs RodzajKontaktu, a potem przekazywać ich instancje do konstruktora, z określeniem implementowanego interfejsu, przez co powstanie jednolita i wspólna droga komunikacji. Ale tutaj wybiegłem poza ramy tego projektu. Generalnie ucz się, czytaj o OOP, zapoznawaj się z wzorcami projektowymi, podpatrzaj u frameworków jak to jest zrobione i pisz, pisz i pisz.

[Oczko21]

Dzieki, wezme sobie te rady do serca i postaram sie to zmienic. Zapytam tylko czy dobrze to rozumiem:
Mam zrobic 2 klasy. Jedna do walidacji formularzy, a druga do zapisywania.
W pierwszym maja byc rozne metody, odnoszace sie do sprawdzania roznych formularzy, a w drugim metody odnoszace sie do zapisy sprawdzanych formularzy i wartosci mam przekazywac przez argumenty tych metod tak?