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

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

В номинации:

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

За время:

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

+2

1
2
3

https://tjournal.ru/analysis/128216-moshenniki-3-0-kak-ne-popastsya-na-udochku-novogo-pokoleniya-prestupnikov-v-sfere-it

https://leonardo.osnova.io/0234cd39-a2ef-c6d8-d8df-87df562f9997/-/scale_crop/600x398/center/

booratihno, 30 Ноября 2019

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

Python / Говнокод #26046

+2

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

>>> from heapq import heappush, heappop
>>> heap = []
>>> data = [1, 3, 5, 7, 9, 2, 4, 6, 8, 0]
>>> for item in data:
...     heappush(heap, item)
...
>>> ordered = []
>>> while heap:
...     ordered.append(heappop(heap))
...
>>> ordered
[0, 1, 2, 3, 4, 5, 6, 7, 8, 9]
>>> data.sort()
>>> data == ordered
True

В «Python» есть стандартный модуль «heapq» с процедурками, которые делают из обычного листа очередь с приоритетом: https://docs.python.org/3.8/library/heapq.html. Всё просто, понятно, удобно и без этих ваших «классов» с «наследованиями». Именно поэтому я за «Python».

gost, 26 Ноября 2019

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

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

+2

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

case Type::Instruction:
        {
          if (o == "movzwl") return OpCode::movzwl;
          if (o == "movzbl") return OpCode::movzbl;
          if (o == "shrb") return OpCode::shrb;
          if (o == "shrl") return OpCode::shrl;
          if (o == "xorl") return OpCode::xorl;
          if (o == "andl") return OpCode::andl;
          if (o == "ret") return OpCode::ret;
          if (o == "movb") return OpCode::movb;
          if (o == "cmpb") return OpCode::cmpb;
          if (o == "movl") return OpCode::movl;
          if (o == "jmp") return OpCode::jmp;
          if (o == "testb") return OpCode::testb;
          if (o == "incl") return OpCode::incl;
          if (o == "sarl") return OpCode::sarl;
          if (o == "decl") return OpCode::decl;
          if (o == "jne") return OpCode::jne;
          if (o == "je") return OpCode::je;
          if (o == "js") return OpCode::js;
          if (o == "subl") return OpCode::subl;
          if (o == "subb") return OpCode::subb;
          if (o == "addl") return OpCode::addl;
          if (o == "addb") return OpCode::addb;
          if (o == "sall") return OpCode::sall;
          if (o == "orl") return OpCode::orl;
          if (o == "andb") return OpCode::andb;
          if (o == "orb") return OpCode::orb;
          if (o == "decb") return OpCode::decb;
          if (o == "incb") return OpCode::incb;
          if (o == "rep") return OpCode::rep;
          if (o == "notb") return OpCode::notb;
          if (o == "negb") return OpCode::negb;
          if (o == "sbbb") return OpCode::sbbb;
          if (o == "pushl") return OpCode::pushl;
          if (o == "retl") return OpCode::retl;
        }

Фрагмент транслятора из ассемблера в ассемблер.

https://github.com/lefticus/x86-to-6502

XYPO3BO3, 24 Ноября 2019

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

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

+2

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

public class AdvancedBoolean {
    private boolean result;
    private DefenitionDictionary.RESULT_CODE code;
    
    
    
//------------------------------------------------------------------------------  
//constructors
//------------------------------------------------------------------------------       
    public AdvancedBoolean() {
        result = true;
    }
        
    public AdvancedBoolean(boolean _result) {
        result = _result;
    }
 
     
    public AdvancedBoolean(boolean _result, DefenitionDictionary.RESULT_CODE _code) {
        result = _result;
        code = _code;
    }

    
    
//------------------------------------------------------------------------------
//get methods
//------------------------------------------------------------------------------     
    public boolean isResult() {
        return result;
    }

    public DefenitionDictionary.RESULT_CODE getCode() {
        return code;
    }


        
//------------------------------------------------------------------------------
//set methods
//------------------------------------------------------------------------------ 
    public void setResult(boolean _result) {
        result = _result;
    }
    
