Вы не зашли.
а зачем ты пишешь в доках название метода а потом описание?
по правилам сначала короткое описание, а потом развернутое. дублировать название метода не надо
Добавлено спустя 55 секунд: Код:
| /** |
| * getPageRange() |
| * Получение ближайшего отрезка номеров страниц для создания навигации вида: (10 11 [12] 13 14). |
| * @return array |
| */ |
| public function getPageRange($range=null) |
@param нету
А точно, вчера же тоже читал про это.
Сейчас.
Мдэ...
2011.10.28 21:52 - первый вид класса навигации моего... и до сих пор мучаюсь
Отредактировано tipsun (2012.01.05 15:03)
Чет никак.
Может как тогда сделать, через
__construct($count, $page, $items)Сделать параметры обязательными, и убрать все эти setCount(), и т.д.
Тогда и ошибки ловить не надо будет.
Я вообще их ловить не умею, ну через try/catch
Ну ловить тоже че,
Код:
| span style="color: #0000BB"><?phptry { throw new Exception('about error');} catch(Exception $e) { echo $e->getMessage(); } |
Это понятно, а как в коде, куда это все.
Код:
| span style="color: #0000BB"><?phppublic function getArray(){ if (null === $this->_count) { trigger_error('Not enough data', E_USER_ERROR); } return array( 'start' => $this->_setStart(), 'stop' => min(($this->_start + $this->_items), $this->_count));} |
Отредактировано tipsun (2012.01.05 18:06)
__construct() и типа рефакторинг. Все вытекающие последствия.
Если чего тут (в теме) архив есть, с него заново начну.
Код:
| span style="color: #0000BB"><?phpclass PageMan{ const DEFAULT_RANGE = 3; private $_count = 0, $_page = 1, $_items = 10, $_pages = 0, $_start = 0; /** * Проверка, является ли число натуральным. * @param integer $var * @return bool */ public function isNatural($var) { if (is_numeric($var) and 0 < $var) return true; return false; } /** * @param integer $count * @param integer $page * @param integer $items */ public function __construct($count=null, $page=null, $items=null) { if ($count and $this->isNatural($count)) $this->_count = (int)$count; if ($page and $this->isNatural($page)) $this->_page = (int)$page; if ($items and $this->isNatural($items)) $this->_items = (int)$items; } /** * Регистрация общего количества записей. * @param integer $count * @return pageMan */ public function setCount($count) { $this->__construct($count); return $this; } /** * Регистрация текущей страницы. * @param integer $page * @return pageMan */ public function setPage($page) { $this->__construct(0, $page); return $this; } /** * Регистрация количества записей на страницу. * @param integer $items * @return pageMan */ public function setItems($items) { $this->__construct(0, 0, $items); return $this; } |
Отредактировано tipsun (2012.01.05 19:07)
это полный ахтунг! весь смысл конструктора просто катится в говно)
зачем ты в getArray делаешь такую проверку? if (null === $this->_count) {
что мешает делать ее в сеттере, а по дефолту задать нормальные значения?
