[PHP] Funkcja budująca tablicę plików do wczytania Opcje

[XP'ek]

Witam wszystkich,

zbudowałem sobie taki skrypcik w którym tworzę Tablicę css'ów do wczytania dla danego controlera wygląda to tak:

controler welcome.php

[PHP] pobierz, plaintext 
    public function __construct()
    {
        parent::__construct();
        $admin = new Admin();
        $admin->load();
 
        View::setCSS('{$base_url}public/css/styles/dashboard.css', "text/css");
    }
[PHP] pobierz, plaintext 

model view.php

[PHP] pobierz, plaintext 
 
    public $_css = array();
    public $_jq = array();
 
    /**
     * przygotowanie tablicy stylu CSS
     */
    public function setCSS($data, $type = false, $media = false) {
        View::$this->_css[] = array("href"=>$data, "type"=>$type, "media"=> $media);
    }
 
    /**
     * przygotowanie tablicy stylu jQuery
     */
    public function setJQ($data) {
        View::$this->_jq[] = array("href"=>$data);
    }
 
    /**
     * przygotowanie tablicy stylu jQuery
     */
    public function gets() {
        $jq = View::$this->_jq;
        $css = View::$this->_css;
        if($css)Smarty::assign("_CSS", $css);
        if($jq) Smarty::assign("_JQ", $jq);
    }
[PHP] pobierz, plaintext 

model admin.php

[PHP] pobierz, plaintext 
    function load(){
        /* Wczytanie domyślnych CSS'ów */
        View::setCSS('public/css/style.css', "text/css");
        View::setCSS('public/css/bootstrap/bootstrap-responsive.css', "text/css");
        View::setCSS('public/css/bootstrap/bootstrap.css', "text/css");
        View::setCSS('public/css/fonts.css', "text/css");
 
        /* Wczytanie domyślnych JS'ów */
        View::setJQ("public/js/html5shiv-printshiv.js");
        View::setJQ("public/js/jquery/jquery-1.6.4.min.js");
        View::setJQ("public/js/jquery/jquery-ui-1.8.16.custom.min.js");
 
        /* przkezanie parametrów */
       View::gets();
    }
[PHP] pobierz, plaintext 

no i oczywiście wszystko było by pięknie ale tak nie jest dostaję zwrotne takie błędy

A PHP Error was encountered

Severity: Notice

Message: Undefined property: Welcome::$_css

Filename: core/Model.php

Line Number: 51

A PHP Error was encountered

Severity: Notice

Message: Indirect modification of overloaded property Admin::$_css has no effect

Filename: models/view.php

Line Number: 52

proszę o pomoc w rozwiązaniu problemu całość ma polegać na tym, że wstawiam sobie w index.tpl

Kod

    <!-- Style -->
    {foreach from=$_CSS item=css}
    <link rel="stylesheet" href="{$css.href}"{if $css.type} type="{$css.type}"{/if}{if $css.media} type="{$css.media}"{/if}>
    {/foreach}

i odpowiednio na moje potrzeby zostaną wczytane te pliki które sobie wpiszę w danym controllers

Ten post edytował XP'ek 4.02.2013, 00:10:51

[Mlodycompany]

w pliku model view.php masz błędy tzn

[PHP] pobierz, plaintext 
 View::$this->_css[] = array("href"=>$data, "type"=>$type, "media"=> $media);
[PHP] pobierz, plaintext 

odwołujesz się do klasy View statycznie ale jednak nie, w sumie to w ogóle nie rozumiem tego zapisu. Popraw go na

[PHP] pobierz, plaintext 
 $this->_css[] = array("href"=>$data, "type"=>$type, "media"=> $media);
[PHP] pobierz, plaintext 

[XP'ek]

Twój sposób nie rozwiązuje błędu.

[nospor]

Poniewaz robisz jeszcze inne bledy, jak np.:
View::setCSS('{$base_url}public/css/styles/dashboard.css', "text/css");
View nie ma metody statycznej setCSS. Ta metoda jest metodą obiektu a nie klasy.

masz stworzyć obiekt $view i przez niego to dodawać
$view->setCSS(....);

Albo masz w klasie View zrobić metodę statycznę poprawnie. Wówczas metoda statyczna moze korzystać ze zmiennych statycznych klasy a nie wlasciwosci obiektu.

Teraz mieszasz ze sobą dwa pojęcia.

[XP'ek]

OK dzięki za podpowiedź zaraz tym się zajmę mam jeszcze jedno pytanie a dokładniej chodzi o budowę drzewka.

np. w controllers welcome
wczytuję bazę

[PHP] pobierz, plaintext 
 public function __construct()
    {
        parent::__construct();
        $admin = new Admin();
        $admin->load();
 
        View::setCSS('{$base_url}public/css/styles/dashboard.css', "text/css");
    }
[PHP] pobierz, plaintext 

ale np. dla funkcji logowanie chce doczytać logowanie.css

[PHP] pobierz, plaintext 
 public function logowanie()
    { 
        View::setCSS('{$base_url}public/css/styles/logowanie.css', "text/css");
    }
[PHP] pobierz, plaintext 

i w tym momencie jak daje gets'a to zostaje mi tylko wczytany logowanie.css jak usprawnić mogę model View'a aby przechwytywał mi w ten sposób tablicę ?

[nospor]

Metoda setCSS wydaje się być ok - wg. kodu z pierwszego posta. Podejrzewam więc, że robiąc poprawki popsułes tę metodę. Pokaż więc kod na nowo z poprawkami

[XP'ek]

ok więc tak wyglądają moje pliki

controllers

[PHP] pobierz, plaintext 
<?php if ( ! defined('BASEPATH')) exit('No direct script access allowed');
 
class Inbox extends CI_Controller {
 
    public function __construct()
    {                
        parent::__construct();                
        $admin = new Admin();
        $admin->admin();
        $admin->lang();
        $admin->uri();
 
        $view = new View();
        $view->setCSS(base_url().'public/css/style.css', "text/css");
        $view->setCSS(base_url().'public/css/bootstrap/bootstrap-responsive.css', "text/css");
        $view->setCSS(base_url().'public/css/bootstrap/bootstrap.css', "text/css");
        $view->setCSS(base_url().'public/css/fonts.css', "text/css");
        $view->setCSS(base_url().'public/css/styles/helper.css', "text/css");
        $view->setCSS(base_url().'public/css/styles/colors.css', "text/css");
        $view->setCSS(base_url().'public/css/styles/agenda.css', "text/css");
        $view->setCSS(base_url().'public/css/size/480.css', "text/css","only all and (min-width: 480px)");
        $view->setCSS(base_url().'public/css/size/768.css', "text/css","only all and (min-width: 768px))");
        $view->setCSS(base_url().'public/css/size/992.css', "text/css","only all and (min-width: 992px)");
        $view->setCSS(base_url().'public/css/size/1200.css', "text/css","only all and (min-width: 1200px)");
 
        /* Wczytanie domyślnych JS'ów */
        $view->setJQ(base_url()."public/js/jquery-1.8.2.min.js");
        $view->setJQ(base_url()."public/js/setup.js");
        $view->setJQ(base_url()."public/js/bootstrap/bootstrap.js");
        $view->setJQ(base_url()."public/js/tinycon.js");
 
        $view->gets();
    }
 
    public function index(){
        $view = new View();
        $view->setCSS(base_url().'public/css/styles/messages.css', "text/css");
 
        $this->smarty->view('inbox/index.tpl');
    }
 
}
[PHP] pobierz, plaintext 

model view.php

[PHP] pobierz, plaintext 
<?php
  class View extends CI_Model{
 
    private $_output = array(
        'status' => 1,
        'message' => '',
        'timestamp' => '',
        'errorCode' => 0,
        'executionTime' => 0,
        'data' => array()
    );
 
    private $_data = array();
    private $_css = array();
    private $_jq = array();
 
    /**
     * przygotowanie tablicy stylu CSS
     */
    public function setCSS($data, $type = false, $media = false) {
        $this->_css[] = array("href"=>$data, "type"=>$type, "media"=> $media);
    }
 
    /**
     * przygotowanie tablicy stylu jQuery
     */
    public function setJQ($data) {
        $this->_jq[] = $data;
    }
 
    /**
     * przygotowanie tablicy stylu jQuery
     */
    public function gets() {
        $jq = $this->_jq;
        $css = $this->_css;
        if($css) $this->smarty->assign("_CSS", $css);
        if($jq)  $this->smarty->assign("_JQ", $jq);
    }
 
    /**
     * Metoda dodaje dane do wyjścia
     * @param mixed $data wartość wysyłana na wyjście
     * @param string $key klucz pod którym umieścić dene
     * @param bool $append flaga czy dodajemy do istniejącej już struktury
     */
    public function setData($data, $key = false, $append = false) {
        if ($key) {
            if ($append) {
                $this->_output['data'][$key][] = $data;
            } else {
                $this->_output['data'][$key] = $data;
            }
        } else {
            if ($append) {
                $this->_output['data'][] = $data;
            } else {
                $this->_output['data'] = $data;
            }
        }
    }
 
 
  }
?>
[PHP] pobierz, plaintext 

ostatecznie zrezygnowałem z modelu admin load i dałem to do __construct

[nospor]

No ale panie kolego... obiekt $view masz tworzyć raz i korzystać z niego tam gdzie chcesz, a nie ty tworzysz x obiektów $view - przecież każdy z nich ma własną tablicę css

[XP'ek]

ale jak w index() wywalę

[PHP] pobierz, plaintext 
$view = new View();
[PHP] pobierz, plaintext 

to automatycznie dostaje błąd

A PHP Error was encountered

Severity: Notice

Message: Undefined variable: view

Filename: controllers/inbox.php

Line Number: 42

możesz mnie nakierować bardziej co jest źle albo jak przerobić to by ominąć kodowo wywołanie $view->gets(); i w inny sposó tworzyć tą tablicę do przekazania

[nospor]

1) Poczytaj o zasięgu zmiennych - w manualu masz to bardzo dobrze wyjaśnione przy zmiennych
2) Możesz użyć wzorca faktory do pobierania $view
3) Możesz użyć wzorca Singleton do pobierania zawsze tego samego $view
4) Obiekt $view możesz przekazywać do innych obiektów, które z niego korzystają
5) No i możesz wkoncu wszystko zamienic na STATIC wowczas będzie się odwoływał przez klase VIEW a nie przez obiekt - wówczas klasa będzie miało wszystko to samo jako static

