developing.name Решение проблем разработки сайтов. uncat php javascript jquery ajax html other C#

PHPShop

Иногда приходится сталкиваться с (полу)коммерческими движками, вот и в PHPShop пришлось покапаться.

Первое впечатление – отличная админка, очень много возможностей, все понятно. Мы конечно визуалы, но нам подавай код.
 

  1. Самое конечно неприятное, это отсутствие табов (лестницы) в коде. Может быть разработчики имеют их, но для клиентов поставляют такой вариант, для "удобства" сторонних кодеров
     
  2. Забавное использование глобальных переменных в функциях, типа: global $_POST, $_SESSION; Господа, они и так глобальные, не нужно их там прописывать.
     
  3. Посдчет SQL запросов: @$SysValue['sql']['num']++; А не проще ли написать (использовать готовый) класс, где будет все подсчитываться. И не использовать, после каждого mysql_query, счетчик.
     
  4. Да, названия таблиц хотелось бы использовать в конфиге, чтобы потом проще править:

    $sql="select id from ".$SysValue['base']['table_name36']." where parent_id=$n";


    но как же понять что за название? Каждый раз в конфиг лезть? Уже 36-е название, в PHPShop работают уникумы, они помнят все названия.
     
  5. Некоторый код ставит в тупик (долго думал):

    if(@$otvet!="")

    $otvet=$otvet;

    else $otvet="";


    Или вот такой:

    if($i==1) $pageOt=$i+@$pageDo;

    else $pageOt=$i+@$pageDo-$i;


    Используется в IF и в ELSE. И подобная кострукция появляется не редко.
     
  6. Проверка почты хороша (с комментарием кодера):

    if(!preg_match("/@/",$_POST['mail_new']))//проверка почты

    {

    $_POST['mail_new']="";

    }


    Кто то придумывает сложную регулярку, а зачем? Все очень просто, главно чтоб собачка была. Тогда почему не strpos?
     
  7. Есть класс по отправки писем, но тем не менее частенько наблюдается генерация писем без его использование. Каждый раз одни и те же headers создаются с нуля.
     
  8. Конечно есть такие моменты: <td style=\"padding:3\"> или так <FONT style=\"font-size:14px;color:red\">. И почему же стили не прописать? Мелочь, а приятно.
     
  9. И конечно же больная тема – шаблоны. Опять какой то не понятный код, с заменой @parameter@ параметров, на html код, частично созданный в скриптах, частично из шаблонов.
     

Есть отдельные файлы, в которых присутствует здравый смысл. Но большинство, вот с такими вещами. Складывается впечатление, что идет текучка кадров-студентов.

Подход конечно верный - главное продать, а внутренности никого не волнуют. В итоге то все работает.

Грусть одолевает...

31.08.2010, 17:05 Рубрика: other

Ссылки по теме:

 

Комментарии

Нет, совсем нет... Будешь первым?

 

Имя*:
e-mail*:
URL:
captcha*
Текст*:
  • phpshop
  • phpshop вход в админку
  • phpshop
  • форум phpshop
  • шаблоны для phpshop
  • phpshop шаблоны вход в админку
  • вход в админку phpshop
  • phpshop форум
  • phpshop не работает
  • phpshop шаблоны
  • @mail@ phpshop
  • phpshop
  • шаблоны phpshop
  • как убрать ссылку phpshop
  • как убрать PHPShop. Все права защищены
  • phpshop админка
  • убрать ссылку phpshop
  • sql запросы в phpshop
  • убрать ссылку phpshop
  • не работает админка phpshop
  • "phpshop"
  • как убрать sysvalue в phpshop
  • как убрать нижнюю ссылку в phpshop
  • phpshop добавить tab
  • phpshop+%D1%88%D0%B0%D0%B1%D0%BB%D0%BE%D0%BD%D1%8B
  • php shop как убрать ссылку
  • phpshop @name@
  • jqeury phpshop
  • адрес phpshop админка
  • ссылка на админку phpshop
  • $SysValue phpshop
  • phpshop какой шаблон используется
  • админка PHPShop
  • ссылка на PHPShop
  • шаблон PHPShop
  • phpshop адрес входа админку
  • как убрать ссылки в подвале на phpshop
  • phpshop убрать ссылку
  • phpshop ссылка на админку
  • phpshop убрать
  • как убрать ссылки в pgpshop
  • как убрать с сайта Все права защищены в phpshop
  • phpshop preg_replace
  • rfr e,hfnm ccskre yf PhpShop
  • как убрать ссылку на phpshop
  • как убрать ссылку на PhpShop в подвале сайта
  • PHPSHOP КАК УБРАТЬ ССЫЛКУ ИЗ ВИТРИНЫ
  • phpshop не работает preg_replace
  • адрес админки phpshop
  • зрзырщз

jQuery plugins:
  • pro.scroll v0.3 - графический скролл для div
  • pro.sres v0.1 - подсказки при поиске
  • pro.tips v0.2 - всплывающие подсказки
  • pro.tube v0.1 - работа с YouTube ссылками
Powered by PRO