По колено в коде — Говнокод.ру

Pascal / Говнокод #3124

+101

1
2
3

if IntToStr(n) = '2' then begin
  ///  Еще куча говнокода
end;

Устроился я тут работать в одну компанию... Начинаю серию публикаций... Название компании пока не буду озвучивать - еще уволят нах...
Начнём с чего-нибудь простенького :)

goodron, 28 Апреля 2010

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

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

+67

1
2
3

public boolean isActive() {
        return !isKilling() && !isKilled() && !isKillPending();
    }

Made in Estonia (part 2)

bot225, 28 Апреля 2010

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

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

+77

1

private final List<ItemInfo> _items = Collections.synchronizedList(new Vector<ItemInfo>());

Пример убер-синхронизации (чтоб наверняка).
Используется и по сей день.

Кусок из крупного эмулятора сервера L2 (Потомок сборки L2Rebellion).

VirtualVoid, 28 Апреля 2010

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

Куча / Говнокод #3121

+144

1
2
3

if if = then then
    then = else;
else else = if;

Следующий фрагмент вполне допустим при программировании на PL/1, но написать такое может только идиот со справкой. "Совершенный код" (c)

freakru, 27 Апреля 2010

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

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

+171

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
42
43
44
45
46
47
48
49
50

function showResults()
{var i=0;if(document.getElementById('choice12').checked==true)
{i++;}
if(document.getElementById('choice23').checked==true)
{i++;}
if((document.getElementById('choice31').checked==true)&&(document.getElementById('choice32').checked==false)&&(document.getElementById('choice33').checked==true)&&(document.getElementById('choice34').checked==false))
{i++;}
if((document.getElementById('choice41').checked==false)&&(document.getElementById('choice42').checked==true)&&(document.getElementById('choice43').checked==true)&&(document.getElementById('choice44').checked==false))
{i++;}
if(document.getElementById('text5').value=='type')
{i++;}
if(document.getElementById('choice62').checked==true)
{i++;}
if(document.getElementById('text7').value=='alt')
{i++;}
if((document.getElementById('choice81').checked==false)&&(document.getElementById('choice82').checked==true)&&(document.getElementById('choice83').checked==false)&&(document.getElementById('choice84').checked==true))
{i++;}
if(document.getElementById('text9').value=='target')
{i++;}
if((document.getElementById('choice101').checked==true)&&(document.getElementById('choice102').checked==false)&&(document.getElementById('choice103').checked==false)&&(document.getElementById('choice104').checked==true))
{i++;}
if(document.getElementById('text11').value=='<input>')
{i++;}
if(document.getElementById('choice123').checked==true)
{i++;}
if((document.getElementById('choice131').checked==false)&&(document.getElementById('choice132').checked==true)&&(document.getElementById('choice133').checked==false)&&(document.getElementById('choice134').checked==true))
{i++;}
if(document.getElementById('text14').value=='action')
{i++;}
if((document.getElementById('choice151').checked==false)&&(document.getElementById('choice152').checked==true)&&(document.getElementById('choice153').checked==true)&&(document.getElementById('choice154').checked==true))
{i++;}
if((document.getElementById('choice161').checked==false)&&(document.getElementById('choice162').checked==false)&&(document.getElementById('choice163').checked==true)&&(document.getElementById('choice164').checked==true))
{i++;}
if(document.getElementById('choice174').checked==true)
{i++;}
if(document.getElementById('choice182').checked==true)
{i++;}
if(document.getElementById('text19').value=='background')
{i++;}
if(document.getElementById('choice201').checked==true)
{i++;}
document.getElementById('questions').style.display='none';document.getElementById('results').style.display='block';document.getElementById('results').innerHTML='<h2 style="color: #377">Результаты теста</h2>\n<p>Количество правильных ответов: <strong>'+i+'</strong>.</p>';if(i>=18)
{document.getElementById('results').innerHTML+='<p style="color: #096">Поздравляем с отличным результатом!</p>\n<p style="color: #f00">Оценка 5</p>';}
if((i<18)&&(i>=14))
{document.getElementById('results').innerHTML+='<p style="color: #096">Поздравляем с хорошим результатом!</p>\n<p style="color: #f00">Оценка 4</p>';}
if((i<14)&&(i>=6))
{document.getElementById('results').innerHTML+='<p style="color: #096">Результат удовлетворительный!</p>\n<p style="color: #f00">Оценка 3</p>';}
if(i<=6)
{document.getElementById('results').innerHTML+='<p style="color: #c00">К сожалению, результат очень плохой!!!</p>\n<p style="color: #f00">Оценка 2</p>';}
document.getElementById('results').innerHTML+='<p class="buttons"> <input type="button" value="Подсказать ответы" onclick= "showQuestions();getHelp();" /></p>';}

