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

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

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

1. PHP (5709)
2. C++ (2740)
3. Куча (2390)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
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 (38)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. JavaScript / Говнокод #3678

   +170

   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

   function varEnb(a) {
   n=true
   try {
   z=eval(a)
   } catch(e) {
   n=false
   }
   if(n==true){
   return true
   } else {
   return false
   }
   }

   Проверяем наличие переменной

   eval, 10 Июля 2010

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

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

   +148

   1. 1

   $isOccupy=($valdet['isOccupy']=='1')?0:1;

   а все по тому что кое кто перепутал логику атрибута=\

   nur, 06 Июля 2010

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

4. PHP / Говнокод #3643

   +176

   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

   //вторая форма
   		$n2 = $_GET['n2'];//Компания
   		$n10 = $_GET['n10'];//Ф.И.О:
   		$n11 = $_GET['n11'];// Должность:
   		$n12 = $_GET['n12'];//Телефон:
   		$n13 = $_GET['n13'];//E-mail:
   		$n14 = $_GET['n14'];//
   		$n15 = $_GET['n15'];
   		$n16 = $_GET['n16'];
   		$n17 = $_GET['n17'];
   		$n18 = $_GET['n18'];
   		$n19 = $_GET['n19'];
   		$n20 = $_GET['n20'];
   		$n21 = $_GET['n21'];
   		$n22 = $_GET['n22'];
   		$n23 = $_GET['n23'];
   		$n24 = $_GET['n24'];
   		$n25 = $_GET['n25'];
   		$n26 = $_GET['n26'];
   		$n27 = $_GET['n27'];
   		$n28 = $_GET['n28'];
   		$n29 = $_GET['n29'];
   		$n30 = $_GET['n30'];
   		$n31 = $_GET['n31'];
   		$n32 = $_GET['n32'];
   		$n33 = $_GET['n33'];
   		$n34 = $_GET['n34'];
   		$n35 = $_GET['n35'];
   		$n37 = $_GET['n37'];//value - другое:
   		$n38 = $_GET['n38'];
   		$n39 = $_GET['n39'];
   		$n40 = $_GET['n40'];//new form - Картонные карты - Тираж(от 10 000 шт.): 
   		$n41 = $_GET['n41'];
   		$n42 = $_GET['n42'];
   		$n43 = $_GET['n43'];
   		$n44 = $_GET['n44'];
   		$n45 = $_GET['n45'];
   		$n46 = $_GET['n46'];
   		$n47 = $_GET['n47'];//new form - Услуги сборки - Количество комплектов: 
   		$n48 = $_GET['n48'];
   		$n49 = $_GET['n49'];
   		$n50 = $_GET['n50'];
   		$n51 = $_GET['n51'];
   		$n52 = $_GET['n52'];
   		$n53 = $_GET['n53'];
   		$n54 = $_GET['n54'];
   		$n55 = $_GET['n55'];
   		$n56 = $_GET['n56'];
   		$n57 = $_GET['n57'];
   		$n58 = $_GET['n58'];
   		$n59 = $_GET['n59'];//new form - Мейлинг - Есть
   		$n60 = $_GET['n60'];
   		$n61 = $_GET['n61'];
   		$n62 = $_GET['n62'];
   		$n63 = $_GET['n63'];
   		$n64 = $_GET['n64'];
   		$n65 = $_GET['n65'];
   		$n66 = $_GET['n66'];
   		$n67 = $_GET['n67'];
   		$n68 = $_GET['n68'];
   		$n69 = $_GET['n69'];
   		$n70 = $_GET['n70'];
   		$n71 = $_GET['n71'];
   		$n72 = $_GET['n72'];
   		$n73 = $_GET['n73'];
   		$n74 = $_GET['n74'];
   		$n75 = $_GET['n75'];
   		$n76 = $_GET['n76'];
   		$n77 = $_GET['n77'];
   		$n78 = $_GET['n78'];
   		$n79 = $_GET['n79'];
   		$n80 = $_GET['n80'];
   		$n81 = $_GET['n81'];
   		$n82 = $_GET['n82'];
   		$n83 = $_GET['n83'];
   		$n84 = $_GET['n84'];
   		$n85 = $_GET['n85'];
   		$n86 = $_GET['n86'];
   		$n87 = $_GET['n87'];
   		$n88 = $_GET['n88'];
   		$n89 = $_GET['n89'];
   		$n90 = $_GET['n90'];
   		
   		$subject2 = "Онлайн просчет";
   		$headers2 = "MIME-Version: 1.0\n";
   		$headers2 .= "Content-type: text/html; charset=utf-8\n";
   		$text2 = "<table border='0' cellspacing='0' cellpadding='0' width='100%' align='center' bgcolor='#f0f0f0'> <tr> 
   		<td><table border='0' cellspacing='0' cellpadding='0' width='600' align='center' bgcolor='#f0f0f0'> <tr> 
   		<td width='178' height='50' valign='center'><font size='2'>&nbsp;&nbsp;&nbsp;Компания* :</font></td> <td 
   		width='329'><input type=TEXT name='n2' value='".$n2."' readonly='readonly' size='49' style='border: 2; 
   		background: #e2e2e2' /></td> </tr> <tr> <td height='50' valign='center'><font

   этот пиздец был найден в коде одного проекта на жумле, в файле темплейта
   в конечном итогде это все отправлялось на мыло

   krasnoukhov, 05 Июля 2010

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

