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

Топ: Оценка кода скриптов
Сообщения
iceman12

wapgid, так красивее, не правда ли

13 Июл 2010, 21:50
iceman12

suhonok, $count = abs("$parse->count");

и накуя кавычки?)))

13 Июл 2010, 21:54
suhonok

iceman12, спросил у больного здоровья)

13 Июл 2010, 21:59
iceman12

О_о не понял тебя))

13 Июл 2010, 22:00
wapgid

iceman12, ну это вопрос удобства наверно? мне лично удобней когда короткие условия if {}полностью в одной строке..
или ты про пробелы рядом с точками? тоже вроде не фатально.. особенно если в редакторе подсветка нормальная..

13 Июл 2010, 22:01
iceman12

85, конечно) ну просто так как то читабельнее для других программистов)

13 Июл 2010, 22:04
wapgid

iceman12, спасиб задумаюсь.. просто никогда не писал с точки зрения что для кого то всегда тока всё себе.. такой вот я жадный гг

13 Июл 2010, 22:06
Akdmeh

if(!empty($cat_info[$category]['descr'])){ $description=$cat_info[$category]['descr']; }
Пробелы поставить - дело вкуса, но всё же было бы симпатичнее - некоторые редакторы делали б перенос строк, а то отсутствие пробелов заносит строку в самую попу - за пределы редактора, а скроллить вправо - убивает
Ты используешь $tpl->load( 'umor_index.tpl' );, но в то же время $mas[]='<div class="onestr"><a href="'.$prefurl.$precat.'/'.$arraycat[$pp]['alt_name'].'/">'.$arraycat[$pp]['name'].'</a></div>';
Лучше бы выносить разметку в шаблон - так было бы более труЪ.
$db - это переопределенный mysqli? А то я не припоминаю метод get_count(); ? Ой, это может быть pdo, если так - сори за необознаность)
while ( $row = $db->get_row() ) - после этого та же ошибка - разметку надо выносить в шаблон.
//
Короче, главная ошибка в том, что с одной стороны - наличие шаблонизатора, но с другой - его вы не придерживаетесь - есть разметка внутри php, что не есть хорошо. Реши - либо использовать mvc по всем правилам, либо не использовать вообще.

13 Июл 2010, 22:07
suhonok

iceman12, в том смысле, что уже не помню, чёт мне помешало, бывало и хуже, код не первой свежести)))

13 Июл 2010, 22:08
Akdmeh

81, дело вкуса, но всё равно симпатичнее;)

13 Июл 2010, 22:09
Ответить на тему