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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2370)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1116)
8. Pascal (648)
9. Python (586)
10. 1C (538)
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 (28)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

2. C++ / Говнокод #17308

   +57

   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

   if (this->IsWhiteFigure())
   		{
   			if (abs(position_x - this->GetPositionX()) == 1 && position_y - this->GetPositionY() == -1)
   			{
   				this->SetPositionX(position_x);
   				this->SetPositionY(position_y);
   			}
   			else
   			{
   				throw IncorrectMoveException();
   			}
   		}
   		else
   		{
   			if (abs(position_x - this->GetPositionX() == 1) && position_y - this->GetPositionY() == 1)
   			{
   				this->SetPositionX(position_x);
   				this->SetPositionY(position_y);
   			}
   			else
   			{
   				throw IncorrectMoveException();
   			}
   		}

   Тест на внимательность. Сможете ли вы догадаться почему это не работает ?

   GreatMASTERcpp, 13 Декабря 2014

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

3. C++ / Говнокод #17306

   +63

   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

   int day (char* input, int begin, int number, int year)
   {
   	switch (input[begin])
   	{
   		case 65:
   		if (input[begin+1] == 112) 
   			if ((year % 4 == 0 && year % 100 != 0) || year % 400 == 0) return 91 + number; 
   			else return 90 + number;
   		else if ((year % 4 == 0 && year % 100 != 0) || year % 400 == 0) return 213 + number; 
   			else return 212 + number;
   		break;
   			
   		case 68:
   		if ((year % 4 == 0 && year % 100 != 0) || year % 400 == 0) return 335 + number; 
   					else return 334 + number;
   		break;
   		
   		case 70:
   		return 31+number;	
   		break;
   		
   		case 74:
   		switch (input[begin+3])
   		{
   			case 117:
   			return number;
   			break;
   			
   			case 101:
   			if ((year % 4 == 0 && year % 100 != 0) || year % 400 == 0) return 152 + number; else return 151 + number;
   			break;
   			
   			case 121:
   			if ((year % 4 == 0 && year % 100 != 0) || year % 400 == 0) return 182 + number; else return 181 + number;
   			break;			
   		}
   		break;
   		
   		case 77:
   		switch (input[begin+2])
   		{		
   			case 114:
   			if ((year % 4 == 0 && year % 100 != 0) || year % 400 == 0) return 60 + number; else return 59 + number;
   			break;
   			
   			case 121:
   			if ((year % 4 == 0 && year % 100 != 0) || year % 400 == 0) return 121 + number; else return 120 + number;
   			break;			
   		}
   		break;
   		
   		case 78:
   		if ((year % 4 == 0 && year % 100 != 0) || year % 400 == 0) return 305 + number; else return 304 + number;
   		break;
   		
   		case 79:
   		if ((year % 4 == 0 && year % 100 != 0) || year % 400 == 0) return 274 + number; else return 273 + number;
   		break;
   		
   		case 83:
   		if ((year % 4 == 0 && year % 100 != 0) || year % 400 == 0) return 244 + number; else return 243 + number;
   		break;
   	}
   }

   Преобразование название месяца в его продолжительность.

   mottoper, 12 Декабря 2014

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

4. Куча / Говнокод #17302

   +129

   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

   " Подсчёт количества документов
   
     DEFINE incr.
       add 1 to ls_summary-c_&1.
     END-OF-DEFINITION.
   
     CASE ls_s08_doc-status.
       WHEN `01`. incr vvr.
       WHEN `02`. incr vvr. incr vpz.
       WHEN `03`. incr prz.
       WHEN `04`. incr prz. incr nnp. incr otk.
       WHEN `05`.
       WHEN `06`. incr prz. incr nnp.
       WHEN `07`. incr prz. incr nnp. incr sgl.
       WHEN `ПД`. incr prz. incr nnp. incr v12.
       WHEN `ПП`.
       WHEN `ПС`. incr prz. incr nnp. incr pns.
       WHEN `НП`.
       WHEN `СФ`. incr prz. incr nnp. incr v12.
       WHEN `ТП`. incr vvr. incr vpz.
       WHEN `ОК`. incr prz. incr nnp. incr v12. incr o12.
       WHEN OTHERS.
     ENDCASE.

   В ABAP нет инкремента. Зато есть макросы, в которые подставляется «что написано» в точности так как написано ^_^

   Yaruson, 12 Декабря 2014

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

