Что сказано в ТЗ топикстартера? - Веб-строительство

Необходимо оценить умение программиста

Strom · 2008-05-20T16:00:03.0000000Z

Необходимо оценить умение программиста выполнившего сскрипты приведенные ниже. Рассматриваю данного человека как потенциального сотрудника. Представленные скрипты это примеры его работ. Парсер данных из Яндекса <?php $globalerror = ''; @ini_set('max_execution_time', 360); function timer($shift = false) { static $first = 0; static $last; $now = preg_replace('#^0(.*) (.*)$#', '$2$1', microtime()); if (!$first) $first = $now; $res = $shift ? $now - $last : $now - $first; $last = $now; return $res; } function noindex($url) { global $globalerror; $url = preg_replace("/http:\/\//i","",trim($url)); $url = preg_replace("/www./i","",$url); $url = preg_replace("/\/$/","",$url); $url = htmlspecialchars($url); $lurl = 'www.'.$url; // XML запрос $request = '<?xml version="1.0" encoding="utf-8"?> <request> <query>url="'.$url.'" | url="'.$lurl.'"</query> </request>'; // POST заголовок $request_header = ' POST /xmlsearch/ HTTP/1.0 Host: xmlsearch.yandex.ru Content-Type: text/plain Content-length: '.strlen($request).' '; $request = $request_header.$request; //устанавливаем соединение $fp = @fsockopen('xmlsearch.yandex.ru',80,$errno,$errstr,30); if ($fp) { //отправляем запрос fwrite($fp,$request); //получаем ответ $xml_respose = ''; while(!feof($fp)){ $xml_respose.= fgets($fp,512); } fclose($fp); } else { $globalerror = "Open socket error"; return; } $xml_respose=substr($xml_respose,strpos($xml_respose,'<?xml')); $globalerror= (preg_match('|<error>([^<]+)</error>|',$xml_respose,$matches)) ? $matches[1] : ''; $result = (preg_match('|<error code="(\d+)">([^<]+)</error>|', $xml_respose, $matches)) ? $matches[2] : 0; return ($globalerror) ? $globalerror : $result; } timer(); $urls = (isset($_POST['urls'])) ? $_POST['urls'] : ''; $badurls = ''; ?> <html> <head> <meta http-equiv="Content-Type" content="text/html; charset=windows-1251"> <title>Yandex XML</title> </head> <body> <h2>Yandex XML URLs</h2> <form method="post" action="xml.php"> <textarea id="urls" name="urls" rows="15" cols="80"><?=$urls?></textarea> <br><input type="submit" value="Проверить URL"> </form> <?php if ($urls) { $list = split("\n", trim($urls)); foreach ($list as $url) { $es = noindex($url = strip_tags(trim($url))); if ($globalerror) break; if (!$es) echo "<b style=\"color:green;\">ok: </b>$url<br>\n"; else { echo "<b style=\"color:red;\">n/a: </b>$url - <span style=\"color:blue;\">$es</span><br>\n"; $badurls .= $url."\n"; } } if ($globalerror) echo "<b style=\"color:red;\">$globalerror</b><br>"; if ($badurls) echo 'Не в индексе:<br><textarea rows="15" cols="80">'.$badurls.'</textarea>'; echo "\n".'<br>Проверка завершена за ' . timer() . ' сек. Проверено ' . count($list) . ' url'; } ?> </body> </html> Парсер рецептов <?php // сайт, слэш в конце обязателен $site='http://vinegret.info/'; $start_num=17877; $other_num=400; set_time_limit(0); error_reporting(0); $file_out=fopen(getcwd()."/parsed.sql","w"); $file_bad=fopen(getcwd()."/parsed_bad.sql","w"); // начало $file=file_get_contents($site.'online.php'); preg_match_all('/(?="><a\shref=\"online.php\?src=).*?(?=">)/i',$file,$matches); foreach ($matches[0] as $item) { $base_cat=str_replace('"><a href="','',$item); // главная категория $file_main_cat=file_get_contents($site.$base_cat); preg_match_all('/(?=d><a\sclass=\'the_link\'\shref=\'online.php\?src=).*?(?=\'>)/i',$file_main_cat,$matches); foreach ($matches[0] as $item) { $main_cat=str_replace("d><a class='the_link' href='",'',$item); // категория $file_cat=file_get_contents($site.$main_cat); preg_match_all('/(?=d><a\sclass=\'the_link\'\shref=\'online.php\?src=).*?(?=\'>)/i',$file_cat,$matches); foreach ($matches[0] as $item) { $cat=str_replace("d><a class='the_link' href='",'',$item); // подкатегория $file_subcat=file_get_contents($site.$cat); // ссылки на рецепты preg_match_all('/(?=d><a\sclass=\'the_link\'\shref=\'online.php\?src=).*?(?=\'>)/i',$file_subcat,$matches); foreach ($matches[0] as $item) { $recept=str_replace("d><a class='the_link' href='",'',$item); // рецепт $file_descript=file_get_contents($site.$recept); // название preg_match_all('/(?=vrn">).*?(?=<)/i',$file_descript,$matches); $title=str_replace('vrn">','',$matches[0][0]); // ингредиенты preg_match_all('/(?=vpb">).*?(?=<)/i',$file_descript,$matches); $ingredient=str_replace('vpb">','',$matches[0][0]); $ingredient="\n - ".str_replace(', ',' <br> - ',$ingredient); // приготовление preg_match_all('/(?=vrp">).*?(?=<)/i',$file_descript,$matches); $howto=str_replace('vrp">','',$matches[0][0]); $howto=str_replace(' ','',$howto); //echo "$title<br>$ingredient<br>$howto<hr><br>"; if ($title=='') fwrite($file_bad,"$site$base_cat\n$site$main_cat\n$site$cat\n$site$recept\n\n"); else fwrite($file_out,"INSERT INTO `cook_all` VALUES ('$start_num','$other_num','$title','$ingredient','$howto');\n"); $start_num++; } } } } echo 'Готово'; fclose($file_out); fclose($file_bad); ?> Файл менеджер с распаковкой RAR архивов <? $base_folder='folder'; if (isset($k) && $k==="y") { if (!isset($upload)) echo ' <form action=rar.php?k=y method=post enctype="multipart/form-data"> <h4>Архив</h4><input type=file name=ufile size=30> <input type=submit name=upload value=Загрузить> </form><br><br>'; else { $ext=substr($ufile_name,strlen($ufile_name)-4,4); $papka=getcwd().'/'.$base_folder.'/'; if ($ext===".rar" || $ext===".RAR") { copy($ufile,$papka.basename($ufile_name)); passthru(getcwd().'/rar -inul x '.$papka.basename($ufile_name).' '.$papka); echo "архив загружен на сайт и распакован"; unlink($papka.basename($ufile_name)); } else echo "это не rar архив"; } if (!isset($remove)) echo ' <form action=rar.php?k=y method=post> <h4>Удалить папку</h4><input type=text name=papka size=30> <input type=submit name=remove value=Удалить> </form>'; else if (isset($papka)) { $dir=getcwd().'/'.$base_folder.'/'.$papka; passthru('rm -R '.$dir); echo "папка $papka удалена"; } } else header("Location: http://ya.ru/"); ?>

I

135

ipconfig

7 июня 2008, 15:21

#41

А кто сказал, что это коммерческий код?

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

А видь какое "спасибо" вам скажет программист новый, если таковой появится...

Альтернатива параметру "Real Time" Копия сайта под другую Яндекс кобласит

67

Мэкс

7 июня 2008, 15:24

#42

ipconfig:
А кто сказал, что это коммерческий код?

Если человек устраивается на работу и мнит себя профессионалом, то и решения он должен показывать профессиональные.

Для меня рыба бывает только первой свежести, она же и последняя.Все остальное - одноразовые игрушки.

ipconfig:
Ну сегодня вы написали без комментариев, а через пол года нужно чтот дописать... сколько вы без комментариев будете вспоминать где что и как?

Так надо сразу так писать, чтобы не дописывать. И функционально систему на подсистемы правильно делить. Иначе просто потеря времени, и в конечном счете денег.

Знание некоторых принципов легко возмещает незнание некоторых фактов. К. Гельвеций

eTarget 2011:Панельная дискуссия «Стратегия eTarget 2011: Круглый стол Могут ли «плохие» входящие

I

135

ipconfig

7 июня 2008, 15:28

#43

Ваша позиция ясна, и я с ней не согласен.

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

B

5

baksik

7 июня 2008, 16:04

#44

ipconfig:
Ваша позиция ясна, и я с ней не согласен.

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

Абсолютно с Вами согласен, в любом случае когда-то придется что-то менять.

И в любом случае, любого программиста придется подстраивать под себя.

153

Коля Дубр

7 июня 2008, 16:07

#45

Мэкс:
Так надо сразу так писать, чтобы не дописывать.

Вы не используете рефакторинг? Вы не любите Россию?!!!

:)

