[MVC] Widok i wyświetlanie danych Opcje

[Asmox]

Otóż mam taki problem. Jest sobie klasa widoku HTML (w odróżnieniu od np. XML)

[PHP] pobierz, plaintext 
class html
{
  private $_templateSrc;
  private $_contents;
 
  public function __construct($templateSrc, $contents) {
    $this->_templateSrc = $templateSrc;
    $this->_contents = $contents;
  }
 
  public function insertCSS($src, $type=null) {
    if(empty($type)) $type = 'all';
    echo '<link href="'.$src.'" media="'.$type.'" rel="stylesheet" type="text/css">'."\n";
  }
 
  public function insertJS($src) {
    echo '<script type="text/javascript" src="'.$src.'"> </script>'."\n";
  }
 
  public function show() {
    Core::loadFile($this->_templateSrc);
  }
}
[PHP] pobierz, plaintext 

I w szablonie jest coś takiego

[PHP] pobierz, plaintext 
<?php $this->insertCSS(Core::DEFAULT_VIEWS_SRC . $this->_view . '/' . 'uklad.css'); ?>
[PHP] pobierz, plaintext 

Ale niestety to wywala błąd:

Fatal error: Using $this when not in object context in [ścieżka]\views\html\templates\default\default.html on line 6

Czy da się to jakoś załatwić, żeby informacje były wyciągane bezpośrednio z widoku? Czyli innymi słowy, żeby nie było tego błędu i wszystko działało :-) ?

[potreb]

$this stosujesz poza klasą? Nie widzę, żebyś tworzył nową instancję klasy

[Asmox]

Jest jeszcze kontroler czy coś takiego articles.

[PHP] pobierz, plaintext 
Core::loadFile('./controllers/controllerInterface.php');
 
class articles implements controllerInterface
{
  private $_router;
  private $_action;
  private $_dataTransfer;
  private $_model;
  private $_modelObj;
  private $_view;
  private $_viewObj;
  private $_template;
  private $_artID;
  private $_contents;
 
  public function __construct(router $router) {
    $this->_router = $router;
    $this->_action = ifNotNull($this->_router->fromRequest('action'), 'view');
    $this->_dataTransfer = ifNotNull($this->_router->fromRequest('dataTransfer'), 'as');
    $this->_artID = ifNotNull($this->_router->fromRequest('id'), 1);
 
    $this->_model = Core::DEFAULT_MODEL_NAME;
    $this->_view = Core::DEFAULT_VIEW_NAME;
    $this->_template = Core::DEFAULT_TEMPLATE_NAME;
 
    //$this->viewAction();
    call_user_method($this->_action . 'Action', $this);
  }
 
  public function setModel($model) {
    $this->_model = $model;
  }
 
  public function setView($view) {
    $this->_view = $view;
  }
 
  public function setTemplate($tpl) {
    $this->_template = $tpl;
  }
 
  public function loadModel() {
    Core::loadClass($this->_model, Core::DEFAULT_MODELS_SRC . $this->_model . '/' . $this->_model . '.php');
    $this->_modelObj = new $this->_model();    
  }
 
  public function loadView() {
    Core::loadClass($this->_view, Core::DEFAULT_VIEWS_SRC . $this->_view . '/' . $this->_view . '.php');
    $templateSrc = Core::DEFAULT_VIEWS_SRC . $this->_view . '/' . 'templates/' . $this->_template . '/' . $this->_template . '.html';
    echo $templateSrc;
    //print_r($this->_contents);
    $this->_viewObj = new $this->_view($templateSrc, $this->_contents); // Tu jest tworzony nowy obiekt
  }
 
  public function loadContents($arrConditions) {
    $this->_contents = $this->_modelObj->getContents($arrConditions);
  }
 
  public function escape() {
    $this->_viewObj->show();
  }
 
  public function viewAction() {
    $this->loadModel();
    $this->loadContents(array('id' => $this->_artID));
    $this->loadView();
    $this->escape();  
  }
}
[PHP] pobierz, plaintext 

[Berg]

Nie da się, starasz się wywołać obiekt z poziomu metody statycznej. Są 2 obejścia problemu (przynajmniej takie o których wiem (IMG:style_emoticons/default/smile.gif) ).

Po pierwsze rozwiązanie niezbyt dobre, możesz w metodzie statycznej utworzyć kod PHP (w Stringu) w którym tworzysz klasę która dziedziczy po klasie html, natomiast w konstruktorze tej klasy umieścić kod z szablonu. Potem inicjujesz taką klasę i po sprawie. Oczywiście ten kod PHP wykonujesz w klasie statycznej za pomocą evala.

Drugie rozwiązanie jest o niebo lepsze - możesz z klasy Core zrobić Singletona, umieścić tam kod metodę magiczną __call która będzie sprawdzała czy dana metoda należy do klasy html i w razie czego ją wywoływał. Oczywiście w takim przypadku w loadFile trzeba będzie przesyłać obiekt klasy html aby klasa Core mogła sprawdzić czy dana metoda znajduje się w klasie i ewentualnie ją wywolać. Metoda show będzie wtedy wyglądała tak:

[PHP] pobierz, plaintext 
public function show() {
   Core::getInstance()->loadFile($this->_templateSrc, $this);
}
[PHP] pobierz, plaintext 

W ogóle na Twoim miejscu zastanowiłbym się czy jest jakiś sens korzystania w tym przypadku z Core, nie możesz dołączyć kodu szablonu bezpośrednio w metodzie show?

[Asmox]

Zapomniałem wspomnieć, że loadClass to tylko loadFile, oraz sprawdzenie, czy klasa istnieje.
Nie wiem tylko, czy wywoływanie z szablonu danych strony z obiektu (czyli używanie $this->content) jest ogólnie nie możliwe, czy to ja gdzieś popełniłem błąd. Bo mam ochotę napisać wszystko od początku. Powód? Nie trzyma się metody KISS i to w takim stopniu, że sam się gubię (IMG:style_emoticons/default/biggrin.gif)

[Berg]

Jeśli wczytujesz szablon z poziomu metody obiektu to nie ma problemu z używanie $this, Twój błąd ewidentnie świadczy o tym że robisz to gdzieś gdzie nie masz dostępu do $this (czyli pewnie w metodzie statycznej).
Kod z szablonu nie musi być nawet wykonywany bezpośrednio w klasie do której chcesz uzyskać dostęp do metody/pobrać zmienną o ile złapiesz metodę za pomocą __call (lub analogicznie zmienną za pomocą __get) i przekierujesz ją do poprawnej referencji do obiektu. Najlepiej gdybyś wrzucil wycinek kodu w którym następuje wczytanie szablonu, wtedy prawdopodobnie będzie można znaleźć jakieś obejście (IMG:style_emoticons/default/smile.gif)

[Asmox]

[PHP] pobierz, plaintext 
class Core
{
/*rożny kod*/
public static function loadFile($src) {
    if(file_exists($src)) require_once($src);
    else {
      $exc = new cannotLoadException('Nie mozna zaladowac pliku.');
      $exc->setSrc($src);
      throw $exc;
    }
  }
}
[PHP] pobierz, plaintext 

Zazwyczaj to było robione w ten sposób, (loadFIle jako loadTemplate)ale zrobiłem do widoku własną funkcję i jest spoko, dzięki :-)

Ten post edytował Asmox 2.09.2010, 20:24:55