Wyświetlanie danych z MySQL, a OOP Opcje

[spokoloko123]

Jak poprawnie wyświetlać dane z bazy danych zgodnie z OOP? Czy konstrukcja:

[PHP] pobierz, plaintext 
while($this->row = mysql_fetch_accoc){
$zmienna = $this->row['title'];    
$zmienna .= $this-row['content'];
}
[PHP] pobierz, plaintext 

, a potem wyświetlenie zmiennej jest poprawna w ujęciu programowania obiektowego? Dlaczego, dlaczego nie?

[bastard13]

W programowaniu obiektowym baza danych to tylko miejsce, w którym dane stałe są zapisywane. Od zarządzania tymi danymi masz modele, czyli klasy, które charakteryzują logikę Twojej aplikacji. Oczywiście musisz posiadać klasę, która stworzy Ci model na podstawie danych z bazy (w Twoim przypadku kolekcje modeli). Może to być osobny obiekt (co jest chyba najlepszym rozwiązaniem) bądź sama klasa modelu może posiadać mechanizm do wyciągania tych danych. Drugie rozwiązanie jest o tyle niezbyt pożądane, że w przypadku, gdy musisz zmienić miejsce przechowywania danych np. z bazy MySQL na pliki XML, to musisz dotykać klas, które powinny być odpowiedzialne jedynie za logikę, a więc niezależne od miejsce przechowywania danych.

[markonix]

Ostatnio też się nad tym głowiłem.
W różnych mini frameworkach tworzona była klasa model i klasa obiektu np. model i obiekt książki.
Np pobranie 10 książek to utworzenie 10 obiektów typu "książka". Wszystko ładnie i pięknie ale gdy robiłem klasę użytkowników gdzie jeden użytkownik miał ponad 20 atrybutów to konstruktor wyglądał nieciekawie i niestety nie udało mi się utrzymać tej konwencji. PDO (mysql_ też) może zwrócić wiersz jako obiekt ale wydaje mi się to sztucznym rozwiązaniem.

Też jestem ciekaw jak to wygląda w praktyce.

[bastard13]

@markonix zawsze możesz zastosować Proxy. Np.
10 książek to kolekcja obiektów. Kolekcja też sama w sobie jest obiektem. I przy tworzeniu tej kolekcji wyciągasz podstawowe informacje nt. książek (np. tytuł, + autor). Natomiast jeżeli potrzebujesz wykonać jakieś bardziej złożone operacje na książce, to dopiero tak naprawdę jest tworzony obiekt.
To samo z użytkownikami.

[spokoloko123]

Gdy do wszystkiego podchodzę od strony obiektu mysqli czyli new mysqli(...) to pojawia się problem i nie jestem zdecydowany do do tego jak go rozwiązać.
Wiadomo, że do mysql łączyć się będę tylko jeden raz, ale obiekty stworzone poza klasą nie są w niej dostępne. Więc moje pytanie jest następujące, czy używać sobie takiemo menadżera, który napisałem:

[PHP] pobierz, plaintext 
class MySQLiQueryManager
	{
		private static $instance;	
		public function __construct()
		{
			if(!self::$instance)
			{
				self::$instance = $this;
				$connect_args = func_get_args();
				if(count($connect_args) != 4)
				{
					echo "Nowy egzemplarz musi zawierać host, nazwię użytkownika i jego hasło oraz bazę danych.";
				}
				self::$instance->db = new mysqli($connect_args[0], $connect_args[1], $connect_args[2], $connect_args[3]);
				if(mysqli_connect_errno())
				{
					echo "Błąd połączenia z baza danych: ".mysqli_connect_error();
					exit;
				}
				self::$instance->db -> set_charset("UTF8");
			}
			else {
				$connect_args = func_get_args();
				if($connect_args)
				{
					echo "Tylko nowy egzemplarz może zawierać połączenie!";
					exit;
				}
			}
		}
		public function query($query)
		{
			return self::$instance->result = self::$instance->db->query($query);
		}
	}
 
	new MySQLiQueryManager("localhost", "root", "", "test");
 
	class Users
	{
		public function getUsers()
		{
			$user = new MySQLiQueryManager;
			$result = $user->query("SELECT * FROM users");
			while($row = $result->fetch_row())
			{
				echo "<pre>";
				print_r($row);
				echo "</pre>";
			}
		}
	}
[PHP] pobierz, plaintext 

i wywoływać ten stary egzemplarz MySQLiQueryManager czy może zostać przy strukturze proceduralnej mysqli na potrzeby użycia w klasach?

[ano]

Ciekawe podejście do singletona (IMG:style_emoticons/default/wink.gif)

A tak serio to nie musisz robić statycznej instancji połączenia z DB. Wystarczy, że do obiektów typu Users będziesz przekazywał (np. w konstruktorze) instancję MySQLiQueryManager.

Np:

[PHP] pobierz, plaintext 
 
	class Users
 
	{
		/**@var MySQLiQueryManager */
		protected $queryManager;
 
		public function __construct(MySQLiQueryManager $queryManager) {
			$this->queryManager = $queryManager;
		}
 
		public function getUsers()
		{
 
			$user = $this->queryManager;
			$result = $user->query("SELECT * FROM users");
 
			while($row = $result->fetch_row())
			{
				echo "<pre>";
				print_r($row);
				echo "</pre>";
 
			}
		}
	}
[PHP] pobierz, plaintext 

Poza tym używanie func_get_args(); w konstruktorze MySQLiQueryManager to BAAARDZO zły pomysł. Co z podpowiadaniem składni w IDE? Po roku już zapomnisz co musisz przekazać w konstruktorze! Zmień to na __construct($username, $password, $host,...)

Kolejna baaardzo słaba rzecz:

[PHP] pobierz, plaintext 
echo "Błąd połączenia z baza danych: ".mysqli_connect_error();
exit;
[PHP] pobierz, plaintext 

To powinno wyglądać np. tak:

[PHP] pobierz, plaintext 
throw new MySQLiQueryManagerException('Błąd połączenia z baza danych: ' . mysqli_connect_error());
[PHP] pobierz, plaintext