Появился какой то ... разработчик. заказчики будьте осторожней.

#
Появился какой то ... разработчик. заказчики будьте осторожней.
Суть такая вот уже второй человек обращается ко мне с прозьбой поправить ЧПУ, мол у хостСМС они не верно работают. Я вообще сначала не поверил.

Когда я увидел что это реально ХостСМС и как организованна структура сайта. я чуть не аху..л , все сделано как попало, документацию разработчик сайта наверное вообще не читал, про xsl не слышал или даже не знает что такое есть и как с ними работать. все сделанно на php и чистом html.

вот кусок кода из его скрипта
<h1>Карта сайта</h1>
                  <ul>
                      <li><a href="/"><span>Главная</span></a></li>
                      <li><a href="/about/"><span>О компании</span></a></li>
                      <li><a href="/for_agents/"><span>Для агентств</span></a></li>
                      <li><a href="/for_tourist/"><span>Для туристов</span></a></li>
                      <li><a href="/mail/"><span>Отправка сообщения</span></a></li>
                      <li><a href="/news/"><span>Новости</span></a></li>
                      <ul>
                      <?
                       $InformationSystem = new InformationSystem();
                       $information_groups_id = 0;
                       $information_systems_id = 17;   
                       // Сортируем по дате
                       $param = array();
                       $param['OrderField']  = 'information_items_date';
                       $param['Order']  = 'DESC';
                       $resource = $InformationSystem->GetInformationItemsFromGroup($information_groups_id,
                                                       $information_systems_id, $param);
                       while($row = mysql_fetch_assoc($resource))
                          {
                       ?><li><a href="/news/?id=<?=$row['information_items_id']?>">
                                 <span><?=$row['information_items_name']?></span></a></li><?

                    }
                  ?>


и так все страницы сайта!!!!!!!! Я извиняюсь но разработчик мудак.
а сами ссылки выглядят примерно так /news/news.php?id=all_news или /news/news.php?group=5&id=56
Вообщем мне пришлось переделывать полностью сайт на нормальный вид.
Так лано бы если он еще правильно все делал, так он про такие теги как title, description, keybord вообще забыл.
Я вообще не понимаю зачем ему хостСмс если он делает все статически....

Вообщем будьте осторожней. И внимательней при выборе фрилансера. Так как потом придется платить при переделки как за новый сайт.
Программист - это тот, чей код работает не только правильно, но и быстро!
#
Re: Появился какой то ... разработчик. заказчики будьте осторожней.
Я молчу про свои переделки! пару раз видел рАбОтУ таких извращенцев... А если попросить HostCMS сделать рейтинг фрилансеров на сайте, что б люди могли сами писать о них отзывы. А то люди приходят от огогогогогогого каких студий и говорят снеси нах сделай по людски. Я думаю умеющим людям работы прибавиться, а выявленные жопотрясы будут курить бамбук!
Skype:ferdinant1988 ICQ:311960596 E-mail: ferdinant@i.ua
#
Re: Появился какой то ... разработчик. заказчики будьте осторожней.

<?php
            if (class_exists('shop'))
            {

               $shop = & singleton('shop');
               $shop_id = 1;
//                $shop->ShowShop($shop_id, 'МагазинГруппыТоваровНаГлавной1');// - на память

               // добудем корневые группы товаров
               $rs_main = $shop->GetGroups($shop_id, 0);
               //прикинем сколько должно быть категорий в колонке для равномерного распределения по 3-м колонкам
               $count_in_col_tbl = ceil(mysql_num_rows($rs_main)/3);
               //пробьем урл магаза
               $structure = & singleton ('Structure');
               $shop_url = '/'.$structure->GetStructurePath(42);
               
               $i = 0;
               while($row = mysql_fetch_assoc($rs_main)){
                  //добудем список категорий конкретной категории
                  $rs = $shop->GetGroups($shop_id, $row['shop_groups_id']);
                  //пробьем урл категории
                  $path = $shop_url.$row['shop_groups_path'].'/';
                  if (mysql_num_rows($rs)){
                     // список каждой категории
                     echo '<h1><a href="#" class="false">'.$row['shop_groups_name'].'</a></h1><ul>';
                     //соберем ссылки на подкатегории дочних корневой категории
                     while($row = mysql_fetch_assoc($rs))
                        echo '<li><a href="'.$path.$row['shop_groups_path'].'/">'.$row['shop_groups_name'].'</a></li>';
                     echo '</ul>';
                  }else{
                     echo '<h1><a href="'.$path.'">'.$row['shop_groups_name'].'</a></h1>';
                  }
                  $i++;
                  if ($count_in_col_tbl == $i){//а не начать ли новую колонку?
                     $i = 0;
                     echo '</td><td width="33%" valign="top">';
                  }
               }
            }
?>      




Ну не *** ли?
Skype:ferdinant1988 ICQ:311960596 E-mail: ferdinant@i.ua
#
Re: Появился какой то ... разработчик. заказчики будьте осторожней.

