Usuwanie podstrony/innej treści, Jak to wykonać zgodnie z obiektowością? Opcje

[kiler129]

Witajcie!
Krótko i na temat - postanowiłem z lekka uporządkować kod swojego CMSu i uczynić go bardziej obiektowym. W związku z tym sprawdziłem jaki wpływa na prędkość ma zbudowanie obiektu i wykonanie metody kontra wykonanie metody statycznej, wyniki z lekka mnie zaskoczyły:

[PHP] pobierz, plaintext 
<?php
class test {
 static public function get() { return array(); }
}
 
class test2 {
 public function get() { return array(); }
}
 
$s = microtime(true);
for($i=0; $i<=1000; $i++) {
 $x = new test;
 $y = $x->get();
 unset($x, $y);
}
echo "Object: ".round((microtime(true)-$s), 4)." ms<br/>";
 
$s = microtime(true);
for($i=0; $i<=1000; $i++) {
 $y = test2::get();
 unset($y);
}
echo "Static: ".round((microtime(true)-$s), 4)." ms<br/>";
 
/*
 Object: 0.0036 ms
 Static: 0.0037 ms
*/
?>
[PHP] pobierz, plaintext 

Różnica praktycznie żadna a wg teorii OOP wykonujemy akcję na danym obiekcie więc chyba najsensowniejszym podejściem jest taka organizacja obsługi podstron:

[PHP] pobierz, plaintext 
<?php
 class subpage {
  protected $id = NULL;
  protected $validId = false;   
 
  function __construct($id=NULL) {
   if($id > 0 && textOperations::isInt($id)) { //basic validation
    $this->validId = true;
    $this->id = $id;
   } else {
    $this->validId = false;
   }
  }
 
  public static getList() { }
  public function get() { }
  public function delete() { }
  public function save($arrayWithParams) { }
 }
 
 $page = new subpage(134);
 $x = $page->get();
 $page->delete();
 unset($page);
?>
[PHP] pobierz, plaintext 

Jedna z metod (akurat w tym przykładzie) jest niezależna od id podstrony bo defakto odnosi się nie do jednej a do listy podstron.
Zastanawia mnie natomiast jedna kwestia - czy obiektowo poprawnie będzie rozwiązać to tak:

[PHP] pobierz, plaintext 
<?php
 class subpage {  
  public static getList() { }
  public static function get($id) { }
  public static function delete($id) { }
  public static function save($id, $arrayWithParams) { }
 }
 
 $x = subpage::get(134);
 subpage::delete(134);
?>
[PHP] pobierz, plaintext 

Druga metoda jest wygodniejsza w użyciu, prędkością się nie różni ale nie wiem czy jest to do końca poprawne.
Z góry dzięki za odpowiedź (IMG:style_emoticons/default/smile.gif)

[kiler129]

Faktycznie, strzeliłem błąd. Przy poprawnej notacji wyniki (dla 10 tys przejść) są już nieco bardziej sensowne:
Object: 0.0358 ms
Static: 0.0166 ms

Tutaj faktycznie logiczniej jest tworzyć obiekt i ew. zostawiać część metod statycznych. A jeśli chodzi o klasy z wszystkimi statycznymi metodami to wydaje mi się, że ma to sens w wypadku gdy coś jest często używane w dużej ilości miejsc kodu gdzie przekazywanie uchwytu nie daje korzyści a jest upierdliwe (np. klasa do cache).

Tylko teraz co z funkcjami typu getMultiById()? Wtedy jest to operacja niby nie na jednej podstronie a na wielu. Nijak tu nie pasuje podanie do konstruktora array(1,3,55,134) więc dobrze aby metoda sama w sobie przyjmowała listę idków jako argument.