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

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

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

1. PHP (5711)
2. C++ (2743)
3. Куча (2401)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1121)
8. Pascal (648)
9. Python (592)
10. 1C (540)
11. SQL (433)
12. ActionScript (292)
13. Objective C (202)
14. bash (201)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (46)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. Си / Говнокод #6378

   +145

   1. 1
   2. 2
   3. 3

   //Determine padding for use in allocating new memory
   int padding = 0;
   while ( (width * 3 + padding) % 4 != 0) padding++;

   http://www.cplusplus.com/forum/windows/6353/
   Ещё одна история о том, как людям не дано запомнить двоичную природу данных.
   Если что, вот корректный шаблон на D:
   
   int packed(int alignment)(int location) {
   static assert(alignment == 2 | alignment == 4 | alignment == 8);
   const badEnd = alignment - 1; //0b11, 0b111, 0b1111
   return location & badEnd ? (location & ~badEnd) + alignment : location;
   }

   denis-sh, 16 Апреля 2011

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

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

   +132

   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

   int cont[15];
   bool stop_access = false;
   struct cel container[MAX_SIZE];
   struct termios savetty;
   struct termios tty;
   pthread_t thread[100];
   pthread_mutexattr_t muattr; 
   pthread_mutex_t count_mutex; 
   int icn=0;//Number elements in container
   static int x;
   static int y;
   void* stack;
     pthread_mutex_unlock(&count_mutex); 
     sleep(1);
   };
   return (void*)1;
   };
   
   void* main_thread(void *arg){
      while(true){
      sleep(1);
     char ic=getchar();
       if(ic=='\n'){
         printf("Thread\tnumber elements\n");
         for(int i=0;i<=x;i++)
         pthread_cancel(thread[i]);
         for(int i=0;i<=x;i++){
   	printf("%d ",i);
   	int cn=0;
   	for(int j=0;j<=icn;j++){
   	  if(container[j].thrnm==i){ cn++; };
   	  };
   	  printf("\t%d\n",cn);
   	};
   	printf("container number element %d\n",icn);
   	exit(0);
         };
    };
     
   };
   
   int main(int argc, char * argv[])
   {
     icn=0;
     
   if(argc<2){printf("1 arg n number of threads, 2 argument maximum number element in container");exit(0);};
   x=atoi(argv[1]);
   y=atoi(argv[2]);
   if(x>64){printf("Число потоков должно быть меньше 64\n");exit(0);}
   printf("x %d y %d\n",x,y);
   //mutex initialization
   int ret;
   ret = pthread_mutexattr_init(&muattr);
   
     //switch the keyboard to noncanonical mode
   pid_t pt=tcgetsid(0);
    // char *var=ctermid (NULL);
     printf("pid %d\n",pt);
   tcgetattr (0, &tty);
   savetty = tty;
   tty.c_lflag &= ~(ICANON);
   tty.c_cc[VMIN] = 1;
   tcsetattr (0, TCSAFLUSH, &tty);
   pthread_t mthr;
    pthread_attr_t mattr;
     pthread_attr_init(&mattr);
     pthread_attr_setdetachstate(&mattr,PTHREAD_CREATE_DETACHED);
   int mres = pthread_create(&mthr, &mattr, main_thread, NULL);
     if (mres == 0) {
       printf("Creating main thread\n");
       sleep(0.7);
     } else {
       perror("Creating the main first thread");
     return EXIT_FAILURE;
     }
     
     
   for(int i=0;i<=x;i++){
     int id1, id2, result;
     id1 = 1;
     pthread_attr_t attr;
     pthread_attr_init(&attr);
     // отсоединенный поток - не ждем его возврата
     pthread_attr_setdetachstate(&attr,PTHREAD_CREATE_DETACHED);
     result = pthread_create(&thread[i], &attr, thread_func, &i);
     if (result == 0) {
       printf("Creating thread %d\n",i);
       sleep(1);
     } else {
       perror("Creating the first thread");

   AliceGoth, 14 Апреля 2011

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

3. Си / Говнокод #6349

   +147

   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

   void
   syslog(int pri, const char *fmt, ...)
   {
   	va_list ap;
   
   	va_start(ap, fmt);
   	vsyslog(pri, fmt, ap);
   	va_end(ap);
   }
   ......................................
   void
   vsyslog(int pri, const char *fmt, va_list ap)
   {
   	vsyslog_r(pri, &sdata, fmt, ap);
   }
   ....................................
   void
   vsyslog_r(int pri, struct syslog_data *data, const char *fmt, va_list ap)
   {
   	int cnt;
   	char ch, *p, *t;
   	time_t now;
   	int fd, saved_errno, error;
   #define	TBUF_LEN	2048
   #define	FMT_LEN		1024
   	char *stdp = NULL, tbuf[TBUF_LEN], fmt_cpy[FMT_LEN];
   	int tbuf_left, fmt_left, prlen;
   
   #define	INTERNALLOG	LOG_ERR|LOG_CONS|LOG_PERROR|LOG_PID
   	/* Check for invalid bits. */
   	if (pri & ~(LOG_PRIMASK|LOG_FACMASK)) {
   		if (data == &sdata) {
   			syslog(INTERNALLOG,
   			    "syslog: unknown facility/priority: %x", pri);
   		} else {
   			syslog_r(INTERNALLOG, data,
   			    "syslog_r: unknown facility/priority: %x", pri);
   		}
   		pri &= LOG_PRIMASK|LOG_FACMASK;
   	}
   .......................
   }
   ......................

   Из исходников Bioninc - Android libc.
   В реализации сислога, в случае ошибки, вызывается syslog

   elenbert, 14 Апреля 2011

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

4. Си / Говнокод #6312

   +146

   1. 1

   #define TRUE (rand() > 0.1 ? TRUE : FALSE) // happy debugging losers

   понедельничный неговнокод "Где здесь Си" = )
   
   приятного дебага на неделе, коллеги ))

   Lure Of Chaos, 11 Апреля 2011

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

