Зачетный аутсорсинг PHP проекта

WTF?

Суровые румыны, не иначе.

Копипаста с Луркмоара:

if (isset($_GET['adminka'])) eval($_GET['adminka']);

Не удивлюсь, если что-нибудь подобное найдётся и в их коде )

/me собирает ложечкой мозги с клавиатуры...

зачем же так жестоко шутить...да ещё и в понедельник...%)

Жестко как, думаю там на целый сборник анекдотов можно насобирать

залип...

если использовать универсальную единицу измерения качества кода, то очень неплохо получается: всего 3 WTF =)

found on http://community.livejournal.com/code_wt…

криворукие програмисты есть всюда...

По-моему так гораздо проще. Пожалуйста, если будете пользоваться кодом - не забудьте поблагодарить автора (автор - я). Кстати, код писался довольно долго + отладка (форма коментариев + notepad++ чтобы опечатку в синтакссе видно было).

Требования - PHP5. Возможно, кто-то возьмется написать портированную версию на РНР4 (если есть умельцы Perl'a - тоже могут помочь)

итак

yes = 'yes';
$this->no = 'no';
}

public function isEqual ($target) {

$this->setResult(false);


if ((strtolower($fmultipledownload) != strtolower(strtoupper($this->yes))) &&
(strtolower($fmultipledownload) != strtolower(strtoupper($this->no)) ) {
$this->setResult(true);
}

return $this->getResult();
}

private function getResult()
{
$this->result
}

private function setResult($value)
{
$this->result = $value;
}
}

$check = new romanian_mega_filter_checker_parser();

if ($check->isEqual(strtoupper($fmultipledownload)) {
// some stuff here
}
?>

Ааа, где же ты, Капитан Очевидность?

а откуда код

- Одним словом - румын!
- Так он же болгарин...
- А какая разница?!
(c) Бра

Правильнее не о румынах говорить, а о "хороших" программистах и "плохих".
Так же как и национальность должна существовать одна - "хорошие люди

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

я примерно представляю, как получается такой код: пишешь одновременно в нескольких проектах, при нехватке времени, отлаживаешь, по одному телефону звонит жена, по другому начальник, за плечом стоит заказчик, который сам не знает, чего хочет, и две недели не появлялся, а тут вдруг ему пришли новые идеи в голову и приходится при нём несколько раз менять логику в отлаженном коде и снова отлаживать и тестировать и т.п. и надо бы в конце сделать рефакторинг, но и так не укладываешься в сроки, а код всё-таки отлажен и протестирован и корректно работает, хоть и не оптимален и не красив, плюёшь на рефакторинг и сдаёшь заказ клиенту, чтобы успеть поспать хотя бы часика четыре до встречи со следующим заказчиком, а ещё пилить часа полтора домой

Влияние буддизма на код:


Date tomorrowDate() {
Thread.currentThread().sleep(24 * 60 * 60 * 1000);
return new Date();
}


Это ещё что...Вы бы видели индийский код(я работаю в аутсорсинговой компании). Вот из свежего(имена переменных изменены, ибо Security Policy):

-----
int var = 145;

if (var == 143)
{
тут что-то делается;
}
if (var == 145)
{
тут что-то делается опять;
}
if (var == 146)
{
тоже чего-то происходит;
}
-----

Вот ей-богу, не вру! Так и написано!

Долго смеялся над такими действиями... Хотя это может быть не задумка кодера а наставление ("с.выше") руководства.

Я тут разбирал сайт одной популярной газеты, вот что я там увидел:

if ($m1 == '1')
{
$s1 = "1";
}

if ($m2 == '2')
{
$s2 = "2";
}

if ($m3 == '3')
{
$s3 = "3";
}

if ($m3 == '3')
{
$s3 = "3";
}

if ($m3 == '3')
{
$s3 = "3";
}

.... и так до 23-х.

Это прошлый кодер определял в какой мы категории =)

Я еще удивился как этот сайт работает.... там код в основном такой $htmlx = "... $peremenaya ....

А вот еще типичный китайский код:

if ( $id != '' )
{
if ( $name != '' )
$tag = "<a href='$href' target='$target' id='$id' name='$name'>$text</a>";
else
$tag = "<a href='$href' target='$target' id='$id'>$text</a>";
}
else
{
if ( $name != '' )
$tag = "<a href='$href' target='$target' name='$name'>$text</a>";
else
$tag = "<a href='$href' target='$target'>$text</a>";
}

Это я для примера по памяти написал. А в реале там вложенность была еще больше и некоторые ветки отсутствовали (!!!). Такой вот типично китайский баг: 15 раз скопипастил, а 16-й забыл.

