PHP фреймворк by Soulgarden

Появляется ощущение, что PHP это фреймворк для создания фреймворков =)

Появляется ощущение, что PHP это фреймворк для создания фреймворков =)

Хорошо сказано. А почему такое ощущение?

soulgarden, не хочу ни коим образом обидеть/наехать и т.п. и т.д. :)

1) public function __construct() {} в контроллере, зачем?
2) if(!defined('APPATH')) header("HTTP/1.0 404 Not Found"); это должно быть ядре, никак не в контроллере
3) Зачем ты понапихал 2 пункт везде? чтоб запретить доступ к директориям клади в них .хтаксесс "deny from all" для нгинкса хз как
4) Слишком дохуя условий в формах, обдумай логику
5)

$application_folder = 'application';

$system_folder = 'system';

//————————————-

$docroot = realpath(dirname(__FILE__)).DIRECTORY_SEPARATOR;

define('APPATH', $docroot.$application_folder.'/', true);
define('SYSPATH', $docroot.$system_folder.'/', true);

Сокращается до:

define('DS', DIRECTORY_SEPARATOR);

define('APPATH', dirname(__FILE__).DS.'application');
define('SYSPATH', APP_DIR.'core'.DS.'system');

Ну это так, ИМХО)

6) Весь bootstrap.php думаю надо вынести в файл конфигурации, в любом удобном тебе виде xml/ini/php array/yaml etc.
7) Почему папка rsc так называется не понял, совершенно не подходит к содержанию :)
8) Все исключения вынеси в один файл, слишком жирно одному классу исключения иметь свой файл :)
9) Тоже самое и с интерфейсами
10) O_o

class NotFoundHttpException extends Exception {
    public function __construct($message = null) {
        header("HTTP/1.0 404 Not Found");
    }
}

заголовок в инсключении? Эмм
_____________________________
В целом мне понравился, сыро но перспективно :)

Скачал, проект. Стиль проверять не буду, это может сделать - http://pear.php.net/package/PHP_CodeSniffer/, открываю index.php:

1) Вижу закрывающий тэг ?>, оно там не надо
2) Вижу 3 require_once, а должен быть 1, все остальное на autoload
3) error_reporting(E_ALL) и подобное позволь тогда настроить в конфигурации
4) Core::init() - статический, настораживает, но пока не знаю почему идем дальше

открываю system/Loader.php

1) Вижу if(!defined('SYSPATH')) header("HTTP/1.0 404 Not Found"), понимаю, что бред проблему доступа и т.п. решай не здесь
2) Опять вижу статический класс, да еще и костыль, если я правильно понял -  if (self::$is_inited == true) return;
3) …

Все схарило.

Вобщем, сделаешь потом апп и покажешь, как оно у тебя все создается в два нажатия кн. мыши ;)

Спустя 32 сек.

А где unit тесты?

Benchmarking soul.test (be patient)
Finished 2731 requests


Server Software:        Apache/2.2.4
Server Hostname:        soul.test
Server Port:            80

Document Path:          /
Document Length:        12 bytes

Concurrency Level:      5
Time taken for tests:   30.5716 seconds
Complete requests:      2731
Failed requests:        0
Write errors:           0
Total transferred:      658412 bytes
HTML transferred:       32784 bytes
Requests per second:    91.02 [#/sec] (mean)
Time per request:       54.935 [ms] (mean)
Time per request:       10.987 [ms] (mean, across all concurrent requests)
Transfer rate:          21.40 [Kbytes/sec] received

ab -t 30 -c 5 http://soul.test/
=/

rider-sx, за шаблонизацию отвечает библиотека phpdude, может быть сейчас включена force перекомпиляция - я пока шаблонизатором не занимался, у меня в планах переработать его. так что ничего страшного

Я знаю, имелось ввиду что я убрал из контроллера вызов шаблонизатора и его использование :) Дабы сравнить быстродействие с тем же уии