Xwab
Форумыnavigate_nextСкрипты

Class Router (v0.2.2) [Нужны рекомендации по коду]
Сообщения
ma3uk

Автор: ma3uk
Версия: 0.2.2

Новый класс роутинга. Очень нужны рекомендации по коду.
Readme на Git устаревшее, пример использования в index.php.

Посмотреть на Github
Скачать


__________
посл.ред. 28 Янв 2013, 4:13; всего 12 раз(а) 06 Окт 2012, 6:56
Akdmeh

Тип переменных в виде preg-шаблона еще не помешал бы

06 Окт 2012, 7:28
ma3uk

Akdmeh, учту, сделаю.

06 Окт 2012, 7:31
wapmorgan

1. phpDoc. Его нет.
2. Применение. Также отсутствует.

06 Окт 2012, 13:54
Akdmeh

Да, толку немного. В любом фреймворке есть своя реализация. А писать фреймворк, опираясь на готовый класс без документации...

Ну как чисто ради опыта - то может нормально, но бесполезно.

06 Окт 2012, 13:59
ma3uk

Обновил первый пост и версию роутера.

07 Окт 2012, 21:20
Okula

ma3uk, для того чтоб IDE читала PHPdoc нужно строить его таким форматом:
/**
* описание функции
*/
Заметь, начинается phpdoc со слеша и двух звёздочек, у тебя же одна, из-за этого описание методов не будет доступно внутри IDE.

07 Окт 2012, 21:37
ma3uk

Okula, спасибо, сейчас исправим.

07 Окт 2012, 21:40
Okula

[php:1:1a62e8e5ff]/*
* @var array Массив маршрутов соответствующих URL.
*/
public $urlPattern;

/*
* @var array Массив маршрутов соответствующих URL.
*/
public $matchedRoutes = array();[/php:1:1a62e8e5ff]

Так какое из свойст для чего используется? Определись.
Зачем ты даёшь свойствам класса значение public если они используются только внутри класса? Понятие "инкапсуляция" тебе знакомо?

07 Окт 2012, 21:46
ma3uk

Okula, когда архив обновлял случайно попал кусок недописанного кода.
добавлено спустя 3 минуты:
Исправлено

07 Окт 2012, 22:00
Ответить на тему