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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
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. Лучший говнокод

   В номинации:

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

   За время:

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

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

   +102

   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 (
        !(this.Department.Length == 4 && 
               "0123456789".Contains(this.Department[0]) && 
               "0123456789".Contains(this.Department[1]) && 
               "0123456789".Contains(this.Department[2]) && 
               "0123456789".Contains(this.Department[3])) &&
        !(this.Department.Length == 4 && 
               "0123456789".Contains(this.Department[0]) && 
               "ABCDEFGHIJKLMNOPQRSTUVWXYZ".Contains(this.Department[1]) && 
               "ABCDEFGHIJKLMNOPQRSTUVWXYZ".Contains(this.Department[2]) && 
               "ABCDEFGHIJKLMNOPQRSTUVWXYZ".Contains(this.Department[3])) &&
        !(this.Department.Length == 5 && 
               "0123456789".Contains(this.Department[0]) && 
               "0123456789".Contains(this.Department[4]) && 
               "ABCDEFGHIJKLMNOPQRSTUVWXYZ".Contains(this.Department[1]) && 
               "ABCDEFGHIJKLMNOPQRSTUVWXYZ".Contains(this.Department[2]) && 
               "ABCDEFGHIJKLMNOPQRSTUVWXYZ".Contains(this.Department[3])))
        {
         return "error";
        }

   Он видимо не знал о существовании регулярных выражений

   NeoN, 02 Мая 2013

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

3. C++ / Говнокод #12871

   +8

   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

   ....
   #define MAX_PROC_LIST 1024
   enum PROCCESS_TYPE
   {
   	 OFFRO,
            GFEER,
            OLLYD,
   	 FLOOT,
             SFOIT,
   };
   
   typedef struct 
   {
   	PROCCESS_TYPE tType;
   	unsigned short uPid;
   	bool bIA32;
   }ACTIVE_PROCCESS_INFO, *PACTIVE_PROCCESS_INFO;
   ....
   
   bool MONITOR_PROCCESS_ENGINE::GetActiveProccessInfo(PACTIVE_PROCCESS_INFO pActiveInfoProccess, const unsigned int nIndex)
   {
   	if(nIndex < 0) return false;
   	PROCESSENTRY32 pe32;
   	HANDLE hProcessSnap = CreateToolhelp32Snapshot( TH32CS_SNAPPROCESS, 0 );
   	bool bFound = false;
   	pe32.dwSize = sizeof( PROCESSENTRY32 );
   	if(!Process32First( hProcessSnap, &pe32 )) return false;
   	char szNameImage[MAX_PATH];
   	ZeroMemory(szNameImage, MAX_PATH);
   	do 
   	{
   		if(pe32.th32ProcessID == aProccessList[nIndex])
   		{
   			bFound = true;
   		        wcstombs(szNameImage, (const wchar_t *)pe32.szExeFile, wcslen(pe32.szExeFile));
   			break;
   		}
   
   	} while( Process32Next( hProcessSnap, &pe32 ));
   	CloseHandle(hProcessSnap);
   
   	if(bFound && strlen(szNameImage)) 
           {
   		  if(!strcmp(szNameImage, szActiveList32[0])) 
   			  pActiveInfoProccess->tType = OFFRO;
   		  else
   		  if(!strcmp(szNameImage, szActiveList32[1])) 
   			   pActiveInfoProccess->tType = GFEER;
   		  else
   		  if(!strcmp(szNameImage, szActiveList32[2]))
   			  pActiveInfoProccess->tType =  OLLYD;
   		  else 
   		  if(!strcmp(szNameImage, szActiveList32[3]))
   			  pActiveInfoProccess->tType =  FLOOT;
   		  else 
   		  if(!strcmp(szNameImage, szActiveList32[4]))
   			  pActiveInfoProccess->tType =  SFOIT;
   		  else  return false;
   		 pActiveInfoProccess->uPid = (unsigned short) aProccessList[nIndex];
   		 pActiveInfoProccess->bIA32 = true; // Fixme x64?
   	}
   	else return false;
   	return true;
   }

   Cи в стиле C++

   not123, 09 Апреля 2013

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

