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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2341)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   В номинации:

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

   За время:

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

2. C# / Говнокод #129

   +3.4

   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

   try
   {
       ...
       if (obj == null)
       {
           LogManager.GetLogger("Log").Warn(msg);
           throw new SoapException("Object not found", SoapException.ServerFaultCode);
       }
   }
   catch (Exception ex)
   {
       LogManager.GetLogger("Log").Error(msg, ex);
       throw new SoapException("Object not found, SoapException.ServerFaultCode, ex);                
   }

   guest, 12 Декабря 2008

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

3. Pascal / Говнокод #113

   −23.1

   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

   if (TableData = 1) or ((TableData = 2) and
       (Pos(FieldDelim + 'Т_НАЦ' + FieldDelim, OpArtFields) > 0)) then
       sSelect := sSelect + ', ' + LMax + 'AR.Percent' + R + ' AR_Percent';
     if (TableData = 1) or ((TableData = 2) and
       (Pos(FieldDelim + 'Т_КАТ' + FieldDelim, OpArtFields) > 0)) then
       sSelect := sSelect + ', ' + LMax + 'AR.ID_ArtCat' + R + ' AR_ID_ArtCat';
     if (TableData = 1) or ((TableData = 2) and
       (Pos(FieldDelim + 'Т_ВЕС' + FieldDelim, OpArtFields) > 0)) then
       sSelect := sSelect + ', ' + LMax + 'AR.Weight' + R + ' AR_Weight';
     if (TableData = 1) or ((TableData = 2) and
       (Pos(FieldDelim + 'Т_ОБЪЕМ' + FieldDelim, OpArtFields) > 0)) then
       sSelect := sSelect + ', ' + LMax + 'AR.Capacity' + R + ' AR_Capacity';
     if (TableData = 1) or ((TableData = 2) and
       (Pos(FieldDelim + 'МИННАЦ' + FieldDelim, OpArtFields) > 0)) then
       sSelect := sSelect + ', ' + LMax + 'AR.MinDiscount' + R + ' AR_MinDiscount';
     if (TableData = 1) or ((TableData = 2) and
       (Pos(FieldDelim + 'Т_МИНКОЛ' + FieldDelim, OpArtFields) > 0)) then
       sSelect := sSelect + ', ' + LMax + 'AR.MinQuantity' + R + ' AR_MinQuantity';
     if (TableData = 1) or ((TableData = 2) and
       (Pos(FieldDelim + 'Т_СТАТУС' + FieldDelim, OpArtFields) > 0)) then
       sSelect := sSelect + ', ' + LMax + 'AR.Status' + R + ' AR_Status';
     if (TableData = 1) or ((TableData = 2) and
       (Pos(FieldDelim + 'Т_ГТД' + FieldDelim, OpArtFields) > 0)) then
       sSelect := sSelect + ', ' + LMax + 'AR.GTD' + R + ' AR_GTD';
     if (TableData = 1) or ((TableData = 2) and
       (Pos(FieldDelim + 'Т_СТРАНА' + FieldDelim, OpArtFields) > 0)) then 
       sSelect := sSelect + ', ' + LMax + 'CO.Name' + R + ' AR_Country';

   Собирание SQL - запроса для выборки полей. И это маленький кусочек :)
   Серьёзная контора с миллиардными оборотами!

   guest, 11 Декабря 2008

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

4. Java / Говнокод #91

   +27

   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

   private int checkTerm(PatternTerm term, int arr[],
         int symbNum, boolean isEditing) {
       if (term.count != INFINITY) {
         int endIdx = Math.min(symbNum + term.count, arr.length);
           int count = 0;
           switch(term.termType) {
             case DIGIT: {
               for(int i = symbNum; i < endIdx; i++) {
                 if (!Character.isDigit((char)arr[i]))
                   return -1;
                 count++;
               }
               break;
             }
             case LETTER: {
               for(int i = symbNum; i < endIdx; i++) {
                 if (!Character.isLetter((char)arr[i]))
                   return -1;
                 count++;
               }
               break;
             }
             case LETTERORDIGIT: {
               for(int i = symbNum; i < endIdx; i++) {
                 if (!Character.isLetterOrDigit((char)arr[i]))
                   return -1;
                 count++;
               }
               break;
             }
             case CHARACTER: {
               for(int i = symbNum; i < endIdx; i++) {
                 if (arr[i] != term.value)
                   return -1;
                 count++;
               }
               break;
             }
         }
         if (!isEditing && count != term.count)
           return -1;
         
         symbNum+=term.count;
       } else {
         int i = 0;
         switch(term.termType) {
           case DIGIT: {
             while (((symbNum + i) < arr.length) &&
                 Character.isDigit((char)arr[symbNum + i])) i++;
             break;
           }
           case LETTER: {
             while (((symbNum + i) < arr.length) &&
                 Character.isLetter((char)arr[symbNum + i])) i++;
             break;
           }
           case LETTERORDIGIT: {
             while (((symbNum + i) < arr.length) &&
                 Character.isLetterOrDigit((char)arr[symbNum + i])) i++;
             break;
           }
           case CHARACTER: {
             while (((symbNum + i) < arr.length) &&
                 (arr[symbNum + i] == term.value)) i++;
             break;
           }
         }
         symbNum+=i;
       }
       return symbNum;
     }

   Паша любит такие конструкции, которые хрен разобрать

   guest, 10 Декабря 2008

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

