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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2373)
4. JavaScript (2031)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (588)
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 (30)
22. Swift (27)
23. Kotlin (14)

1. 1C / Говнокод #3131

   −124

   1. 1

   ЭтоЗаказПокупателя = ?(ТипЗнч(СсылкаНаОбъект) = Тип("ДокументСсылка.ЗаказПокупателя"), Истина, Ложь);

   мощь ))))

   luns, 29 Апреля 2010

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

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

   +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

   #define SPI_BUF_SIZE                                0x400
   #define SDRAM_SPI_0_BUF_MISO           (void*)0x100000
   #define SDRAM_SPI_0_BUF_MOSI           SDRAM_SPI_0_BUF_MISO - SPI_BUF_SIZE
   #define SDRAM_SPI_1_BUF_MISO           SDRAM_SPI_0_BUF_MOSI - SPI_BUF_SIZE
   #define SDRAM_SPI_1_BUF_MOSI           SDRAM_SPI_1_BUF_MISO - SPI_BUF_SIZE
   #define SDRAM_SPI_2_BUF_MISO           SDRAM_SPI_1_BUF_MOSI - SPI_BUF_SIZE
   #define SDRAM_SPI_2_BUF_MOSI           SDRAM_SPI_2_BUF_MISO - SPI_BUF_SIZE
   #define SDRAM_SPI_3_BUF_MISO           SDRAM_SPI_2_BUF_MOSI - SPI_BUF_SIZE
   #define SDRAM_SPI_3_BUF_MOSI           SDRAM_SPI_3_BUF_MISO - SPI_BUF_SIZE
   
   void test()
   {
   void *buf = SDRAM_SPI_3_BUF_MOSI;
   void *buf1 = (void*)0x100000 - 0x400*7;
   *buf = *buf1 = 0xAA;
   
   printf("buf=%.2x \tbuf1=%.2x\n ",*buf, buf1);
   return;
   }

   Расскажу преамбулу. Код писался под проц atmel at91sam7x
   В этом процессоре есть звено SPI (serial pereferial interface). Но так как звено было спроектировано с небольшим косячком (который кстати в эррате был явно описан) буффер для каждой линии нужно было выделять в сдраме.
   Сказано - сделано. Выделили. а так как средства отладки только DBGU и JTAG.
   Все волосы на жопе вырвал пока нашел в чем ошибка.
   Внимание, вопрос. В чем гавно?

   ursus, 29 Апреля 2010

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

3. PHP / Говнокод #3129

   +170

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

   $order = strtolower($order);
   if ($order=='id' OR $order=='created_by' OR $order=='created_at' OR
   	$order=='action')
   {
   	$order = $order;
   }
   else
   {
   	$order = 'id';
   }

   Надо крепче спать и внимательней писать.

   azzz, 29 Апреля 2010

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

