[MVC] Czy dobrze interpretuję ?, ciągle to mvc :( Opcje

[matix]

Witam,

Ciągle zastanawiam się, czy dobrze interpretuję zasadę MVC.

Chciałem napisać coś takiego:

KONTROLLER: Index.Controller.php:

Pobiera $_GET['view'] i zwraca $this -> setView():

[PHP] pobierz, plaintext 
<?
class aController 
{
  private $sView;
 
		function __construct() 
	{
		$this -> sView = $_GET['view'];
				self::setView();
	}
 
 
	function setView()
	{
		require_once('view/'.$sView);
	}
}
?>
[PHP] pobierz, plaintext 

Następnie MODEL pobiera dane z mysql z danego projektu i zwraca:

[PHP] pobierz, plaintext 
<?
class db 
{
	function __get() {
		while (...) 
		{
			$oArray[] = $db -> id;
		}
 
		return $oArray;
	}
 
}
?>
[PHP] pobierz, plaintext 

W ostateczności Widok pobiera wszystko i generuje HTML:

[HTML] pobierz, plaintext 
Klient o ID: <?=$db['id'];?><br/>
Pobrał .... <?=$db['nazwa'];?>
 
To jest przykładowa tresc pliku widoku.
[HTML] pobierz, plaintext 

Męczy mnie także pytanie, jak to wszystko połączyć, zeby działało?
Chodzi o większą ilość widoków, lub modeli.

Proszę mnie nie odsyłać do artykułów, bo z nich nic się (niestety) nie można dowiedzieć.

Pozdrawiam, Matix: )

Ten post edytował matix 25.05.2007, 22:03:54

[koodlaty]

Kod odbierajacy i filtrujacy dane z POST/GET (i wszelkie dane pochodzace od użytkownika) powinien znajdować się w metodach kontrolera, bardzo esencjonalnie zasady konstrukcji wzorca MVC podał ActivePlayer.
Polecam lekture dokumentacji Zend Framefork-a - kopalnia wiedzy o wzorcach i OOP.

Źle natomiast zrozumiałeś zastosowanie metod statycznych.
Nie ma powodu by Twoja metoda _set() była statyczna. Jej statyczne wykorzystanie i tak nie będzie mozliwe bo odnosi się do pól obiektu. Metody statyczne wywolywane są bez tworzenia obiektu, nie mogą więc odwolywać się do jego pól.

Opisując Ci sposób tworzenia metody statycznej chcialem powiedzieć że statyczne wywolanie self::setView(); było niepotrzebne i gdyby setView() odnosiło się do pól klasy jej działanie okazać by się moglo inne niż oczekiwane.

W tym konkretnym przypadku zamiast

[PHP] pobierz, plaintext 
<?php
self::setView();
?>
[PHP] pobierz, plaintext 

powinno być

[PHP] pobierz, plaintext 
<?php
$this -> setView();
?>
[PHP] pobierz, plaintext 

to wszystko.

Konstrukcja:

[PHP] pobierz, plaintext 
<?php
public static function _set() {
		switch($this -> _sType) {
			case 1: $this -> _aType = 'rss'; break;
			case 2: $this -> _aType = 'news'; break;
		}
 
		return $this -> _aType;
	}
?>
[PHP] pobierz, plaintext 

Nie bedzie mogla być wywolywana statycznie, poza tym jest to bezzasadne.