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

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

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

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

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

   В номинации:

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

   За время:

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

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

   +133

   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

   if (wParam == MI_CMDLINEBEGIN)
             return xstrcpyW((void *)lParam, wpCmdLineBegin);
           if (wParam == MI_CMDLINEEND)
             return xstrcpyW((void *)lParam, wpCmdLineEnd);
           if (wParam == MI_SHOWMODIFY)
             return moCur.dwShowModify;
           if (wParam == MI_STATUSPOSTYPE)
             return moCur.dwStatusPosType;
           if (wParam == MI_STATUSUSERFORMAT)
             return xstrcpynW((void *)lParam, moCur.wszStatusUserFormat, MAX_PATH);
           if (wParam == MI_WORDBREAKCUSTOM)
             return moCur.dwWordBreakCustom;
           if (wParam == MI_PAINTOPTIONS)
             return moCur.dwPaintOptions;
           if (wParam == MI_EDITSTYLE)
             return moCur.dwEditStyle;
           if (wParam == MI_RICHEDITCLASS)
             return moCur.bRichEditClass;
           if (wParam == MI_AKELADMINRESIDENT)
             return moCur.bAkelAdminResident;
           if (wParam == MI_DATELOGFORMAT)
             return xstrcpynW((void *)lParam, moCur.wszDateLogFormat, 128);
           if (wParam == MI_DATEINSERTFORMAT)
             return xstrcpynW((void *)lParam, moCur.wszDateInsertFormat, 128);
           if (wParam == MI_AKELUPDATEROPTIONS)
             return xstrcpynW((void *)lParam, moCur.wszAkelUpdaterOptions, MAX_PATH);
           if (wParam == MI_URLCOMMAND)
             return xstrcpynW((void *)lParam, moCur.wszUrlCommand, MAX_PATH);
           if (wParam == MI_TABNAMEFIND)
             return xstrcpynW((void *)lParam, moCur.wszTabNameFind, MAX_PATH);
           if (wParam == MI_TABNAMEREP)
             return xstrcpynW((void *)lParam, moCur.wszTabNameRep, MAX_PATH);
           if (wParam == MI_ONTOP)
             return moCur.bOnTop;
           if (wParam == MI_STATUSBAR)
             return moCur.bStatusBar;
           if (wParam == MI_KEEPSPACE)
             return moCur.bKeepSpace;
           if (wParam == MI_WATCHFILE)
             return moCur.bWatchFile;
           if (wParam == MI_SAVETIME)
             return moCur.bSaveTime;
           if (wParam == MI_SINGLEOPENFILE)
             return moCur.bSingleOpenFile;
           if (wParam == MI_SINGLEOPENPROGRAM)
             return moCur.dwSingleOpenProgram;
           if (wParam == MI_TABOPTIONSMDI)
             return moCur.dwTabOptionsMDI;
           if (wParam == MI_EXECUTECOMMAND)

   И такого больше сотни строк.
   Akelpad.

   gost, 25 Ноября 2014

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

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

   +133

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

   int size;
   
   size = EXPR;
   
   if (size > INT_MAX || size <= 0) {
       return NULL;
   }
   
   // ...

   Ндя. Семь лет уже. Теперь всё понятно...

   bot, 23 Ноября 2014

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