5. 1C / Говнокод #17253

   −122

   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

   Для Индекс = 1 По 4 Цикл
   
   	Если Индекс = 1 Тогда
   		<Действие1...>;
   	ИначеЕсли Индекс = 2 Тогда
   		<Действие2...>;
   	ИначеЕсли Индекс = 3 Тогда
   		<Действие3...>;
   	ИначеЕсли Индекс = 4 Тогда
   		<Действие4...>;
   	КонецЦикла;	
   
   КонецЦикла

   facepeeler, 04 Декабря 2014

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

6. JavaScript / Говнокод #17248

   +159

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

   request.done(function(data){			 
   			 if (data){
                                           <...много кода с кучей вложенных if else ..>
   				    	noty({type: "success", text: 'Заказ взят с биржи'});
   				  	setTimeout(location.reload(), 2000);
   				  }
   		   	 } else if (data.error) {
   				 console.log(data);
   				 noty({type: "error", text: data.error["message"]});				
   			 }
   	});

   i had n1ce debug time

   heyzea1, 03 Декабря 2014

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

7. Си / Говнокод #17234

   +133

   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

   bool findImageToleranceIn(CTSInfo *info, bitmap *imageToFind, int32_t *x, int32_t *y, int32_t x1, int32_t y1, int32_t x2, int32_t y2, uint16_t tolerance)
   {
       int I, J, XX, YY;
       info->tol = tolerance;
       int dX = (x2 - x1) - (imageToFind->width - 1);
       int dY = (y2 - y1) - (imageToFind->height - 1);
       for (I = 0; I < dY; ++I)
       {
           for (J = 0; J < dX; ++J)
           {
               for (YY = 0; YY < imageToFind->height; ++YY)
               {
                   for (XX = 0; XX < imageToFind->width; ++XX)
                   {
                       rgb32* pixel = &imageToFind->pixels[YY * imageToFind->width + XX];
                       rgb32* targetPixel = &info->targetImage->pixels[(YY + I) * info->targetImage->width + (XX + J)];
                       if (pixel->a != 0)
                       {
                           if (!(*info->ctsFuncPtr)(info, pixel, targetPixel))
                           {
                               goto Skip;
                           }
                       }
                   }
               }
               *x = J + x1;
               *y = I + y1;
               return true;
               Skip:
               continue;
           }
       }
       *x = -1;
       *y = -1;
       return false;
   }

   В чём здесь сакральный смысл GoTo?

   Cynicrus, 01 Декабря 2014

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

