MVC - klasy modelu - pojedyncze obiekty i listy Opcje

[marcini82]

Witam!

Pytanie dotyczy klas modelu w rozumieniu wzorca MVC, a ma zastosowanie chyba do prawie kazdego rodzaju danych przechowywanych w bazie. Otoz jesli mamy jakiekolwiek obiekty (uzytkownikow, ogloszenia, wiadomosci, itp.), to zazwyczaj istnieje potrzeba przeprowadzania operacji na pojedynczym obiekcie (dodanie, usuniecie, edycja itp.) oraz na grupie obiektow, spelniajacych okreslone kryteria (np. wyswietlenie listy uzytkownikow, ogloszen itp.). Jaka tworzycie tutaj strukture klas?
Czy robicie dwie klasy, z ktorych jedna reprezentuje pojedynczy obiekt a druga liste takich obiektow (np. Uzytkownik i ListaUzytkownikow)? A moze jedna klasa (np. Uzytkownik) posiada metody obslugujace obie mozliwosci, np. pobierz($id) dla pojedynczego obiektu i pobierzListe() dla calej listy obiektow? A moze jeszcze inaczej?

EDIT: Wlasciwie to ten temat powinienem dac do PHP > Programowanie obiektowe. Moze moderator zechcialby go przeniesc?

Ten post edytował marcini82 3.03.2007, 14:58:48

[strife]

Przenoszę do PHP > Programowanie obiektowe

[Strzałek]

Zobacz sobie model w Cake, poczytaj o ActiveRecord.

Ja mam klasę NewsContainer, z metodami findById, findByTitle, findAll etc.

[Athlan]

Strzałek dobrze prawi imho... model users powinien obsługiwać metody takie jak findAll, get($mId, $bByName = false), setEmail(), setParams( jakies dane usera ) itp itd dotyczącego usera. Mogę zarzucić przykład z mojego modelu obsługującego filmiki w bazie danych (skomny, ale przedstawi działanie):

[PHP] pobierz, plaintext 
<?php
 
class movies_Vmodel extends Vmodel
{
  public function getLastModified($iTime)
  {
	$oDb = $this->getSource();
 
	$oDb->setOrder("movie_time_edit ASC");
	$oDb->setWhere("movie_time_edit > " . (time() - $iTime));
 
	$rResult = $oDb->dbSelect('movies', true, array('users' => 'movie_owner = user_id'));
 
	$oDb->clear();
 
	return $oDb->fetchAll($rResult);
  }
 
  function getUserMovies($iUser)
  {
	$oDb = $this->getSource();
 
	$oDb->setWhere("movie_owner = " . (int)$iUser);
 
	$rResult = $oDb->dbSelect('movies', true);
 
	$oDb->clear();
 
	return $oDb->fetchAll($rResult);
  }
 
  function setMovie($iUser, $sDescription, $sCode)
  {
	$oDb = $this->getSource();
 
	$oDb->movie_owner = $iUser;
	$oDb->movie_text = $sDescription;
	$oDb->movie_code = $sCode;
	$oDb->movie_time = time();
	$oDb->movie_time_edit = time();
 
	$rResult = $oDb->dbInsert('movies');
 
	if($rResult)
	  $oDb->Execute("UPDATE users SET user_count_movies = user_count_movies + 1 WHERE user_id = " . $iUser);
 
	$oDb->clear();
  }
 
  function getMovie($iId)
  {
	$oDb = $this->getSource();
 
	$oDb->setWhere("movie_id = " . (int)$iId);
 
	$rResult = $oDb->dbSelect('movies', true, array('users' => 'movie_owner = user_id'));
 
	$oDb->clear();
 
	return $oDb->fetch(1, $rResult);
  }
 
  function unMovie($iId, $aData)
  {
	$oDb = $this->getSource();
 
	$oDb->setWhere("movie_id = " . (int)$iId);
 
	$rResult = $oDb->dbDelete('movies');
 
	$oDb->clear();
 
	$oDb->Execute("UPDATE users SET user_count_movies = user_count_movies - 1 WHERE user_id = " . $aData['movie_owner']);
 
	return $rResult;
  }
 
  function setMovieDesc($iId, $sDesc)
  {
	$oDb = $this->getSource();
 
	$oDb->setWhere("movie_id = " . (int)$iId);
	$oDb->movie_text = substr($sDesc, 0, 40);
	$oDb->movie_time_edit = time();
 
	$rResult = $oDb->dbUpdate('movies');
 
	$oDb->clear();
 
	return $rResult;
  }
}
 
?>
[PHP] pobierz, plaintext 

[Strzałek]

Trochę mało wygodnie to masz. Przy tabelach które nie potrzebują joinów u mnie wystarczy

[PHP] pobierz, plaintext 
<?php
 
 class NewsContainer extends DatabaseMapper {
   public $table = "news";
 }
 
?>
[PHP] pobierz, plaintext 

I juz mam wszystko czego potrzebuję

[PHP] pobierz, plaintext 
<?php
 
$news = new NewsContainer();
 
$news -> findAll();
//lub
$fields = array('id', 'title', 'time', 'comments', 'intro');
$criteria = array('sort_by' => 'id', 'sort_type' => 'DESC', 'limit' => '10');
 
$news -> findAll($fields, $criteria);
 
 
$news -> findById($id);
$news -> findByTitle('Tytuł newsa');
 
$news -> deleteById($id);
 
$nowyNews = array('title' => 'Super tutuł', 'text' => 'tresc', 'time' => time());
 
