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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5706)
2. C++ (2739)
3. Куча (2371)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (586)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Haskell (29)
22. Swift (27)
23. Kotlin (14)

1. PHP / Говнокод #7984

   +164

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7
   8. 8

   $rsAnswers = CFormAnswer::GetList( $QUESTION_ID, $by="s_id", $order="desc", $arFilter, $is_filtered ); 
   $z = 0; 
   while ($arAnswer = $rsAnswers->Fetch()) 
   { 
   	$z = 1; 
   } 
   
   if ($z == 1) {

   Когда там накопилось 100500 элементов сайт ВНЕЗАПНО начал лагать...

   Tairesh, 26 Сентября 2011

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

2. Java / Говнокод #7983

   +80

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28
   29. 29

   if (k <= 1)
       		{
       			if ((s.equals("2"))||(s.equals("3"))||(s.equals("4")))
       			{
       				txtCommonPeople.setText("ЧЕЛОВЕКА");
       			}
       			else
       			{
       				txtCommonPeople.setText("ЧЕЛОВЕК");
       			}
       		}
       		else
       		{
       				if (s.charAt(k-2) != 1) 
       				{
       					if (((s.charAt(k-1) == 2) || (s.charAt(k-1) == 3) || (s.charAt(k-1) == 4)))
       					{
       						txtCommonPeople.setText("ЧЕЛОВЕКА");
       					}
       					else
       					{
       						txtCommonPeople.setText("ЧЕЛОВЕК");
       					}
       				}
       				else
       				{
       					txtCommonPeople.setText("ЧЕЛОВЕК");
       				}
       		}

   chaoswithin, 26 Сентября 2011

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

3. C# / Говнокод #7982

   +128

   1. 1
   2. 2
   3. 3
   4. 4

   XmlNodeList list = xdoc.GetElementsByTagName("Customer");
   for (int i = 0; i < list.Count; i++)
   {
   XmlElement cl = (XmlElement)xdoc.GetElementsByTagName("Customer")[i];

   Написано расовым индусом.
   http://www.c-sharpcorner.com/UploadFile/vimalkandasamy/Xml03032009055848AM/Xml.aspx - чисто индусский бложик. Там много подобного.

   koodeer, 26 Сентября 2011

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

4. PHP / Говнокод #7981

   +154

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7
   8. 8

   <?php
   $temp='Исходная строка';
   $search='Поисковый запрос';
   $result=str_ireplace($search, $temp);
   if ($result!=$temp)return true;
   else
   return false;
   ?>

   Вот так я проверял на наличие подстроки в строке! *ROFL*

   QarezZ, 25 Сентября 2011

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

5. Java / Говнокод #7980

   +146

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13

   public class HelloWorld {
   
   	public static void main(String[] args) {
   		HelloWorld hw = new HelloWorld();;;;;;;;;;;;;;;;;;;;;;;;;
   		hw.doHw();
   	}
   	
   	public void doHw()
   	{
   		System.out.println("Hello, world");
   	}
   
   }

   Не совсем понимаю что это, но javac HelloWorld.java не выдал никаких ошибок. Возможно кто-то про это писал но все же..

   redenemy, 25 Сентября 2011

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

6. Си / Говнокод #7979

   +139

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14

   int x = 0;
   	for (i = 4; i <= 31; i++ && x++)
   		if (code.l & (1 << i))
   		  {
   		    if (x == 0)
   		      {
   			info.bios_info->characteristics = (char**) malloc (sizeof (char*));
   			info.bios_info->characteristics[x] = characteristics[i-3];
   		      }else
   		      {
   		        realloc (info.bios_info->characteristics, (sizeof(char*) * x));
   			info.bios_info->characteristics[x] = characteristics[i - 3];
   		      }
   		  }

   realloc и массив указателей типа char*
   
   http://www.linux.org.ru/forum/development/6786393

   sayidandrtfm, 25 Сентября 2011

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

7. C++ / Говнокод #7978

   +151

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6

   void f_ai(intrusive_ptr<serial> i);
   //...
   static serial ai;
   serial* ii=&ai;
   //...
   f_ai(ii);

   Компилируется, а потом грохается. С++ такой Си...

   CKrestKrestGovno, 25 Сентября 2011

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

8. Pascal / Говнокод #7977

   +102

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28
   29. 29
   30. 30
   31. 31
   32. 32
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62
   63. 63
   64. 64
   65. 65
   66. 66
   67. 67
   68. 68
   69. 69
   70. 70
   71. 71
   72. 72
   73. 73
   74. 74
   75. 75
   76. 76
   77. 77
   78. 78
   79. 79
   80. 80
   81. 81
   82. 82
   83. 83
   84. 84
   85. 85
   86. 86
   87. 87
   88. 88
   89. 89
   90. 90
   91. 91
   92. 92

   const n = 50;
   type vec = array [1..n*4] of system.word;
   function vprod(const a, b: vec): Cardinal;
     var i: longInt;
     begin
       result := 0;
       for i := 1 to high(vec) do inc(result, a[i] * b[i]);
     end;
   function vprod_asm1(const a, b: vec): Cardinal; assembler;
     asm
        push ebx
        push ecx
        push edx
        push esi
        push edi
        xor ebx, ebx
        mov ecx, n*4
        mov esi, a
        mov edi, b
        xor eax, eax
        cld
     @@l:
        mov   ax, word ptr [esi]
        lea   esi, [esi+2]
        mul   word ptr [edi]
        lea   edi, [edi+2]
        shl   edx, 16
        mov   dx, ax
        add   ebx, edx
        dec   ecx
        jne   @@l
        mov   eax, ebx
        pop edi
        pop esi
        pop edx
        pop ecx
        pop ebx
     end;
   function vprod_asm2(const a, b: vec): Cardinal; assembler;
     asm
       push ebx
       push ecx
       push edx
       push esi
       push edi
       xor ebx, ebx
       mov ecx, n*4
       mov esi, a
       mov edi, b
       xor eax, eax
       cld
     @@l:
       lodsw
       movzx   edx, WORD PTR [edi]
       imul    edx
       lea     edi, [edi+2]
       add     ebx, eax
       loop  @@l
       mov   eax, ebx
       pop edi
       pop esi
       pop edx
       pop ecx
       pop ebx
     end;
   function vprod_mmx (const a, b: vec): Cardinal; assembler;
     var muls: record l, h: Cardinal end;
     asm
       push ebx
       push ecx
       push esi
       push edi
       mov ecx, n
       mov esi, a
       mov edi, b
       xor eax, eax
       lea     ebx, muls
     @@l:
       db $0F,$6F,$06  // movq    mm0, [esi]
       db $0F,$F5,$07  // pmaddwd mm0, [edi]
       lea     esi, [esi+8]
       db $0F,$7F,$03  // movq    [ebx], mm0
       lea     edi, [edi+8]
       add     eax, [ebx]
       add     eax, [ebx+4]
       loop    @@l
       db $0F,$77 // emms
       pop edi
       pop esi
       pop ecx
       pop ebx
     end;

   По просьбам трудящихся публикую модифицированную версию примера MMXTEST.PAS из комплекта компилятора TMT Pascal. Программа находит скалярное произведение двух векторов. Далее должен быть основной блок с фрагментами типа for i := 1 to 100000 do vprod(a, b); , которые я не стал публиковать ввиду ограничений. Функция vprod_asm1 — почти оригинальный код TMT, функция vprod_asm2 — мой оптимизированный вариант. Результаты запуска на двух машинах (таймер получал по RDTSC):
   AMD K6-2-333 МГц, FSB 66 МГц.
   
   Delphi7:
   Pascal = 0.550 sec.
   Asm x86 (original) = 1.034 sec.
   Asm x86 (optimized) = 0.490 sec.
   Asm MMX = 0.130 sec.
   С директивой $O- первый результат 0.853 sec.
   Замена loop на dec ecx + jne увеличивает результаты на 0,015 c.
   
   FPC:
   Pascal = 1.387 sec.
   Asm x86 (original) = 1.199 sec.
   Asm x86 (optimized) = 0.510 sec.
   Asm MMX = 0.124 sec.
   
   TMT:
   Pascal = 0.914 sec.
   Asm x86 (original) = 1.037 sec.
   Asm x86 (optimized) = 0.494 sec.
   Asm MMX = 0.126 sec.
   
   VP:
   Pascal = 0.520 sec.
   Asm x86 (original) = 1.033 sec.
   Asm x86 (optimized) = 0.493 sec.
   Asm MMX = 0.146 sec.
   С директивами $Q+,R+ первый результат 1.032 sec.
   С директивой $Speed- первый результат 0.731 sec.
   
   ------------------------------
   Celeron 1,86 ГГц, FSB 533 МГц.
   
   Delphi7:
   Pascal = 0.025 sec.
   Asm x86 (original) = 0.091 sec.
   Asm x86 (optimized) = 0.082 sec.
   Asm MMX = 0.044 sec.
   
   TMT:
   Pascal = 0.109 sec.
   Asm x86 (original) = 0.087 sec.
   Asm x86 (optimized) = 0.079 sec.
   Asm MMX = 0.042 sec.

   inkanus-gray, 25 Сентября 2011

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

9. JavaScript / Говнокод #7976

   +147

   1. 1
   2. 2
   3. 3
   4. 4

   var i = new Array(new Array());
   i[0][0] = 'num';
   
   alert(i);

   матрица в js

   jQuery, 25 Сентября 2011

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

10. C++ / Говнокод #7975

    +165

    1. 1
    2. 2
    3. 3
    4. 4

    temp = new LIST;
    temp = ptr->next;
    ptr->next = ptr;
    ptr = temp;

    Так меняют указатели местами при сортировке списка.

    brainfucker, 25 Сентября 2011

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

• ← влево
• вправо →

• 2107
• …
• 786
• 785
• 784
• 783
• 782
• 781
• 780
• 779
• 778
• …
• 1