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

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

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

1. PHP (5711)
2. C++ (2742)
3. Куча (2397)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1120)
8. Pascal (648)
9. Python (591)
10. 1C (540)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (44)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. Си / Говнокод #7393

   +109

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

   int rnd() {
     srand( rand()*rand() );
     int r = 0;
     for (int i=0;i<=10;i++)
       r=rand();
     srand( ++r - time(NULL) );
     return rand()/r;
   }

   NEED MOAR RANDOM NUMBERS!!!

   Fai, 29 Июля 2011

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

3. bash / Говнокод #7390

   −107

   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

   #!/usr/bin/env zsh
   
   A=( q w e r t y u i o p a s d f g h j k l z x c v b n m )
   B=( Q W E R T Y U I O P A S D F G H J K L Z X C V B N M )
   C=( 0 1 2 3 4 5 6 7 8 9 A B C D E F ) #hex
   F=( 1 2 4 )
   D=40
   E=10
   
   Char::GetRandA(){
       echo ${A[$(shuf -n 1 -i 1-${#A[@]})]}
   }
   
   Char::GetRandB(){
       echo ${B[$(shuf -n 1 -i 1-${#B[@]})]}
   }
   
   Char::GetRandC(){
       echo ${C[$(shuf -n 1 -i 1-${#C[@]})]}
   }
   
   Char::GetRandF(){
       echo ${F[$(shuf -n 1 -i 1-${#F[@]})]}
   }
   
   Main::Matrix(){
       for i in {1..$D}; do
           for b in {A,B,C}; do
               echo -n '\033[0;3'$(Char::GetRandF)"m$(Char::GetRand$b) "
           done
       done
   }
   
   #Run
   for i in {1..$E}; do
       Main::Matrix
   done

   дело было вечером...

   james, 29 Июля 2011

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

4. Pascal / Говнокод #7384

   +146

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

   function factorial( n: Integer ): Int64;
   begin
     if n=0 then Result := 1
     else             Result := n*factorial(n-1);
   end;
   
   factorial(-1); //<--ЗАЦИКЛИВАЕТ

   =D

   Fai, 29 Июля 2011

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

5. C++ / Говнокод #7356

   +147

   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
   93. 93
   94. 94
   95. 95
   96. 96
   97. 97

   int offset_800=0;
   	if(SCREEN_WIDTH == 800) offset_800 = 20;
   
   	addItem(TNM_GAMELOFT_LOG_GREEN_ITEM, done, complete_log[lang][0], TNM_ItemCoords[TNM_GAMELOFT_LOG_GREEN_ITEM][0]*((float)1200/(float)SCREEN_WIDTH)-35-offset_800/*-offset_shtamp*3*/, TNM_ItemCoords[TNM_GAMELOFT_LOG_GREEN_ITEM][1]*((float)720/(float)SCREEN_HEIGHT)/*-offset_shtamp*3*/, -1, ITEM_INVISIBLE)
   		->attachTo2dItem(pItemListBG);
   	addItem(TNM_GAMELOFT_LOG_RED_ITEM, done, complete_log[lang][1], TNM_ItemCoords[TNM_GAMELOFT_LOG_RED_ITEM][0]*((float)1200/(float)SCREEN_WIDTH)-35-offset_800/*-offset_shtamp*3*/, TNM_ItemCoords[TNM_GAMELOFT_LOG_RED_ITEM][1]*((float)720/(float)SCREEN_HEIGHT)/*-offset_shtamp*3*/, -1, ITEM_INVISIBLE)
   		->attachTo2dItem(pItemListBG);
   	
   //#ifdef WIN32
   
   	addItem(TNM_CHEAT_WIN, main_all, 30, TNM_ItemCoords[TNM_CHEAT_WIN][0],TNM_ItemCoords[TNM_CHEAT_WIN][1], 31, ITEM_INVISIBLE);
   	addItem(TNM_CHEAT_LOSE, main_all, 30, TNM_ItemCoords[TNM_CHEAT_LOSE][0],TNM_ItemCoords[TNM_CHEAT_LOSE][1], 31, ITEM_INVISIBLE);
   //#endif
        int offY=0;
   	int offset=0,
   		offsetmailX=0;
   	
   	if(SCREEN_WIDTH ==800||SCREEN_WIDTH == 1024)
   		offset=2;
   	
   	if(SCREEN_WIDTH==1366||SCREEN_WIDTH == 1440){
   		offY=4;
   		offset=2;
   	}
   	if( SCREEN_WIDTH==1680)
   	{ 
   		offY=7;
   		offset=2;
   		offsetmailX=60;
   	}
   		if( SCREEN_WIDTH==1920)
   	{ 
   		offY=7;
   		offset=5;
   		offsetmailX=60;
   	}
   
   		int y_800=0;
   
   		if(SCREEN_WIDTH == 1440) offsetmailX = 20;
   		if(SCREEN_HEIGHT == 800) {
   			offset = 2;
   			offY = 1;
   			
   		}
   		if(SCREEN_WIDTH == 2560) offset=2;
   		if(SCREEN_WIDTH == 1024) offY+=1;
   		if(SCREEN_WIDTH == 800&&(Text::getInstance().getLanguage()== Text::LANG_SC||Text::getInstance().getLanguage()== Text::LANG_KR)) {
   			offY-=1;
   			y_800 = 3;
   		}
   
   		
   
   		if((SCREEN_WIDTH == 1680||SCREEN_HEIGHT == 1080)&&(Text::getInstance().getLanguage()== Text::LANG_SC||Text::getInstance().getLanguage()== Text::LANG_KR)){
   			//offset-=2;
   			offY-=2;
   		}
   		if((SCREEN_HEIGHT == 1200)&&(Text::getInstance().getLanguage()== Text::LANG_SC||Text::getInstance().getLanguage()== Text::LANG_KR))
   			offY-=2;
   
   	addText(TNM_MESSAGE_SENDER_ITEM, STR_SENDER, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MESSAGE_SENDER_ITEM][0], TNM_ItemCoords[TNM_MESSAGE_SENDER_ITEM][1]/*-offY*/, ITEM_INVISIBLE, LEFT)
   		->attachTo2dItem(pItemListBG);
   	
   	
   
   	addText(TNM_MESSAGE_SUBJECT_ITEM, STR_SUBJECT, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MESSAGE_SUBJECT_ITEM][0], TNM_ItemCoords[TNM_MESSAGE_SUBJECT_ITEM][1]-offset-offY*1.5f+y_800, ITEM_INVISIBLE, LEFT)
   		->attachTo2dItem(pItemListBG);
   	addText(TNM_MESSAGE_TIME_ITEM, STR_RECEIVED, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MESSAGE_TIME_ITEM][0], TNM_ItemCoords[TNM_MESSAGE_TIME_ITEM][1]-offY*1.07f,  ITEM_INVISIBLE, LEFT)
   		->attachTo2dItem(pItemListBG);
   	//addText(TNM_MAIL_TITLE_ITEM, "Sender: \nRecieved: \nSubject: ", fontS, TNM_ItemCoords[TNM_MAIL_TITLE_ITEM][0], TNM_ItemCoords[TNM_MAIL_TITLE_ITEM][1], ITEM_INVISIBLE, LEFT|TOP);
   	int space = lang == Text::LANG_IT ? 10 : 0;
   	if(lang == Text::LANG_KR)
   		space = 30;
   	int spaceY=0;
   
   	
   	if(SCREEN_WIDTH!=1680&&SCREEN_WIDTH!=1920)
   		switch( pGame->getCurrentLanguage() )
   	{
   	case Text::LANG_KR:
   		spaceY = 2;
   		break;
   	case Text::LANG_SC:
   		spaceY = 2;
   		break;
   	case Text::LANG_JP:
   		spaceY = 2;
   		break;
   	}
   
   	addText(TNM_MAIL_TITLE1_ITEM, m_mailTitle, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MAIL_TITLE1_ITEM][0] + space, TNM_ItemCoords[TNM_MAIL_TITLE1_ITEM][1], ITEM_INVISIBLE, LEFT|TOP)
   		->attachTo2dItem(pItemListBG);
   	addText(TNM_MAIL_TITLE2_ITEM, m_mailTitle, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MAIL_TITLE2_ITEM][0] + space, TNM_ItemCoords[TNM_MAIL_TITLE2_ITEM][1]-offY*1.07f, ITEM_INVISIBLE, LEFT|TOP)
   		->attachTo2dItem(pItemListBG);
   	addText(TNM_MAIL_TITLE3_ITEM, m_mailTitle, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MAIL_TITLE3_ITEM][0] + space, TNM_ItemCoords[TNM_MAIL_TITLE3_ITEM][1]-offset-offY*1.5f+spaceY, ITEM_INVISIBLE, LEFT|TOP)
   		->attachTo2dItem(pItemListBG);

   Еще один малый кусок. По аналогии с выложеным ранее.

   BlaZe, 27 Июля 2011

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

6. Java / Говнокод #7355

   +147

   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

   private static Random rnd = new Random();
   
     public static int getRand(int start, int end) {
       boolean check = false;
       int number = 0;
       if (start >= end) {
         final int temp = end;
         end = start;
         start = temp;
       }
       while (!check) {
         number = Math.abs(rnd.nextInt() % end);
         if ((number >= start) && (number < end)) {
           check = true;
         }
       }
       return number;
     }

   эпический метод, найден в исходниках игрушки

   Lure Of Chaos, 27 Июля 2011

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

7. JavaScript / Говнокод #7349

   +149

   1. 1

   var save = if($('input#remmepls').attr('checked="checked"')){1}else{0};

   Думаю и так всё понятно

   TRANE73, 26 Июля 2011

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

8. C# / Говнокод #7306

   +976

   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

   public void OpredelenieKletki(TipiReljefaMiraVelikogo tipKletkiEtt)
   {
   tipKletki = tipKletkiEtt;
    
   #region огромное назначение координат ректанглам вырезания из разнобоя
    
   if (kameraEtojKletki.sasUvelicenie)
   {
       switch (tipKletki)
       {
           case TipiReljefaMiraVelikogo.zelenaRovnaAA:
               malOcered = 1; kadrSasVirezanija = new Rectangle(0, kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
               smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
           case TipiReljefaMiraVelikogo.zelenaRovnaBB:
               malOcered = 1; kadrSasVirezanija = new Rectangle(0, 316 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
               smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
           case TipiReljefaMiraVelikogo.zelenaRovnaCC:
               malOcered = 1; kadrSasVirezanija = new Rectangle(0, 632 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
               smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
           case TipiReljefaMiraVelikogo.zelenaRovnaDD:
               malOcered = 1; kadrSasVirezanija = new Rectangle(0, 948 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
               smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
    
           case TipiReljefaMiraVelikogo.zelenaRovnaVertPalkaAA:
               malOcered = 1; kadrSasVirezanija = new Rectangle(0, 1264 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
               smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
           case TipiReljefaMiraVelikogo.zelenaRovnaVertPalkaAb:
               malOcered = 1; kadrSasVirezanija = new Rectangle(0, 1580 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
               smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
    
           case TipiReljefaMiraVelikogo.zelenaRovnaVertPalkaBB:
               malOcered = 1; kadrSasVirezanija = new Rectangle(162, 1264 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
               smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
           case TipiReljefaMiraVelikogo.zelenaRovnaVertPalkaBc:
               malOcered = 1; kadrSasVirezanija = new Rectangle(162, 1580 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
               smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
   
        ... ... ...

   Класс который достает из спрайтшита нужную текстуру и передает её на отрисовку.

   ArtFeel, 21 Июля 2011

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

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

   +164

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

   function calculateBaggageFields(selectedNormalOutboundOption, selectedNormalInboundOption,
   				numOriginalNormalBagsOutboundField, numOriginalNormalBagsInboundField,
   				numAdditionalBagsField, numAdditionalBagsOutboundField, numAdditionalBagsInboundField,
   				selectedOutsizeOutboundOption, selectedOutsizeInboundOption,
   				totalForThisPassengerField,
                                   selectedExcessKilosOutboundOption,selectedExcessKilosInboundOption,
                                   numOriginalExcessKilosOutboundField,numOriginalExcessKilosInboundField,
                                   numAdditionalExcessKilos,numAdditionalExcessKilosOutbound,numAdditionalExcessKilosInbound)

   продакшн-говно за работой!

   hrls, 14 Июля 2011

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

10. Java / Говнокод #7244

    +147

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

    if(!xmlDate.equals(null))
    {
    ...
    }
    else
    {
            return null;
    }

    Для полноты картинки смотрим метод equals в XMLGregorianCalendar.java. Стажеры такие стажеры...

    Art, 14 Июля 2011

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

11. SQL / Говнокод #7240

    −851

    1. 1

    facepalm.sql

    В таблице платежей несколько полей с цифрами - _amount и _value.
    В связаной таблице есть поле PaymentAmount.
    Угадайте, какому полю платежа оно соответствует 7:

    wiz, 13 Июля 2011

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

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

• 1976
• …
• 1229
• 1228
• 1227
• 1226
• 1225
• 1224
• 1223
• 1222
• 1221
• …
• 1