Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

1C / Говнокод #2804

−131.6

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26

ТаблицаДоговоров = ДоговораМенеджера.Выгрузить();
ТаблицаДоговоров.Свернуть("Договор");

		
ЗапросПоДоговорам = новый Запрос;
ЗапросПоДоговорам.УстановитьПараметр("НаДату", Дата);
ЗапросПоДоговорам.УстановитьПараметр("СписокДоговоров", ТаблицаДоговоров.ВыгрузитьКолонку("Договор"));

ЗапросПоДоговорам.Текст = "ВЫБРАТЬ
								  |	ОтветственныеПоДоговорамСрезПоследних.Договор,
								  |	ОтветственныеПоДоговорамСрезПоследних.Приложение,
								  |	ОтветственныеПоДоговорамСрезПоследних.ВидОтветственного,
								  |	ОтветственныеПоДоговорамСрезПоследних.Ответственный,
								  |	СтатусыДоговоровСрезПоследних.Статус,
								  |	ПриложенияДоговоровСрезПоследних.КонецПриложения
								  |ИЗ
								  |	РегистрСведений.ОтветственныеПоДоговорам.СрезПоследних(&НаДату, Договор В (&СписокДоговоров)) КАК ОтветственныеПоДоговорамСрезПоследних
								  |		ВНУТРЕННЕЕ СОЕДИНЕНИЕ РегистрСведений.СтатусыДоговоров.СрезПоследних(&НаДату, Договор В (&СписокДоговоров)) КАК СтатусыДоговоровСрезПоследних
								  |		ПО ОтветственныеПоДоговорамСрезПоследних.Договор = СтатусыДоговоровСрезПоследних.Договор
								  |			И ОтветственныеПоДоговорамСрезПоследних.Приложение = СтатусыДоговоровСрезПоследних.Приложение
								  |		ВНУТРЕННЕЕ СОЕДИНЕНИЕ РегистрСведений.ПриложенияДоговоров.СрезПоследних КАК ПриложенияДоговоровСрезПоследних
								  |		ПО ОтветственныеПоДоговорамСрезПоследних.Договор = ПриложенияДоговоровСрезПоследних.Договор
								  |			И ОтветственныеПоДоговорамСрезПоследних.Приложение = ПриложенияДоговоровСрезПоследних.Приложение";
								  
		
		ТаблицаДоговоровИСтатусов = ЗапросПоДоговорам.Выполнить().Выгрузить();

так вот пишет наш ведущий программист. Зачем сворачивать???

Sameccc, 16 Марта 2010

Комментарии (8)

PHP / Говнокод #2792

+161

1
2
3
4
5
6
7

$file=file('list_less.dat'); 
 for ($i=0;$i<count($file);$i++)
 {
    $f_abbr=substr($file[$i], 0, strpos($file[$i], '|'));
    $f_name=substr($file[$i], strpos($file[$i], '|')+1, strlen($file[$i]));
    $select.='<option value='.$f_abbr.'>'.$f_name;
  }

почему то
$file=file('list_less.dat');
for ($i=0;$i<count($file);$i++)
{
$expl=explode('|', $file[$i]
$f_abbr=$expl[0];
$f_name=$expl[1]
$select.='<option value='.$f_abbr.'>'.$f_name;
}
мне показалось слишком просто о_О =)(=
Писал вроде трезвый

/*Найдено в одном проекте*/

NFL, 14 Марта 2010

Комментарии (8)

Java / Говнокод #2791

+81.6

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26

public void calculate(){
        jTextField1.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner1.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner13.getValue()))));
        jTextField2.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner2.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner14.getValue()))));
        jTextField3.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner3.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner15.getValue()))));
        jTextField4.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner4.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner16.getValue()))));
        jTextField5.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner5.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner17.getValue()))));
        jTextField6.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner6.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner18.getValue()))));
        jTextField7.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner7.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner19.getValue()))));
        jTextField8.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner8.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner20.getValue()))));
        jTextField9.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner9.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner21.getValue()))));
        jTextField10.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner10.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner22.getValue()))));
        jTextField11.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner11.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner23.getValue()))));
        jTextField12.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner12.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner24.getValue()))));
    }

x_X

ChakuZa, 14 Марта 2010

Комментарии (8)

1C / Говнокод #2771

−126.4

1
2

НСтрока = ДокументОбъект.Товары.Добавить();
НСтрока.ЕдиницаИзмерения = Справочники.ЕдиницыИзмерения.НайтиПоНаименованию("шт",,,НСтрока.Номенклатура);

luns, 12 Марта 2010

Комментарии (8)

PHP / Говнокод #2761