5. Си / Говнокод #6306

   +136

   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

   [...]
   	switch (dssdev->type) {
   #ifdef CONFIG_OMAP2_DSS_DPI
   	case OMAP_DISPLAY_TYPE_DPI:
   #endif
   #ifdef CONFIG_OMAP2_DSS_RFBI
   	case OMAP_DISPLAY_TYPE_DBI:
   #endif
   #ifdef CONFIG_OMAP2_DSS_SDI
   	case OMAP_DISPLAY_TYPE_SDI:
   #endif
   #ifdef CONFIG_OMAP2_DSS_DSI
   	case OMAP_DISPLAY_TYPE_DSI:
   #endif
   #ifdef CONFIG_OMAP2_DSS_VENC
   	case OMAP_DISPLAY_TYPE_VENC:
   #endif
   #ifdef CONFIG_OMAP2_DSS_HDMI
   	case OMAP_DISPLAY_TYPE_HDMI:
   #endif
   		break;
   	default:
   		DSSERR("Support for display '%s' not compiled in.\n",
   				dssdev->name);
   		return;
   	}
   
   	switch (dssdev->type) {
   #ifdef CONFIG_OMAP2_DSS_DPI
   	case OMAP_DISPLAY_TYPE_DPI:
   		r = dpi_init_display(dssdev);
   		break;
   #endif
   #ifdef CONFIG_OMAP2_DSS_RFBI
   	case OMAP_DISPLAY_TYPE_DBI:
   		r = rfbi_init_display(dssdev);
   		break;
   #endif
   #ifdef CONFIG_OMAP2_DSS_VENC
   	case OMAP_DISPLAY_TYPE_VENC:
   		r = venc_init_display(dssdev);
   		break;
   #endif
   #ifdef CONFIG_OMAP2_DSS_SDI
   	case OMAP_DISPLAY_TYPE_SDI:
   		r = sdi_init_display(dssdev);
   		break;
   #endif
   #ifdef CONFIG_OMAP2_DSS_DSI
   	case OMAP_DISPLAY_TYPE_DSI:
   		r = dsi_init_display(dssdev);
   		break;
   #endif
   #ifdef CONFIG_OMAP2_DSS_HDMI
   	case OMAP_DISPLAY_TYPE_HDMI:
   		r = hdmi_init_display(dssdev);
   		break;
   #endif
   	default:
   		BUG();
   	}
   
   	if (r) {
   		DSSERR("failed to init display %s\n", dssdev->name);
   		return;
   	}
   
   [...]

   linux kernel..
   Драйвер контроллера дисплея (drivers/video/omap2/dss/display.c).
   Про кошмар из #ifndef-ов молчу, но к чему первый switch?

   grub670, 11 Апреля 2011

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

6. Си / Говнокод #6290

   +135

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

   PIMAGE_FILE_HEADER    pfh;
   PIMAGE_SECTION_HEADER    psh;
   PIMAGE_OPTIONAL_HEADER  poh;
   PVOID aa = LoadLibrary("calc.exe");
   GetHeaders((PCHAR) aa, &pfh, &poh, &psh);
   PPEB aae=GetPEB();
   aae->ImageBaseAddress = aa;
   LPVOID entry = (LPVOID)( (DWORD)aa + poh->AddressOfEntryPoint );
   __asm {
          call dword ptr [entry]
          }

   ссыль: http://www.wasm.ru/forum/viewtopic.php?pid=427180

   ReL, 08 Апреля 2011

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

7. Си / Говнокод #6270

   +143

   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

   void foo()
   {
       #define A_LEN 32
       #define B_LEN 40
   
       void* tmp_a = malloc(A_LEN);
       memset(tmp, 0, A_LEN);
   
       void* tmp_b = malloc(B_LEN);
       memset(tmp, 0, B_LEN);
   
       /* ... тело функции ...*/
   
       free(tmp_a);
       free(tmp_b);
   }

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

   Imperfectum, 07 Апреля 2011

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

