Witajcie serdecznie.

Jako, że ciągle próbuję walczyć z programowaniem obiektowym w PHP i chęcią rozgryzienia tego tematu, dziś napisałem jakiś prosty skrypt a'la rejestracja.

Ogółem działa, przy prosiłbym doświadczonych użytkowników o krytykę i ewentualne poprawki - no chyba, że całość jest nie w ten deseń, jaki być powinno.

Z góry dziękuję.

Poniżej kody z dwóch plików, które napisałem:
Plik z klasą:

[PHP] pobierz, plaintext 
<?php
$db=mysql_connect("localhost", "root", "krasnal");
$db_sel = mysql_select_db("projekt");
 
function oczysc($zmienna)
{
	$zmienna = strip_tags(trim($zmienna));
	return $zmienna;
}
################################ KLASY
class Rejestr
{
  public function form($login, $email)
 
	{
	 echo '
	<form action="" method="post">
	Login:<input type="text" name="login" value='.$login.'><br>
	Hasło:<input type="password" name="pass"><br>
	Potwierdź hasło:<input type="password" name="repass"><br>
	E-mail:<input type="password" name="email" value='.$email.'><br>
	<input type="submit" name="rejestr" value="Rejestruj">
	</form>
	';
	}
 
 
	public function sprawdz($login, $pass, $repass, $email)
	{
	$login=oczysc($login);// czy tego jakoś kurcze nie mozna 
	$pass=oczysc($pass);    // zautomatyzowac? :-) 
	$repass=oczysc($repass); // jakis 'myk' ze wszystkie przekazane dane
	$email=oczysc($email);   // filtrujemy przez oczysc
 
 
	$zap=mysql_query("Select id from uzytkownik where login='$login'");
	if((mysql_num_rows($zap)>0) || (strlen($login)<3))
	 {
		$blad .= "- Login zajęty, lub podano za krótki.<br>";
	 }
 
 
	$zap=mysql_query("Select id from uzytkownik where email='$email'");
	if((mysql_num_rows($zap)>0))
	 {
		$blad .= "- E-mail już zarejestrowany, lub podano niepoprawny.<br>";
	 }
 
	if(($pass!=$repass) || (strlen($pass))<4)
	 {
	 $blad .= "- Błąd przy podawaniu hasła.<br>";
	 }
 
	if(!empty($blad)) 
	{
	echo $blad;
	$this->form(oczysc($login), oczysc($email));
	}
	else
	echo 'Zarejestrowano poprawnie!';
	// mysql_query -> dodawanie uzytkownika
 
 
	}
 
}
?> 
[PHP] pobierz, plaintext 

Oraz plik główny:

[PHP] pobierz, plaintext 
<?php
  include_once('klasy.php');
	$rejestruj = new Rejestr();
 
 
 
	if($_SERVER['REQUEST_METHOD']=="POST")
	{
	 if(isset($_POST['rejestr']))
	  {
		 $rejestruj->sprawdz($_POST['login'], $_POST['pass'], $_POST['repass'], $_POST['email']);
 
		}
 
	}
	else
	{
 
 
 $rejestruj->form(oczysc($login), oczysc($email));	
	}
 
 
 
?> 
[PHP] pobierz, plaintext 

Zwykle funkcję zamknięte w klasie, nie ma tutaj o czym pisać.

Co do obiektówki, możesz generować html w klasie, ale w Twoim przypadku mógłby być statyczny w innym pliku. Funkcje nie powinny niczego wyświetlać a jedynie zwracać pewne wartości.

Czyli jednym słowem, oprócz słowa class, to za dużo nie ma wspólnego z obiektówką?

A i owszem.

Ja klasę definiuję jako coś co ma pewne właściwości i można wykonywać na tym jakieś operacje, weźmy dla przykładu klasę: kot

Kot przechowuje pewne właściwości, zmienne i stałe.
Stałymi będą np. rasa, płeć
Zmiennymi: waga, wzrost, kolor sierści (z biegiem czasu się zmienia) etc.

A teraz funkcje, które możemy wykonywać na obiekcie "kot" (czyli mamy już konkretnego kotka)
np. karmienie, usypianie, zabawa

i funkcje prywatne, które będą wykonywały się automatycznie przy wykonaniu innych funkcji np. :
po użyciu funkcji: karmienie uruchamiamy automatycznie np. tycie, trawienie (bo bez karmienia nie można)


No i to tak w skrócie ode mnie.

Trafiło na kota, bo akurat mnie zaczepiał przy pisaniu. ;p

Skoro to początki obiektówki to zapamiętaj sobie: podstawą OOP jest obiekt, nie klasa - podobnie jak podstawą budownictwa są budynki, nie ich plany/szkice. Dlatego na swój kod patrz przez pryzmat jego interfejsu, a nie implementacji.
Jedna z podstawowych zasad to tworzenie obiektów, które zajmują się jednym konkretnym zagadnieniem, czyli przykładowo: walidacja danych, obsługa błędów w formularzu, sam formularz, zapisanie użytkownika - to wszystko to osobne obiekty.

btw: jest 2010 rok, funkcje mysql_*() już naprawdę powinny odejść w niepamięć. PDO może i cudem nie jest, ale w porównaniu do pierwszego... deklasuje.

Witam.

Dziękuję za odpowiedzi i chęci zobrazowania sprawy.

Nie ukrywam, że trochę już przeczytałem o obiektówce, gorzej z "wdrożeniem" tej wiedzy.

Może odwrócę trochę sprawę i napiszę co chcę uzyskać - obiektowo, bo strukturalnie to już działa.

W bardzo uproszczonej formie wyglądałoby to tak - ma być to strona radia internetowego, a w niej:
- użytkownik strony, który może:
* zarejestrować się
- przy rejestracji walidacja danych
* pisać pozdrowienia / zamówienia
* może edytować swoje dane

- od strony zarządzania stroną
* przyjmować pozdrowienia i przenosić do archiwum
* edytować użytkowników

Jeżeli możecie podpowiedzieć mi, jak to zrobić obiektowo, będę wdzięczny.