Klasa news - czy to można nazwać OOP? Opcje

[adriano321]

Jestem dopiero co zaczynającym "programistą" (haha) PHP. Programowanie proceduralne nie sprawia mi już większych kłopotów więc chcę wkroczyć na wyższy poziom. Niestety same podstawy tj. teoria nie wystarczy, żeby napisać coś samemu czyt. dobrze napisać. Więc tu się zwracam z prośbą do Was: Czy to coś poniżej ma jakiś związek z OOP? I czy z własnych doświadczeń możecie coś zaproponować, jakieś rozwiązania dalsze?

Klasa ObsługaNewsa z założenia ma być dostępna tylko z poziomu panelu administracyjnego.

Kolejne pytanie:
Jak rozwiązać problem wyświetlania kilku newsów na jednej stronie? Utworzyć kilka egzemplarzy klasy News? Tu zastanawiam się jak zrobić to, aby wykonać tylko jedno zapytanie do bazy danych?

Czy można się w jakiś sposób pozbyć dodawania $db w parametrach metod??

[PHP] pobierz, plaintext 
<?php
class News {
 
	public $id;
	public $tytul;
	public $tresc;
	public $autor;
	public $data;
	public $kategoria;
 
	function __construct() 
	{
	}	
 
}
 
class ObslugaNews extends News { 
 
	public function dodajNewsa ($db) 
	{
		$sql = "INSERT INTO `news` (`id` ,`title` ,`text` ,`date` ,`edit_date` ,`id_author` ,`id_kategory`) VALUES ('','$this->tytul','$this->tresc','$this->data','','$this->autor','$this->kategoria')";
 
		if (@!$db->query($sql))  echo " z tego!";
	}
 
	public function edytujNewsa ($id,$db) 
	{
		$sql="UPDATE `news` SET `title`=$this->tytul ,`text`=$this->tresc ,`edit_date`='aktualna_data' ,`id_kategory`=$this->kategoria WHERE `news`.`id`=$id LIMIT 1";
 
		if (@!$db->query($sql))  echo " z tego!";
	}
 
