Zwracanie wartości przez model Opcje

[janusz.master]

Mam sobie dwie klasy: userData - zawiera dane uzytkownika, oraz userBase - wszelkie operacje na bazie danych zwiazane z uzytkownikiem. I mam pytanie: co powinna zwracać funkcja klasy userBase, pobierania uzytkownika z bazy? Powinna zwracać tablicę z danymi czy lepiej byłoby aby zwracała obiekt klasy userData - uzytkownika??

Mam taki kod:

[PHP] pobierz, plaintext 
<?php
class userData extends zasob{
 
}
 
class userBase extends model{
	public function add(userData $user)
	{
		$this->db->insert($this->ust['table'], $user->getDataAssoc());
	}
	public function update(userData $user)
	{
		$this->db->setWhere(array('login'=>$user->login));
		$this->db->update($this->ust['table']);
	}
	public function delete(userData $user)
	{
		$this->db->setWhere(array('login'=>$user->login));
		$this->db->delete($this->ust['table']);
	}
	public function getByLogin(userData $user) // pobiera 1 uzytkownika na podstawie loginu
	{
		$this->db->setWhere(array('login'=>$user->login));
		$result = $this->db->select($this->ust['table'], array('*'));
		$user = new userData;
		$user->fetch($result);
		return $user;
	}
	public function getUsers(userData $user) // pobiera uzytkownikow na podstawie wzorca
	{
		$this->db->setWhere($user->getDataAssoc());
		$result = $this->db->select($this->ust['table'], array('*'));
 
		$users = '';
		foreach ($result as $user)
		{
			$user = new userData;
			$user->fetch($result);
			$users[] = $user;
		}
		return $users;
	}
}
?>
[PHP] pobierz, plaintext 

No i jak widać przy pobieraniu z bazy przy każdego rodzaju zasobie muszę tworzyć w funkcji nowy obiekt (obiekty) i zapisywać do nich informacje z bazy. Sposób z przekazywaniem tablicy wydaje się być dużo prostszy i mniejszy objętościowo dlatego pytam jak wy to robicie i jakie są tego zalety/wady.

[Cysiaczek]

Zdecydowanie obiekt. Jest to logiczne, że klasa zarządzająca tymi obiektami wypełnia je w jakiś sposób i zwraca ich kolekcję. Po co Ci tablica z danymi? W każdym poleceniu i tak musiałbyś przepisywać tablice na obiekt, więc tak, jak jest teraz - jest dobrze. Jedynie metoda userBase::getUsers() powinna zostać rozbita na dwie metody.

1. userBase::loadUsers() - Odpowiada za pobranie danych z bazy i umieszczenie obiektów w kolekcji
2. userBase::getUsers() - zwraca jedynie utworzoną kolekcję, a w przypadku jej braku wywołuje userBase::loadUsers()

Pozdrawiam.

[janusz.master]

2. userBase::getUsers() - zwraca jedynie utworzoną kolekcję, a w przypadku jej braku wywołuje userBase::loadUsers()

Nie za bardzo rozumiem - getUsers uruchamia loadUsers a gdy ta nie zwroci nic to znow ja wywoluje?? Nie kumam

Przez to co powiedziałeś, zrozumiałem ze ma to wygladac mniej wiecej tak:

[PHP] pobierz, plaintext 
<?php
class userBase extends model{
	protected $users = array();
 
// .....
 
	protected function loadUsers(userData $user)
	{
		$this->db->setWhere($user->getDataAssoc());
		$result = $this->db->select($this->ust['table'], array('*'));
 
		$users = '';
		foreach ($result as $user)
		{
			$user = new userData;
			$user->fetch($result);
			$users[] = $user;
		}
		$this->users = $users;
		return $users;		
	}
 
	public function getUsers(userData $user)
	{
		if($this->users) return $this->users;
		return $this->loadUsers($user);
	}
}
?>
[PHP] pobierz, plaintext 

Dobrze mysle??

[Cysiaczek]

Tak, właśnie coś takiego

Pozdrawiam