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

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

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

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

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

   +136

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

   // Делаем из префикса количество хостов (без .0 и броадкаста)
   numips = pow(2.0, (double)(32 - slashnet)) - 2;
   
   // Делаем префикс из маски сети
   slashnet = 32 - ((int)log2((double)(0xFFFFFFFF - vnetconfig->nm)) + 1);

   Всё те же, всё оттуда же...

   raorn, 12 Мая 2010

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

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

   +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

   int check_process(pid_t pid, char *search) {
     char file[1024], buf[1024];
     FILE *FH=NULL;
     int rc, ret=0;
   
     snprintf(file, 1024, "/proc/%d/cmdline", pid);
     rc = check_file(file);
     if (!rc) {
       // cmdline exists
       ret = 1;
       if (search) {
         // check if cmdline contains 'search'
         FH = fopen(file, "r");
         if (FH) {
   	bzero(buf, 1024);
   	while(fgets(buf, 1024, FH)) {
   	  char *p;
   	  while((p = memchr(buf, '\0', 1024))) {
   	    *p = 'X';
   	  }
   	  buf[1023] = '\0';
   	  if (strstr(buf, search)) {
   	    ret = 0;
   	  }
   	}
   	fclose(FH);
         }
       } else {
         ret = 0;
       }
     } else {
       ret = 1;
     }
     return(ret);
   }

   Суровые калифорнийские студенты суровы.

   raorn, 11 Мая 2010

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

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

   +136

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

   if (init) {
   } else {
     // thread is not initialized, run first time local state setup
     ...
   }

   raorn, 11 Мая 2010

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

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

   +135

   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

   #define atfl_calc_opcode_(op) \
   ((op) == (ATFL_OP_PAGE_STORE|ATFL_BUF1|ATFL_PAGE_ERASE) ? 0x83 : (\
   	(op) == (ATFL_OP_PAGE_STORE|ATFL_BUF2|ATFL_PAGE_ERASE) ? 0x86 : (\
   	(op) == (ATFL_OP_PAGE_STORE|ATFL_BUF1|ATFL_PAGE_NO_ERASE) ? 0x88 : (\
   	(op) == (ATFL_OP_PAGE_STORE|ATFL_BUF2|ATFL_PAGE_NO_ERASE) ? 0x89 : (\
   	(op) == (ATFL_OP_PAGE_LOAD|ATFL_BUF1) ? 0x53 : (\
   	(op) == (ATFL_OP_PAGE_LOAD|ATFL_BUF2) ? 0x55 : (\
   	(op) == (ATFL_OP_BUF_READ|ATFL_BUF1) ? 0xD4 : (\
   	(op) == (ATFL_OP_BUF_READ|ATFL_BUF2) ? 0xD6 : (\
   	(op) == (ATFL_OP_BUF_WRITE|ATFL_BUF1) ? 0x84 : (\
   	(op) == (ATFL_OP_BUF_WRITE|ATFL_BUF2) ? 0x87 : (\
   	(op) == (ATFL_OP_PAGE_ERASE) ? 0x81 : (\
   	(op) == (ATFL_OP_PAGE_ERASE|ATFL_PAGE_ERASE) ? 0x81 : (\
   	0)))))))))))))

   Угадайте зачем.

   dmzlj.livejournal.com, 07 Мая 2010

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

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

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

   // народ на протяжении 5+ лет жаловался что постоянно что-то в этой библиотеке Х
   // криво потому что часто (с их слов) попытки вызова ее функций просто обламываются:
   
   int module_main_function()
   {
     int rc = 0;
     // ...
   
     rc = libX_do_work( ... )
     if (rc != OK) {
       // error!!!
     }
   
     // ...
   }
   
   // теперь заглянем в инициализацию этого модуля:
   
   int module_init()
   {
      int rc; 
      // ... кучи дежурного копи-паста для инициализации скипнуты ...
      rc = libZ_init();
      if (rc != OK)
      {
        /* error handling */
      }
   
      rc = libY_init();
      if (rc != OK)
      {
        /* error handling */
      }
   
      libX_init();   /* лопата */
      if (rc != OK)
      {
        /* error handling */
      }
   
      rc = libA_init();
      if (rc != OK)
      {
        /* error handling */
      }
   
      // ... и т.д. и т.п.
   }

   меня жалобы одного отдела начали потиху доставать что наша библиотека (назовем ее libX) у них в модуле постоянно не работает и шо ваабще за херня. как встроили ее - так и не работает. иногда. в 80+ модулях работает как часы - ни один другой отдел не жалуется - а вот в ихнем очень специальном модуле нет.
   
   ну я к ним подвалил. они меня сразу в рабочую функцию тыкать стали - вот тут дескать не работает. я им методично: покаж где инициализация. я когда в инициализацию заглянул, почти сразу начал роготять (давно так не смеялся, наболело за годы).
   им минут пять понадобилось что бы в своем копи-паст параде рассмотреть эту глюку.
   
   ЗЫ ответ: на строке с лопатой, не хватает "rc = ". у библиотеки есть внешняя конфигурация на отсутствии коей она у них обламывалась постоянно. и исправить скрипты свои что бы чужая конфигурация не убивалясь им похоже религия не позволяет.

   Dummy00001, 04 Мая 2010

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

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

   +124

   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 f()
   {
   char d[14] = "Hello, error!";
   }
   
   void h()
   {
   char d[14];
   std::cout<<d;
   }
   
   int main()
   {
   f();
   h();
   }

   Кто отгадает, что делает этот код(а еще лучше почему он вобще работает) тому ни чего не будет.

   hromjo, 02 Мая 2010

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

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

   +136

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

   memset( fc->key, 0, KEY_LEN+1);
   memset( fc->value, 0, VALUE_LEN+1);
   
   strncpy(fc->key, key, strlen( key) + 1);
   strncpy(fc->value, value,strlen( value) + 1);

   совершенное непонимание работы ф-ии strncpy

   benderlog, 30 Апреля 2010

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

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

   +135

   1. 1

   while( (*(curr_pos++) != 0x0a) && ((UINT)curr_pos - (UINT)buffer < (UINT)length) );

   Ищем новую строку

   dokonvog, 30 Апреля 2010

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

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

   +144

   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

   #include <stdio.h>
   
   int
   predicat
   (
   int
   i
   )
   {
       return
       i
       ==
       2
       ;
   }
   
   int 
   main
   (
   )
   {
       int
       boolean
       =
       0
       ;
       for
       (
       int
       i
       =
       0
       ;
       i
       <
       3
       &&
       !
       (
       boolean
       =
       predicat
       (
       i
       )
       )
       ;
       ++
       i
       )
       ;
       printf
       (
       "%i"
       ,
       boolean
       )
       ;
       return
       0
       ;
   }

   Есть мнение что этот код идеален - максимальная экономия горизонтального простанства. Компилируется (-std=C99)

   idkravitz, 29 Апреля 2010

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

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

    +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

    #define SPI_BUF_SIZE                                0x400
    #define SDRAM_SPI_0_BUF_MISO           (void*)0x100000
    #define SDRAM_SPI_0_BUF_MOSI           SDRAM_SPI_0_BUF_MISO - SPI_BUF_SIZE
    #define SDRAM_SPI_1_BUF_MISO           SDRAM_SPI_0_BUF_MOSI - SPI_BUF_SIZE
    #define SDRAM_SPI_1_BUF_MOSI           SDRAM_SPI_1_BUF_MISO - SPI_BUF_SIZE
    #define SDRAM_SPI_2_BUF_MISO           SDRAM_SPI_1_BUF_MOSI - SPI_BUF_SIZE
    #define SDRAM_SPI_2_BUF_MOSI           SDRAM_SPI_2_BUF_MISO - SPI_BUF_SIZE
    #define SDRAM_SPI_3_BUF_MISO           SDRAM_SPI_2_BUF_MOSI - SPI_BUF_SIZE
    #define SDRAM_SPI_3_BUF_MOSI           SDRAM_SPI_3_BUF_MISO - SPI_BUF_SIZE
    
    void test()
    {
    void *buf = SDRAM_SPI_3_BUF_MOSI;
    void *buf1 = (void*)0x100000 - 0x400*7;
    *buf = *buf1 = 0xAA;
    
    printf("buf=%.2x \tbuf1=%.2x\n ",*buf, buf1);
    return;
    }

    Расскажу преамбулу. Код писался под проц atmel at91sam7x
    В этом процессоре есть звено SPI (serial pereferial interface). Но так как звено было спроектировано с небольшим косячком (который кстати в эррате был явно описан) буффер для каждой линии нужно было выделять в сдраме.
    Сказано - сделано. Выделили. а так как средства отладки только DBGU и JTAG.
    Все волосы на жопе вырвал пока нашел в чем ошибка.
    Внимание, вопрос. В чем гавно?

    ursus, 29 Апреля 2010

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

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

• 112
• …
• 18
• 17
• 16
• 15
• 14
• 13
• 12
• 11
• 10
• …
• 1