5. Pascal / Говнокод #87

   −12.7

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

   for i:=sk[(npl-1)*5+1] to sk[(npl-1)*5+2]-1 do
      begin
      l1:=0;l2:=0;l3:=0;l4:=0;l5:=0;l6:=0;l7:=0;l8:=0;l9:=0;l10:=0;
     for j:=1 to 100 do g1[j]:=0;
      for k:=sk[(npl-1)*5+1] to sk[(npl-1)*5+2] do
        if (k<>i )and
      (((abs(xyz^[pxyz^[i]].x-xyz^[pxyz^[k]].x)< dd) or  ((lx-abs(xyz^[pxyz^[i]].x-xyz^[pxyz^[k]].x))<dd))
      and ((abs(xyz^[pxyz^[i]].y-xyz^[pxyz^[k]].y)<dd) or  ((ly-abs(xyz^[pxyz^[i]].y-xyz^[pxyz^[k]].y))<dd))
      and ((abs(xyz^[pxyz^[i]].z-xyz^[pxyz^[k]].z)<dd) or  ((lz-abs(xyz^[pxyz^[i]].z-xyz^[pxyz^[k]].z))<dd)))
      then

   это только заголовок цикла ..

   guest, 09 Декабря 2008

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

6. Python / Говнокод #84

   −187.2

   1. 1
   2. 2

   respons+="""<span class="gray">коэффициент:</span> %s<br />""" % str(koeff[0][5])
   respons+="""<br />"""

   Формирование "аяксового" ответа

   guest, 09 Декабря 2008

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

7. PHP / Говнокод #68

   +22.9

   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

   $result = mysql_query("SELECT * FROM `Wallpapers` WHERE `Title` = '".$Title."'");
   $row = mysql_fetch_assoc($result);
   $wID=$row['ID'];
   $Wallpaper1=$row['Wallpaper1'];
   $Wallpaper2=$row['Wallpaper2'];
   $Wallpaper3=$row['Wallpaper3'];
   $Wallpaper4=$row['Wallpaper4'];
   $Wallpaper5=$row['Wallpaper5'];
   
   if ( $_SESSION['userid'] )
   	{ 
   	if ($Wallpaper1 != NULL) {
   		$size='';
   		$size=getimagesize("images/wallpapers/".$Wallpaper1);
   		$dHTML .= '<a class="preview" href="'.$image_path.'show/'.$Title.'/?w=1" style="text-decoration: underline;" onClick="alerts('.$wID.'); countIp();">'.$size[0].'x'.$size[1].'</a> ';
   	}
   	if ($Wallpaper2 != NULL) {
   		$size='';
   		$size=getimagesize("images/wallpapers/".$Wallpaper2);
   		$dHTML .= '<a class="preview" href="'.$image_path.'show/'.$Title.'/?w=2" style="text-decoration: underline;" onClick="alerts('.$wID.'); countIp();">'.$size[0].'x'.$size[1].'</a> ';
   	}
   	if ($Wallpaper3 != NULL) {
   		$size='';
   		$size=getimagesize("images/wallpapers/".$Wallpaper3);
   		$dHTML .= '<a class="preview" href="'.$image_path.'show/'.$Title.'/?w=3" style="text-decoration: underline;" onClick="alerts('.$wID.'); countIp();">'.$size[0].'x'.$size[1].'</a> ';
   	}
   	if ($Wallpaper4 != NULL) {
   		$size='';
   		$size=getimagesize("images/wallpapers/".$Wallpaper4);
   		$dHTML .= '<a class="preview" href="'.$image_path.'show/'.$Title.'/?w=4" style="text-decoration: underline;" onClick="alerts('.$wID.'); countIp();">'.$size[0].'x'.$size[1].'</a> ';
   	}
   	if ($Wallpaper5 != NULL) {
   		$size='';
   		$size=getimagesize("images/wallpapers/".$Wallpaper5);
   		$dHTML .= '<a class="preview" href="'.$image_path.'show/'.$Title.'/?w=5" style="text-decoration: underline;" onClick="alerts('.$wID.'); countIp();">'.$size[0].'x'.$size[1].'</a> ';
   	}
   	}

   guest, 08 Декабря 2008

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

