Snad spravne pouziti Zend_Logu

[meldik]

Ahoj,

prosim vas o radu/kontrolu ... Mam toto spravne napsane?

class JMLib_Core_Action extends Zend_Controller_Action
{
    protected $_logger;

    public function log($message, $priority)
    {
        $this->_logger = new Zend_Log();
        $writer = new Zend_Log_Writer_Stream(JMLib_Application::getInstance()->getAppPath('LOG') . '/error_log.log');
        $this->_logger->addWriter($writer);
        return $this->_logger->log($message, $priority);

    }
}

S tim, ze pak pouziti mam v controleru jako

class JMLib_Com_AdminController extends JMLib_Core_Action
{
    public function noneAction()
    {
        $this->log($message, 3);
    }
}

[Kevujin]

Ahoj,

prosim vas o radu/kontrolu ... Mam toto spravne napsane?

class JMLib_Core_Action extends Zend_Controller_Action
{
    protected $_logger;

    public function log($message, $priority)
    {
        $this->_logger = new Zend_Log();
        $writer = new Zend_Log_Writer_Stream(JMLib_Application::getInstance()->getAppPath('LOG') . '/error_log.log');
        $this->_logger->addWriter($writer);
        return $this->_logger->log($message, $priority);

    }
}

tak se podívej, co to vlastně dělá. při každým zavolání metody log vytváříš všechno znovu. vezmi si, že tu metodu log zavoláš třeba víckrát a pokaždý znovu inicializuješ proměnnou _logger. to asi není úplně správně, co? inicializuj ji jen jednou.

class JMLib_Core_Action extends Zend_Controller_Action
{
    protected $_logger = null;

    public function log($message, $priority)
    {
        if (null === $this->_logger) {
          $this->_logger = new Zend_Log();
          $writer = new Zend_Log_Writer_Stream(JMLib_Application::getInstance()->getAppPath('LOG') . '/error_log.log');
          $this->_logger->addWriter($writer);
        }
        
        return $this->_logger->log($message, $priority);

    }
}

[meldik]

To jsem si vůbec neuvědomil. Děkuji moc.

[Sniper]

a mozna jeste lepsi je to nedavat do controlleru ale udelat si budto plugin nebo v bootstrapu _initLog metodu, kde si vytvoris ten objekt logu a ulozis ho do Zend_Registry. Pak ho totiz muzes pouzivat i v modelu apod. a ne jenom v instancich tvojeho upravenyho kontroleru a potomcich. Zrovna na tohle mi prijde Zend_Registry idealni