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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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. SQL / Говнокод #2351

   −861.7

   1. 1
   2. 2
   3. 3

   select ...
   where ISNULL(e2e.ItemId, -1) = (case when @itemId < 1 then ISNULL(e2e.ItemId, -1) else @itemId end) 
   and ISNULL(e2e.LeftId, -1) = (case when @parentItemId < 1 then ISNULL(e2e.LeftId, -1) else @parentItemId) end

   workgss, 29 Декабря 2009

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

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

   +132.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
   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

   var ChildListWithCondition =
                   (from list in ChildList
                    join requestedRelatedEntityIds in EntitiesIdInList on list.ParentEntityId equals requestedRelatedEntityIds
                    join requestedEntityType in EntityTypeIdToTake on list.EntityTypeId equals requestedEntityType)
                    .Select(list => 
                        new LayoutDataOutputStructure()
                        {
                            ParentEntityId = list.ParentEntityId,
                            EntityId = list.EntityId,
                            FieldId = list.FieldId,
                            FieldValue = list.FieldValue,
                            EntityTypeId = list.EntityTypeId,
                            RelationTypeToParent = list.RelationTypeToParent,
                            FieldValueId = list.FieldValueId
                        })
                   .GroupBy(item => item.ParentEntityId)
                   .Select(group => new
                       {
                           ParentEntityId = group.Key,
                           GroupEntityTypeId = group.GroupBy(item => item.EntityTypeId)
                               .Select(group2 => new
                                                    {
                                                       EntityTypeId = group2.Key,
                                                       EntityRelation = group2.Select(item => item.RelationTypeToParent).FirstOrDefault(),
                                                       GroupEntityId = group2.GroupBy(group3 => group3.EntityId)
                                                           .Select(group3 => new
                                                                                 {
                                                                                     EntityId = group3.Key, 
                                                                                     Fields = group3.GroupBy(group4 => group4.FieldId)
                                                                                       .Select(group4 => new { FieldId = group4.Key, FieldValues = group4 })
                                                                                       
                                                                                 })
                                                    }
                                   )
                       }
                   )
                   .ToList();

   workgss, 29 Декабря 2009

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

3. Java / Говнокод #2349

   +78.2

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

   public void pause(){
           Object lock=new Object();
           synchronized(lock){
               try{
                   lock.wait(1000);
               }catch(Exception e){
                   System.out.println(e.getMessage());
               }
           }
       }

   _jk_, 29 Декабря 2009

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

4. C++ / Говнокод #2348

   +64.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

   int main(int argc, char **argv){
     QApplication app(argc, argv);
     QWebView view;
     view.load(QUrl("http://www.linuxcertif.com/"));
     view.show();
     app.exec();
     QWebSettings::setIconDatabasePath("/tmp");
     QWebSettings::iconForUrl(QUrl("http://www.linuxcertif.com/"));
     QIcon icon = QWebSettings::iconForUrl(QUrl("http://www.linuxcertif.com/"));
     QPushButton button;
     button.setIcon(icon);
     button.show();
     return app.exec();
   }

   смотрим на восьмую строку и понимаем, что она не делает ничего....
   ошибка то мелкая, было бы не смешно, если тока этот код не распологался бы на багтрекере вебикита - https://bugs.webkit.org/show_bug.cgi?id=29440
   еще в вебките забавно, что если в этом семпле седьмую строку переместить сразу за второй, семпл работает...

   LuCiFer, 29 Декабря 2009

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

5. PHP / Говнокод #2347

   +155.9

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

   <?php 
   
   $Fnm = $mosConfig_absolute_path .'/components/'.$option.'/cron.php';
   $inF = fopen($Fnm,"w");
   fwrite($inF,'<?php $last_cron_date='.$last_cron_date.';?>');
   fclose($inF);

   всеми любимая джумла... а точнее один из её компонентов... строка 1846-ая...
   
   почему собственно гадость:
   1. а если нету доступа на запись...
   2. не безопасно
   3. если в файле что-нибудь будет не так - обвалится весь сайт, потому как ошибко php
   4. я настороженно отношусь к файлам по 2246 - строк ... уж больно много всего...

   nicolay-punin, 29 Декабря 2009

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

6. Куча / Говнокод #2346

   +124.3

   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

   Functions called:
    fflush (4 times)
    delete (11 times)
    fclose (1 times)
    free (11 times)
    fread (3 times)
    fopen (1 times)
    vsnprintf (4 times)
    new (11 times)
    SysReallocMem (652 times)
    SysFreeMem (13185 times)
    SysGetMem (13188 times)
    lstrlenA (1 times)
    lstrcpynA (4 times)
    calloc (1 times)
    strlen (98 times)
    realloc (1 times)
    strdup (1 times)
    malloc (8 times)
    memcpy (4 times)
   Resource types used:
    object (11 allocs, 11 max)
    memory block (13199 allocs, 4007 max)
    file stream (1 allocs, 1 max)
    file handle (1 allocs, 1 max)

   Nemerle, 29 Декабря 2009

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

