Forum PHP.pl

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 

Dane do połączenia z bazą powinny zostać przekazane np do konstruktora a nie na stałe zapisane w ciele klasy.
Metody __set i __get nic nie wnoszą i są tutaj niepotrzebne
Brak flirtowania danych (patrz zaloguj)
spr_stan powinna zwracać true/false w zależności czy zalogowany czy nie, zamiast wyrzucać jakieś komunikaty.

Właśnie dane do połączeni z bazą dałem na stałe (na tą chwilę), ponieważ zastanawiałem się jak zrobić, aby w kilku klasach można było z bazy korzystać. Bo za każdym razem przekazywać dane np do konstruktora to trochę nie poręczne, myślałem nad dziedziczeniem (tylko nie wiem czy to ma sens) oraz użycie zmiennej globalne i tutaj nie do końca wiedziałem jak to uczynić
Co do __set i __get to racja na chwilę obecną nic nie wnoszą.
Co spr_stan to słusznie

Przyłączam się do pytania z połączeniem z bazą - jak to najlepiej zrobić, żeby można było korzystać z połączenia w kilku klasach?

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

W dużym ułatwieniu można choćby użyć singleton'a, którego nie polecam.

Drugim wyjściem jest napisanie jakieś klasy, która będzie trzymała cały config i połączenie z bazą w sobie i wtedy z każdego miejsca się tam dostaniesz.

Ten post edytował Fifi209 20.03.2013, 14:27:38

Ten kod jest niepoprawny z kilku powodów:

1. Wspomniane już robienie "dwulicowej klasy".
2. Pakowanie widoku w model.
3. Niepoprawne wywoływanie funkcji w konstruktorze.
4. Mieszanie polskich nazw z angielskimi.
5. Niekompletny kod.
6. Brak komentarzy przez co potem się pogubisz.

Przykładowe poprawne rozwiązanie (propo bazy danych):

[PHP] pobierz, plaintext 
class User {
 
private $conn;
public static $userTablePrefix = 'prefiks_';
 
public function __construct($conn) {
$this->conn = $conn;
}
 
}
[PHP] pobierz, plaintext 

Ten post edytował pyro 20.03.2013, 15:03:31

Korzystając z tematu. Mam pytanie, dlaczego niektóre zmienne składowe daje się w nawiasy klamrowe, np.


$variable = "{$this ->producerFirstName}";


Czytam książke "PHP Obiekty, wzorce, narzędzia" i tu autor często stosuję tę konwencję - niestety bez wspominek o konieczności ich stosowania.

Z mojego domysłu używa się ich do seperacji, ciągów znaków od innych typów danych, myle się?.

http://php.net/manual/en/language.types.string.php -> curly syntax