Как-то попросили меня добавить на один сайт пару плюшечек. До этого сайт разрабатывался партнерами с золотыми сертификатами, и не так уж давно. И вот начал я разбираться, что где находится и что же я увидел. Просто мрак, словно разработчики обучались на 4-5 версии БУС и о том, что поменялось ведать не ведают.
Вот так выглядит раздел фотогалереи:
А вот так файл инклюд:
Ни компонентов, ни кеша. Чистая работа с инфоблоками.
А еще например, файлики со стилями лежат в /css, скрипты в /js, картинки в /img.
Есть ли у кого какие мысли, по поводу того, как можно более объективно оценивать работу разработчиков того или иного партнера. А то человек приходит, видит кучу сертификатов, кучу красивых портфолио. Потом заказывает у них сайт. А через некоторое время обращается к другому партнеру/разработчику для изменения/обновления сайта, а тут такая гадость.
Вот так выглядит раздел фотогалереи:
<? require($_SERVER["DOCUMENT_ROOT"]."/bitrix/header.php"); $APPLICATION->SetTitle("Раздел"); ?> <? $APPLICATION -> IncludeFile("iblock/studio_foto_section.php", array("IB" => 2, "SECTION" => $_REQUEST['SECTION_ID'], "SECTION_CODE" => $_REQUEST['SECTION_CODE']));?> <?require($_SERVER["DOCUMENT_ROOT"]."/bitrix/footer.php");?> |
<? if (CModule::IncludeModule("iblock")) { $IB = intval($IB); $SECTION = intval($SECTION); $arFilter = array( "IBLOCK_ID" => $IB, "ACTIVE" => "Y", ); if (strlen($SECTION_CODE)>0) { $arFilter['CODE']=$SECTION_CODE; } else { $arFilter['ID']=$SECTION; } $rsSection = CIBlockSection::GetList(array("SORT" => "ASC"), $arFilter, false, array('UF_*')); if ($arSection = $rsSection -> GetNext()) { ... $rsElement = CIBlockElement::GetList(array("SORT" => "ASC"), $arFilter, false, false, $arSelect); if ($rsElement -> SelectedRowsCount() > 0) { while ($arElement = $rsElement -> GetNext()) { $rsFile = CFile::GetByID($arElement['PREVIEW_PICTURE']); $arFile = $rsFile -> GetNext(); $w = $arFile['WIDTH']; $h = $arFile['HEIGHT']; if ($w > $h) { $nw = 224; $nh = ceil($h / ($w / $nw)); } else { $nh = 139; $nw = ceil($w / ($h / $nh)); } $file = CFile::ResizeImageGet($arElement['PREVIEW_PICTURE'], array('width'=>$nw, 'height'=>$nh), BX_RESIZE_IMAGE_PROPORTIONAL, true); $files[] = $file; ?> <li> <a href="<?=$arElement['DETAIL_PAGE_URL'];?>" style="background:url('<?=$file['src'];?>'); display: inline-block; width: 224px; height: 139px;"> <? if ($arElement['PROPERTY_VIDEO_VALUE'] > 0 || strlen($arElement['PROPERTY_VIDEO_CODE_VALUE']) > 0):?> <b></b> <? endif;?> </a> </li> ... } ?> |
А еще например, файлики со стилями лежат в /css, скрипты в /js, картинки в /img.
Есть ли у кого какие мысли, по поводу того, как можно более объективно оценивать работу разработчиков того или иного партнера. А то человек приходит, видит кучу сертификатов, кучу красивых портфолио. Потом заказывает у них сайт. А через некоторое время обращается к другому партнеру/разработчику для изменения/обновления сайта, а тут такая гадость.
Кроме того, это не самый вопиющий случай — сплошь и рядом встречаются PHP без API, прямые SQL-запросы к БД итд
Это рынок.
Можно купить за 300 рублей абибас
Можно купить за 3000 настоящий адидас
Можно купить за 30.000 настоящее дольче, которое по качеству будет не лучше абибаса
Можно купить за 30.000 купить шмотку от ноунейм мастера, сделанную вручную, которая будет лучше качеством и визуально, чем все остальные, да ещё будет ИНДИВИДУАЛЬНОЙ и УНИКАЛЬНОЙ
заказчику следует начинать в этом ориентироваться, а не вестись на все объявления в контексте "сделаем интернет-магазин под ключ за 3 часа и 3000р"
А если понадобится вынести всю статику на отдельный сервер, что будете делать? Или об этом не задумывались? Структура статичных файлов проекта:
Бывает по жестче код, чем Вы показали.
Недавно клиент спросил, в чем разница между золотым и сертифицированным партнером. Я ему сказал разница только в продажах и открытых проектах. Разницы в качестве работы возможно и нет! Даже порой хуже бывает у золотых, чем у сертифицированных.
К сожалению, клиент никак это не сможет отследить (качество работы)! Роман указал про монитор качества. Так вот Приходили клиенты которые даже его не запускали, И это почти каждый клиент. Т.к. партнеры при обучении клиента зачастую опускают такие моменты (как монитор качества). Это я понял когда клиенты пришли от партнеров на доработки.
Согласен с Ваней, тут уже "Оценивать работу конкретного разработчика нужно уметь самому партнеру"
Сам наблюдал сей "лол".
Это чудо под названием "Монитор качества" сейчас наблюдает у меня на последнем проекте 4 модифицированных файла ядра (ни одного файла ядра не трогал кроме как на чтение), рапортует об отсутствии резервного копирования (при мастер-слейв репликации базы, синхронизации файловой системы между двумя серверами фронт-энда и ежедневном бекапе) и ругается на не включенную проактивную защиту (на "Старте", ага).
Если бы я работал в студии, то потратил бы своё время не на то, чтобы соблюсти требования этого "монитора качества", а на то, чтобы он не выдавал того, что с моей точки зрения является ложными срабатываниями.
А если бы меня это реально парило, то я бы ещё и выдавал объяснения клиентам в стиле: "У нас всё правильно, это просто в битриксе работают такие чудаки".
Говорить "не парьтесь" - некорректно - это просто отмазка.
Вот если бы мы знали, что стоимость разработки была дорогой,
и обещали клиенту другое и кеш и все такое
а сделали такое как вы привели, тогда запасаемся камнями.