4. ActionScript / Говнокод #3128

   −86

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

   for (i = 0; i < 3; i++) {
       if (smth) {
        blnCardValue = true;
        i = 4;
       }
      }

   клевый способ выйти из цикла

   dimas_art, 29 Апреля 2010

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

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

   +73

   1. 001
   2. 002
   3. 003
   4. 004
   5. 005
   6. 006
   7. 007
   8. 008
   9. 009
   10. 010
   11. 011
   12. 012
   13. 013
   14. 014
   15. 015
   16. 016
   17. 017
   18. 018
   19. 019
   20. 020
   21. 021
   22. 022
   23. 023
   24. 024
   25. 025
   26. 026
   27. 027
   28. 028
   29. 029
   30. 030
   31. 031
   32. 032
   33. 033
   34. 034
   35. 035
   36. 036
   37. 037
   38. 038
   39. 039
   40. 040
   41. 041
   42. 042
   43. 043
   44. 044
   45. 045
   46. 046
   47. 047
   48. 048
   49. 049
   50. 050
   51. 051
   52. 052
   53. 053
   54. 054
   55. 055
   56. 056
   57. 057
   58. 058
   59. 059
   60. 060
   61. 061
   62. 062
   63. 063
   64. 064
   65. 065
   66. 066
   67. 067
   68. 068
   69. 069
   70. 070
   71. 071
   72. 072
   73. 073
   74. 074
   75. 075
   76. 076
   77. 077
   78. 078
   79. 079
   80. 080
   81. 081
   82. 082
   83. 083
   84. 084
   85. 085
   86. 086
   87. 087
   88. 088
   89. 089
   90. 090
   91. 091
   92. 092
   93. 093
   94. 094
   95. 095
   96. 096
   97. 097
   98. 098
   99. 099
   100. 100

   ICQ::logInIcq( )
   {
         TLV   clid1,clid,clm,clmin,clless,clbild,cldistr,cllang,clcontr,t1;
         WSA   ws;
         int   packn;
         unsigned char st1[40];
   
      port=ControlIcqPropert.portIcqServer;        
      host=(char*)ControlIcqPropert.IcqServer;
      if((strlen(host)==0)|(port==0)) 
      {
         port=DEFAULT_SERVER_PORT;
         host=DEFAULT_SERVER_HOST;
      }
      if(strlen(password)<3) return 0;
   
      pn=sock.Connect( host,port );      
      packn=sock.Receive( ( char * )uc,sizeof( uc ),0,0 );
   
      if(  packn <= 0  ){return 1;}
      //SetIndic(10);
      tUin.type=0x0100;
      tUin.lenght[0]=0x00;//htons( ( BYTE )strlen( uin ) );
      tUin.lenght[1]=strlen(uin);
      memcpy( tUin.data,uin,strlen( uin ) );
      flap.chan_id=0x01;
      flap.Sequence+=0x0100;
      flap.datalenght[0]=0x00;
      flap.datalenght[1]=0x04;
      flap.data[0]=0x00;
      flap.data[1]=0x00;
      flap.data[2]=0x00;
      flap.data[3]=0x01;
      packn=sock.Send( ( const char * )&flap,10,0 );
      flap.Sequence+=0x0100;
      flap.chan_id=0x02;
      t1.type=0x1700;
      t1.lenght[0]=0x00;
      t1.lenght[1]=0x06;
      t1.data[0]=0x00;
      t1.data[1]=0x00;
      t1.data[2]=0x00;
      t1.data[3]=0x00;
      t1.data[4]=0x00;
      t1.data[5]=0x00;
      int lnum=0;
      lnum+=10;
      memcpy( &st1,&t1,lnum );
      memcpy( &st1[lnum],&tUin,strlen(uin)+4 );
      lnum+=strlen(uin)+4;
      
      unsigned char u1[4];
      u1[0]=0x00;
      u1[1]=0x4B;
      u1[2]=0x00;
      u1[3]=0x00;
      
      memcpy( &st1[lnum],&u1,4 );
      lnum+=4;
      memcpy( flap.data,&st1,lnum );
      flap.datalenght[0]=0x00;
      flap.datalenght[1]=0x1B;
      lnum+=6;
      packn=sock.Send( ( char * )&flap,lnum,0 );
      //memcpy( &flap,NULL,4096 );
      packn=sock.Receive( ( char * )&inflap,sizeof( inflap ),0 );
      memcpy( &uc,&inflap,packn );
      memcpy( &snac,&uc[6],packn-6 );
      memcpy( key,&snac.data[2],snac.data[1] );
      int ii;
      MD5_CTX Md5Ctx;
      HASH hash;
      MD5Init  (  &Md5Ctx  );
      MD5Update(  &Md5Ctx, (  unsigned char*  )password, strlen( password ) );
      MD5Final ( ( uchar*  )hash, &Md5Ctx  );
      MD5Init  (  &Md5Ctx  );
      MD5Update(  &Md5Ctx, (  unsigned char*  )key, snac.data[1] );
      MD5Update(  &Md5Ctx, (  unsigned char*  )hash, 16 );
      MD5Update(  &Md5Ctx, (  unsigned char*  )AIM_MD5_STRING, strlen( AIM_MD5_STRING ) );
      MD5Final ( (  uchar*  )hash, &Md5Ctx  );
   
      flap.chan_id=0x02;
      flap.Sequence+=0x0100;
      flap.datalenght[0]=0x00;
      flap.datalenght[1]=0x72;
      t1.lenght[0]=0x00;               //Not Lenght!!! it is SNAC!!!
      t1.lenght[1]=0x02;
      memcpy( uc,&t1,10 );
      memcpy( &uc[10],&tUin,13 );
      TLV auth_h;
      auth_h.type=0x2500;
      auth_h.lenght[0]=0x00;
      auth_h.lenght[1]=0x10;
      memcpy( &auth_h.data,&hash,16 );
      memcpy( &uc[23],&auth_h,20 );
      uc[43]=0x00;
      uc[44]=0x4c;
      uc[45]=0x00;
      uc[46]=0x00;
      clid1.type=0x0300;

   Много лет назад, будучи молодым программером писал абсолютное гавно. Была поставлена задача реализовать протокол оскар 11 в одной бухгалтерской системе. Полное отсутствие архитектуры приложения, бизнес-слоя. В общем мне тогда сказали что так писать не надо и с треском выгнали. После этого так перестал писать :-)

   ursus, 29 Апреля 2010

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