8. C++ / Говнокод #17207

   +53

   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

   BOOL IsNameInExpression(PCWSTR Expression, PCWSTR Name, DWORD len)
   {
   	WCHAR c, b;
   
   	while (c = *Expression++) 
   	{
   		if (c == L'*') 
   		{
   __loop:
   			switch (*Expression)
   			{
   			case '*':
   				Expression++;
   				goto __loop;
   			case 0:
   				return TRUE;
   			}
   
   			if (len)
   			{
   				do 
   				{
   					if (IsNameInExpression(Expression, Name++, len--)) return TRUE;
   				} while (len);
   			}
   
   			return FALSE;
   		}
   
   		if (!len--)
   		{
   			return FALSE;
   		}
   
   		b = *Name++;
   
   		if (c != L'?' && b != c) 
   		{
   			return FALSE;
   		} 
   	}
   
   	return !len;
   }
   
   BOOL IsNameInExpression(PCWSTR Expression, PUNICODE_STRING Name)
   {
   	RtlUpcaseUnicodeString(Name, Name, FALSE);
   	return IsNameInExpression(Expression, Name->Buffer, Name->Length >> 1);
   }

   zhukas, 28 Ноября 2014

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

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

   +160

   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

   var action = $(this).attr("action");
   var id_taxi = $('#id_taxi').val();
   var id_car = $('#id_car').val();
   var car_number = $('#number-car').val();
   var car_code = $('#number-car-code').val();
   <....>
   if ( action == "add" ){
       var url = '/api/4/*****/service/taxi/addAvto';
       var data_send={ 'id_taxi':id_taxi, 'car_number':car_number, 'model':model, 'color':color, 'licence_number': licence_number, 'id_type_car_body': id_type_car_body, 'type_car_comfort' : comfort, 'car_options' : option, 'isActive': is_active };
   
   }else if ( action == "edit"  ){
       var url = '/api/4/****/service/taxi/updateAvto';
       var data_send={ 'id_car': id_car, 'car_number':car_number, 'model':model, 'color':color, 'licence_number': licence_number,'id_type_car_body': id_type_car_body,  'type_car_comfort' : comfort, 'car_options' : option, 'isActive': is_active};
   
   }
   <...>
   var request = $.ajax({
     url: url,  
     type: "POST",
     dataType: 'json',
     data: data_send
   });

   Сабмитим формы.
   И так на всем проекте, 3к строк JS кода, которые подобным образом описывают ВСЕ формы на проекте.
   web service architecture, йобана

   heyzea1, 27 Ноября 2014

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