$news -> save($nowyNews);
 
?>
[PHP] pobierz, plaintext 

Proste prawda ?

Do każdej rzeczy będziesz teraz pisał findAll, findBy(.*), ja u siebie mam abstrakcyjna klasę modelu, która mam podstawowe metody - find, save, update, delete + dodatkowo, __call które obsługuje findByTitle odpalając metodę find z odpowiednimi argumentami. Taki sposobem powstał mały DatabaseMapper. Bez żadnych wodotrysków i zapytań z joinami. Jeżeli potrzebuję do NewsContainer'a łaczyć tabele, dopisuje sobie wtedy metodę i recznie piszę zapytanie.

No chyba że mam tego więcej. Wtedy mamy Propel lub PHP Doctrine

[Athlan]

@Strzałek, owszem, przyznam się, że obsługa bazy danych to kicha u mnie mi tam wygodnie jest, ale w przykladze chodzilo głownie o to, aby pokazać możliwości modelu (filmy to filmy itp).

Co do tej obsługi... w Rapide budowana jest całkiem nowa już

[marcini82]

Dzieki za odpowiedzi. Jak sobie czytam o ActiveRecord to widze, ze to jest mniej wiecej to, co od jakiegos czasu zazwyczaj stosowalem, nie nazywajac tego w ten sposob

@Strzalek - ciekawa koncepcja tego DatabaseMapper. Wlasnie mysle, czy da sie stworzyc cos tak uniwersalnego dla projektu nad ktorym siedze :-)

Ten post edytował marcini82 5.03.2007, 07:20:53

[Turgon]

Strzałek: Mam kilka wątpliwości. Otóż też napisałem takiego podobnego DatabaseMappera, który ładuje na podstawie takiej:
load(fieldName,fieldValue);
I tak sie zastanawiam, czy to ma sens w takiej formie ? Bo wtedy Obiekt, który należy do grupy TurActiveRecord i jest odpalany przez fabrykę. Ma tą zaletę, że nie trzeba się bawić w rozdział mapperów dla baz?

[Strzałek]

Nie wiem o co Ci chodzi. Za dużo filozofii i sztuki dla sztuki widzę u Ciebie. Mam Controller w którym jak potrzebuję to tworzę sobie nowy obiekt np. NewsContainer. Jak nie potrzeba to nie tworzę i tyle.

[Turgon]

O może tak. Mam taki problem, że u mnie Database Mapper działa tylko dla jednego obiektu. Jak proponujesz to przerobić by pracowało z wieloma rekordami? Jak zrobić hasOne i hasMany ? Tylko tyle mi potrzeba ?

[Strzałek]

Aaaa, hehe. Napisałem wyżej że ominąłem relacje. Można je napisać i nie jest to wcale trudne. Po prostu przy find(.*) musisz dodatkowo obsłużyć relacje. Dodajesz właściwości $hasMany, $hasOne np.:

[PHP] pobierz, plaintext 
<?php
 class NewsContainer extends DatabaseMapper {
   public $table = 'news';
   public $hasOne = 'author';
 }
?>
[PHP] pobierz, plaintext 

Dalej automatycznie się wszystko dzieje.

[Turgon]

Eeee... Poprosiłbym jakieś dokładniejsze przykłady ? Dla mnie trzeba łopatologicznie. Najchętniej bym poczytał coś w stylu artykułów o DBObject Hwaka

[marcini82]

Nie do konca Turgon rozumiem, w czym tkwi twoj problem, i co ma oznaczac to $hasOne i $hasMany... Jesli NewsContainer ma okreslona tabele news to w DatabaseMapper budujesz uniwersalne zapytanie w oparciu o te tabele i dostajesz zestaw rekordow...
Moze gdybys wytlumaczyl tak lopatologicznie, na jakims przykladzie, to ktos bylby w stanie ci pomoc.

[Strzałek]

Chodzi o relacje. Tabele buduje się relacyjnie. Np.

News
id
title
text
author

Comments
id
news_id
text

Users
id
login
pass

I w tabeli news trzymamy w polu author, numer id usera, natomiast w comments trzymamy komentarze gdzie w news id znajduje się id newsa do którego dodano komentarz. Żeby coś takiego uzyskać są potrzebne relacje has.... To tak w telegraficznym skrócie. Ale to już wyższa szkoła jazdy. W swoim czasie się dowiesz o co chodzi.

[marcini82]

Chodzi o relacje.

A no to w takim razie ja bym zrobil cos takiego:

[PHP] pobierz, plaintext 
<?php
class NewsContainer extends DatabaseMapper {
   public $table = 'news';
   public $joins = Array('author' => 'Users.id');
 }
?>
[PHP] pobierz, plaintext 

Jak mamy w $joins okreslone kolumny na ktorych robimy zlaczenia to mozemy to wykorzystac w uniwersalnej metodzie DatabaseMappera. A jak trzeba cos bardziej wyrafinowanego wyciagnac niz proste zlaczenie 2-3 tabel to lepiej w NewsContainer nadpisac metode z DatabaseMapper i wklepac nowe zapytanie.
Wlasciwie to ja bym tego nie nazwal wyzsza szkola jazdy bo to raczej podstawy relacyjnych baz danych
Napisanie dobrej i uniwersalnej struktury klas to juz co innego...

Sposobu praktycznego wykorzystania tego $hasOne nadal nie zalapalem, ale to moze nie jest takie istotne...

Ten post edytował marcini82 11.03.2007, 12:24:32