4. C# / Говнокод #17077

   +133

   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

   [DataContract]
           class outColumnsList
           {
   ...
               [DataMember]
               public decimal? FK_ID_OBJECT { get; set; }
   ...
               [DataMember]
               public decimal? IS_PK { get; set; }
               [DataMember]
               public decimal? IS_EDIT { get; set; }
   ..
           }
   ....
                   List<outColumnsList> l = new List<outColumnsList>();
                   while (r.Read())
                   {
                       outColumnsList l1 = new outColumnsList();
                       if (r.GetOracleValue(0).ToString()!="null")
                       l1.ID_OBJECT = Convert.ToDecimal(r.GetOracleValue(0).ToString());
                       l1.NM_DISPLAY = r.GetOracleValue(1).ToString();
                       l1.NM_SOURCE = r.GetOracleValue(2).ToString();
                       if (r.GetOracleValue(3).ToString() != "null")
                       l1.DISPLAY_SIZE = Convert.ToDecimal(r.GetOracleValue(3).ToString());
                       if (r.GetOracleValue(4).ToString() != "null")
                       l1.FK_ID_OBJECT = Convert.ToDecimal(r.GetOracleValue(4).ToString());
                       if (r.GetOracleValue(5).ToString() != "null")
                       l1.ORDER_DISPLAY = Convert.ToDecimal(r.GetOracleValue(5).ToString());
                       if (r.GetOracleValue(6).ToString() != "null")
                       l1.IS_EDIT = Convert.ToDecimal(r.GetOracleValue(6).ToString());
                       if (r.GetOracleValue(7).ToString() != "null")
                       l1.IS_PK = Convert.ToDecimal(r.GetOracleValue(7).ToString());
                       l1.FK_FIELD = r.GetOracleValue(8).ToString();
                       l1.NM_SOURCE_FK = r.GetOracleValue(10).ToString();
                       l.Add(l1);
                   }

   это потрясающе. в качестве идентификатора использовать дробные числа... хорошо хоть не с плавающей запятой.
   а так же, их же, для булевых....
   
   а то, как они из базы читались, так это вообще сказка.
   все ToString(), и все ConvertToDecimal... и проверка, на null еще тоже на уровне. про IsDBNull явно не слышал ничего

   Lokich, 07 Ноября 2014

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

5. C# / Говнокод #16997

   +133

   1. 1
   2. 2
   3. 3

   TopPassGridBox.ItemsSource = CollectionViewSource.GetDefaultView((from t in Edit_Curent_Pass.PassFieldList
   	where Edit_Curent_Pass.PassFieldList.IndexOf(t) == 0 || Edit_Curent_Pass.PassFieldList.IndexOf(t) == 1 || Edit_Curent_Pass.PassFieldList.IndexOf(t) == 2
   	select t).ToList());

   Я так и не понял, что хотели этим сказать...

   SantePaulinum, 30 Октября 2014

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

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

   +133

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

   if (strcmp(request_name, REQUEST_1) == 0)
               function_1();
           else if (strcmp(request_name, REQUEST_2) == 0)
               function_2();
           else if (strcmp(request_name, REQUEST_3) == 0)
               function_3();
           /* И так далее */
           else if (strcmp(request_name, REQUEST_N) == 0)
               function_n();

   Собственно, как можно ЭТО рефакторить в более приемлемый вид и можно ли заменить на branchless?

   codemonkey, 15 Октября 2014

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

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

   +133

   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

   /*Checks whether the path exists and the path is directory, otherwise creates a new directory*/
   RET_VALS check_create_directory(const char* const dir_path, void (*print_func)(int, char *,...))
   {
   	RET_VALS ret_val;
   	struct stat sb;
   
   	ret_val = RET_OK;
   	if (NULL != dir_path)
   	{
   		if(NULL != print_func)
   		{
   			print_func(DBG_INFO, "%s - Checking %s existence\n", __FUNCTION__, dir_path);
   		}
   		else
   		{
   			dbgprintln("Checking %s existence", dir_path);
   		}
   		if (0 != stat(dir_path, &sb) || (false == S_ISDIR(sb.st_mode)))
   		{
   			ret_val |= RET_DIR_MISSING;
   			if(NULL != print_func)
   			{
   				print_func(DBG_INFO, "%s - %s is missing\n", __FUNCTION__, dir_path);
   			}
   			else
   			{
   				dbgprintln("%s is missing", dir_path);
   			}
   			errno = 0;
   			if (0 == mkdir(dir_path, S_IRWXU | S_IRGRP | S_IXGRP | S_IROTH | S_IXOTH))
   			{
   				ret_val |= RET_DIR_CREATE_OK;
   				if(NULL != print_func)
   				{
   					print_func(DBG_INFO, "%s - %s is successfully created\n", __FUNCTION__, dir_path);
   				}
   				else
   				{
   					dbgprintln("%s is successfully created", dir_path);
   				}
   			}
   			else
   			{
   				char err_msg[_K];
   
   				sprintf(err_msg, "Failed to create %s, error %04d - %s", dir_path, errno, strerror(errno));
   				ret_val |= RET_DIR_CREATE_FAILED;
   				//dbg_ffln(DBG_ERROR, "Failed to create %s", dir_path);
   				if(NULL != print_func)
   				{
   					print_func(DBG_INFO, "%s - %s\n", __FUNCTION__, err_msg);
   				}
   				else
   				{
   					dbgprintln("%s", err_msg);
   				}
   			}
   		}
   		else
   		{
   			ret_val |= RET_DIR_ALREADY_EXIST;
   			if(NULL != print_func)
   			{
   				print_func(DBG_INFO, "%s - %s already exists\n", __FUNCTION__, dir_path);
   			}
   			else
   			{
   				dbgprintln("%s already exists", dir_path);
   			}
   		}
   	}
   	else
   	{
   		ret_val = RET_DIR_MISSING;
   		if(NULL != print_func)
   		{
   			print_func(DBG_ERROR, "%s - No directory name is provided", __FUNCTION__);
   		}
   		else
   		{
   			dbgprintln("No directory name is provided");
   		}
   	}
   	return ret_val;
   }

   Продолжаем раскопки. Вообще весь .с файл можно сюда выложить.

   codemonkey, 12 Октября 2014

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

