WAP Мастер Форум
- Вы не зашли.
title
Description
Body
#81 2011.12.20 19:19
Re: Функция навигации по страницам
Математика у меня в pageMan::pageRange() неправильная, мне кажется.
Хотя там все работает верно.
Если расчитать все четко не получится, то увы. Будет как есть.
Отредактировано tipsun (2011.12.20 19:20)
Неактивен
#82 2011.12.21 05:13
Re: Функция навигации по страницам
Я с телефона, и половины темы не осилил! Код обрезает ппц! Убери для вап версии, в стиле (забыл как называеется) скроулинг
Я буду лучше голоден, но между креслом и рулем и на дороге.
Неактивен
#83 2011.12.21 08:02
#84 2011.12.23 11:54
#85 2011.12.23 15:28
Re: Функция навигации по страницам
Единственное, что в голову приходит - это оптимизация.
Сейчас поправки внесу и выложу.
Вот. (Этот гребаный виндовс завирусованный, после восстановления мои архивы пропали! >_<)
Код:
|
|
Отредактировано tipsun (2011.12.23 16:02)
Неактивен
#86 2012.01.04 07:24
Re: Функция навигации по страницам
Akdmehp написал:
255, tipsun, смотрю навигацию в "живом" режиме.
Приятно, что используется
phpDoc - это плюс.
Итак, не совсем понял, зачем
делать так: $_count=null;
приравнивание переменной значению null равно её
удалению. То есть, ты
создаешь переменную, удаляя
её. Проще либо приравнивать
нулю или значениям по-
умолчанию, либо так: private $_count, $_page,
$_pages, $_start, $_tmpPage =
1, $_items = 10;
Не хватает для наглядности
примера использования.
Плохо, что нету конструктора, так было бы намного
нагляднее, советую добавить
конструктор.
Также по правилам хорошего
тона имя класса делаем с
большой. в phpDoc после * @param mixed
$count указываем что эта
переменная значит - по
правилам хорошего тона, ну и
после return указываем, что
именно возвращает. Советую использовать в
классе систему try/catch, это
будет намного логичнее, чем
генерировать ошибки.
Пример конструктора на твоем
коде, хотя я все равно бы код рефакторил:
__construct($count, $page,
$items);
Просмотрел. В принципе,
работы очень много, все можно
переделать короче и лучше, используя в два раза меньше
методов, так как это в свою
очередь запутывает код. В общем, работай дальше!
Благодаря Gemorroj'ю (насчет phpDoc) плюс заработал 
Добавлено спустя 5 минут 51 секунду:
Я вот уже думаю насчет рефактора кода и конструктор, в принципе, добавить не проблема.
Добавлю конструктор а переменные буду обрабатывать методами, которые принимают данные.
try/catch даже не знаю че такое, придется учить.
Насчет $var = null; даже не знал.
Отредактировано tipsun (2012.01.04 07:34)
Неактивен
#87 2012.01.04 07:31
Re: Функция навигации по страницам
>> приравнивание переменной значению null равно её
удалению.
нет, это не так. если переменная существует (даже со значением null), под нее уже выделена память.
в остальном все так.
ссылки на стандарты кодирования
http://pear.php.net/manual/ru/standards.php
http://framework.zend.com/manual/ru/cod … ndard.html
я бы советовал придерживаться зендовских соглашений. PEAR отличается в мелочах.
Неактивен
#88 2012.01.04 07:41
#89 2012.01.04 08:54
#90 2012.01.04 15:40
Re: Функция навигации по страницам
Gemorroj написал:
phpDoc не совсем верные
@return object замени на @return pageMan
@param mixed $count на @param integer $count
и тд. так же убери pageMan::getTable() и проч, как будто ты описываешь статические методы.
а так, не вдаваясь в подробности выглядит норм.
Сделано.
Отредактировано tipsun (2012.01.04 15:50)
Неактивен