    public void setCode(DefenitionDictionary.RESULT_CODE _code) {
        code = _code;
    }
    
    
    
//------------------------------------------------------------------------------
//other methods
//------------------------------------------------------------------------------     
    public String getLabel() {
        return code.getResultLabel();
    }
}

Без комментариев

Veers, 19 Ноября 2019

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

Си / Говнокод #26034

+2

1
2

Comparing structs with, let's say, memcmp, does not work,
 as you end up comparing the "unspecified" padding bytes as well — you must compare member-by-member.

While writing this post, the author observed that some verions of GCC (experimentally, >= 4.7, < 8.0) do not zero padding if an empty intializer list is passed, under certain a certain code pattern; if an entire struct (i.e. sizeof(STRUCTNAME)) is subsequently memcpy'd after assigment of its members, and this intermediate buffer is what is used by the code going forward. This appears to be based on how optimization passes interact with GCC's built-in memcpy, since passing -fno-builtin-memcpy returns the behavior to the expected.
https://www.nccgroup.trust/us/about-us/newsroom-and-events/blog/2019/october/padding-the-struct-how-a-compiler-optimization-can-disclose-stack-memory/

3.14159265, 18 Ноября 2019

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

Си / Говнокод #26027

+2

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

#include <stdio.h>
#include <inttypes.h>
#include <string.h>

// endian dependend
#define PUT3(a,b,c) (((uint64_t)a<<8*0) | ((uint64_t)b<<8*1) | ((uint64_t)c<<8*2))

void testswitch(uint64_t x)
{
  switch (x) 
  { 
    case PUT3('a','b','c'): printf("abc\n"); 
      break; 
    case PUT3('d','e','f'): printf("def\n"); 
      break; 
    case PUT3('g','h','i'): printf("ghi\n"); 
      break; 
    default: printf("Choice other than abc, def and ghi\n"); 
      break;   
   }
}

int main() 
{ 
   uint64_t x = 0;
   char a[] = "abc";
   memcpy(&x, a, sizeof(a)-1);
   testswitch(x);

   char b[] = "def";
   memcpy(&x, b, sizeof(a)-1);
   testswitch(x);

   char c[] = "ghi";
   memcpy(&x, c, sizeof(a)-1);
   testswitch(x);
   return 0; 
}

switch для строк!

Перечитывал несвежие говнокоды, где я выкладывал творчество вконтактоолимпиадников https://govnokod.ru/23170#comment388376

j123123, 12 Ноября 2019

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

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

+2

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18

Дали тестовое задание.
Написать хеш таблицу с открытой адресацией. Главное качество, а не скорость, сказали.
Реализовал на C++.
Самая тягомотина - это тестирование многопоточности.
Есть операции: 
find, equalRange, operator [] - тот же equalRange, но возвращает все найденные вхождения, insert, remove, extend, size, capacity.

И вот надо каждую относительно другой проверять.  
Только это выходит (9 - 1)^2 тестов. Помимо остальных, уже реализованных.

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

Сойдет ли просто проверять время блокировки (shared_lock, и lock_guard) 
каждого параллельного текущему потоку относительно времени блокировки\старта\окончания блокировки текущего?
кроме связки (access - insert)?
Тем более, что операции записи\чтения проверяются в немногопоточном исполнении?

OlegUP, 08 Ноября 2019

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

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

+2

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

https://habr.com/ru/post/472970/

Критика протокола и оргподходов Telegram. Часть 1, техническая: опыт написания клиента с нуля

в Full самая наркомания, с точки зрения сетевика: длина,
sequence number, причем НЕ ТОТ, что в основном MTProto,
тело, CRC32. Да, всё это поверх TCP. Который предоставляет
нам надежный транспорт в виде последовательного потока байт,
никакие последовательности не нужны, тем более контрольные
суммы. Окей, мне сейчас возразят, что в TCP 16-битная
контрольная сумма, так что искажение данных случается.
Отлично, только у нас вообще-то криптографический протокол
с хэшами длиннее 16 байт, все эти ошибки — и даже
более — будут отловлены на несовпадении SHA уровнем выше.
Никакого смысла в CRC32 поверх этого — НЕТ.

