Witam

Od jakiegos czasu sledze to co dzieje sie na tym subforum. Kupilem ksiazke i powoli ucze sie OOP. Mam w szkole projekt do zrobienia i rzucilem sie na gleboka wode (jak dla mnie) z tematem katalogu filmowego w OOP. Ehh nie moge juz zmienic tego wyboru. Mieszajac wiedze z ksiazek i tego forum "cos" pisze. Przedstawie wam krotko 3 klasy jakie napisalem i zalezy mi ta tym zeby ktos powiedzial czy zmierzam w dobra strone lub czy powinienem na problem spojrzec z innej lepszej strony

okrojona klasa DB

[PHP] pobierz, plaintext 
<?php
function fetch_row($query_result)
   {
	  $query_result = (empty($query_result)) ? $this -> query_result : $query_result;
	  $array = mysql_fetch_array($query_result);
	  if(!$array)
	  {
		 return false;
	  }
	  else
	  {
		 return $array;
	  }
   }
 
public function ask($query){ 
$result = mysql_query($query) or die("Zapytanie niepoprawne");
return $result;
}
?>
[PHP] pobierz, plaintext 

Klasa Movie

[PHP] pobierz, plaintext 
<?php
class Movie 
	{
 
		private $id = null;
		private $title = null;
		private $premiere = null;
		// ... i wiecej nie bede na forum pisal wszystkich
 
 
		public function __construct(array $row)
		{
			isset($row['id'])? $this->set__('id',$row['id']): '';
			isset($row['title'])? $this->set__('title',$row['title']): '';
			isset($row['premiere'])? $this->set__('premiere',$row['premiere']): '';
			// ...
		}
 
		public function set__($var,$val) 
		{
			$this->$var = $val;
		}
 
		public function get__($var) 
		{
			return $this->$var;
		}
	//-
	}
?>
[PHP] pobierz, plaintext 

Klasa MovieManager (ktora ma pobierac i zarazem wyswietlac info o filmioe liste itp wykorzystujac klase Templates)

[PHP] pobierz, plaintext 
<?php
include('class.DB.php');
include('class.Movie.php');
//include('class.Templates.php');
	class MovieManager 
	{
 
		public $db;
		public $tpl;
			 
		function __construct($db) 
		{
			$this->db=$db;
			$this->tpl= new Templates;
		} 
 
		public function getMovie($id)
		{
 
			$sql="SELECT * FROM db_movies WHERE id='".$id."' LIMIT 1";  
			$result = $this->db->ask($sql);
 
			$row = array();
			while($row = $this->db->fetch_row($result)) 
			{  
			$row['id'] = $result['id'];
			$row['title'] = $result['mv_title'];
			$row['premiere'] = $result['mv_premiere'];
			// ...
 
			//return new Movie($row);
			$mve = new Movie($row);
			}
			return $mve;
		}
 
		public function getSevenMovies()
		{
 
			$sql="SELECT * FROM db_movies LIMIT 7";  
			$result = $this->db->ask($sql); 
 
			$evts = array();
			$row = array();
 
			while($Temp = $this->db->fetch_row($result)) 
			{		 
				$row['id']	  = $Temp['id'];
				$row['title']	= $Temp['mv_title'];
 
				$evts[] = new Movie($row);
			}
			return $evts;
		}
 
		public function printMovieDetails()
		{
		 print "> ".$evt->get__('title')." < <br />"; // nie dziala
		}
 
		public function printMoviesList()
		{
 
			$this->tpl->load('commingMovies');
									$this->tpl->blocks[] = array(...);
			$this->tpl->parse();
			foreach( (array)$this->getCommingMovies() as $this->evts ) 
			{
			$this->tpl->display(); // - nie widac textow z bazy
			print $this->evts->get__('title')."<br />"; //OK
			}
		}
 
	//-
	}
?>
[PHP] pobierz, plaintext 

Prosze o ocene i pomoc ( jak wyswietlac elementy? )

Możesz zrobić klasę Movie w taki sposób (prostrzy):

[PHP] pobierz, plaintext 
<?php
class Movies{
	$fields = array('id', 'title', 'premiere');
	$data = array();
 
	public function __construct(array $data){
		$this->setData($data);
	}
 