8. Java / Говнокод #16829

   +133

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

   /**
   	 * return angle of otrezok near the center, with circle point as a second part of the otrezok
   	 * 
   	 * @param center
   	 * @param circlePoint
   	 * @param len
   	 * @return
   	 */

   someone, 09 Октября 2014

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

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

   +133

   1. 1
   2. 2

   //str will destroy
   char* strdup_(char* str){

   wat?

   LispGovno, 07 Октября 2014

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

10. Куча / Говнокод #16802

    +133

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

    lc = $(subst A,a,$(subst B,b,$(subst C,c,$(subst D,d,$(subst E,e,$(subst F,f,$(subst G,g,$(subst H,h,$(subst I,i,$(subst J,j,$(subst K,k,$(subst L,l,$(subst M,m,$(subst N,n,$(subst O,o,$(subst P,p,$(subst Q,q,$(subst R,r,$(subst S,s,$(subst T,t,$(subst U,u,$(subst V,v,$(subst W,w,$(subst X,x,$(subst Y,y,$(subst Z,z,$1))))))))))))))))))))))))))
    
    VAR = MixedCaseText
    LOWER_VAR = $(call lc,$(VAR))
    
    all:
            @echo $(VAR)
            @echo $(LOWER_VAR)

    как реализовать портабельно lowercase функцию в GNU Make.
    
    как же я тебя временами лублу, мэйк.
    
    ЗЫ было случайно найдено в http://stackoverflow.com/questions/664601/in-gnu-make-how-do-i-convert-a-variable-to-lower-case

    Dummy00001, 06 Октября 2014

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

11. C# / Говнокод #16705

    +133

    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

    namespace pck_pkx
    {
        public partial class Form1 : Form
        {
            public Form1()
            {
                InitializeComponent();
            }
    
            private void button3_Click(object sender, EventArgs e)
            {
                if (textBox1.Text == "" || textBox2.Text == "")
                {
                    MessageBox.Show("Выберите файлы для слияния!");
                }
                else
                {
                    FileStream fs = new FileStream(textBox2.Text, FileMode.Open, FileAccess.Read);
                    BinaryReader br = new BinaryReader(fs);
                    BinaryWriter bw = new BinaryWriter(new FileStream(textBox1.Text, FileMode.Open, FileAccess.Write));
                    bw.BaseStream.Seek(0, SeekOrigin.End);
                    bw.Write(br.ReadBytes(Convert.ToInt32(fs.Length)));
                    MessageBox.Show("Успешно!");
                }
            }
    
            private void button1_Click(object sender, EventArgs e)
            {
                OpenFileDialog ofd = new OpenFileDialog();
                ofd.Filter = "Angelica engine files|*.pck";
                if (ofd.ShowDialog() == DialogResult.OK)
                {
                    textBox1.Text = ofd.FileName;
                }
            }
    
            private void button2_Click(object sender, EventArgs e)
            {
                OpenFileDialog ofd = new OpenFileDialog();
                ofd.Filter = "Angelica engine files|*.pkx";
                if (ofd.ShowDialog() == DialogResult.OK)
                {
                    textBox2.Text = ofd.FileName;
                }
            }
        }
    }

    Сука, до слез.
    http://kn1fe-zone.ru/index.php?threads/pck-pkx.399/

    DesmondHume, 16 Сентября 2014

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

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

• 1572
• …
• 744
• 743
• 742
• 741
• 740
• 739
• 738
• 737
• 736
• …
• 1