Forum PHP.pl

Witam. Moje pytanie jest następujące...

czy lepiej będzie oprzeć model (np. taki NewsContainer) na metodach w stylu:
getNewsByID()
getAllNews()
deleteNewsByID()

czy moze lepiej wszystko oprogramowac jako Iteratory (oczywiscie w role wchodzi tylko php5) i w php operowac na bazie sql jak na normalnych tablicach.

Owszem ladnie wyglada to drugie rozwiazanie... jednak probowalem je 'wdrazac' i szlo mi bardzo ciezko :/

Co wy o tym sadzicie.

Wg mnie powinienes wprowadzac wg czego jako argument najlepiej jako tablica bardzo elastyczne i sie nie napiszesz za duzo (rozne zachodza potrzebny wtedy sie przydaje ze mozna dac tablice)
np
array( 'id' => 1 );
Wybierze newsa z id 1
array( 'id' => 3, 'user' => 'hwao' )
Wybierrze cos z id 3 i 2 warunek ze musi jeszcze byc user hwao :-)

hwao: troche źle mnie zrozumiałeś (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
$newsList = new NewsContainer();
 
// Pobiera newsa o ID 1
$news = $newsList[1];
 
// Edytuje pobranego newsa.
$news['title'] = 'Inny tytul';
 
// Usuwa newsa o ID 2
unset($newsList[2]); 
?>
[PHP] pobierz, plaintext 

I wszystko to automatycznie wprowadza odpowiednie zmiany w bazie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Da się to uzyskać np. poprzez interfejs ArrayAccess

To tutaj nie masz iteratorów tylko ArrayAccess właściwie. IMHO twój przykład jest technicznie fajny, ale mało zrozumiały - przegięcie z bajerami. Ja bym wolał:

[PHP] pobierz, plaintext 
<?php
$news = $newsList->findById(1);
 
$news->title = 'foo';
// ... lub ...
$news->setTitle('foo');
 
$newsList->delete(1);
?>
[PHP] pobierz, plaintext 

$news['title'] jest możliwe ale bardzo zakręcone i tak naprawdę nic nie daje poza dezorientacją. $news->title jest już znacznie lepsze, jeżeli nie chcemy pisać metody setXXX() dla każdego atrybutu. Gorzej, jeżeli ustawienie tego atrybutu wymaga jakiś nietrywialnych operacji - wtedy w metodzie __set() musiałbyś mieć paskudnego switcha.

Natomiast usuwanie realizowane za pomocę destruktorów jest bardzo ciekawe, ale trochę bym się obawiał. Powody:
1) Destruktory będą wywoływane same z siebie na końcu skryptu.
2) Zawsze można utworzyć obiekt $news ręcznie (czyli bez odpowiednika w bazie danych), i co wtedy zrobi destruktor?
3) Jeżeli wystąpi jakiś problem przy zapytaniu DELETE, to jak poinformować użytkownika? Destruktor rzucający wyjątek?

Chociaż, jeżeli uda się ominąć trudności techniczne, jest to bardzo ciekawy pomysł. Rozwiązuje problem istnienia obiektu $news po usunięciu newsa - łączy usunięcie obiektu z usunięciem rekordu z bazy danych, a to jest duża zaleta.