Wyswietlanie elementow, pierwsze klasy katalogu filmowego Opcje

[k4myk85]

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? )

Ten post edytował k4myk85 21.03.2008, 12:57:03

[-=Peter=-]

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).

Ten post edytował -=Peter=- 21.03.2008, 13:39:07

[k4myk85]

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

[-=Peter=-]

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