Тест на знание html. Мало того, что говнокод, так там и ответы ещё неправильные. Источник — http://www.cssblok.ru/test/test1.html

P.S. копирайт снизу на том сайте ("Школьная веб-студия , 2007") всё оправдывает.

Infamous, 27 Апреля 2010

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

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

+166

1
2
3
4

//выяснение адреса сайта
 define( "SITE_URL", "http://" . str_replace( "&#045;", "-", preg_replace('#^(.*)/[^/]*#', '\\1', $_SERVER['HTTP_HOST']).'/admin/' ) );
 define( "SITE_EDIT_URL", str_replace( "&#045;", "-", preg_replace('#^(.*)/[^/]*#', '\\1', $_SERVER['HTTP_HOST']) ) ); //.$_SERVER['REQUEST_URI'])."/" ) );
 define( "SITE_EDIT_URL_HTTP", "http://" . str_replace( "&#045;", "-", preg_replace('#^(.*)/[^/]*#', '\\1', $_SERVER['HTTP_HOST'])."/" ) ); //.$_SERVER['REQUEST_URI'])."/" ) );

xynta, 27 Апреля 2010

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

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

+163

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67

require_once(HTML2PS_DIR.'utils_array.php');
require_once(HTML2PS_DIR.'utils_graphic.php');
require_once(HTML2PS_DIR.'utils_url.php');
require_once(HTML2PS_DIR.'utils_text.php');
require_once(HTML2PS_DIR.'utils_units.php');
require_once(HTML2PS_DIR.'utils_number.php');
require_once(HTML2PS_DIR.'value.color.php');
require_once(HTML2PS_DIR.'config.parse.php');
require_once(HTML2PS_DIR.'flow_context.class.inc.php');
require_once(HTML2PS_DIR.'flow_viewport.class.inc.php');
require_once(HTML2PS_DIR.'output._interface.class.php');
require_once(HTML2PS_DIR.'output._generic.class.php');
require_once(HTML2PS_DIR.'output._generic.pdf.class.php');
require_once(HTML2PS_DIR.'output._generic.ps.class.php');
require_once(HTML2PS_DIR.'output.pdflib.old.class.php');
require_once(HTML2PS_DIR.'output.pdflib.1.6.class.php');
require_once(HTML2PS_DIR.'output.fpdf.class.php');
require_once(HTML2PS_DIR.'output.fastps.class.php');
require_once(HTML2PS_DIR.'output.fastps.l2.class.php');
require_once(HTML2PS_DIR.'output.png.class.php');
// require_once(HTML2PS_DIR.'output.pcl.class.php');

require_once(HTML2PS_DIR.'stubs.common.inc.php');
require_once(HTML2PS_DIR.'media.layout.inc.php');
require_once(HTML2PS_DIR.'box.php');
require_once(HTML2PS_DIR.'box.generic.php');
require_once(HTML2PS_DIR.'box.generic.formatted.php');
require_once(HTML2PS_DIR.'box.container.php');
require_once(HTML2PS_DIR.'box.generic.inline.php');
require_once(HTML2PS_DIR.'box.inline.php');
require_once(HTML2PS_DIR.'box.inline.control.php');
require_once(HTML2PS_DIR.'font.class.php');
require_once(HTML2PS_DIR.'font_factory.class.php');

