[MVC]Kontroler-Model-Widok, Kilka pytan Opcje

[MateuszS]

wydaje mi sie ze z nazewnictwem nie jest zle, zrobilem sobie taki config ze stalymi

[PHP] pobierz, plaintext 
/* Glowna sciezka do aplikacji */
define("ROOT", realpath(dirname(__FILE__) . "/../../"));
 
 
/* Klasy kontrolerow */
define("CONTROLLERS", ROOT . "/app/controllers/");
 
 
/* Klasy modeli */
define("MODELS", ROOT . "/app/models/");
 
 
/* Klasy widoku */
define("VIEW", ROOT. "/app/view/");
 
 
/* Koncowka pliku klasy */
define("FCLASS", ".class.php");
[PHP] pobierz, plaintext 

i w sumie __autoload() dziala elegancko, moge sie teraz odwolac do dowolnej klasy w kazdej klasie ktorej chce (oczywiscie oby tylko zgodnie z MVC)

//02.03

Hey, postanowiłem coś SPRÓBOWAĆ napisać w oparciu o MVC, puki co mam VC, napisałem bardzo prosty system wybierania szablonu, ustawiania i uruchamiania aplikacji, wiadomo, teraz na nic się to nie nada, ale chodzi mi to o samą metodę a raczej metody, czy dobrze są ustawione, czy kontroler dobrze jest napisany itp. Jezeli cos jest zle, a pewnie tak, to prosze o jakies podpowiedzi jak to poprawic. Po poprawie dorzuce model. Pozdro

index.php

[PHP] pobierz, plaintext 
<?php
 
require_once("app/config/config.php");
 
function __autoload($filename) {
	if(require_once(CONTROLLERS . $filename . FCLASS)) return true;
	if(require_once(MODELS . $filename . FCLASS)) return true;
	if(require_once(VIEW . $filename . FCLASS)) return true;
}
 
try {
 
	$ob = new Controller();
	$ob->run();
 
 
} catch(Exception $e) {
	echo "Blad: ".$e->getMessage();
}
?>
[PHP] pobierz, plaintext 

app/controllers/ViewController.class.php

[PHP] pobierz, plaintext 
<?php
 
class viewController 
{
	public $themeFolder;
	public $themeDirectory;
 
	public function theme($folder) 
	{
		$dir = VIEW . $folder;
		if(@!opendir($dir))
			throw new Exception("Nie ma stylow w podanym folderze");
 
		$this->themeFolder = $folder;
		$this->themeDirectory = $dir;	
 
	}
 
	public function getTheme() 
	{
		return $this->themeFolder;
	}
 
	public function getThemeDir() 
	{
		return $this->themeDirectory;
	}
 
	public function display($file) 
	{
		if(@!include($this->getThemeDir() . "/" . $file))
			throw new Exception("Nie znaleziono pliku widoku (".$file.")");
	}
 
}
 
?>
[PHP] pobierz, plaintext 

app/controllers/Controller.class.php (to moglbym wrzucic do jakiegos folderu Core np. ale czekam puki co na opinie

[PHP] pobierz, plaintext 
<?php
 
class Controller
{
	public $model;
	public $view;
 
	public function __construct()
	{
		$this->view = new viewController;		
		// + obiekt modelu
	}
 
	public function setTheme()
	{
		$this->view->theme("theme1"); 
	}
 
	public function setHome()
	{
		$this->view->display("form.phtml");
	}
 
	public function run()
	{
		$this->setTheme();
		$this->setHome();
 
	}
 
}
 
?>
[PHP] pobierz, plaintext 

app/view/theme1/form.phtml

[HTML] pobierz, plaintext 
<form action="index.php" method="POST">
	<input type="submit" name="wyslij" value="Zaloguj" />
</form>	
[HTML] pobierz, plaintext 

[marcio]

W klasie controller masz metode run() ktora wywoluje 2 publiczne metody ale je daj na protected lub private jesli klasa ma byc "final" bo jak nie to sie mija z celem.
Modelu i widoku raczej nie ustawiaj na sztywno do jednego atrybutu jak cos daj chociac do array zebys mogl korzystac z kilku modeli jesli bedzie taka potrzeba.

[MateuszS]

Jak dam run() protected to chyba nie bede mogl jej wywolac z poza klasy nie? a przeciez musze ja wywolac poza klasa.

Ten post edytował MateuszScirka 3.03.2010, 19:02:33

[marcio]

Jak dam run() protected to chyba nie bede mogl jej wywolac z poza klasy nie? a przeciez musze ja wywolac poza klasa.

http://pl.wikipedia.org/wiki/Hermetyzacja
Czy ja mowilem o metodzie run().... nie.Poczytaj a bedziesz wiedzial o co kaman.

[MateuszS]

No tak, rozumiem o co chodzi, powinienem dac metodom setTheme i setHome protected. Poza tym co zmienic itp?

[marcio]

No tak, rozumiem o co chodzi, powinienem dac metodom setTheme i setHome protected. Poza tym co zmienic itp?

Nom wlasnie o to mi chodzi a tak na reszte nie lukalem.