Ебать дебилы

j123123, 27 Октября 2019

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

Си / Говнокод #25997

+2

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

// https://godbolt.org/z/dMT7v3

unsigned div_eq(unsigned a, unsigned b)
{
  ALWAYS_TRUE(a == b);
  return a/b;
}

unsigned div(unsigned a, unsigned b)
{
  return a/b;
}


int test_array(unsigned char a[10])
{
  for (int i = 1; i < 10; i++)
  {
    ALWAYS_TRUE(a[i-1] <= a[i]);
  }
  return a[0] <= a[2];
}

Refinement type
Можно этой хуйней ассерты позаменять попробовать, и компилятор возможно что-то сможет за счет этого соптимизировать

j123123, 26 Октября 2019

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

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

+2

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

$jsxss="onabort,oncanplay,oncanplaythrough,ondurationchange,onemptied,onended,onerror,onloadeddata,onloadedmetadata,onloadstart,onpause,onplay,onplaying,onprogress,onratechange,onseeked,onseeking,onstalled,onsuspend,ontimeupdate,onvolumechange,onwaiting,oncopy,oncut,onpaste,ondrag,ondragend,ondragenter,ondragleave,ondragover,ondragstart,ondrop,onblur,onfocus,onfocusin,onfocusout,onchange,oninput,oninvalid,onreset,onsearch,onselect,onsubmit,onabort,onbeforeunload,onerror,onhashchange,onload,onpageshow,onpagehide,onresize,onscroll,onunload,onkeydown,onkeypress,onkeyup,altKey,ctrlKey,shiftKey,metaKey,key,keyCode,which,charCode,location,onclick,ondblclick,oncontextmenu,onmouseover,onmouseenter,onmouseout,onmouseleave,onmouseup,onmousemove,onwheel,altKey,ctrlKey,shiftKey,metaKey,button,buttons,which,clientX,clientY,detail,relatedTarget,screenX,screenY,deltaX,deltaY,deltaZ,deltaMode,animationstart,animationend,animationiteration,animationName,elapsedTime,propertyName,elapsedTime,transitionend,onerror,onmessage,onopen,ononline,onoffline,onstorage,onshow,ontoggle,onpopstate,ontouchstart,ontouchmove,ontouchend,ontouchcancel,persisted,javascript";
$jsxss = explode(",",$jsxss);
foreach($_GET as $k=>$v)
{
	if(is_array($v))
	{
		foreach($v as $Kk=>$Vv)
		{
			$Vv = preg_replace ( "'<script[^>]*?>.*?</script>'si", "", $Vv );
			$Vv = str_replace($jsxss,"",$Vv);
			$Vv = str_replace (array("*","\\"), "", $Vv );
			$Vv = strip_tags($Vv);
			$Vv = htmlentities($Vv, ENT_QUOTES, "UTF-8");
			$Vv = htmlspecialchars($Vv, ENT_QUOTES);
			$_GET[$k][$Kk] = $Vv;
		}
	}
	ELSE
	{
		//Сначала удаляем любые скрипты для защиты от xss-атак
		$v = preg_replace ( "'<script[^>]*?>.*?</script>'si", "", $v );
		//Вырезаем все известные javascript события для защиты от xss-атак
		$v = str_replace($jsxss,"",$v);
		//Удаляем экранирование для защиты от SQL-инъекций
		$v = str_replace (array("*","\\"), "", $v );
		//Экранируем специальные символы в строках для использования в выражениях SQL
		$v = mysql_real_escape_string( $v );
		//Удаляем другие лишние теги.	
		$v = strip_tags($v);
		//Преобразуем все возможные символы в соответствующие HTML-сущности
		$v = htmlentities($v, ENT_QUOTES, "UTF-8");
		$v = htmlspecialchars($v, ENT_QUOTES);
		//Перезаписываем GET массив
		$_GET[$k] = $v;
	}
	
}

[colo=blue]https://habr.com/ru/post/470193/[/color]

Интересно, все эти люди сами до всего этого доходят, или их кто-то этому учит?

gpyrou_nemyx, 05 Октября 2019

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

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