4. JavaScript / Говнокод #12817

   +151

   1. 1

   if (Scene.generatedFast || (typeof generatedFast != "undefined" && generatedFast) )

   Lure Of Chaos, 28 Марта 2013

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

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

   +146

   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

   getInsideText($part1[0],'<string>','</string>',1,true);
   
   function getInsideText($str,$fstr,$lstr,$limit=0,$trim=true){
       $temp_arr=array();
       $lcnt=0;
       while(strpos($str,$fstr)!==false && ($limit ? $lcnt<$limit : true)){
           $fpos=($fstr ? strpos($str,$fstr)+strlen($fstr) : 0);
           $str=substr($str,$fpos);
           $lpos=strpos($str,$lstr);
           $val=($lpos!==false ? substr($str,0,$lpos) : $str);
           $temp_arr[]=($trim ? trim($val) : $val);
           $str=substr($str,$lpos+strlen($lstr));
           $lcnt++;
       }
       return ($limit==1 ? (isset($temp_arr[0]) ? $temp_arr[0] : '') : $temp_arr);
   }

   Конечный автомат своими силами.

   alexx, 21 Марта 2013

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

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

   +16

   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

   class Context;
   
   class AbstractState
   {
   	Context * m_context;
   
   protected:
   	Context * context() const { return m_context; }
   
   public:
   	AbstractState(Context * context) : m_context(context) { };
   	virtual ~AbstractState() { }
   	virtual void doSomething() = 0;
   };
   
   class Context
   {
   	std::unique_ptr<AbstractState> m_state;
   
   public:
   	enum State
   	{
   		State1,
   		State2,
   	};
   	Context() { switchToState(State1); }
   	void switchToState(State newState);
   	void doSomething() { m_state->doSomething(); }
   	void someCleanup() { }
   };
   
   class ConcreteState1 : public AbstractState
   {
   public:
   	ConcreteState1(Context * context) : AbstractState(context) { }
   	virtual void doSomething()
   	{
   		context()->switchToState(Context::State2);
   		context()->someCleanup();
   	}
   };
   
   class ConcreteState2 : public AbstractState
   {
   public:
   	ConcreteState2(Context * context) : AbstractState(context) { }
   	virtual void doSomething()
   	{
   		context()->switchToState(Context::State1);
   		context()->someCleanup();
   	}
   };
   
   void Context::switchToState(State newState)
   {
   	switch(newState)
   	{
   	case State1:
   		m_state.reset(new ConcreteState1(this));
   		return;
   	case State2:
   		m_state.reset(new ConcreteState2(this));
   		return;
   	}
   }

   Бывает, на меня находит состояние "сначала делай, потом думай", благо результат был быстро обнаружен отладчиком.

   Xom94ok, 10 Марта 2013

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

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

   −169

   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

   DECLARE @IsPerson BIT  
    ,@ContactTypeID INT  
   
    SET @ContactTypeID = CASE WHEN @IsPerson = 0 THEN 3 ELSE 4 END  
   
    IF (LEN(ISNULL(@FirstName,'')) <= 0 AND LEN(ISNULL(@LastName,'')) <= 0 AND (LEN(@EntityName) > 0 OR LEN(@TradingAs) > 0))  
    BEGIN  
     SET @IsPerson = 0  
    END   
    ELSE  
    BEGIN  
     SET @IsPerson = 1  
    END

   Такая себе Stored Procedure...
   Magic numbers - это всё фигня. Меня всегда интересовало, почему тип контакта всегда 4?
   И, да, строки идут именно в такой очерёдности.

   kore_sar, 28 Февраля 2013

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

8. JavaScript / Говнокод #12660

   +155

   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

   //...
                   else if (data.indexOf('none') == -1)
                   {
                       // Экспорт в работе
                       // Повторять запрос раз в nSec секунд.
                       nSec = 3;
                       document.getElementById("export_loading").show();
                       setTimeout("loadFile({{subject_id}}, '')", nSec*1000);
                   }
   //...
   <div class='loader' id="export_loading" style="visibility: hidden; display: inline-block; text-align: center; margin-right: 13px;"><img src='/app/images/loading.gif'/> идёт формирование списка авторов</div>
   <div id="messages_container" style="visibility: hidden; padding: 0px 30px 0 35px;">
   </div>

   Чувак вообще прикалывается)

   greevex, 27 Февраля 2013

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

9. PHP / Говнокод #12645

   +146

   1. 1

   strtotime(date('Y-m-d H:i:s'))

   xscript, 25 Февраля 2013

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

10. JavaScript / Говнокод #12641

    +152

    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

    function fixMootoolsJSON(thing) {
        var i, member, pattern = /^"\[.*\]"$/, copy;
        if (thing instanceof Array) {
            for (i = 0; i < thing.length; i++) {
                member = thing[i];
                if (typeof member == "string" && pattern.test(member)) {
                    thing[i] = fixMootoolsJSON(JSON.decode(member));
                }
            }
        } else if (typeof thing == "object") {
            copy = { };
            for (i in thing) {
                if (thing.hasOwnProperty(i)) {
                    copy[i] = fixMootoolsJSON(JSON.decode(thing[i]));
                }
            }
            for (i in copy) {
                if (copy.hasOwnProperty(i)) {
                    thing[i] = copy[i];
                }
            }
        }
        return thing;
    }

    http://outsourceror.blogspot.co.il/2011/04/mootools-intrudes-on-native-json-and.html

    
    Но вы не подумайте, оказалось, что я был первый (в нашем небольшом коллективе), кто это заметил, а (в нашем небольшом коллективе) было принято JSON прям как есть в базу сохранять...

    wvxvw, 24 Февраля 2013

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

11. PHP / Говнокод #12606

    +142

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

    $arr = date("w", strtotime($date_bron));
    $date = $arr;
    if ($arr == 0)
         $date = 0;

    Система бронирования столика для бара...

    unst, 17 Февраля 2013

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

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

• 1963
• …
• 57
• 56
• 55
• 54
• 53
• 52
• 51
• 50
• 49
• …
• 1