6. C++ / Говнокод #3126

   +87

   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
   98. 98

   float Chislo(char *a, int i)
     {
      int j=i;
      int k=0;
      int r;
      float chis=0;
      while (a[j-1]!=' ' && a[j-1]!='>' && a[j-1]!='-' && a[j-1]!='+' && a[j-1]!='(')
        {j--;
                //тут перегоняем символ a[j] в цифру r
         if (int(a[j]==48)) r=0;
          else if (int(a[j]==49)) r=1;
           else if (int(a[j]==50)) r=2;
            else if (int(a[j]==51)) r=3;
             else if (int(a[j]==52)) r=4;
              else if (int(a[j]==53)) r=5;
               else if (int(a[j]==54)) r=6;
                else if (int(a[j]==55)) r=7;
                 else if (int(a[j]==56)) r=8;
                  else if (int(a[j]==57)) r=9;
                    else goto lab8;
         //ShowMessage(a[j]);
   
         chis=chis + r*pow(10,k);
         k++;
        };
      //j--;            //дальше надо обратно до точки и дольше до пробела
      while (a[j]!='.')
        {if (a[j]==' ' || a[j]==':') goto lab8;
         j++;
        };
      j++;
      r=0;
      k=-1;
      
         k--;
         j++;
        };
   
      lab8:
   
      return chis;
     }
   
   // была еще и похощая функция возвращающая результат в виде AnsiString
   
   // вот сколько переменных понадобивлось автору в более чем 700-строчной процедуре
   
   void Progon()
   {
    //
     AnsiString TempR1, TempR2, TempR3, Com,Sport,Team1,Team2, Team, URL1, URL2, page,Champ,Date;
     AnsiString day, month, year, Res, temp, otdyh1, otdyh2;
     int dat;
     //TempR1 = NMHTTP1->Body;
   
      float tot,TM,TB,bol1,bol2,srbol,ver,value,proc;
      AnsiString totS,f1S,ind1S,ind2S, f1ch1S, t_ch1S, f1ch2S, t_ch2S, f1ch3S, t_ch3S, f1ch4S, t_ch4S;
      float fora1,ind1,ind2, f1ch1, t_ch1, f1ch2, t_ch2, f1ch3, t_ch3, f1ch4, t_ch4;
      AnsiString Score1, Score2, s1ch1S, s2ch1S, s1ch2S, s2ch2S, s1ch3S, s2ch3S, s1ch4S, s2ch4S;
   
      AnsiString FIO, ScoreS, RebS, AstS, MinS, Amplua, BlkS, StlS;
      AnsiString TotalS, RebBookS, AstBookS, StlBookS, BlkBookS;
      float Total, RebBook, AstBook, BlkBook, StlBook;
      float PtssNRebsBook, PtssNAstsBook, RebsNAstsBook, PtssNRebsNAstsBook, Book3ptTeam1, Book3ptTeam2;
      int Score, count;
   
      int str1, str2, ind, indd, ig_pos;
      bool win;
   
   // В коде куча такого
   
    while (bukva2[i2]!='/') i2++;
                         i2++;
                         while (bukva2[i2]!='/') i2++;
                         i2++;
                         while (bukva2[i2]!='/') i2++;       // // обошли stl
   
   int *Ptss;
     Ptss = new int[count];
   
     int *Rebs;
     Rebs = new int[count];
   
     int *Asts;
     Asts = new int[count];
   
     int *Blks;
   
   
   while ( ! ( ((bukva[i]=='H') &&(bukva[i+1]=='o') && (bukva[i+2]=='m') && (bukva[i+3]=='e')) || ((bukva[i]=='R') &&(bukva[i+1]=='o') && (bukva[i+2]=='a') && (bukva[i+3]=='d')))) i--;
            while (bukva[i]!='a') i--;
            while ((bukva[i-1]!='>') && (bukva[i-1]!=' ')) i--;
   
   // Внутри циклов всюду работа с экселем
    else
                                {Form1->App.OlePropertyGet("ActiveSheet").OlePropertyGet("Cells", str1, str2+14).OlePropertyGet("Font").OlePropertySet("Color", -16777216);
                                 Form1->App.OlePropertyGet("ActiveSheet").OlePropertyGet("Cells", str1, str2+15).OlePropertyGet("Font").OlePropertySet("Color", -16777216);
                                };

   Как -то по фрилансу заказчик хотел доработать то ли парсер, то ли еще что-то (по предоставленным исходникам я так и не смог понять что это за прога, названная "Автониггер" и написанная каким -то студентом толи специально ради выноса мозгов, то ли по еще каким-то причинам вышеобозначенными выкладками, которые вполне возможно содержат в себе какойто гипнотический смысл).

   fenryr, 29 Апреля 2010

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

7. Pascal / Говнокод #3125

   +97

   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

   /// До этой строчки еще больше 1000 строк кода (И это всё в ОДНОЙ процедур)
    finally
       if Main.RecordCount > 0
       then Main.First;
       Main.EnableControls;
       Panel2.Color:=clLime;
       lbTimeSpend.Caption:='Âðåìÿ çàòðà÷åíî:'+TimeToStr(time()-TimeWork);
     end;
   
     Exit;
   
     try
       if TmpCollect_Skd.IsSelectAll then
       begin
         Askd_count := TmpCollect_Skd.RxDBGrid1.DataSource.DataSet.RecordCount;
       end
    /// После этой строки еще более 1000 строк кода всё в этой же процедуре!

   Продолжим. Вот так люди используют Exit;

   goodron, 28 Апреля 2010

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

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

   +101

   1. 1
   2. 2
   3. 3

   if IntToStr(n) = '2' then begin
     ///  Еще куча говнокода
   end;

   Устроился я тут работать в одну компанию... Начинаю серию публикаций... Название компании пока не буду озвучивать - еще уволят нах...
   Начнём с чего-нибудь простенького :)

   goodron, 28 Апреля 2010

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

9. Java / Говнокод #3123

   +67

   1. 1
   2. 2
   3. 3

   public boolean isActive() {
           return !isKilling() && !isKilled() && !isKillPending();
       }

   Made in Estonia (part 2)

   bot225, 28 Апреля 2010

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

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

    +77

    1. 1

    private final List<ItemInfo> _items = Collections.synchronizedList(new Vector<ItemInfo>());

    Пример убер-синхронизации (чтоб наверняка).
    Используется и по сей день.
    
    Кусок из крупного эмулятора сервера L2 (Потомок сборки L2Rebellion).

    VirtualVoid, 28 Апреля 2010

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

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

• 2107
• …
• 307
• 306
• 305
• 304
• 303
• 302
• 301
• 300
• 299
• …
• 1