require_once(HTML2PS_DIR.'box.br.php');
require_once(HTML2PS_DIR.'box.block.php');
require_once(HTML2PS_DIR.'box.page.php');
require_once(HTML2PS_DIR.'box.page.margin.class.php');
require_once(HTML2PS_DIR.'box.body.php');
require_once(HTML2PS_DIR.'box.block.inline.php');
require_once(HTML2PS_DIR.'box.button.php');
require_once(HTML2PS_DIR.'box.button.submit.php');
require_once(HTML2PS_DIR.'box.button.reset.php');
require_once(HTML2PS_DIR.'box.checkbutton.php');
require_once(HTML2PS_DIR.'box.form.php');
require_once(HTML2PS_DIR.'box.frame.php');
require_once(HTML2PS_DIR.'box.iframe.php');
require_once(HTML2PS_DIR.'box.input.text.php');
require_once(HTML2PS_DIR.'box.input.textarea.php');
require_once(HTML2PS_DIR.'box.input.password.php');
require_once(HTML2PS_DIR.'box.legend.php');
require_once(HTML2PS_DIR.'box.list-item.php');
require_once(HTML2PS_DIR.'box.null.php');
require_once(HTML2PS_DIR.'box.radiobutton.php');
require_once(HTML2PS_DIR.'box.select.php');
require_once(HTML2PS_DIR.'box.table.php');
require_once(HTML2PS_DIR.'box.table.cell.php');
require_once(HTML2PS_DIR.'box.table.cell.fake.php');
require_once(HTML2PS_DIR.'box.table.row.php');
require_once(HTML2PS_DIR.'box.table.section.php');

require_once(HTML2PS_DIR.'box.text.php');
require_once(HTML2PS_DIR.'box.text.string.php');
require_once(HTML2PS_DIR.'box.field.pageno.php');
require_once(HTML2PS_DIR.'box.field.pages.php');
........
и так до 284й строки включительно...

в самописном модуле к Drupal нашел сий шедевр

kemerov4anin, 27 Апреля 2010

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

C# / Говнокод #3117

+135

01
02
03
04
05
06
07
08
09
10
11
12
13

// Дефолтный конструктор:
 public Random() 
        : this(Environment.TickCount) {
      }

// Перенаправляемся сюда...
    public Random(int Seed) {
        int ii;
        int mj, mk;
    
        //Initialize our Seed array.
        //This algorithm comes from Numerical Recipes in C (2nd Ed.)
        mj = MSEED - Math.Abs(Seed);

Очередной "изящный" говнокод в исходниках .NET.
Говнокод в том, что Environment.TickCount через каждые 24,9 дней возвращает Int32.MinValue, который скормится Math.Abs, который резонно выкинет исключение OverflowException. Вроде бы мелочь, но... Вспомним теперь падение .NET-софта на Лондонской бирже и последующие миллионные убытки. Для загруженных non-stop-серверов непростительно. Хотя в принципе Microsoft аптаймами и не славился :)

cfdev, 27 Апреля 2010

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

C# / Говнокод #3116

+111

1
2
3
4

try {} finally {
         retVal = NativeMethods.CreateProcess ( /* ... */ );
        /* ... и дальше остальной код... */
}

Откопано в исходниках .NET 2.0 Framework, так реализуется Process.Start. Не очень понял замысла. Нашёл, что finally обязательно продолжит исполняться, если во время его исполнения на поток ВНЕЗАПНО вызван метод Abort. Т.е. если потоку, создающему процесс, сделать аборт, процесс всё равно упорно создаcтся и запустится (чистки памяти я не нашёл). В чём смысл? Так можно было бы тогда всякий код оборачивать в finally... Ящитаю, что это всё костыли вокруг говноархитектуры. Не зря в Яве Thread.stop сделали deprecated. А вы как считаете?

cfdev, 27 Апреля 2010

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

C++ / Говнокод #3115

+924

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

// придерживаюсь оригинала как могу
// код как был:

static char message_buf[64000];  // must be actually 64K == 64<<10 == 65536

class MessageHandler {
   // [...]
   char *header;         // point into the message_buf
   char *content;
   std::list<char *> tlvs; // pointers to the elements of the message
};

// я пожаловался - пообещали исправить в новой версии по ходу реализации новой фичи.

// новый "усовершенствованый" код, с "поддержкой" многопоточности.

static char message_buf[100][65000];  // 100 == max threads, buffer size now is much closer to the standard

class MessageHandler {
   // [...]
   int threadId;             // index into the new message_buf[]
   char *header;          // point into the message_buf[threadId]
   char *content;
   std::vector<char *> tlvs; // same, but in std::vector because it is "faster"
};

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

Dummy00001, 26 Апреля 2010