e_notice, собаки в коде yii и тп.

[pirrat]

В проекте установил свой error_handler, вместо дефолтного в yii.

Код: Выделить всё

define('YII_ENABLE_ERROR_HANDLER',false);
function exception_error_handler($errno, $errstr, $errfile, $errline )
{
    throw new ErrorException($errstr, 0, $errno, $errfile, $errline);
}
set_error_handler("exception_error_handler"); 

После чего начались сыпаться ошибки в коде фреймворка.
они не особо критичны, но не приятны, а данный обработчик игнорирует собаки и выбрасывает эксепшены.
да и вообще всем известно что код с собаками не трууу код.)

вот 2ру примеров:

A session had already been started - ignoring session_start()
../CHttpSession.php(100)
...
00100: @session_start();
....

Как вариант можно было написать так:

Код: Выделить всё

if(!isset($_SESSION))
            session_start(); 

__
2 ошибка:

filemtime() [<a href='function.filemtime'>function.filemtime</a>]: stat failed for ../runtime/cache/0d64df092bfa9f7fb45391eb639abd67.bin
../CFileCache.php(104)
...
00104: if(($time=@filemtime($cacheFile))>time())
....

можно же было проверить файл на существование и не было бы проблем:

Код: Выделить всё

        
if(!file_exists($cacheFile))
            return false;
$time=filemtime($cacheFile);
if($time>time())
... 

и я думаю таких примеров можно ещё найти...
Разработчики планируют рефакторить код на предмет таких вещей?

[samdark]

Думаю, стоит сделать небольшой тест на производительность — с собакой и без. По идее без должно быть быстрее. Если это действительно так — всё найденное поправлю.