[ Поиск ] - [ Пользователи ] - [ Календарь ]
Полная Версия: Код ревью ABC-фреймворка
Страницы: 1, 2, 3
twin
Цитата (chee @ 21.04.2016 - 12:57)
Ты серьезно думаешь, что то что я предлагаю с композером и архивом с уже сгенерированым композером автозагрузчиком это переусложенение?

Я не вижу в этом смысла. Я, если чесно, вообще не вижу смысла в композеровском автозагрузчике. Чем он так знаменит?

Мой автозагрузчик нормально работает, для чего лишние движения? Там есть опция, кто захочет, сам сделает. Я может попозже тоже сделаю. Когда более менее в порядок приведу. А то каждый раз генерить надоест.

_____________
Если вам недостаточно собственных заблуждений, можно расширить их мнениями экспертов.

Нужно уважать мнение оппонета. Ведь заблуждаться - его святое право.

Настаивал, настаиваю и буду настаивать на своем. На кедровых орешках.

user posted image
twin
Цитата (chee @ 21.04.2016 - 12:54)
Можешь создать темку на ревью моей системки, я буду рад АДЕКВАТНОЙ критике.

Я пробовал еще когда баттл был. Не получилсь по двум причинам.

Первая - я не смог поставить. А так не интересно.
И вторая - я запутался в твоих ветках. Смотришь код, разбираешь, выносишь свои замечания, а оказывается я не туда смотрел и ты уже всё переделал в другой ветке.

Я тогда еще сказал - ты приведи все в подобающий вид, тогда и обсудим.


_____________
Если вам недостаточно собственных заблуждений, можно расширить их мнениями экспертов.

Нужно уважать мнение оппонета. Ведь заблуждаться - его святое право.

Настаивал, настаиваю и буду настаивать на своем. На кедровых орешках.

user posted image
chee
Цитата (twin @ 21.04.2016 - 18:28)
Я тогда еще сказал - ты приведи все в подобающий вид, тогда и обсудим.

Ок

Идем дальше, по пути выполнения кода

https://github.com/abc-framework/abc-framew...onfigurator.php

Взляни на эту строчку

https://github.com/abc-framework/abc-framew...Abc/Abc.php#L43

и на метод

https://github.com/abc-framework/abc-framew...gurator.php#L40

То есть ты прокидываешь эти параметры бесчисленное количество раз и получается, что абстракция в итоге все равно протекла.

Исправить ситуацию можно так: Инициализацию конфигуратора перенсти из AbcProcessor в ABC\Abc, а уже в конструктор AbcProcessor передавать объект конфигуратора, плюс можно туда передавать и контейнер.

Примерно так

public static function startApp($appConfig = [], $siteConfig = [])
{
if (!empty(self::$process)) {
throw new \LogicException('Only one process');
}

self::$configurator = new AbcConfigurator($appConfig, $siteConfig);
self::$container = new Container;
self::$autoload = __DIR__ . self::$autoload;
self::autoloadSelector();
self::$process = new AbcProcessor($configurator, $container);
self::$process->startApp();
}


_____________
Люди, имеющие низкий уровень квалификации, делают ошибочные выводы, принимают неудачные решения и при этом неспособны осознавать свои ошибки в силу низкого уровня своей квалификации
Быстрый ответ:

 Графические смайлики |  Показывать подпись
Здесь расположена полная версия этой страницы.
Invision Power Board © 2001-2024 Invision Power Services, Inc.