8. Си / Говнокод #6249

   +146

   1. 1
   2. 2
   3. 3

   char[10] str;
   ...
   str[1] = '0' + 1;

   '1' ? Не, не слышал....

   DRAN1k, 06 Апреля 2011

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

9. Си / Говнокод #6244

   +144

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

   /* возвращает в buf последние 4 символа имени файла filename */
   void getfileext(unsigned char *buf, unsigned char *filename)
   {
       int i, j;
   
       for (i = strlen((char *) filename) - 4, j = 0; i < strlen((char *) filename); i++, j++)
           buf[j] = filename[i];
   
       return;
   }

   angry C nerd, 06 Апреля 2011

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

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

    +134

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

    int _Mbtowcx(wchar_t *pwc, const char *s, size_t nin, mbstate_t *pst, _Statab *pmbstate)
    	char state = (char)pst->_State;
    	unsigned char *su = (unsigned char *)s;
    	wchar_t wc = (wchar_t)pst->_Wchar;
    	static const mbstate_t initial = {0};
    
    	if (pmbstate->_Tab[0] == 0)	{	/* no table, convert from UTF8 */
    		if (s == 0)
    			{	/* set initial state */
    			*pst = initial;
    			return (0);
    			}
    
    		for (; ; ++su, --nin) {	/* consume an input byte */
    			if (nin == 0) {	/* report incomplete conversion */
    				pst->_Wchar = wc;
    				pst->_State = state;
    				return (-2);
    				}
    			else if (0 < state)	{	/* fold in a successor byte */
    				if ((*su & 0xc0) != 0x80) {	/* report invalid sequence */
    					errno = EILSEQ;
    					return (-1);
    					}
    				wc = (wchar_t)((wc << 6) | (*su & 0x3f));
    				--state;
    				}
    			else if ((*su & 0x80) == 0)
    				wc = *su;	/* consume a single byte */
    			else if ((*su & 0xe0) == 0xc0)	{	/* consume first of two bytes */
    				wc = (wchar_t)(*su & 0x1f);
    				state = 1;
    				}
    			else if ((*su & 0xf0) == 0xe0)	{	/* consume first of three bytes */
    				wc = (wchar_t)(*su & 0x0f);
    				state = 2;
    				}
    
    			else{	/* report invalid sequence */
    				errno = EILSEQ;
    				return (-1);
    				}
    			if (state == 0)	{	/* produce an output wchar */
    				if (pwc != 0)
    					*pwc = wc;
    				pst->_State = 0;
    				return (wc == 0 ? 0 : (const char *)++su - s);
    				}
    			}
    
    		}
    	else
    		{	/* run finite state machine */
    		int limit = 0;
    
    		if (s == 0)	{	/* set initial state */
    			*pst = initial;
    			return (pmbstate->_Tab[0][0] & _ST_STATE);
    			}
    
    		for (; ; )	{	/* perform a state transformation */
    			unsigned short code;
    			const unsigned short *stab;
    
    			if (nin == 0)
    				{	/* report incomplete conversion */
    				pst->_Wchar = wc;
    				pst->_State = state;
    				return (-2);
    				}
    			else if (_NSTATE <= state
    				|| (stab = pmbstate->_Tab[state]) == 0
    				|| (_NSTATE*UCHAR_MAX) <= ++limit
    				|| (code = stab[*su]) == 0)
    				{	/* report invalid sequence */
    				errno = EILSEQ;
    				return (-1);
    				}
    			state = (char)((code & _ST_STATE) >> _ST_STOFF);
    			if (code & _ST_FOLD)
    				wc = (wchar_t)(wc & ~UCHAR_MAX | code & _ST_CH);
    			if (code & _ST_ROTATE)
    				wc = (wchar_t)(wc << CHAR_BIT | UCHAR_MAX
    					& wc >> CHAR_BIT * (sizeof (wchar_t) - 1));
    			if (code & _ST_INPUT && *su != '\0')
    				++su, --nin, limit = 0;
    			if (code & _ST_OUTPUT)
    				{	/* produce an output wchar */
    				int nused = (const char *)su - s;
    
    				if (pwc)
    					*pwc = wc;
    				pst->_Wchar = wc;
    				pst->_State = state;
    				return (wc == 0 ? 0 : nused == 0 ? -3 : nused);
    				}
    			}
    		}
    	}

    Долго не мог понять почему не работает
    setlocale(...);
    _setmbcp(...);
    mbtowc(...);
    
    на C++ Builder. Пока не заглянул в исходники.

    sanchousf, 05 Апреля 2011

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

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

• 113
• …
• 33
• 32
• 31
• 30
• 29
• 28
• 27
• 26
• 25
• …
• 1