Przykładowa klasa artykułów a jej konstrukcja, Temat początkującego Opcje

[Barton]

Witam,

Zaczynam przygodę z programowaniem obiektowym. Jako przewodnik służy mi książka 'PHP. Obiekty, wzorce, narzędzia' Matta Zandstry. Jednak nie mogę tam znaleźć jednoznacznej odpowiedzi na dość istotne pytanie.

Wyobraźmy sobie hipotetyczną podstronę która ma wyświetlać różne artykuły z bazy danych. I tutaj nasuwa się pytanie: w jaki sposób powinna być skonstruowana? Czy jedna klasa, odpowiadająca za artykuły powinna je wyświetlać, edytować, dodawać etc. na podstawie parametru $id? Np.:

[PHP] pobierz, plaintext 
$artykuly->wyswietl($id);
$artykuly->edytuj($id);
[PHP] pobierz, plaintext 

Czy może powinna istnieć osobna klasa reprezentująca jako taki artykuł. Mam tutaj na myśli sytuację w której z każdego pobranego artykułu tworzy się nowy obiekt:

[PHP] pobierz, plaintext 
$artykul[2]->wyswietl();
$artykul[2]->edytuj();
$artykul[3]->usun();
[PHP] pobierz, plaintext 

I w tej sytuacji każdy artykuł jest osobnym obiektem.

Które rozwiązanie jest lepsze? Odpowiedź na to pytanie myślę że dość mocno rozjaśni mi ideę OOP. Jeśli temat jest w nieodpowiednim dziale to proszę o przeniesienie do odpowiedniego.

Pozdrawiam.

Ten post edytował Barton 18.04.2012, 12:32:40

[viking]

Pomyśl w tym przypadku o artykule jako pojedynczym wierszu z bazy danych na którym dokonujesz różnych operacji (czyli np setId, setAuthor, getPublishDate). Czyli rozwiązanie numer 2 i w przyszłości znacznie ułatwisz sobie pracę.

[Fifi209]

setId? Id ma być unikalne i jego nie zmieniamy, broń boże

[mortus]

Wyświetlanie i edycja to zadania dla zupełnie innej klasy, a w przypadku edycji nawet dla kilku klas, którym trzeba odpowiednie dane przekazać (chodzi o klasy widoku i formularza). Natomiast to co powinna realizować klasa artykułu to interakcja pomiędzy aplikacją a bazą danych - de facto jest to tzw. model danych, będący jednym z elementów wzorca MVC (model-widok-kontroler). Model ten, jak już wspomniałem powinien odpowiadać za interakcję/wymianę danych pomiędzy aplikacją a bazą danych, czyli:

[PHP] pobierz, plaintext 
class Article {
  private $_db;
  public function __construct($db) {
    $this->_db = $db;
  }
  public function fetchAll() {
    $sql = "SELECT * FROM `articles`";
    $results = $this->_db->query($sql);
    $articles = array();
    while($row = $this->_db->fetchAssoc($results)) {
      $articles[] = $row;
    }
    return $articles;   
  }
  public function find($article_id) {
    $sql = "SELECT * FROM `articles` WHERE `id` = {$article_id}";
    $results = $this->_db->query($sql);
    $article = $this->_db->fetchAssoc($results));
    return $article;   
  }
  public function update($data) {
    // a tutaj aktualizacja danych, przy czym $data to tablica danych
    // zbudowana w oparciu o formularz 
  }
}
[PHP] pobierz, plaintext 

To tak w wielkim skrócie, bo powyższej klasie jeszcze bardzo wiele brakuje. Natomiast wyświetlanie powinno już odbywać się w innym miejscu, a to zależy od architektury aplikacji (we wzorcu MVC wyświetlanie odbywa się w widoku, przy czym kontroler przekazuje do niego odpowiednie dane). Niemniej wyglądałoby to tak:

[PHP] pobierz, plaintext 
// wyświetlanie wszystkich artykułów
$articleModel = new Article($db);
$articles = $articleModel->fetchAll();
foreach($articles as $article) {
  echo $article['title'];
  echo $article['content'];
}
// lub też jednego na podstawie id przekazanego w URL
$articleModel = new Article($db);
$article = $articleModel->find($_GET['id']);
echo $article['title'];
echo $article['content'];
[PHP] pobierz, plaintext 

EDIT:
Powyższa klasa i sposób jej wykorzystania to tylko przykłady, które w rzeczywistości wymagają jeszcze trochę (lub raczej dużo) pracy.

Ten post edytował mortus 18.04.2012, 18:10:33

[viking]

setId? Id ma być unikalne i jego nie zmieniamy, broń boże

[SQL] pobierz, plaintext 
SELECT id, title FROM articles
[SQL] pobierz, plaintext 

[PHP] pobierz, plaintext 
foreach.... $article->setId($row->id)
[PHP] pobierz, plaintext 

W widoku wybranie z tego.

[mortus]

@viking: Jeśli już, to metoda powinna się nazywać raczej getId(), a nie setId() - get = pobierz, set = ustaw.

[viking]

Naprawdę? Chyba ustawiasz danymi z bazy (set) które pobierasz w widoku (get). Nazywa się to (polskie tłumaczenie sobie odpuszczę) domain object / model. Mówimy w swoich przykładach zastosowań o 2 różnych rzeczach. Ty zrobiłeś z tej klasy bezpośredni mapper. Ja mówię o takim podejściu http://martinfowler.com/eaaCatalog/dataMapper.html
http://stackoverflow.com/a/7109595
http://mwop.net/blog/202-Model-Infrastructure.html

Ten post edytował viking 19.04.2012, 05:34:12