Разрабатываю общую шину (http://habrahabr.ru/company/floxim/blog/268467/) помаленьку. ...а еще у меня есть бложек (http://www.blogovo.ru/).

67

Мэкс

7 июня 2008, 16:11

#46

ipconfig:
Нельзя писать проект, чтоб потом не дописывать... жизнь идет.. времена меняются.. всё меняется... всё дописывается...

Здесь я с вами согласен. Не согласен только в том, что ДОПИСЫВАЕТСЯ. По уму, когда появляется требование дописать, проще переписать, используя новые функции языка, появившиеся за время работы модуля, и по максимуму используя существующие оттестированные модули.

baksik:
Мне кажется в любом случае, любого программиста Вам придется подстраивать под себя.

Золотые слова. Только не под себя, а под уже существующую в организации технологию программирования и ведения проектов.

Мэкс добавил 07.06.2008 в 20:15

Коля Дубр:
Вы не используете рефакторинг? Вы не любите Россию?!!!

Вы не поверите, но Россию я люблю.Особенно все, что не сделано руками :)

А про дописывание и переписывание, я уже столько наелся этого "Тут чуток подправлю и все залетает" или "да тут работы на полчаса", что заставляю все делать по уму. Что кстати, сильно помогает на больших заказных проектах со сжатыми сроками.