+160.4

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41

<?php
 $ok = true;
        $argsbackup = $args;
        for ($i = 0, $j = count($args); $i < $j; $i++) {
            $arg = array_shift($args);
            $type = array_shift($signature);
            switch ($type) {
                case 'int':
                case 'i4':
                    if (is_array($arg) || !is_int($arg)) {
                        $ok = false;
                    }
                    break;
                case 'base64':
                case 'string':
                    if (!is_string($arg)) {
                        $ok = false;
                    }
                    break;
                case 'boolean':
                    if ($arg !== false && $arg !== true) {
                        $ok = false;
                    }
                    break;
                case 'float':
                case 'double':
                    if (!is_float($arg)) {
                        $ok = false;
                    }
                    break;
                case 'date':
                case 'dateTime.iso8601':
                    if (!is_a($arg, 'IXR_Date')) {
                        $ok = false;
                    }
                    break;
            }
            if (!$ok) {
                return new IXR_Error(-32602, 'server error. invalid method parameters');
            }
        }

предыстория. IRX_Library - XML-RPC server

вот так оно проверяет параметры к функции. так жестко по массиву ходить не стоит... (((

nicolay-punin, 11 Марта 2010

Комментарии (8)

PHP / Говнокод #2741

+154.1

1

$style = ( U_AREA != 'admin' and !defined('U_AREA_MOD') ) ?  'public/templates/' . URegistry::$settings['default_template']['value'] :  'sources/skin_acp/' . SKIN_ACP;

ололо

ololo, 09 Марта 2010

Комментарии (8)

Java / Говнокод #2715

+83.5

1
2
3
4

public void connect(boolean numberOfTransactions)
{
...
}

Че туда передавать???

Nike83, 04 Марта 2010

Комментарии (8)

JavaScript / Говнокод #2705

+167.6

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31

function BOOK_list() {
 document.getElementById('rbLoadedId_list').style.display='block';
 document.getElementById('rbLoadedId_blogs').style.display='none';
 document.getElementById('rbLoadedId_free').style.display='none';
 document.getElementById('rbLoadedId_add').style.display='none';
 document.getElementById('rbLoadedId_friend').style.display='none';
 document.getElementById('rbLoadedId_award').style.display='none';

 document.getElementById('0rbLoadedId_list').className='act';
 document.getElementById('0rbLoadedId_blogs').className='book';
 document.getElementById('0rbLoadedId_free').className='book';
 document.getElementById('0rbLoadedId_add').className='book';
 document.getElementById('0rbLoadedId_friend').className='book';
 document.getElementById('0rbLoadedId_award').className='book';
    }

  function BOOK_blogs() {
 document.getElementById('rbLoadedId_list').style.display='none';
 document.getElementById('rbLoadedId_blogs').style.display='block';
 document.getElementById('rbLoadedId_free').style.display='none';
 document.getElementById('rbLoadedId_add').style.display='none';
 document.getElementById('rbLoadedId_friend').style.display='none';
 document.getElementById('rbLoadedId_award').style.display='none';

 document.getElementById('0rbLoadedId_list').className='book';
 document.getElementById('0rbLoadedId_blogs').className='act';
 document.getElementById('0rbLoadedId_free').className='book';
 document.getElementById('0rbLoadedId_add').className='book';
 document.getElementById('0rbLoadedId_friend').className='book';
 document.getElementById('0rbLoadedId_award').className='book';
    }

JS-табы по-китайски. Взято отсюда — http://vkurse.net.ru/js/info.js
Это только первые 2 функции, в источнике ещё больше

Infamous, 03 Марта 2010

Комментарии (8)

SQL / Говнокод #2697

−169

1
2
3
4

SELECT NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL,
       type_table_of_name_tag_value ()
  FROM DUAL
 WHERE dummy = 'Z'

andrew, 01 Марта 2010

Комментарии (8)

ActionScript / Говнокод #2689

−186.1

1
2
3
4
5
6

var topItem:Object;
var rowNum:int;
var rowCount:int;
...
else if (!topItem && !rowNum == rowCount)
...

Чтобы не утомлять вас догадками, во что же это превратится и в каком порядке произойдут операции: это условие выполниться только когда:
topItem == null
rowNum != 0
rowCount == 0
Как следует из названий переменных, человек, написавший это ожидал, что количество строк может быть меньше, чем порядковый номер одной из строк...
Взято, опять же из Flex Framework mx.controls::Tree.
Скорее всего автор имел в виду следующее:

else if (!topItem && rowNum !== rowCount)

Но булевые переменные, они ж такие коварные :)

wvxvw, 27 Февраля 2010

Комментарии (8)

Говнокод: по колено в коде.