	public function pobierzNewsa ($id,$db) 
	{
		$sql="SELECT `id` ,`title` ,`text` ,`date` ,`edit_date` ,`id_author` ,`id_kategor
y` FROM `news` WHERE id='$id' LIMIT 1";
 
		if (@!$wynik=$db->query($sql))  echo " z tego!";
		$wynik = $wynik->fetch_array();
 
		return $wynik;
	}
 
	public function ustawNewsa ($tytul,$tresc,$kategoria) 
	{
		$this->tytul = strip_tags(ltrim(addslashes($tytul)));
		$this->tresc = strip_tags(addslashes(ltrim($tresc)));
		$this->kategoria = strip_tags(ltrim(addslashes($kategoria)));
	}
}
?>
[PHP] pobierz, plaintext 

Adriano.

[Cysiaczek]

Nie zniechęcaj się, ale to jest kod strukturalny ujęty w klasy (pseudo oop). Nie jest to nic złego, ale aby nie psuć pojęcia OOP, to nazywajmy to poop (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)

Błędy, które od razu rzucają się w oczy.

1. Nazewnictwo klas i metod. Pisz angielskie wersje
2. W tym wypadku News jest jedną tabelą, więc dobrze, aby był tak traktowany (to sie nazywa mapowanie relacyjno obiektowe - tu w podstawowej wersji na jednej tabeli). Częśc metod, jak np. obsługaNews:: pobierzNewsa($id) może być w samej klasie News. Ni eoznacz to bynajmniej, że ta klasa obsługaNews jest zbędna.

[PHP] pobierz, plaintext 
<?php
class News
{
	public $id;
	public $tytul;
	public $tresc;
	public $autor;
	public $data;
	public $kategoria;
 
	function __construct($data)
	{
		// Wykorzystamy to, że php ma dynamiczne typowanie zmiennych
		if(is_integer($data))
		{
			$sql="SELECT * FROM news WHERE id='$id' LIMIT 1";
			$result=$db->query($sql);
			$current=$result->fetchRowAsArray();
		}
		elseif(is_array($data))
		{
			$current=& $data;
		}
 
		//Tu już Twoja wyorraźnia niech zadziała, bo da się to zrobić ładniej, ale ja nie 
chce zaciemniać kodu
		$this->id=$current['id'];
		//itd.
	}
 
	public function save()
	{
		// zapisa obiektu do bazy danych
	}
 
}
 
class NewsBase
{
	private $collection;
 
	public function __construct()
	{
		$this->collection=new NewsCollection(); // Obiekt kolekcji
	}
 
	public function loadSomeNews($howMany, $order)
	{
		$stack=array();
		$sql="SELECT * FROM news WHERE ORDER BY $order LIMIT $howMany";
		$result=$db->query($sql);
		foreach($result->fetch_array() as $row)
		{
			$obj=new News($row);
			$stack[]=$obj;
			$this->collection->add($obj);
		}
		return $stack;
	}
 
	public function findById($id)
	{
		if($this->collection->has($id))
		{
			return $this->collection->get($id);
		}
		else
		{
			$obj=new News($id);		
			$this->collection->add($obj);
			return $obj;
		}
	}
 
	function saveCollection()
	{
		foreach($this->collection as $obj)
		{
			$obj->save();
		}
	}
}
?>
[PHP] pobierz, plaintext 

Pisane z palca, bez sprawdzenia, traktuj to jako pseudokod (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pozdrawiam.

p.s To nie jest jedyna metoda, anie też nie zawsze najlepsze. Jest za to szybka w implementacji i daje dość duże pole manewru.

[Whisller]

@cysiaczek
Ja osobiscie nie wstawiał bym pobierania danych newsa w klasie reprezentujacej jego obiekt (News), bo i po co ma ona wiedziec jak i skad pobierac te dane?

Pozwolicie ze przedstawie swoja idee (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?
 
/**
 * Klasa reprezentujaca pojedynczy news
 *
 */
 
class News
{
	/**
	 * W konstruktorze mozemy ustawic dane obiektu. Dla newsa to np. title, body, vi
sibility etc.
	  *
	 * @param array $args
	 */
	public function __construct(array $args)
	{
		isset($args['id'])		   ? $this->setId($args['id'])				   : '';
		isset($args['title'])	  ? $this->setTitle($args['title'])			: '';
		isset($args['body'])	   ? $this->setBody($args['body'])			  : '';
		isset($args['visibility']) ? $this->setVisibility($args['visibility']) : $this->setVisibility(false);
	}
 
	/**
	 * @param int $id
	 */
	public function setId($id)
	{
		$this->id = (int)$id;
	}
 
	/**
	 * @param string $title
	 */
	public function setTitle($title)
	{
		$this->title = $title;
	}
 
	/**
	 * @param string $body
	 */
	public function setBody($body)
	{
		$this->body = $body;
	}
 
	/**
	 * @param mixed $visibility
	 */
	public function setVisibility($visibility)
	{
		$this->visibility = (int)$visibility;
	}
 
	private $id			= null;
	private $title	  = null;
	private $body	   = null;
	private $visibility = null;
}
 
/**
 * Klasa reprezentujaca managera newsow
 *
 */
class NewsManager
{
	/**
	 * W zaleznosci do tego czy przekazany zostaje ID newsa wykonujemy jego edycje l
ub tez dodajemy nowy
	 *
	 * @param object $news Obiekt reprezentujacy pojedynczy news
	 */
	public function save(News $news)
	{
		if ( $news->getId() != null ) {
			$this->edit($news);
		} else {
			$this->add($news);
		}
	}
 
	private function edit(News $news)
	{
		// Tutaj edycja
	}
 
	private function add(News $news)
	{
		// Tutaj dodanie nowego
	}
 
	private function delete($id)
	{
		// Usuniecie
	}
 
	/**
	 * @param int $id ID newsa ktory chcemy pobrac
	 * @return object Obiekt klasy News
	 */
	public function getOne($id)
	{
		// Tutaj wyciagamy informacje na temat newsa, z pliku, bazy etc.
		// ...
 
		$args = array();
		$args['id']		 = $result['id'];
		$args['title']	  = $result['title'];
		$args['body']	   = $result['body'];
		$args['visibility'] = $result['visibility'];
 
		return new News($args);
	}
 
 
	/**
	 * @return array Tablica obiektow klasy News
	 */
	public function getMany()
	{
		$result = array();
 
		foreach( $news AS $oneNews ) {
			$args['id']		 = $result['id'];
			$args['title']	  = $result['title'];
			$args['body']	   = $result['body'];
			$args['visibility'] = $result['visibility'];
 
			$result[] =  new News($args);
		}
 
		return $result;
	}
}
 
 
// Wyciagniecie pojedynczego newsa 
$newsManager = new NewsManager();
$oneNews = $newsManager->getOne(4);
// Teraz odwolujac sie do danych newsa robimy to przy uzyciu $oneNews->getTitle(), ->getBody(), ...
 
 
// Wyciagniecie kilku newsow
$newsmanager = new NewsManager();
foreach( $newsManager->getMany() AS $oneNews ) {
	// Tutaj takze mozemy sie odwolywac jak do listingu powyzej tzn. $oneNews->getTitle(), ->getBody()... i bedziemy mieli dane newsa na ktorym w danej interacji operujemy.
}
?>
[PHP] pobierz, plaintext 

To chyba tyle, wydaje mi sie ze nie jest to najgorsze rozwiazanie z kilku wzgledow.
1. Mozemy w latwy sposob robic kopie obiektow - przy np. porownywaniu obiektow przed i po edycji
2. Nie wazne skad pobierzemy dane w NewsManager'rze bo i tak operujemy na obiekcie
3. Nie jestesmy uzaleznieny od nazewnictwa w bazie danych poniewaz stosujemy getery wiec gdy nam sie zmienia nazwa w bazie np. z title na title_a ( a uwierzcie mi takie rzeczy sie zdarzaja (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ) zmieniamy to tylko w managerze

[Cysiaczek]

Twoje jest bardzie oop ;]
Tylkp, że mi przyświecała inna idea. Chodziło o pokazanie, do czego przydaje się klasa NewsManager/NewsBase etc.

[adriano321]

ooo właśnie o to mi chodziło. Dzięki wielkie za przykłady (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

To raczej niewiele ma wspólnego z OOP, ale jak pozbyć się $db (odwołania do bazy danych) w liście parametrów metody?

[PHP] pobierz, plaintext 
<?php
public function pobierzNewsa ($id,$db)	 //  <- to $db
	{
		$sql="SELECT `id` ,`title` ,`text` ,`date` ,`edit_date` ,`id_author` ,`id_kategor
y` FROM `news` WHERE id='$id' LIMIT 1";
 
		if (!$wynik=$db->query($sql))  echo " z tego!";
		$wynik = $wynik->fetch_array();
 
		return $wynik;
	}
?>
[PHP] pobierz, plaintext 

Robiąc podobnie jak Cysiaczek bez $db:

[PHP] pobierz, plaintext 
<?php
public function loadSomeNews($howMany, $order)
	{
		$stack=array();
		$sql="SELECT * FROM news WHERE ORDER BY $order LIMIT $howMany";
	 [b]   $result=$db->query($sql); [/b]   // <- Chodzi mi o to
		foreach($result->fetch_array() as $row)
		{
			$obj=new News($row);
			$stack[]=$obj;
			$this->collection->add($obj);
		}
		return $stack;
	}
?>
[PHP] pobierz, plaintext 

wywala mi błąd:

Kod

Fatal error: Call to a member function query() on a non-object in

w linijce:

Kod

  if (!$wynik=$db->query($sql))  echo " z tego!";

Czy da się to jakoś obejść?

[Whisller]

@adriano321
Możesz przekazać handler połączenia w konstruktorze

[PHP] pobierz, plaintext 
<?php
public function __construct($dbh) 
{
	$this->dbh = $dbh;
}
?>
[PHP] pobierz, plaintext 

I wtedy odwołujesz się w metodzie poprzez $this->dbh np. $this->dbh->query(), $this->dbh->fetch() etc.
Albo też możesz użyć singletona ( http://www.phppatterns.com/docs/design/singleton_pattern ) lub rejestru( http://www.phppatterns.com/docs/design/the_registry ).

Ten post edytował Whisller 12.02.2008, 17:44:05

[adriano321]

No to teraz zaprezentuję całą klasę news(na podstawie klasy Whisllera, dzięki chłopaki), może komuś się w przyszłości przyda (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) . Jakby ktoś miał jakieś zastrzeżenia to proszę o przedstawienie ich, będą one na pewno bardzo pomocne w dalszej pracy.

[PHP] pobierz, plaintext 
<?php
class News
{
	private $id = null;
	private $title = null;
	private $text  = null;
	private $date = null;
	private $editDate = null;
	private $idAuthor = null;
	private $idKategory = null;
 
 
 
	public function __construct(array $args)
	{
		isset($args['id'])		 ? $this->setId($args['id'])				 : '';
		isset($args['title'])	  ? $this->setTitle($args['title'])		   : '';
		isset($args['text'])	   ? $this->setText($args['text'])			 : '';
		isset($args['date'])	   ? $this->setDate($args['date'])			 :  '';
		isset($args['editDate'])   ? $this->setEditDate($args['editDate'])	 :  '';
		isset($args['idAuthor'])   ? $this->setIdAuthor($args['idAuthor'])	 :  '';
		isset($args['idKategory']) ? $this->setIdKategory($args['idKategory']) :  '';
	}
	 
	public function setId($id)
	{
		$this->id = (int)$id;
	}
 
	public function setTitle($title)
	{
		$this->title = $title;
	}
 
 
	public function setText($text)
	{
		$this->text = $text;
	}
 
	public function setDate($date)
	{
		$this->date = $date;
	}
 
	public function setEditDate($editDate)
	{
		$this->editDate = $editDate;
	}
 
	public function setIdAuthor($idAuthor)
	{
		$this->idAuthor = (int)$idAuthor;
	}
 
	public function setIdKategory($idKategory)
	{
		$this->idKategory = $idKategory;
	}
 
	public function getId() 
	{
		return  $this->id;
	}
 
	public function getTitle() 
	{
		return  $this->title;
	}
 
	public function getText() 
	{
		return  $this->text;
	}
 
	public function getDate() 
	{
		return  $this->date;
	}
 
	public function getEditDate() 
	{
		return  $this->editDate;
	}
 
	public function getIdAuthor() 
	{
		return  $this->idAuthor;
	}
 
	public function getIdKategory() 
	{
		return  $this->idKategory;
	}
 
}
 
 
class NewsManager
{
	public $db;
	 
	function __construct($db) {
		$this->db=$db;
	} 
	 
	public function save(News $news)
	{
		if ($this->getId($news->getId())) {
			$this->edit($news);
		} else {
			$this->add($news);
		}
	}
 
	private function getId($id) 
	{
		$sql="SELECT id FROM news WHERE id='$id'";
		$result = $this->db->query($sql);
 
		if ($result->num_rows == 1) {
			return true;
		} else {
			return false;
		}
	}
 
	private function edit(News $news)
	{
		$sql="UPDATE `news` SET `title`='".$news->getTitle()."' ,`text`='".$news->getText()."' ,`editDate`='".$news->getEditDate()."' ,`idKategory`='".$news->getIdKategory()."' WHERE `news`.`id`='".$news->getId()."' LIMIT 1";
 
		if ($this->db->query($sql)) echo "tak";
		else echo $this->db->error;
	}
 
	private function add(News $news)
	{
		$sql = "INSERT INTO `news` VALUES ('','".$news->getTitle()."','".$news->getText()."','".$news->getDate()."','','".$news->getIdAuthor()."','".$news->getIDKategory()."')";
 
		if ($this->db->query($sql))  echo "tak";
		else echo $this->db->error;
	}
 
	private function delete($id)
	{
		$sql="DELETE FROM `news` WHERE `news`.`id`=$id LIMIT 1";
		if ($this->db->query($sql)) echo "tak";
		else echo $this->db->error;
	}
 
	public function getOne($id)
	{
   
 
			$sql="SELECT * FROM `news` WHERE `id`=$id LIMIT 1";
 
			$result = $this->db->query($sql);
			$result = $result->fetch_array();
 
			$args = array();
 
			$args['id']		=   $result['id'];
			$args['title']	 =   $result['title'];
			$args['text']	  =   $result['text'];
			$args['date']	  =   $result['date'];
			   $args['editDate']  =   $result['editDate'];
			$args['idAuthor']  =   $result['idAuthor'];
			$args['idKategory']=   $result['idKategory'];
 
		return new News($args);
	}
 
	public function getMany($howMany,$startFrom)
	{
		$sql="SELECT * FROM `news` WHERE `news`.`id`=$id ORDER BY id DESC LIMIT $startFro
m, $howMany";
 
		$result = $this->db->query($sql);
 
		$news = array();
 
		while( $temp = $result->fetch_array() ) {
			$args = array();
 
			$args['id']		=   $temp['id'];
			$args['title']	 =   $temp['title'];
			$args['text']	  =   $temp['text'];
			$args['date']	  =   $temp['date'];
			$args['editDate']  =   $temp['editDate'];
			$args['idAuthor']  =   $temp['idAuthor'];
			$args['idKategory']=   $temp['idKategory'];
 
 
			$news[]=new News($args);
 
		}
		return $news;
	}
 
	public function getNewsList($sort) // ASC or DESC
	{
		$sql = "SELECT `id`,`title` FROM `news` ORDER BY id ".$sort;
		$result = $this->db->query($sql);
		$temp = array();
		while ($abc = $result->fetch_array()) 
		{
			$temp[]=array('id' => $abc['id'], 'title' => $abc['title']);
		}
 
		return $temp;
	}
}
?>
[PHP] pobierz, plaintext 

Adrian.

Ten post edytował adriano321 14.02.2008, 18:28:12

[Whisller]

@adriano321
1. Nie rób echo w metodzie klasy (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Bo dajmy na to będziesz chciał komunikat o błędzy zapisać do pliku albo też wysłać mailem, co wtedy? Zwracaj go w postaci stringa albo też wyrzucaj wyjątek.
2. W metodzie getOne nie musisz wartości przypisywać to tablicy $args jeśli nazwy pól w bazie danych są takie same jak indeksy w tablicy argumentów przyjmowanej przez konstruktor klasy News.
Tzn. możesz sobie skrócić pracę pisząc
$r = mysql_fetch_assoc($sth);
return new News( $r );
3. Zamiast robić metodę getNewsList można by pokusić się o to aby klasa NewsManager dziedziczyła po jakiejś klasie - lub też otrzymywała obiekt klasy - w której określał byś jakie pola chcesz pobrać oraz kryteria do WHERE, tak mi się wydaje.
A chesz zostawić tą metodę tak jak jest to daj albo sprawdzanie czy zmienna $sort jest ustawiona na DESC albo ASC lub też daj wartość domyślną dla tego argumentu( public function getNewsList($sort='ASC') ).

[revyag]

A nie byłoby czytelniej zamiast tych wszystki setCosTam i getCosTam, zrobić tak?

[PHP] pobierz, plaintext 
<?php
public function set__($var,$val) {
	$this->{$var} = $val;
}
public function get__($var) {
	return $this->{$var}
}
?>
[PHP] pobierz, plaintext 

a potem np.

[PHP] pobierz, plaintext 
<?php
isset($args['id']) ? $this->set__('id',$args['id']) : '';
?>
[PHP] pobierz, plaintext 

zamiast n metod dla n właściwości są 2.

[adriano321]

1. Nie rób echo w metodzie klasy smile.gif Bo dajmy na to będziesz chciał komunikat o błędzy zapisać do pliku albo też wysłać mailem, co wtedy? Zwracaj go w postaci stringa albo też wyrzucaj wyjątek.
2. W metodzie getOne nie musisz wartości przypisywać to tablicy $args jeśli nazwy pól w bazie danych są takie same jak indeksy w tablicy argumentów przyjmowanej przez konstruktor klasy News.
Tzn. możesz sobie skrócić pracę pisząc
$r = mysql_fetch_assoc($sth);
return new News( $r );
3. Zamiast robić metodę getNewsList można by pokusić się o to aby klasa NewsManager dziedziczyła po jakiejś klasie - lub też otrzymywała obiekt klasy - w której określał byś jakie pola chcesz pobrać oraz kryteria do WHERE, tak mi się wydaje.
A chesz zostawić tą metodę tak jak jest to daj albo sprawdzanie czy zmienna $sort jest ustawiona na DESC albo ASC lub też daj wartość domyślną dla tego argumentu( public function getNewsList($sort='ASC') ).

1. To jest na razie wersja robocza, zapomniałem usunąć echa, mam zamiar wyrzucać wyjątki ale to dopiero później dopiszę.
2. Też tak na początku myślałem ale po $db->fetch_array() otrzymywałem tablicę:

Kod

Array ( [0] => 8 [id] => 8 [1] => CSM [title] => CSM [2] => CSM [text] => CSM [3] => 2012-01-01 05:01:20 [date] => 2012-01-01 05:01:20 [4] => 2042-01-01 06:01:20 [editDate] => 2042-01-01 06:01:20 [5] => 231 [idAuthor] => 231 [6] => 2 [idKategory] => 2 )

// yyy nie no rzeczywiście przecież to nie mało by żadnego znaczenia (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) hehe dobra to już wiem co i jak.

@revyag - dobra uwaga (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

[Black-Berry]

Tak czytam i widzę, że siedzimy w tym samym temacie. Przeczytaj mój wątek na temat klasy db_driver i jak 'singleton' rozwiązuje sprawę, z przekazywaniem obiektu $db:

http://forum.php.pl/widocznosc_obiektu_wew...asy_t87343.html

[Cysiaczek]

Można też tak, co pozwala na zachowanie interfejsu (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
function lcfirst($str)
{
	return strtolower(substr($str, 0, 1)) . substr($str, 1);
}
 
class callHelper
{
	protected $data;
 
	public function __call($method, $args)
	{
 
		if(!method_exists($this, $method))
		{
			$pre=substr($method, 0, 3);
			$post=substr($method, 3);
			$propertyName=lcfirst($post);
 
			if($this->has($propertyName))
			{
				if($pre=='set')
				{
					return $this->setProperty($propertyName, $args[0]);
				}
				if($pre=='get')
				{
					return $this->getProperty($propertyName);
				}
			}
			else
			{
				throw new Exception('Niezdefiniowana metoda '.__CLASS__.'::'.$method);
			}
		}
 
		return call_user_func_array(
		array($this, $method),
		$args
		);
	}
 
	public function setProperty($key, $value)
	{
		$this->data[$key]=$value;
	}
 
	public function getProperty($key)
	{
		return $this->data[$key];
	}
 
	public function has($key)
	{
		if(isset($this->data[$key]))
		{
			return true;
		}
		return false;
	}
}
?>
[PHP] pobierz, plaintext 

[adriano321]

Czy to jest dobrze? Albo raczej czy to jest zgodne ze wzorcem Singleton?

[PHP] pobierz, plaintext 
<?php
class Database extends mysqli 
{  
 
  private static $oInstance = false; 
 
  private function __construct() 
  {
  } 
 
  public static function getinstance($dbHost, $dbUser, $dbPass, $dbBaseName) 
  { 
	if ( self::$oInstance == false ) 
	{ 
	  self::$oInstance = new mysqli($dbHost, $dbUser, $dbPass, $dbBaseName);
	} 
	return self::$oInstance; 
  } 
}
 
$db = Database::getinstance('aaa','aaa','aaa','aaa');
?>
[PHP] pobierz, plaintext 

Ten post edytował adriano321 16.02.2008, 12:38:44

[Whisller]

@revyag
Jasne że można ale tylko wtedy gdy masz pewność że nie będziesz przeprowadzał w setach jakiś walidacji/filtracji etc. choćby rzutowania. Co wtedy? Nie zrobisz w __set switch z wyłapywaniem nazw pól i odpaleniem odpowiedniego rzutowania (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
--
Edit
Ok mozna by napisac metody dla wartosci przy ustawioniu ktorych przeprowadzamy walidacje etc ale jakos osobiscie przyzwyczailem sie do takiego sposobu pisania + z tego co slyszalem ( nie przeprowadzalem wlasnych testow ) metody magiczne typu __set __get __call etc. sa wolniejsze.

Ten post edytował Whisller 18.02.2008, 09:48:08

[revyag]

@Whisller ale ja nie podałem __set tylko set__ (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) To mała ale zasadnicza różnica. I to wszystko o czym mówisz da się zrobić. np. jako trzeci parametr podawać typ parametru.
__set wykorzystałbym do czegoś innego.

[Cysiaczek]

Może i odrobinę wolniejsze, ale jak masz mieć 100 setterów i getterów, to lepiej chyba magiczne dać (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif) Ja tam je polubiłem - zwłaszcza __call, którego implementację dawałem i która jest bardzo uniwersalna.

[Sedziwoj]

Jak dla mnie te wszystkie magiczne metody robią burdel w kodzie i tyle.
Chociaż tak prosta rzecz jak podpowiadanie nie będzie w tedy działać i nikt mi nie powie że to będzie upierdliwe. Moim zdaniem klasa powinna mieć jawny interfejs a nie jakieś sztuczki. (w sumie to brak typizacji argumentów [co pociąga brak możliwości definicji metod o tej samej nazwie a różnych argumentach] w PHP też mnie irytuje)

A nie wiem mi się nie zdarzyło abym w jednej klasie musiał pisać 100 metod get i set...