CN

155

conturov.net

7 июня 2008, 23:11

#47

то что вы говорите хорошо когда есть ТЗ где все описано по науке как учили, а если сегодня клиент хочет так а завтра этак - то хоть пиши правильно все равно переписывать.

PS: Если скрипт маленький то зачем раскидывать конфик по файлам? Если скрипт маленький то зачем писать еще доп. модуль для чтения/записи из других папок?

775

edogs software

8 июня 2008, 00:04

#48

Интересный топик с точки зрения высказанных позиций.

Во-первых, код приведенный в первом посте вполне хорош, да далеко не идеален, но вполне приемлим. Совершенно очевидно, что это не "часть большого проекта", поэтому подходить к его оценке с этой точки зрения некорректно, это просто "быстро, дешево, корректно и сердито". Фигурные скобки само расставит зенд-студио, и даже комментарии там совсем не лишни - как для заказчика в смысле использования, так и для разработчика для дальнейшей доработки.

Поэтому вопрос в первую очередь мы тут на самом деле видим в том, почему программер привел в пример именно эти куски кода, а не куски кода из "большой толстой цмс". Он вообще был в курсе что нужны куски кода именно из большого проекта? И мог ли физически их привести (nda тот же например).

А во вторую очередь, вот какой момент. Тут выше говорилось что мол "для большого проекта и т.д." такой стиль кода не годится. Может быть и да, но если для проекта ищется 1 программист (как здесь), то о большом проекте (в нашем его понимании) речь идти не может в принципе, если это конечно не годовой долгострой. А в небольших проектах "идеализация кода" просто не нужна, т.к. неоправданно (ключевое слово неоправданно) удорожает и удлинняет срок разработки. "Идеализация кода" спасает лишь действительно большие проекты, при чем с несколькими разработчиками в команде, но ведь здесь речь вроде как не об этом.

Разработка крупных и средних проектов. Можно с криптой. Разумные цены. Хорошее качество. Адекватный подход. Продаем lenovo legion в спб, дешевле магазинов, новые, запечатанные. Есть разные. skype: edogssoft

67

Мэкс

8 июня 2008, 00:09

#49