8. Си / Говнокод #29055

   0

   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

   int IsEven(long long int number){
   	if (number == 0) return 0; 
   	long long int loc_num = 0;
   	update: 
   	
   	if (number == loc_num+1) {return 1;
   	} else if (number == (loc_num+1)*(-1)) { return 1;
   	} else {
   		if (number == loc_num+2) {return 0;
   		} else if (number == (loc_num+2)*(-1)) { return 0;
   		} else {
   			if (number == loc_num+3) {return 1;
   			} else if (number == (loc_num+3)*(-1)) { return 1;
   			} else {
   				if (number == loc_num+4) {return 0;
   				} else if (number == (loc_num+4)*(-1)) { return 0;
   				} else {
   					if (number == loc_num+5) {return 1;
   					} else if (number == (loc_num+5)*(-1)) { return 1;
   					} else {
   						if (number == loc_num+6) {return 0;
   						} else if (number == (loc_num+6)*(-1)) { return 0;
   						} else {
   							if (number == loc_num+7) {return 1;
   							} else if (number == (loc_num+7)*(-1)) { return 1;
   							} else {
   								if (number == loc_num+8) {return 0;
   								} else if (number == (loc_num+8)*(-1)) { return 0;
   								} else {
   									if (number == loc_num+9) {return 1;
   									} else if (number == (loc_num+9)*(-1)) { return 1;
   									} else {
   										if (number == loc_num+10) {return 0;
   										} else if (number == (loc_num+10)*(-1)) { return 0;
   										} else {
   												loc_num+=10; 
   												goto update;
   											}
   										}
   									}
   								}
   							}
   						}
   					}
   				}
   			}
   		}
   	}

   TurboLyudoed, 14 Ноября 2024

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

9. C# / Говнокод #28997

   0

   1. 1

   Тип объекта, который инициализируется безумно.

   https://learn.microsoft.com/ru-ru/dotnet/api/system.lazy-1?view=net-8.0#--------------
   Обожаю автоматический перевод в документации
   
   В оригинале, кстати, вот так:
   https://learn.microsoft.com/en-us/dotnet/api/system.lazy-1?view=net-8.0#type-parameters
   Заметьте, что ссылка тоже побилась

   kezzyhko, 11 Октября 2024

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

10. 1C / Говнокод #28996

    0

    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

    //Таб пропущенные
    	УсловиеОтбора = "ABANDON";
    	ДлинаСтрОтбора = 32;
    	Для с=1 По ДлинаСтрОтбора - СтрДлина(УсловиеОтбора) Цикл
    		УсловиеОтбора = УсловиеОтбора + " ";
    	КонецЦикла;		
    	ТабПропущенные = Таб_queue.Скопировать(Новый Структура("Событие",УсловиеОтбора));
    	
    	//Таб всё же дозвонились
    	УсловиеОтбора = "CONNECT";
    	ДлинаСтрОтбора = 32;
    	Для с=1 По ДлинаСтрОтбора - СтрДлина(УсловиеОтбора) Цикл
    		УсловиеОтбора = УсловиеОтбора + " ";
    	КонецЦикла;	
    	ТабДозвонились = Таб_queue.Скопировать(Новый Структура("Событие",УсловиеОтбора));

    В таблице какого то черта стояла фиксированная строка длинной 32 символа.
    Вот так "изящно" обошли это.

    NioGoth, 08 Октября 2024

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

11. C# / Говнокод #28985

    0

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

    private void UpdateRowInfo()
    {
    	//код
    	if (condition);
    	{
    		//код
    	}
    	//код
    }

    Наткнулся на вот такое в одном проекте. Точка с запятой после if считается пустым statement, а всё что внутри фигурных скобок - просто блок, от if'а независящий. Всё, как IDEшка об этом сообщает - точка с запятой стала серого цвета (на сером фоне, ага). Угадайте, сколько времени искался этот баг

    kezzyhko, 02 Сентября 2024

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

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

• 1963
• …
• 326
• 325
• 324
• 323
• 322
• 321
• 320
• 319
• 318
• …
• 1