	public function setData($data){
		foreach($data as $k => $v){
			if(in_array($k, $this->fields))
				$this->data[$k] = $v;
			}
		}
	}
 
	public function __get($name){
		$method = 'get'.ucfirst(strtolower($name));
 
		if(method_exists($this, $method)){
			call_user_func(array($this, $method));
		}else{
			if(in_array($name, $this->fields){
				return $this->data[$name];
			}else{
				//wyrzucenie wyjatku? zwrocenie false?
				return false;
			}
		}
	}
 
	public function __set($name, $value){
		$method = 'set'.ucfirst(strtolower($name));
 
		if(method_exists($this, $method)){
			call_user_func(array($this, $method), $value);
		}else{
			if(in_array($name, $this->fields){
				$this->data[$name] = $value;
			}else{
				//wyrzucenie wyjatku? zwrocenie false?
				return false;
			}
		}
 
	}
}
?>
[PHP] pobierz, plaintext 

Możesz ten szkielet wyodrębnić w klasie abstrakcyjnej i Movie by po nim dziedziczyło.

[PHP] pobierz, plaintext 
<?php
public function printMovieDetails()
{
		 print "> ".$evt->get__('title')." < <br />"; // nie dziala
}
?>
[PHP] pobierz, plaintext 

To Ci nie działa, bo zmienna $evt jest wzięta z powietrza, nie została zdeklarowana, czy podana jako parametr do metody. Pozatym widzę wiele błędów w klasie MovieManager, używasz atrybutu "evt", a nie jest on zdefinowany w ciele klasy, stąd te błędy.

[PHP] pobierz, plaintext 
<?php
foreach( (array)$this->getCommingMovies() as $this->evts ) {}
?>
[PHP] pobierz, plaintext 

Takie coś nie powinno mieć miejsca ($this->evts nie jest zdeklarowane jako atrybut klasy)... Użyj poprostu:

[PHP] pobierz, plaintext 
<?php
foreach( (array)$this->getCommingMovies() as $evts ) {}
?>
[PHP] pobierz, plaintext 

Jeśli już używasz klasy widoku (Templates), to taka metoda jak printMovieDetails jest niepotrzebna. Metody w klasach mają nic nie wyświetlać, tylko zwracać. Wyświetlaniem zajmuje się widok (chyba że nie stosujesz MVC).

Dzieki bardzo za ta odpowiedz. Licze na kolejne...

Nie uzywam framework'ow. Chce nauczyc sie oop tylko troszeczke zle boisko wybralem (szkola) ;p. Ten sposob na klase Movie jest ciekawy. Dzisiaj w nocy posiedze nad MovieManager bo faktycznie sajgon z tej klasy. Co do wyswietlania elementow tablicy... obiektow to zrobic osobna klase ktora bedzie dziedziczyla z Templates i odpowiednio umieszczala dane w tabelkach z plikow tpl? Nie chce umieszczac kodu html w klasach bo czytalem ze to jest 'nieladne' tak jak uzywanie print i echo (tak jest w dosyc sporej ilosci ksiazek na temat php oop).


Przy okazji pozdrawiam i Zycze Wesolych Swiat i mokrego Dyngusa

To co chcesz zaimplementować przypomina DAO (taki wrzorzec).

Oto jak bym widział klasę MoviesManager (oczywiście uproszczoną):

[PHP] pobierz, plaintext 
<?php
class MoviesManager{
	private $hDb = null;
 
	public function __construct(){
		$this->hDb = Database::instance();
	}
 
	/**
	 * Zwraca podany obiekt filmu
	 */
	public function getMovie($id){
		$aMovie = $this->hDb->query("SELECT * FROM movies WHERE id='$id';");
 
		$movie = new Movie($aMovie);
		return $movie;
	}
 
	/**
	 * Zwraca tablice obiektow
	 */
	public function getAllMovies($limit = null){
		$l = '';
		if(is_numeric($limit)){
			$l = ' LIMIT '.$limit;
		}
		$query = 'SELECT * FROM movies'.$limit.';';
		//zakladam ze Database::getAll zwraca to samo co metoda rozszerzenia PEAR DB::getA
ll
 
		$aMovies = $this->hDb->getAll($query);
 
		$movies = array();
 
		foreach($aMovies as $aMovie){
			$movies[] = new Movie($aMovie);
		}
 
		return $movies;
	}	
}
 
//zastosowanie
$mm = new MoviesManager();
 
$movie = $mm->getMovie(1);//pobiera film o id=1
$movies = $mm->getAllMovies(7);//pobiera siedem filmow
?>
[PHP] pobierz, plaintext