and_07 пишет:$front = Router::getInstance();
0) Не люблю инстации, они плодят рекурсивные ссылки на объекты, лучше сделать абстрактный класс роутера и обращаться к нему откуда угодно: Router::<method>();
1) getParams() возвращает сразу все параметры, которые придется потом перебирать либо в экшне контроллера, либо передавать модели, которая так же будет вынуждена уметь их перебирать, так теряется инкапсуляция.
2) Нет поддержки модулей, стоящих выше контроллеров, которых в одном модуле может быть несколько.
3) В синглтоне, если уж на то пошло, нужно еще повесить заглушку __clone, иначе всетаки сделать еще одну копию объекта получится )))
|