5. Куча / Говнокод #3606

   +133

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

   Below is a list of the warning message(s) produced when checking your document. 
    Line 91, Column 31: NET-enabling start-tag requires SHORTTAG YES 
   
           alt="Правильный CSS!" /> </a>                                  ✉ 
   
   The sequence <FOO /> can be interpreted in at least two different ways, depending on the DOCTYPE of the document...

   Валидатор ругается на иконку, подаренную валидатором

   bugmenot, 30 Июня 2010

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

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

   +194

   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

   function gopic(piciden) {
   piciden=piciden*1;
     if (piciden<0) {
       eval("document.risunki.ris0"+piciden+".disabled=true")
       eval("document.risunki.mark0"+piciden+".disabled=true")
       eval("mark=document.risunki.mark0"+piciden+".options[document.risunki.mark0"+piciden+".selectedIndex].value")
       }
     else {
       eval("document.risunki.ris"+piciden+".disabled=true")
       eval("document.risunki.mark"+piciden+".disabled=true")
       eval("mark=document.risunki.mark"+piciden+".options[document.risunki.mark"+piciden+".selectedIndex].value")
       }
     }
   
   function check_data(elem){
   a=0
   }

   3.14159265, 30 Июня 2010

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

7. SQL / Говнокод #3541

   −862

   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

   SELECT wine_name,wine_year,cost FROM Wines 
   INNER JOIN storage ON wines.wine_ID=storage.wine_ID
   INNER JOIN pricelist ON storage.wine_ID=pricelist.wine_ID
   WHERE fabric_ID=(SELECT fabric_ID FROM fabrics WHERE Name="Toscana Italy")
   AND count>0 ORDER BY wine_year ASC LIMIT 1,1000;
   
   SELECT wine_name,cost,count FROM wines
   INNER JOIN storage ON wines.wine_ID=storage.wine_ID
   INNER JOIN vendorprices ON storage.wine_ID=vendorprices.wine_ID
   WHERE count > 0 ORDER BY wine_name DESC LIMIT 1,1000;
   
   SELECT wine_name,cost,count FROM wines
   INNER JOIN storage ON wines.wine_ID=storage.wine_ID
   INNER JOIN vendorprices ON storage.wine_ID=vendorprices.wine_ID
   WHERE count = 0 ORDER BY wine_name DESC LIMIT 1,1000;

   Извращаюсь как могу лишь бы запросы напоминали двух-критериальные. Впрочем таковыми они от этого не становятся. Думаю еще добавить по три-четыре подзапроса и побольше джойнов, жаль условных конструкций не знаю(собственно сутки-двое изучаю SQL)
   Институт сказал "Надо", студент ответил... ну впрочем неважно что ответил, но сделал...

   Ivan0x32, 23 Июня 2010

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

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

   +146

   1. 1
   2. 2
   3. 3
   4. 4

   if (cur1!=NULL){
   	while(cur1!=NULL){
   	...
   }

   Вот такие вот потрясающие проверки в духе "нуавдруг?"

   tony, 22 Июня 2010

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

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

   +118

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

   public static IEnumerable<Tuple<A1, A2, A3, A4>> 
   			CrossProduct<A1, A2, A3, A4>
   			(IEnumerable<A1> arg1, IEnumerable<A2> arg2,
   			IEnumerable<A3> arg3, IEnumerable<A4> arg4)
   		{
   			foreach (var a1 in arg1)
   				foreach (var a2 in arg2)
   					foreach (var a3 in arg3)
   						foreach (var a4 in arg4)
   							yield return Make.Tuple(a1, a2, a3, a4);
   		}

   Интересно, а что этот человек будет делать, когда нужно будет n ступенек? :-)

   tinynick, 21 Июня 2010

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