7. Python / Говнокод #2345

   −409.2

   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

   import urllib, sys, string
   from string import maketrans
   
   bbb = 0
   
   f = urllib.urlopen("http://www.pythonchallenge.com/pc/def/equality.html")
   while 1:
       buf = f.read(200000)
       if not len(buf):
           break
       for x in range(len(buf)):
           if buf[x] in string.ascii_lowercase:
              if buf[x+1] in string.ascii_uppercase:
                  if buf[x-1] in string.ascii_uppercase:
                      if buf[x+2] in string.ascii_uppercase:
                          if buf[x-2] in string.ascii_uppercase:
                              if buf[x+3] in string.ascii_uppercase:
                                  if buf[x-3] in string.ascii_uppercase:
                                      if buf[x+4] in string.ascii_lowercase:
                                          if buf[x-4] in string.ascii_lowercase:
                                              bbb = x
       sys.stdout.write(buf)
       print(buf[bbb-3:bbb+4])

   http://stackoverflow.com/questions/1972693/python-challenge-level-3

   ods, 29 Декабря 2009

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

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

   +59.5

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

   //и теперь шлём запрос письма
                       reply = QString("<iq type='get' from='%1' to='%2' id='mail-request-%3'>"\
                           "<query xmlns='google:mail:notify' %4 %5/></iq>")
                               .arg(from)
                               .arg(to)
                               .arg(id)
                               .arg((lastCheck.value(QString("%1").arg(account),"")=="")?QString(""):QString("newer-than-time='%1'").arg(lastCheck.value(QString("%1").arg(account))))
                               .arg((lastTid.value(QString("%1").arg(account),"")=="")?QString(""):QString("newer-than-tid='%1'").arg(lastTid.value(QString("%1").arg(account))));
                       stanzaSender->sendStanza(account, reply);

   Раскопано в недрах плагинов к psi:
   Особенно радует шедевральная конвертация int'а в строку при помощи QString("%1").arg(account),"")
   Плюс ко всему комменты на русском в интернациональном проекте это немножко моветон

   Sauron, 29 Декабря 2009

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

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

   +135.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
   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

   public static BitmapImage ConvertBitmapToBitmapImage(Bitmap bitmap)
           {
               var bitMapImage = new BitmapImage();
               var ms = new MemoryStream();
   
               try
               {
                   bitmap.Save(ms, ImageFormat.Bmp);
                   bitMapImage.BeginInit();
                   bitMapImage.StreamSource = ms;
                   bitMapImage.EndInit();
               }
               catch (NotSupportedException e)
               {
                   try
                   {
                       bitMapImage = new BitmapImage();
                       bitmap.Save(ms, ImageFormat.Png);
                       bitMapImage.BeginInit();
                       bitMapImage.StreamSource = ms;
                       bitMapImage.EndInit();
                   }
                   catch (NotSupportedException e2)
                   {
                       bitMapImage = new BitmapImage();
                       bitmap.Save(ms, ImageFormat.Bmp);
                       ms.Write(ms.ToArray(), 78, (int)(ms.Length - 78));
                       bitMapImage.BeginInit();
                       bitMapImage.StreamSource = ms;
                       bitMapImage.EndInit();
                   }
               }
   
               return bitMapImage;
           }

   wpf + System.Drawing + sql и хз что со всем этим делать )))))

   walash, 28 Декабря 2009

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

10. PHP / Говнокод #2342

    +168.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
    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

    function check($pass) 
    { if (strpos($pass,'0') || strpos($pass,'1') 
        || strpos($pass,'2') || strpos($pass,'3') 
        || strpos($pass,'4') || strpos($pass,'5') 
        || strpos($pass,'6') || strpos($pass,'7') 
        || strpos($pass,'8') || strpos($pass,'9')) 
        { 
         if (strpos($pass,'a') || strpos($pass,'b') || strpos($pass,'c') 
             || strpos($pass,'d') || strpos($pass,'e') || strpos($pass,'f') 
             || strpos($pass,'g') || strpos($pass,'h') || strpos($pass,'i') 
             || strpos($pass,'j') || strpos($pass,'k') || strpos($pass,'l') 
             || strpos($pass,'m') || strpos($pass,'n') || strpos($pass,'o') 
             || strpos($pass,'p') || strpos($pass,'q') || strpos($pass,'r') 
             || strpos($pass,'s') || strpos($pass,'t') || strpos($pass,'u') 
             || strpos($pass,'v') || strpos($pass,'w') || strpos($pass,'x') 
             || strpos($pass,'y') || strpos($pass,'z')) 
             { 
              if (strpos($pass,'A') || strpos($pass,'B') || strpos($pass,'C') 
                  || strpos($pass,'D') || strpos($pass,'E') || strpos($pass,'F') 
                  || strpos($pass,'G') || strpos($pass,'H') || strpos($pass,'I') 
                  || strpos($pass,'J') || strpos($pass,'K') || strpos($pass,'K') 
                  || strpos($pass,'M') || strpos($pass,'N') || strpos($pass,'O') 
                  || strpos($pass,'P') || strpos($pass,'Q') || strpos($pass,'R') 
                  || strpos($pass,'S') || strpos($pass,'T') || strpos($pass,'U') 
                  || strpos($pass,'V') || strpos($pass,'W') || strpos($pass,'X') 
                  || strpos($pass,'Y') || strpos($pass,'Z')) 
                  {                if (ctype_lower(substr($pass,0,1)) || ctype_upper(substr($pass,0,1)) || is_numeric(substr($pass,0,1))) 
                       {                        if (ctype_lower(substr($pass,0,1))) 
                               {                             if (!ctype_lower(substr($pass,-1))) 
                                    {                                  return $pass;                                 }                            } 
                           if (ctype_upper(substr($pass,0,1))) 
                               { 
                                if (!ctype_upper(substr($pass,-1))) 
                                    { 
                                     return $pass; 
                                    } 
                               } 
                           if (is_numeric(substr($pass,0,1))) 
                               { 
                                if (!is_numeric(substr($pass,-1))) 
                                    { 
                                     return $pass; 
                                    } 
                               }                    } 
                  } 
             } 
        } }

    без коментариев. (из одного вап чата)

    [email protected], 28 Декабря 2009

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

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

• 2104
• …
• 230
• 229
• 228
• 227
• 226
• 225
• 224
• 223
• 222
• …
• 1