Początki programowanie OOP, Przedstawienie kodu napisanego w oop php Opcje

[Michal_PB1]

Jestem nowy na tym forum więc witam wszystkich.

Teraz do sedna postanowiłem rozpocząć przygodę z programowaniem obiektowym i mam pytanie czy ten kod poniżej na sens programowania obiektowego ? Proszę o wyrozumiałość to początki, ale uwagi mile widziane.

[PHP] pobierz, plaintext 
class User
	{
		private $mysql;
		private $host = "localhost";
		private $uzytkownik = "user";
		private $baza = "baza";
		private $haslo = "haslo";
		private $prefiks = "prefiks_";
 
		function __construct() {
 
			$this -> spr_stan();
			$this -> mysql = new mysqli ($this -> host, $this -> user, $this -> pass, $this -> db);
 
		}
 
		function __set ($nazwa, $wartosc) {
			$this -> $nazwa = $wartosc;
		}
 
		function __get ($nazwa) {
			return $this -> $nazwa;
		}
 
		function zaloguj($login, $haslo) {
 
			$sql = "SELECT * FROM ".$this -> prefiks."account WHERE login = $login AND password = $haslo ;";
			$sqlQry = $this -> mysql -> query($sql);
			$ile = $sqlQry -> num_rows;
			if ($ile == '1') $_SESSION['stan'] = '1';;		
 
		}
 
		function wyloguj() {
 
			$_SESSION['stan'] = '0';
			header('Location: index.php');
 
		}
 
		function zmien_haslo($email, $haslo) {
 
		}
 
		function spr_stan() {
 
			if($_SESSION['stan'] == '1')
				echo '<script>powiadomienie(\'Zalogowano\', \'accept\');</script>';
			else
			{				
 
				echo '<script>powiadomienie("Aby przeglądać Panel Admina musisz się <button onClick=\"show_login();\">", "information");</script>';
				exit;
			}
 
		}
 
	}
[PHP] pobierz, plaintext 

[Dipter]

Prócz tego co wypisał Tobie Fifi209, to ad. komunikatów, które wyrzucasz podczas wystąpienia czegokolwiek - od tego są wyjątki + do tego polecam wpis na blogu Zyxa.

Co do trzymania jakichś właściwości klasy (w twoim przypadku User) wykorzystaj do tego kontener/tablicę (lecz tracisz w tym przypadku brak supportu ze strony IDE), bądź po prostu ustaw je jako zmienne w klasie i za pomocą IDE wygeneruj metody set/get.

Swoją drogą klasa User, raczej powinna określać danego użytkownika jako byt, nie jako forma autoryzacji. Ponadto staraj się używać konwencji PSR-0/1/2 i angielskiego nazewnictwa.

Ad. przekazywania połączenia z bazą danych - Najlepiej jest to zrobić poprzez przekazanie obiektu jako argument choćby konstruktora. Stwórz abstrakcyjną klasę zajmującą się "Modelem" czyli warstwą do obsługi danych i ustal w niej metodę do przekazania połączenia. Nigdy zaś nie powinieneś dziedziczyć w stylu Użytkownik - Baza danych.

Ten post edytował Dipter 20.03.2013, 14:14:21