10. Си / Говнокод #3421

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

    #include <iostream>
    #include <string.h>
    using namespace std;
    int strsearch(char* from, char* what);
    int getnumberafter(char* from, char* what);
    int strch(char* a, char* b, int pos);
    
    int strsearch(char* from, char* what)
    {
        int n=strlen(from)-strlen(what)+1;
        int i;
        for(i=0; i<n; i++)
    	if(strch(from, what, i)==strlen(what)) return i;
        return 0;
    }
    int getnumberafter(char* from, char* what)
    {
        int p, ret=0;
        if((p=strsearch(from,what))==0) return 0;
        p+=strlen(what);
        while(from[p]<'0'&&from[p]>'9')
        {
    	p++;
    	if(p>=strlen(from)) return 0;
        }
        while(from[p]>='0'&&from[p]<='9'&&p<strlen(from))
        {
    	ret=ret*10+(from[p++]-'0');
        }
        return ret;
    }
    int strch(char* a, char* b, int pos)
    {
        int i,n=min(strlen(a)-pos,strlen(b));
        for(i=0; i<n; i++)
    	if(a[i+pos]!=b[i]) return i;
        return i;
    }
    int main(int argc, char* argv[])
    {
        int i;
        double max=1,vol=0;
        for(i=0; i<argc; i++)
        {
    	cout << i << ": " << argv[i] << endl;
    	if(strch(argv[i],"type",0)==4) {  max=getnumberafter(argv[i],"max=");}
    	if(strch(argv[i],"values",0)==6) { vol=(getnumberafter(argv[i],"=")+getnumberafter(argv[i],","))/2;}
        }
        cout << vol/max << endl;
        return 0;
    }

    Задача из чего-то типа этого:
    numid=2,iface=MIXER,name='Master Playback Volume'
    ; type=INTEGER,access=rw---R--,values=2,min=0,max=31,step=0
    : values=31,31
    | dBscale-min=-46.50dB,step=1.50dB,mute=0
    Получить значение в процентах.
    #./a.out `amixer cget numid=2`
    0,67

    Tanger, 08 Июня 2010

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

11. C++ / Говнокод #3412

    +157

    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

    void foo()
    {
     if (!wxFile::Exists(defDir + resourcePath)) return false;
     HBITMAP *bmp = new HBITMAP;
     wxString path = defDir + resourcePath;
     (*bmp) = (HBITMAP)LoadImage(0, path.c_str(), IMAGE_BITMAP, 0, 0, LR_LOADFROMFILE | LR_CREATEDIBSECTION);
     if (bmp != NULL) (*pMap)[resourceName] = bmp; else return false;
     return true;
    }
    
    bool SXXXXBitmapStorage::releaseItem(wxString resourceName)
    {
     ScrollBitmapHash::iterator iterator = pMap->find(resourceName);
     if (iterator == pMap->end()) return false;
     if (iterator->second != NULL) delete (HBITMAP*)iterator->second;
     pMap->erase(resourceName);
     return true;
    }
    
    HDC* SXXXXBitmapStorage::adjustToDC(wxString resourceName)//we must delete this object
    {
     HBITMAP *bmp = this->getItem(resourceName);
     if (bmp == NULL) return NULL;
     HDC *pDC = new HDC();
     (*pDC) = CreateCompatibleDC(0);
     SelectObject(*pDC, *bmp);
     return pDC;
    }

    И вы не поверите, этот код работал! Правда, конечно же, была утечка ресурсов

    Hsilgos, 07 Июня 2010

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

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

• 1975
• …
• 1133
• 1132
• 1131
• 1130
• 1129
• 1128
• 1127
• 1126
• 1125
• …
• 1