<title><?php
$title = '';
if (isset($GLOBALS['row_item'])){
   if (isset($GLOBALS['row_item']['shop_items_catalog_seo_title']) && $GLOBALS['row_item']['shop_items_catalog_seo_title'] != '')
      $title = $GLOBALS['row_item']['shop_items_catalog_seo_title'];
   else if (isset($GLOBALS['row_item']['shop_items_catalog_name']) && $GLOBALS['row_item']['shop_items_catalog_name'] != '')
      $title = $GLOBALS['row_item']['shop_items_catalog_name'];
   else if (isset($GLOBALS['row_item']['information_items_seo_title']) && $GLOBALS['row_item']['information_items_seo_title'] != '')
      $title = $GLOBALS['row_item']['information_items_seo_title'];
   else if (isset($GLOBALS['row_item']['information_items_name']) && $GLOBALS['row_item']['information_items_name'] != '')
      $title = $GLOBALS['row_item']['information_items_name'];
}
if ($title == '' && isset($GLOBALS['row_group'])){
   if (isset($GLOBALS['row_group']['shop_groups_seo_title']) && $GLOBALS['row_group']['shop_groups_seo_title'] != '')
      $title = $GLOBALS['row_group']['shop_groups_seo_title'];
   else if (isset($GLOBALS['row_group']['shop_groups_name']) && $GLOBALS['row_group']['shop_groups_name'] != '')
      $title = $GLOBALS['row_group']['shop_groups_name'];
}
if ($title == '') $title = $GLOBALS['structure']['structure_menu_name'];
echo $title;
?></title>
<meta name='yandex-verification' content='67f83a51d573cbe2' />
<meta name="verify-v1" content="9K3tCfbm1l144UKH3+ep25FUgP8cgoAyfn7KrUE8bds=" >
<meta name="msvalidate.01" content="AC482BDBAADDEF50AC995A8963801724" />
<?php
if ((CURRENT_STRUCTURE_ID != 42) || !preg_match('/page-(\d+)\/?$/',$_SERVER['REDIRECT_URL'],$match) || $match[1] == '1'){
   echo '<meta name="description" content="';
   $kernel->show_description();
   echo "\">\n";
   
   echo '<meta name="keywords" content="';
   $kernel->show_keywords();
   echo "\">\n";
   ;
}
?>


Skype:ferdinant1988 ICQ:311960596 E-mail: ferdinant@i.ua
#
Re: Появился какой то ... разработчик. заказчики будьте осторожней.
govnokod.ru/php

суда постите) Вмести посмеемся)
Программист - это тот, чей код работает не только правильно, но и быстро!
#
Re: Появился какой то ... разработчик. заказчики будьте осторожней.
Вы почитайте что он делает в коментах к скрипту...
Skype:ferdinant1988 ICQ:311960596 E-mail: ferdinant@i.ua
#
Re: Появился какой то ... разработчик. заказчики будьте осторожней.

      <?php
            // формирование списка фирм
            $db = &singleton("database");
            $sql = "SELECT i.information_items_id, i.information_items_name, i_p.information_propertys_items_value FROM information_items_table as i
                  LEFT JOIN information_propertys_items_table as i_p ON i.information_items_id = i_p.information_items_id
                  LEFT JOIN information_propertys_table as p ON i_p.information_propertys_id = p.information_propertys_id
                  WHERE information_propertys_items_dir_id = 2 AND i_p.information_propertys_items_value IS NOT NULL AND i_p.information_propertys_items_value <> ''  ORDER BY i.information_items_id
            ";
      
            if ($result = $db->query($sql)) {
               while ($row = mysql_fetch_assoc($result)) {
                  $firms[$row["information_items_id"]]["name"] = $row["information_items_name"];
                  $firms[$row["information_items_id"]]["phone"][] = $row["information_propertys_items_value"];
                  
                   $firms[$row["information_items_id"]]["path"] = $InformationSystem->GetPathItem($row["information_items_id"], '');
               }
               
               $xml_firms = "<firms_list>\r\n";
               
               foreach ($firms as $id => $firm) {
                  $xml_firms .= "<firm id=\"".$id."\">\r\n";
                  $xml_firms .= "<name>".htmlspecialchars($firm["name"])."</name>\r\n";
                  $xml_firms .= "<path>".$firm["path"]."</path>\r\n";
                  foreach ($firm["phone"] as $phone) {
                     $xml_firms .= "<phone>$phone</phone>\r\n";
                  }
                  $xml_firms .= "</firm>\r\n";
               }
               
                $information_system_id = 21;
                $xml_firms .= $InformationSystem->GenXml4InformationSystem($information_system_id);
               
               
               
               $xml_firms .= "</firms_list>\r\n";
               if (isset($property_news["external_xml"])) $property_news["external_xml"] .= $xml_firms;
               else $property_news["external_xml"] = $xml_firms;
            }
            // --------------------------------------------

             $InformationSystem = & singleton('InformationSystem');
             $external_propertys_news = array();
      
            $InformationSystem->ShowInformationSystem(22, false, 'СписокНовостейНаГлавнойSite', 5, 0, $external_propertys_news, $property_news);
      ?>
Skype:ferdinant1988 ICQ:311960596 E-mail: ferdinant@i.ua
#
Re: Появился какой то ... разработчик. заказчики будьте осторожней.
ferdinant, прошу объяснений по поводу приведённого выше куска кода? что же в нём не так?
#
Re: Появился какой то ... разработчик. заказчики будьте осторожней.
а контакты разработчика? чтобы не попасть..
#
Re: Появился какой то ... разработчик. заказчики будьте осторожней.
xandr99 писал(а):
ferdinant, прошу объяснений по поводу приведённого выше куска кода? что же в нём не так?


для этой задачи есть доп свойства ShowInformationSystem, вы посути увеличили количество запросов к БД.  

вот

$property['select'] массив ($element) с дополнительными параметрами для задания дополнительных условий отбора информационных элементов

http://hostcms.ru/api/HostCMS_5/InformationSystem.html#methodShowInformationSystem
Программист - это тот, чей код работает не только правильно, но и быстро!
Авторизация