Do wyboru do koloru

[XP'ek]

ok dzięki faktycznie wystarczyło zmienić

private $_css = array();
private $_jq = array();

na

private static $_css = array();
private static $_jq = array();

ale pozostaje kwestia jak najlepiej rozwiązać opcję z View::gets(); by pobierać elementy bo jeśli zostawię nawet tak

[PHP] pobierz, plaintext 
<?php if ( ! defined('BASEPATH')) exit('No direct script access allowed');
 
class Inbox extends CI_Controller {
 
    public function __construct()
    {                
        parent::__construct();                
        $admin = new Admin();
        $admin->admin();
        $admin->lang();
        $admin->uri();
 
        View::setCSS(base_url().'public/css/style.css', "text/css");
        View::setCSS(base_url().'public/css/bootstrap/bootstrap-responsive.css', "text/css");
        View::setCSS(base_url().'public/css/bootstrap/bootstrap.css', "text/css");
        View::setCSS(base_url().'public/css/fonts.css', "text/css");
        View::setCSS(base_url().'public/css/styles/helper.css', "text/css");
        View::setCSS(base_url().'public/css/styles/colors.css', "text/css");
        View::setCSS(base_url().'public/css/styles/agenda.css', "text/css");
        View::setCSS(base_url().'public/css/size/480.css', "text/css","only all and (min-width: 480px)");
        View::setCSS(base_url().'public/css/size/768.css', "text/css","only all and (min-width: 768px))");
        View::setCSS(base_url().'public/css/size/992.css', "text/css","only all and (min-width: 992px)");
        View::setCSS(base_url().'public/css/size/1200.css', "text/css","only all and (min-width: 1200px)");
 
        /* Wczytanie domyślnych JS'ów */
        View::setJQ(base_url()."public/js/jquery-1.8.2.min.js");
        View::setJQ(base_url()."public/js/setup.js");
        View::setJQ(base_url()."public/js/bootstrap/bootstrap.js");
        View::setJQ(base_url()."public/js/tinycon.js");
 
        View::gets();
    }
 
    public function index(){
        View::setCSS(base_url().'public/css/styles/messages.css', "text/css");
 
        $this->smarty->view('inbox/index.tpl');
    }
 
}
 
[PHP] pobierz, plaintext 

to nie wczyta mi SetCSS z index'a znowuż nie chce w każdej funkcji rozpoczynać od View::gets(); bo wtedy owszem to działa i wszystko co mam nad gets'em jest poprawnie wczytane

próbowałem wykorzystać http://ellislab.com/codeigniter/user-guide...re_classes.html

[PHP] pobierz, plaintext 
<?php
  class MY_Controller extends CI_Controller {
 
    function __construct()
    {
        parent::__construct();
        View::gets();
    }
 
  }
?>
[PHP] pobierz, plaintext 

i w controllerze wywołałem extends MY_C.. zamiast CI ale znowu dostaje

A PHP Error was encountered

Severity: Notice

Message: Undefined property: Inbox::$_jq

Filename: models/view.php

Line Number: 66

A PHP Error was encountered

Severity: Notice

Message: Undefined property: Inbox::$_css

Filename: models/view.php

Line Number: 67

[nospor]

znowuż nie chce w każdej funkcji rozpoczynać od View::gets()

A kto ci każe w każdej funkcji? Robić to masz raz, tuż przed generowaniem głównego szablonu

[XP'ek]

no tak zgadza się ale to muszę teraz przerobić $this->smarty->view
ponieważ wstawienie View::gets(); w classę odpowiadającą za wczytanie i ustawienie smarty dla CodeIgniter to niestety pokazuje mi identyczny błąd.

Ten post edytował XP'ek 4.02.2013, 16:55:51

[nospor]

No a metodę gets zmieniłeś na statyczną? Zmieniłeś w niej odwołania do css i jq? Z komunikatu błędu widać jakby nie....

[XP'ek]

nie nie zmieniłem na statyczną i nie wiem jak zmienić odwołanie ponieważ $this-> mi nie działa

[nospor]

no skoro masz funkcje statyczną, to już nie $this-> a self::$zmienna
no takie podstawy trzeba ci wyjaśniać?