10. JavaScript / Говнокод #17174

    +156

    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

    var move_container_after = function(toMove, toAfter) {
    	  $(toMove).insertAfter(toAfter);
    };
    
    var move_container_before = function(toMove, toBefore) {
    	  $(toMove).insertBefore(toBefore);
    };
    
    move_container_before(".toggle.basic1-tog", ".basic1 .store-order-item-group-action-list");
    move_container_after(".toggle.basic1rn-tog", ".basic1rn .store-order-item-description");
    
    move_container_before(".toggle.basic3-tog", ".basic3 .store-order-item-group-action-list");
    move_container_after(".toggle.basic3rn-tog", ".basic3rn .store-order-item-description");
    
    move_container_before(".toggle.bundle1-tog", ".bundle1 .store-order-item-group-action-list");
    move_container_after(".toggle.bundle1rn-tog", ".bundle1rn .store-order-item-group-action-list");
    
    move_container_before(".toggle.bundle12-tog", ".bundle12 .store-order-item-group-action-list");
    move_container_after(".toggle.bundle12rn-tog", ".bundle12rn .store-order-item-group-action-list");
    
    move_container_before(".toggle.bundle3-tog", ".bundle3 .store-order-item-group-action-list");
    move_container_after(".toggle.bundle3rn-tog", ".bundle3rn .store-order-item-group-action-list");
    
    move_container_before(".toggle.bundle6-tog", ".bundle6 .store-order-item-group-action-list");
    move_container_after(".toggle.bundle6rn-tog", ".bundle6rn .store-order-item-group-action-list");
    
    move_container_before(".toggle.desktop1-tog", ".desktop1 .store-order-item-group-action-list");
    move_container_after(".toggle.desktop1rn-tog", ".desktop1rn .store-order-item-description");
    
    move_container_before(".toggle.desktop12-tog", ".desktop12 .store-order-item-group-action-list");
    move_container_after(".toggle.desktop12rn-tog", ".desktop12rn .store-order-item-description");
    
    move_container_before(".toggle.desktop3-tog", ".desktop3 .store-order-item-group-action-list");
    move_container_after(".toggle.desktop3rn-tog", ".desktop3rn .store-order-item-description");
    
    move_container_before(".toggle.desktop6-tog", ".desktop6 .store-order-item-group-action-list");
    move_container_after(".toggle.desktop6rn-tog", ".desktop6rn .store-order-item-description");
    
    move_container_before(".toggle.desktopbus12-tog", ".desktopbus12 .store-order-item-group-action-list");
    move_container_after(".toggle.desktopbus12rn-tog", ".desktopbus12rn .store-order-item-description");   
    
    move_container_before(".toggle.desktopbus6-tog", ".desktopbus6 .store-order-item-group-action-list");
    move_container_after(".toggle.desktopbus6rn-tog", ".desktopbus6rn .store-order-item-description");
    
    move_container_before(".toggle.premium1-tog", ".premium1 .store-order-item-group-action-list");
    move_container_after(".toggle.premium1rn-tog", ".premium1rn .store-order-item-description");
    
    move_container_before(".toggle.premium12-tog", ".premium12 .store-order-item-group-action-list");
    move_container_after(".toggle.premium12rn-tog", ".premium12rn .store-order-item-description");
    
    move_container_before(".toggle.premium3-tog", ".premium3 .store-order-item-group-action-list");
    move_container_after(".toggle.premium3rn-tog", ".premium3rn .store-order-item-description");
    
    move_container_before(".toggle.premium6-tog", ".premium6 .store-order-item-group-action-list");
    move_container_after(".toggle.premium6rn-tog", ".premium6rn .store-order-item-description");
    
    move_container_before(".toggle.premiumbus12-tog", ".premiumbus12 .store-order-item-group-action-list");
    move_container_after(".toggle.premiumbus12rn-tog", ".premiumbus12rn .store-order-item-description");
    
    move_container_before(".toggle.premiumbus6-tog", ".premiumbus6 .store-order-item-group-action-list");
    move_container_after(".toggle.premiumbus6rn-tog", ".premiumbus6rn .store-order-item-description");

    "Перенос" формы, для каждого ID, в HTML есть соответствующее колово форм подобного типа
    
    <div class="toggle basic1-tog">Auto Renew: Off
    <form method="POST" action="https://sites.fastspring.com/store/api/order">
    <input type="hidden" name="operation" value="update"/>
    <input type="hidden" name="destination" value="checkout"/>
    <input type="hidden" name="product_1_path" value="/basicsubscription1monthrn"/>
    <input type="hidden" name="product_2_path" value="/basicsubscription1month"/>
    <input type="hidden" name="product_2_quantity" value="0"/>
    <input type="submit" value="Turn On"/>
    </form>
    </div>
    <div class="toggle basic1rn-tog">Auto Renew: On
    <form method="POST" action="https://sites.fastspring.com/store/api/order">
    <input type="hidden" name="operation" value="update"/>
    <input type="hidden" name="destination" value="checkout"/>
    <input type="hidden" name="product_1_path" value="/basicsubscription1month"/>
    <input type="hidden" name="product_2_path" value="/basicsubscription1monthrn"/>
    <input type="hidden" name="product_2_quantity" value="0"/>
    <input type="submit" value="Turn Off"/>
    </form>
    </div>

    IRIDON, 25 Ноября 2014

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

11. Java / Говнокод #17170

    +74

    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

    if (convertView == null || convertView != null) {
                    if (Position2.get(position)) {
                        t.setVisibility(0);
                        b.setVisibility(4);
                        mlp.setMargins(15, 0, 60, 0);
                        y.setBackgroundColor(getResources().getColor(R.color.app_text_white));
                    } else {
                        mlp.setMargins(60, 0, 15, 0);
                        t.setVisibility(4);
                        b.setVisibility(0);
                        y.setBackgroundColor(getResources().getColor(R.color.app_text_white));                    
                    }
                } else {
                    view = convertView;
                }

    Индусы

    vinnizp, 25 Ноября 2014

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

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

• 1970
• …
• 410
• 409
• 408
• 407
• 406
• 405
• 404
• 403
• 402
• …
• 1