Początki z programowaniem obiektowym - klasa artykuł, Proszę o opinie sugestie Opcje

[czarek1986]

Witam

Od jakiegoś czasu interesuje się programowaniem obiektowym, ale od niedawna zdecydowałem się stworzyć coś własnego. Będzie to mój pierwszy obiektowy (przynajmniej w części) CMS. chciałbym przedstawić wam klasę artykułu do oceny. Prosze o konstruktywną krytykę (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
 
class Artykul {
 
//zawiera unikalny numer identyfikujący artykuł w bazie danych
private $id;
//zawiera nazwę autora tworzacego artykuł
private $autor;
//zawiera nazwę kategorii do której przyporządkowany jest artykuł
private $kategoria;
//zaiwera tytuł artykułu
private $tytul;
//zawiera wstępny tekst artykułu
private $naglowek;
//zawiera główną treść artykułu
private $tekst_artykulu;
//zawiera datę utworzenia artykułu !!Unixowy znacznik czasu
private $utworzony;
//zawiera datę ostatniej modyfikacji artykułu !!Unixowy znacznik czasu
private $zmodyfikowany;
//zawiera datę ostatniej publikacji artykułu !!Unixowy znacznik czasu
private $opublikowany;
//określa czy artykuł jest nowo utworzony, czy został wczytany z bazy danych
private $new;
 
 
 
public function __construct($id=NULL)
{
  if($id)
  {
   $this->id_art = $id;
   $db = Database::GetInstance();
   $sql = "SELECT * 
       FROM j_artykuly
       WHERE id = $id";
 
   $art = $db->select($sql);
 
   if(!$art)
    throw new Exception('Wybrany artykuł nie istnieje');
 
   $this->new = false;
   $this->set($art[0]);
 
  }
  else
   $this->new = true;
 
}
 
/**
* Metoda zwraca wartość jednego wybranego pola klasy
*
* @param name - nazwa pola w klasie, której wartość chcemy uzyskać
**/
 
public function getOne($name)
{
  if(property_exists(get_Class($this), $name))
   return $this->$name;
  else
   throw new Exception('Klasa '.get_class($this).' nie posiada pola '.$name);
}
 
/**
*
* Metoda zwraca wartości wszystkich pól klasy
*
* @return tablica z danymi wszystkich pól;
*
**/
 
public function getAll()
{
 
  $arData = array(
        'id'         => $this->id,
        'autor'        => $this->autor,
        'kategoria'     => $this->kategoria,
        'tytul'        => $this->tytul,
        'naglowek'     => $this->naglowek,
        'tekst_artykulu' => $this->tekst_artykulu,
        'utworzony'      => $this->utworzony,
        'zmodyfikowany'  => $this->zmodyfikowany,
        'opublikowany'   => $this->opublikowany,
       );
 
  return $arData;
}
 
 
/**
*
* Metoda służy do ustawienia wartości pól w klasie
*
[email="*@Param"]*@Param[/email] arData- tablica przechowująca nazwy pól i odpowiadające im wartości
*
**/
 
public function set(array $arData)
{
  foreach($arData as $key => $value)
  {
   if($key == 'id' && $this->new)
    throw new Exception("Wartość pola 'id' może być nadana tylko automatycznie");
 
   if(property_exists(get_Class($this), $key))
   {
    $this->$key = $value;
   }
   else
    throw new Exception('Klasa '.get_class($this).' nie posiada pola '.$key);
  }
}
 
 
 
/**
*
* Metoda Tworząca artykuł w bazie danych
*
**/
 
public function createArt()
{
  if($this->new)
  {
   $this->checkRequiredData();
 
   $db = Database::getInstance();
   if($db->Insert('j_artykuly', $this->getAll()) > 0);
    return true;
  }
 
  throw new Exception('Bierzący artykuł już istnieje w bazie danych. Można go tylko zaktualizować');
}
 
 
/**
*
* Metoda aktualizująca dane artykułu w bazie danych
*
**/ 
 
public function updateArt()
{
  if(!$this->new)
  {
   $this->checkRequiredData();
 
   $db = Database::getInstance();
   $arConditions['id']=$this->id;
   if($db->Update('j_artykuly', $this->getAll(), $arConditions) > 0);
    return true;
  }
 
  throw new Exception('Musisz najpierw utworzyć artykuł w bazie danych aby móc go edytować');
}
 
 
/**
*
* Metoda usuwająca artykuł z bazy danych
*
**/ 
 
public function delArt()
{
  if(!$this->new)
  {
   $this->checkRequiredData();
 
   $db = Database::getInstance();
   $arWhere = array("id = ".$this->id);
   if($db->delete('j_artykuly', $arWhere) > 0)
    return true;
  }
 
  throw new Exception('Nie można usunąć bierzącego artykułu, ponieważ nie został jeszcze zapisany w bazie danych.');
}
 
 
/**
*
* Metoda sprawdzająca czy pola, które nie mogą być puste w bazie danych, są wypełnione danymi
*
**/
 
private function checkRequiredData()
{
  $arData = array();
  $arData = $this->getAll();
  $arEmpty = array();
  foreach($arData as $key => $value)
  {
   if($key=='autor' || $key=='kategoria' || $key=='tytul' || $key=='naglowek' || $key=='utworzony')
   {
    if(!$value)
     $arEmpty[]=$key;
   }
  }
 
  if(count($arEmpty) > 0)
  {
   $empty = implode(', ', $arEmpty);
   throw new Exception('Wymagane pola '.$empty.' są puste.');
  }
 
  return true;
}
 
 
}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował erix 25.08.2009, 11:26:21

Powód edycji: [erix] przeniosłem

[thek]

A po co dodatkowy parametr new? ID w zupełności go zastępuje (IMG:style_emoticons/default/smile.gif) W końcu jeśli jest tam NULL, to spełnia on tę samą rolę. Sprawdzasz więc jedynie jaką wartość ma ID i w razie czego podejmujesz decyzję czy robić UPDATE czy INSERT w bazie. To sprawia, że createArt i updateArt możesz zastąpić jedną metodą. Konstruktor w zasadzie działałby podobnie. Brak id -> nowy z wartościami domyślnymi, prawidłowe id -> pobiera artykuł, nieprawidłowe id -> nowy + error. Kwestia funkcji do wyświetlania wszystkiego jest tak naprawdę sporna. Można użyć funkcji getOne, ale używanie funkcji w funkcji to zawsze minimalny narzut czasowy przy wykonywaniu. Więc patrząc od strony wydajności odwołanie się prosto do obiektu pozwala na ominięcie tworzenia zmiennych tymczasowych, odwołania do funkcji, powrotu do funkcji macierzystej, niszczenie zmiennych tymczasowych na stosie itp. Nie jest więc takie głupie jak się może wydawać bo oszczędza nam zarówno czas jak i pamięć serwera. Poza tym popatrz na getOne... Wywołujesz w niej za każdym razem funkcje i warunki. To spowalniałoby getAll za każdym razem.
Zastanawiałbym się także nad kategoria i tagi. Wszystko zależnie od tego jak masz zamiar rozwiązać to od strony bazy. Do przemyślenia jest bowiem choćby to, czy artykuł może być dodany do jednej, czy wielu kategorii. Bo jeśli do wielu to musisz nieco inaczej zapytania popisać. To samo tyczyło by ewentualnego dodania do klasy zmiennej dotyczącej tagów.