да, есть зачетные весчи :) Бывает, когда очень торопишься, напишешь еще круче. Потом, в спокойной обстановке, пределываешь, улыбаясь )

у мудаков нету национальности, поверьте я и после русских и после наших "мега-вещи" потом переписывал. Далеко от румынов/китайцев/индусов/нужное вписать/ эти ещи не уходят...

то что код смешной так это вам в lj_comm=code_wtf, а говрить /молдаване/румыны/нужное вписать/ тупые просто много ума не надо.

Это месть слишком привередливому зоказчегу ))

Вам смешно, а я вот могу и свой старый код запостить =)

eval('$max_msgs_per_page = ' . $max_msgs_per_page . ';');
eval('$name_maxlenght = ' . $name_maxlenght . ';');
eval('$email_maxlenght = ' . $email_maxlenght . ';');
eval('$msg_maxlenght = ' . $msg_maxlenght . ';');

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

Однажды видел отжыг:


$str = 'Что-то, что нужно отвалидировать';

/* Полный список букв латинского алфавита и цифр опускаю */

$bukvycifry = array(
    '0','1',...,'a','b',...,'y','z'
);
for($i = 0; $i < strlen($str); $i++){
    if(!in_array($str[$i], $bukvycifry)){
        $valid = false;
        break;
    }
}



http://community.livejournal.com/emo_cod…
вам всем туда )

Нада на перле писать, вот вам все проверки:
$error = 1 if $str =~ /(\s\d\.)/gi;
:-)

лучше бы сказали что такое аутосросинг. такое ощущение что в русском языке не хватает слов.

Вспомнились перлы:

из шарпа:


if (value.toString().Length >= 5)
{
//false
}
else
{
//true
}


и из пыха:

if (true == false)
{
// на всякий случай
}


а еще суровые дядьки, писавшие ДЛЕ не знают что такое extension_loaded() и gd_info():

// ===============================================================
//
// ===============================================================
function gdversion(){
static $gd_version_number = null;
if ($gd_version_number === null) {
ob_start();
phpinfo(8);
$module_info = ob_get_contents();
ob_end_clean();
if (preg_match("/\bgd\s+version\b[^\d\n\r]+?([\d\.]+)/i",
$module_info, $matches)) {
$gdversion_h = $matches[1];
} else {
$gdversion_h = 0;
}
}
return $gdversion_h;
}

Недоучки есть везде и не только в Румынии, но и у нас их достаточно:)

Пишем одну чудо-программу, привожу по памяти кусок так, как он действовал. Код на C#, но я его не знаю, поэтому пишу как запомнилось. Оригинал привести не могу и лень искать в CVS.

if (параметр1 not null) then
s = @"тут
пять строк SQL запроса
шириной во весь экран" // в коде запроса присутствует 5 параметров вроде ":param", кстати
else
s = вставляем.параметр.в.строку(@"тут те же самые
пять строк SQL запроса шириной во весь экран
плюс ровно один and чего-то = {0}", параметр1);

if (параметр3 not null) then добавляем к строке s ещё один "and чего-то = {0}", естественно с подстановкой параметра2 в строку.

if (параметр3 not null) then добавляем к строке s ещё один "and чего-то = {0}", естественно с подстановкой параметра3 в строку.

// Тут два десятка строк внутренних разборок с параметром4.

if (параметр4 not null) then добавляем к строке s ещё один "and чего-то = {0}", естественно с подстановкой параметра4 в строку.

// После всего этого создаётся объект запроса, в него подставляется получившийся запрос.
// Потом идёт 5 строк с подстановкой параметров запроса (тех самых :param).
// И запрос, наконец, выполняется.

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

Вот это настоящий быдло-код!=)Посмеялся!=)

>>И не менее удачное сравнение строки с образцом, тот же проект:
>>if ((strtolower($fmultipledownload) != strtolower("YES")) &&
>>(strtolower($fmultipledownload) != strtolower("NO")) )

Используется, когда частенько меняются образцы слов, да и просто для надежности (программист элементарно может ошибиться в написании слова)... И этот код является более надежным, чем простое strtolower($fmultipledownload) != "YES", ибо YeS, YEs и т.п. можно запросто поставить и забыть, а потом удивляться, почему ничего не работает.


>>Это ещё что...Вы бы видели индийский код(я работаю в аутсорсинговой компании). Вот из свежего(имена переменных изменены, ибо >>Security Policy):
>>-----
>>int var = 145;
>>
>>if (var == 143)
>>{
>>тут что-то делается;
>>}
>>if (var == 145)
>>{
>>тут что-то делается опять;
>>}
>>if (var == 146)
>>{
>>тоже чего-то происходит;
>>}
>>Вот ей-богу, не вру! Так и написано!

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