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

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

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

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

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

   В номинации:

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

   За время:

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

2. Java / Говнокод #4783

   +70

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

   if(url.getProtocol().equals("http") || true)
   {
      // ...
   }

   Индийский код добрался и до Android...

   Highlander, 29 Ноября 2010

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

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

   +145

   1. 1

   return !!$this->db->where('id', $for_id)->update($for, $row);

   ояебал, у них наверное где-то склад с веществами

   DrFreez, 28 Ноября 2010

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

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

   +68

   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

   package bytestring;
   
   public class Main {
   
       public static void main(String[] args) {
           String source = new String("A ya sdelal etu hren s perevorotom stroki s ispolzovaniem bayta");
   
           byte bytes[] = source.getBytes();
   
           ////////////////////////////////////////////////////////////////////////
   
           int i, j;
           int length, first, last;
           byte a;
   
           length = bytes.length;
   
           //Переворачиваем всю строку
           for(i = 0; i < length / 2; i++) {
               a = bytes[i];
               bytes[i] = bytes[length - i - 1];
               bytes[length - i - 1] = a;
           }
   
           //Переворачиваем каждое слово строки
           first = 0;
           for(i = 1; i <= length; i++)
               if(i == length || bytes[i] == ' ') {
                   last = i - 1;
                   for(j = first; j <= first + (last - first) / 2; j++) {
                       a = bytes[j];
                       bytes[j] = bytes[first + last - j];
                       bytes[first + last - j] = a;
                   }
                   first = i + 1;
               }
   
           ////////////////////////////////////////////////////////////////////////
   
           char destination[] = new char[bytes.length];
           for(i = 0; i < bytes.length; i++)
               destination[i] = (char) bytes[i];
   
           System.out.println(String.copyValueOf(destination));
       }
   }

   hedgecrab, 28 Ноября 2010

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

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

   +159

   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

   // TODO: use Virtual memory instead of heap!
   #ifndef __CHUNK_H__
   #define __CHUNK_H__
   #include <windows.h>
   #include "../JuceLibraryCode/JuceHeader.h"
   
   class Chunk {
   
   public:
   	enum CHUNK_DIRECTION {CHUNK_UNKNOWN = 0, CHUNK_IN, CHUNK_OUT};
   	Chunk (DWORD ChunkSize, WORD id, CHUNK_DIRECTION chunkDirection);
   	~Chunk ();
   	// override
   	virtual void eventChunkIsEmpty () { 
   		Logger::outputDebugString(T("empty")); 
   	}
   	virtual void eventChunkIsFull () {
   		Logger::outputDebugString(T("full"));
   	}
   	virtual void eventChunkOverrun () {
   		Logger::outputDebugString(T("overrun"));
   	}
   	DWORD getData (WORD *data, DWORD size) {
   		if (data == 0) return 0;
   		if (cs.tryEnter ()) { // if it's true, we locked.. (TODO: check, i'm not sure about that)
   			if (((size + nReadCounter) > nWriteCounter) || size == 0) { cs.exit(); return 0; }
   			memcpy (data, pBuffer + nReadCounter, size*sizeof(WORD));
   			nReadCounter += size;
   			if (nReadCounter == nWriteCounter) { eventChunkIsEmpty() ; nWriteCounter = 0; nReadCounter = 0; }
   			cs.exit ();
   			return size;
   		}
   		return 0;
   	}
   	DWORD putData (WORD *data, DWORD size) {
   		if (data == 0) return 0;
   		if (cs.tryEnter ()) { // if it's true, we locked.. (TODO: check, i'm not sure about that)
   			if ((size + nWriteCounter) > nSize) { eventChunkOverrun(); cs.exit (); return 0; }
   			memcpy (pBuffer + nWriteCounter, data, size*sizeof(WORD));
   			nWriteCounter += size;
   			if (nWriteCounter == nSize ) eventChunkIsFull();
   			cs.exit ();
   			return size;
   		}
   		return 0;
   	}
   	inline DWORD getSize () {
   		return nSize;
   	}
   	// TODO: add check for nWriteCounter?
   	inline bool setSize (DWORD ChunkSize) {
   		if (bExchangeIsActive) return false;
   		nSize   = ChunkSize;
   		// TODO: add result check.
   		pBuffer = (WORD*) realloc ((void*)pBuffer, ChunkSize*sizeof(WORD));
   		if (pBuffer) return true;
   		return false;
   	}
   	inline DWORD getReadCounter () { return nReadCounter; }
   	inline DWORD getWriteCounter () { return nWriteCounter; }
   	juce_UseDebuggingNewOperator
   protected:
   	bool            bExchangeIsActive;
   	CHUNK_DIRECTION cdDirection;
   	DWORD           nSize;
   	DWORD           nWriteCounter;
   	DWORD           nReadCounter;
   	WORD            nChunkId;
   	WORD           *pBuffer;
   	CriticalSection cs;
   };
   #endif
   // EOF
   #include "Chunk.h"
   Chunk::Chunk (DWORD ChunkSize, WORD id, CHUNK_DIRECTION chunkDirection) {
   	bExchangeIsActive = false;
   	cdDirection       = chunkDirection;
   	nSize             = ChunkSize;
   	nWriteCounter     = 0;
   	nReadCounter      = 0;
   	nChunkId          = id;
   	pBuffer           = (WORD*) malloc (ChunkSize*sizeof(WORD));
   	zeromem (pBuffer, ChunkSize*sizeof(WORD));
   }
   Chunk::~Chunk () {
   	if (pBuffer) free (pBuffer);
   }
   // EOF

   Посвящается всем изобретателям велосипедов и просто неудачникам.. :(

   neudachnik, 27 Ноября 2010

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

6. PHP / Говнокод #4769

   +168

   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

   function cleanUrl( $url ){
   	
   	$new_url = str_replace( " ", "-", $url );
   	$new_url = str_replace( "/", "-", $new_url );
   	$new_url = str_replace( "--", "-", $new_url );
   	$new_url = str_split( $new_url );
   	$new_url = preg_grep( "<[A-Za-z0-9-_]+>", $new_url );
   	$new_url = trim( implode( $new_url ) );
   	$new_url = strtolower($new_url);
   	
   	// Was still getting cases of double hyphens
   	$new_url = str_replace( "--", "-", $new_url );
   	
   	return $new_url;
   	
   }// end function - cleanUrl

   Спасаю "Говнокод.ру" пехапе шедевром.

   Yurik, 27 Ноября 2010

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

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

   +165

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

   if (count($arResult[$j]["DATA"])!=1) $btw_max = count($arResult[$j]); else $btw_max = 0;
   
   				if ($btw_max ==2) $btw_max =1;
   
   
   
   				for ($k=0;$k<$btw_max;$k++){...}

   Суть проблемы, есть массив $arResult[$j]["DATA"], последний элемент которого лишний, его выводить не надо, для этого чувак сделал следующее
   Вместо того, чтобы это значения не добавлять в массив при его заполнении, либо исключив последний элемент при выводе. В общем труЪ говнокод :D

   minnigaliev-r, 21 Ноября 2010

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

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

   +112

   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

   sSQLTitle := 'SELECT  CASE WHEN (VTSS.WRKORDNO IS NOT NULL)'
       +' THEN VTSS.WRKORDNO ELSE VTW.WRKORDNO END AS '''+RSNumDok+''','
       +' CASE WHEN (VTSS.GRECNO IS NOT NULL) THEN VTSS.GRECNO ELSE VTW.GRECNO END AS '''+RSSchet+''','
       +' CASE WHEN (VTSS.STYPE IS NOT NULL) THEN VTSS.STYPE ELSE VTW.STYPE END AS '''+RSDocType+''','
       +' CASE WHEN (VTSS.ServD IS NOT NULL) THEN VTSS.ServD ELSE VTW.ServD END AS '''+RSOpen+''','
       +' CASE WHEN (VTSS.BILLD IS NOT NULL) THEN VTSS.BILLD ELSE VTW.BILLD  END AS '''+RSClose+''','
       +' CASE WHEN (VTSS.FULLNAME IS NOT NULL) THEN VTSS.FULLNAME ELSE VTW.FULLNAME  END AS '''+RSClient+''','
       +' CASE WHEN (VTSS.PAYNAME IS NOT NULL) THEN VTSS.PAYNAME ELSE VTW.PAYNAME  END AS '''+RSPlatelwik+''','
       +' CASE WHEN (VTSS.VIN IS NOT NULL) THEN VTSS.VIN ELSE VTW.VIN  END AS '''+RSVIN+''','
       +' CASE WHEN (VTSS.GNUMER IS NOT NULL) THEN VTSS.GNUMER ELSE VTW.GNUMER  END AS '''+RSGosNomer+''','
       +' CASE WHEN (VTSS.HSMANID IS NOT NULL) THEN VTSS.HSMANID ELSE VTW.HSMANID  END AS '''+RSOtvetstv+''','
       +' CASE WHEN (VTSS.LBACK IS NOT NULL) THEN VTSS.LBACK ELSE VTW.LBACK  END AS '''+RSReturn+''','
       +' VTSS.FULLSUMMA AS '''+RSSumItems+''','
       +' VTW.SUMMA AS '''+RSSumWork+'''';
   with DataMdl.adsADODataSet do
     begin
       Active := false;
       if rbAllDate.Checked then
       {$REGION 'Поиск по всем датам'}
         case rgZakazType.ItemIndex of
           0: //поиск по открытым заказ-заявкам
           CommandText:=sSQLTitle
             +' FROM'
             +' (SELECT  VTS.WRKORDNO,'
          (...)
             +' SUM(SUMMA) AS FULLSUMMA'
             +' FROM'
             +' (SELECT GS.WRKORDNO,'
             +' GB.GRECNO,'
             + sSQLCase
             +' CONVERT(varchar(10), GS.ServD, 104) AS ServD,'
             +' CONVERT(varchar(10), GB.BILLD,104) AS BILLD,'
             +' CASE'
             +'	WHEN C.FNAME IS NULL THEN C.LNAME'
             +'	WHEN C.LNAME IS NULL THEN C.FNAME'
             +' ELSE C.FNAME+'' ''+C.LNAME'
             +'	END AS FULLNAME,'
   //          +' C.LNAME + '' '' + C.FNAME AS FULLNAME,'
             +' CS.PNAME AS PAYNAME,'
             +' V.SERIALNO AS VIN,'
             +' V.LicNo AS GNUMER,'
             +' GB.HSMANID,'
             +' CASE WHEN (GS.CREDIT IS NULL)OR (GS.CREDIT <> 1) THEN NULL ELSE '''+RSReturn+''' END AS LBACK,'
             +' GW.NAME,'
             +' SUM(GW.ORDNUM) AS '''+RSQuantity+''','
             +' SUM(GW.ORDNUM)*GW.UNITPR AS SUMMA'
             +' FROM GSAL'+sPostfix+' GS LEFT OUTER JOIN'
             +' GROW'+sPostfix+' GW ON (GS.GSALID = GW.GSALID)  LEFT OUTER JOIN'
             +' GBIL'+sPostfix+' GB ON (GS.GSALID = GB.GSALID) LEFT OUTER JOIN'
             +' VEHI V ON (GS.VEHIID = V.VEHIID) LEFT OUTER JOIN'
             +' CUST C ON (GS.CUSTNO = C.CUSTNO) RIGHT OUTER JOIN'
             +' (SELECT     CustNo, Lname + '' '' + FNAME AS PNAME'
             +' FROM CUST WITH (NOLOCK))'
             +' CS ON (GB.CUSTNO = CS.CUSTNO)'
             +' WHERE (GS.STATUS = ''A'') AND (GW.RTYPE IN (1,2))'  +sSQLWhere
             +' GROUP BY  GW.ITEM, GW.NAME, GW.UNITPR,'
             +' GS.WRKORDNO, GB.Grecno, GS.ServD, GB.BILLD, C.LNAME, C.FNAME,'
             +' CS.PNAME, V.SERIALNO, V.LicNo, GB.HSMANID, GS.CREDIT, GW.RECMTIME, GS.STYPE)'
             +' VTS'
             +' CASE WHEN (GS.CREDIT IS NULL)OR (GS.CREDIT <> 1) THEN NULL ELSE '''+RSReturn+''' END AS LBACK,'
             +' GW.NAME,'
             +' SUM(GW.ORDNUM) AS '''+RSQuantity+''','
             +' SUM(GW.ORDNUM)*GW.UNITPR AS SUMMA'
             +' FROM GSAL'+sPostfix+' GS LEFT OUTER JOIN'
             +' GROW'+sPostfix+' GW ON (GS.GSALID = GW.GSALID)  LEFT OUTER JOIN'
             +' GBIL'+sPostfix+' GB ON (GS.GSALID = GB.GSALID) LEFT OUTER JOIN'
             +' VEHI V ON (GS.VEHIID = V.VEHIID) LEFT OUTER JOIN'
             +' CUST C ON (GS.CUSTNO = C.CUSTNO) RIGHT OUTER JOIN'
             +' (SELECT     CustNo, Lname + '' '' + FNAME AS PNAME'
          
        ...

   ... и так далее. НАПИСАНО ВРУЧНУЮ! Всего 420 строк, 4 подзапроса, связанных через full outer join. 16 тысяч записей 4-x ядерный mssql server обрабатывал таким образом минуты две. Я уж промолчу, как он по строкам в Delphi разбит...
   Два дня врубался, как оно работает, переписал за два часа. 58 строк, включая каждое поле из select с новой строки :)

   artemlight, 16 Ноября 2010

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

9. Pascal / Говнокод #4627

   +104

   1. 1
   2. 2

   if doc_lang='UA' then filename:=ExtractFileDir(Application.ExeName)+'/docs/sales_slip_MS_UA.xlt';
   if doc_lang='RU' then filename:=ExtractFileDir(Application.ExeName)+'/docs/sales_slip_MS_RU.xlt';

   Классика жанра, и так пол проги((((((

   Nikitiy_II, 14 Ноября 2010

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

10. C# / Говнокод #4623

    +117

    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

    private XmlNode GetListItemsXML(bool _flag, string _param, SPList _list)
            {
                SPQuery oEmptyQuery = new SPQuery();
                SPListItemCollection spliColl;
                XmlReaderSettings sett = new XmlReaderSettings();
                Stream stream;
                XmlReader reader;
                XmlDocument listItems = new XmlDocument();
                string str = "";
                try
                {
                    switch (_flag)
                    {
                        case true:
                            //select division
                            oEmptyQuery.Query = "<Where><Eq><FieldRef Name=\"URL\"/><Value Type=\"Text\">" + _param + "</Value></Eq></Where>";
                            spliColl = _list.GetItems(oEmptyQuery);
    
                            str = c_my_headFile + spliColl.Xml;
                            str.Replace("\r\n", "");
                            stream = new MemoryStream(new UTF8Encoding(true).GetBytes(str));
    
                            
                            sett.ConformanceLevel = ConformanceLevel.Auto;
                            reader = XmlReader.Create(new StringReader(str), sett);
    
                            
    
                            listItems.LoadXml(str);
                            return listItems.SelectSingleNode("xml").ChildNodes[1].ChildNodes[0];
                            //return listItems.SelectSingleNode("xml").ChildNodes[1];
                        case false:
                            //select articles
                            //_x0420__x0430__x0437__x0434__x04
                            oEmptyQuery.Query = "<Where><Eq><FieldRef Name=\"_x0420__x0430__x0437__x0434__x04\"  LookupId=\"TRUE\" /><Value Type=\"Lookup\">" + _param + "</Value></Eq></Where>";
                            spliColl = _list.GetItems(oEmptyQuery);
    
                            str = c_my_headFile + spliColl.Xml;
                            str.Replace("\r\n", "");
                            stream = new MemoryStream(new UTF8Encoding(true).GetBytes(str));
    
                            
                            sett.ConformanceLevel = ConformanceLevel.Auto;
                            reader = XmlReader.Create(new StringReader(str), sett);
    
                            
    
                            listItems.LoadXml(str);
                            //return listItems.SelectSingleNode("xml").ChildNodes[1].ChildNodes[0];
                            return listItems.SelectSingleNode("xml").ChildNodes[1];
                    }
                }
                catch (XmlException)
                {
                    return null;
                }
                catch (Exception)
                {
                    return null;
                }
                return null;
            }

    Метод, у которого говорящая сигнатура. Увидев ее можно сразу понять что делает этот мегакусок.
    Если кто-нибудь поймет зачем создается экземпляр ридера - объясните. Блин я не понял, честно... xD
    (SPQuery и др. это опять же SharePoint API.)

    dreaktor, 14 Ноября 2010

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

11. ActionScript / Говнокод #4585

    −159

    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

    if( !ruleWithLongName1 )
    {
    	return;
    }
    if( !ruleWithLongName2 )
    {
    	return;
    }
    if( !ruleWithLongName3 )
    {
    	return;
    }
    if( !ruleWithLongName4 )
    {
    	return;
    }
    if( !ruleWithLongName5 )
    {
    	return;
    }

    клевый кусочек (повторяется в нескольких функциях) имена условий изменены, но скажу что они не Boolean типа

    Werdn, 11 Ноября 2010

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

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

• 1970
• …
• 359
• 358
• 357
• 356
• 355
• 354
• 353
• 352
• 351
• …
• 1