Все

Gemorroj · 2012.01.05 08:01

а зачем ты пишешь в доках название метода а потом описание?
по правилам сначала короткое описание, а потом развернутое. дублировать название метода не надо
Добавлено спустя 55 секунд:

Код:

1

2

3

4

5

6

/**

* getPageRange()

* Получение ближайшего отрезка номеров страниц для создания навигации вида: (10 11 [12] 13 14).

* @return array

*/

public function getPageRange($range=null)

@param нету

tipsun · 2012.01.05 08:36

А точно, вчера же тоже читал про это. sad Сейчас.

tipsun · 2012.01.05 09:23

Мдэ... 2011.10.28 21:52 - первый вид класса навигации моего... и до сих пор мучаюсь big_smile

Отредактировано tipsun (2012.01.05 11:31)

tipsun · 2012.01.05 13:20

Чет никак.
Может как тогда сделать, через __construct($count, $page, $items)
Сделать параметры обязательными, и убрать все эти setCount(), и т.д.
Тогда и ошибки ловить не надо будет.
Я вообще их ловить не умею, ну через try/catch

Gemorroj · 2012.01.05 14:26

я не понимаю проблемы.

tipsun · 2012.01.05 14:27

Ну ловить тоже че,

Код:

1

span style="color: #0000BB"><?phptry { throw new Exception('about error');} catch(Exception $e) { echo $e->getMessage(); }

Это понятно, а как в коде, куда это все.

Gemorroj · 2012.01.05 14:31

в каком коде?

tipsun · 2012.01.05 14:35

Код:

1

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 14:36)

tipsun · 2012.01.05 14:55

__construct() и типа рефакторинг. Все вытекающие последствия.
Если чего тут (в теме) архив есть, с него заново начну.

Код:

1

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 15:06)

Gemorroj · 2012.01.05 15:27

это полный ахтунг! весь смысл конструктора просто катится в говно)
зачем ты в getArray делаешь такую проверку? if (null === $this->_count) {
что мешает делать ее в сеттере, а по дефолту задать нормальные значения?

WAP Мастер Форум

#91 2012.01.05 08:01

Re: Функция навигации по страницам

Код:

#92 2012.01.05 08:36

Re: Функция навигации по страницам

#93 2012.01.05 09:23

Re: Функция навигации по страницам

#94 2012.01.05 13:20

Re: Функция навигации по страницам

#95 2012.01.05 14:26

Re: Функция навигации по страницам

#96 2012.01.05 14:27

Re: Функция навигации по страницам

Код:

#97 2012.01.05 14:31

Re: Функция навигации по страницам

#98 2012.01.05 14:35

Re: Функция навигации по страницам

Код:

#99 2012.01.05 14:55

Re: Функция навигации по страницам

Код:

#100 2012.01.05 15:27

Re: Функция навигации по страницам

Дополнительно