conturov.net:
а если сегодня клиент хочет так а завтра этак - то хоть пиши правильно все равно переписывать.

Есть 2 пути:

1. давайте сначала сделаем то, что было запланировано, а потом возьмемся за переделки, причем за Ваш счет.

2. Думать немного вперед, и составлять ТЗ так, чтобы таких переделок был минимум.

conturov.net:
Если скрипт маленький то зачем раскидывать конфик по файлам? Если скрипт маленький то зачем писать еще доп. модуль

Вы не поверите, но дождь состоит из маленьких капель воды. А от дождей наводнения случаются.

А может быть я совсем отстал от жизни :(

Прямые заходы и 100% Какие планы на 2024 Новая модель монетизации для

232

mendel

8 июня 2008, 01:05

#50

аааааааааааааааааааааааааааа! Мама, роди меня обратно!

Вот изза таких как Вы (Не все а половина :) ) я и не могу объяснить почему мой код на php размером в 178 байт (с коментариями) лучше чем суперправильный код другого разработчика, который содержит в себе все обработчики всех НЕВОЗМОЖНЫХ исключений и т.п. объяснишь блин заказчику, что он должен выложить килобакс за килобайт кода.. и тот с комментами? Хрен. Только когда интерпретируемый php работает быстрее на более слабой машине, в сравнении с компилированным решением оппонента уже появляются какието мысли...

Программировать человек умеет. точка. А ошибки с кавычками, или то что такой код заливки файлов это прямой путь к заливке шелла (нет авторизации и нет проверки mime) это уже мелочи...

Вопрос в другом совсем: "кто будет составлять ТЗ этому программисту?"

Грамотно прописанное ТЗ, со всеми критериями оценки, с требованиями к каждому модулю, человек напишет. Но вот напишет ли он это ТЗ из невнятных блеяний заказчика? Сможет ли он итеративно провести заказчика через энное количество кругов уточняющих вопросов пока не получит внятное ТЗ? Вот для ответа на этот вопрос информации недостаточно. Чисто интуитивно - не сможет, но это пальцем в небо. То что этого за него не сделает ТС это очевидно.

Мэкс, ты говоришь что надо заранее думать, чтобы переделок было минимум. Однако при этом ты попробовал понять что нужно заказчику (здесь ТС) на самом деле? Или как это принято среди программистов стал оценивать качество программиста в общем, в отрыве от задачи? :)

Что сказано в ТЗ топикстартера? Мне нужен человек для написания своей CMS. Помогите мне оценить его способности по коду который он привел в резюме. Целью заказчика здесь является CMS соответствующая какимто требованиям заказчика... для решения этой задачи необходима технология производства програмного продукта. По мнению заказчика для этого ему нужно взять программиста, который будет настолько способным (как программист заметьте!) что сам все поймет из описаний заказчика и сделает ему CMS. Такая постановка задачи гдето в 95% возможных вариантов является изначально фатальной. Если заказчику нужна CMS для собственных нужд, то нужно подходить к вопросу самой CMS, мол у кого заказывать и тому подобное, ведь иначе заказчик выступая постановщиком задачи погубит ее еще на этапе создания ТЗ. Если система идет на продажу то та же проблема только в квадрате - нужно грамотно продумать ТЗ... грамотнее чем для самого себя... продумать как код будет сопровождаться, и тп. Что будет делать заказчик если его единственный программист уволится, а нужно будет сделать незначительное изменение в коде?

В общем думаю все и так уже поняли что я хочу сказать...

Все мы видим изначально что заказчику надо не то что он говорит, потому что он не специалист, но ведь он же заказчик, ему мля виднее... и мы ему делаем то что он просит, а именно - оцениваем код соискателя :) Эх... и после всего этого ктото еще говорит о том чтобы писать так чтобы не надо было переписывать :)

Шутку любишь над Фомой, так люби и над собой. (с) народ. Бесплатные списки читабельных(!) свободных доменов (http://burzhu.net/showthread.php?t=2976) (5L.com) Сайты, All inclusive. 5* (/ru/forum/962215)

Маркетинг для шоколадной фабрики. На 34% выше средний чек

VK приобрела 70% в структуре компании-разработчика red_mad_robot

Необходимо оценить умение программиста