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

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

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

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

1. C# / Говнокод #1654

   +129

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

   private string              fEmail;
   ...
   public string Email  
   { 
       get {return fEmail = fEmail != null ? fEmail: string.Empty;}
       set {fEmail=value;}
   }

   чтобы проверить на нулл переменную fEmail, ее нужно зачем то присвоить самой же себе, долго вкуривал, но менять не стал. Наверное это заморочки с энхибернэйт
   
   З.Ы. версия фреймворка для которого было написано это чудо 1.1

   jackman, 21 Августа 2009

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

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

   +135.1

   1. 1

   private string[] months = new string[12] {"Январь","Февраль","Март", "Апрель","Май","Июнь","Июль","Август","Сентябрь","Октябрь","Ноябрь","Декабрь"};

   безусловно сильный ход. особенно для .NET

   stan, 21 Августа 2009

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

3. C# / Говнокод #1647

   +135.1

   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

   if ((Owner as Form1).ZakazdataGrid.Columns.Count == 0)
               {
                   (Owner as Form1).ZakazdataGrid.Columns.Add("KodTov", "");
                   (Owner as Form1).ZakazdataGrid.Columns.Add("NazvaTov", "Товар");
                   (Owner as Form1).ZakazdataGrid.Columns.Add("Kolvo", "Кол-во");
                   (Owner as Form1).ZakazdataGrid.Columns.Add("NazvaEd", "Ед.");
                   (Owner as Form1).ZakazdataGrid.Columns.Add("Suma", "Сумма");
                   (Owner as Form1).ZakazdataGrid.Columns.Add("KolEd", "");
                   (Owner as Form1).ZakazdataGrid.Columns.Add("Sostavnoy", "");
                   (Owner as Form1).ZakazdataGrid.Columns.Add("Sostav", "");
                   (Owner as Form1).ZakazdataGrid.Columns["Sostavnoy"].Visible = false;
                   (Owner as Form1).ZakazdataGrid.Columns["Sostav"].Visible = false;
                   (Owner as Form1).ZakazdataGrid.Columns["KodTov"].Visible = false;
                   (Owner as Form1).ZakazdataGrid.Columns["KolEd"].Visible = false;
                   (Owner as Form1).ZakazdataGrid.Columns["NazvaTov"].Width = (int)((Owner as Form1).ZakazdataGrid.Width * 0.5);
                   (Owner as Form1).ZakazdataGrid.Columns["NazvaTov"].SortMode = DataGridViewColumnSortMode.NotSortable;
                   (Owner as Form1).ZakazdataGrid.Columns["Kolvo"].Width = (int)((Owner as Form1).ZakazdataGrid.Width * 0.15);
                   (Owner as Form1).ZakazdataGrid.Columns["Kolvo"].SortMode = DataGridViewColumnSortMode.NotSortable;
                   (Owner as Form1).ZakazdataGrid.Columns["NazvaEd"].Width = (int)((Owner as Form1).ZakazdataGrid.Width * 0.15);
                   (Owner as Form1).ZakazdataGrid.Columns["NazvaEd"].SortMode = DataGridViewColumnSortMode.NotSortable;
                   (Owner as Form1).ZakazdataGrid.Columns["Suma"].Width = (int)((Owner as Form1).ZakazdataGrid.Width * 0.2);
                   (Owner as Form1).ZakazdataGrid.Columns["Suma"].SortMode = DataGridViewColumnSortMode.NotSortable;
                   (Owner as Form1).ZakazdataGrid.RowsDefaultCellStyle.Font = new Font("Times New Roman", FontSize - 5);
                   (Owner as Form1).ZakazdataGrid.ColumnHeadersDefaultCellStyle.Font = new Font("Times New Roman", FontSize - 5);
                   (Owner as Form1).ZakazdataGrid.AlternatingRowsDefaultCellStyle.BackColor = Color.PowderBlue;
                   (Owner as Form1).ZakazdataGrid.RowsDefaultCellStyle.BackColor = Color.LightSkyBlue;
               }

   Отсюда: http://sql.ru/forum/actualthread.aspx?tid=688754
   :-)

   VMcL, 21 Августа 2009

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

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

   +129.7

   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

   [StructLayout(LayoutKind.Explicit, Size=264)]
   public struct STRRET
   {
        [FieldOffset(0)]
        public UInt32 uType;    // One of the STRRET_* values
   
        [FieldOffset(4)]
        public IntPtr pOleStr;    // must be freed by caller of GetDisplayNameOf
   
        [FieldOffset(4)]
        public IntPtr pStr;        // NOT USED
   
        [FieldOffset(4)]
        public UInt32 uOffset;    // Offset into SHITEMID
   
        [FieldOffset(4)]
        public IntPtr cStr;        // Buffer to fill in (ANSI)
   }

   Обертка над структурой STRRET. Доставляет тут http://www.pinvoke.net/default.aspx/Structures/STRRET.html

   dethtroll, 20 Августа 2009

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

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

   +135.1

   1. 1

   this.txtGroupSubject.BackColor = System.Drawing.Color.FromArgb(((int)(((byte)(255)))), ((int)(((byte)(255)))), ((int)(((byte)(192)))));

   Виетнамский кастинг :)

   bugotrep, 18 Августа 2009

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

6. C# / Говнокод #1604

   +135.2

   1. 1

   Continue = (Continue) ? true : false;

   CADakaRK, 17 Августа 2009

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

7. C# / Говнокод #1600

   +139

   1. 1

   dgridPFF1.Rows[1].Cells[1].Value = UtilHelper.Number2String(Convert.ToDouble(UtilHelper.StringToInvariantCulture(dgridPFF1.Rows[0].Cells[1].Value), UtilHelper.getCultureInfo()) / Convert.ToDouble(UtilHelper.StringToInvariantCulture(dgridPFF2.Rows[0].Cells[0].Value),UtilHelper.getCultureInfo()), 2); // String.Format(UtilHelper.getCultureInfo(),UtilHelper.NUMBERIC_FORMAT_2,Convert.ToDouble(dgridPFF1.Rows[0].Cells[1].Value) / Convert.ToDouble(dgridPFF2.Rows[0].Cells[0].Value, UtilHelper.getCultureInfo()));

   Виетнамский jungle code :)

   bugotrep, 16 Августа 2009

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

8. C# / Говнокод #1599

   +138.9

   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

   public static void loadDataToDataGridView(DataGridView objDataGridView, List<object> objResultSet, IList objects)
           {
               if (objDataGridView == null || objResultSet == null) return;
   
               objDataGridView.Rows.Clear();
               for (int i = 0; i < objResultSet.Count; i++)
               {
                   for (int columnIndex = 0; columnIndex < ((List<object>)objResultSet[i]).Count; columnIndex++)
                   {
                       if (columnIndex == 0)
                       {
                           objDataGridView.Rows.Add();
                       }
                       objDataGridView.Rows[i].Cells[columnIndex].Value = ((List<object>)objResultSet[i])[columnIndex];
                   }
                   if (objects != null)
                   {
                       objDataGridView.Rows[i].Tag = objects[i];
                   }
               }
           }
   
           public static void loadDataToDataGridView(DataGridView objDataGridView, List<object> objResultSet)
           {
               loadDataToDataGridView(objDataGridView, objResultSet, null);
           }
   ...
   use of this code:
   ...
               List<object> values = new List<object>();
               List<object> value = new List<object>();
               if (_objPriceFormationFormula.RateVsEuro == 0)
               {
                   value.Add(UtilHelper.Number2String(1.00,2));// String.Format(UtilHelper.getCultureInfo(),UtilHelper.NUMBERIC_FORMAT_2,1.00));//1.00
               }
               else 
               {
                   value.Add(UtilHelper.Number2String(_objPriceFormationFormula.RateVsEuro,2));// String.Format(UtilHelper.getCultureInfo(),UtilHelper.NUMBERIC_FORMAT_2,_objPriceFormationFormula.RateVsEuro));
               }
               
               values.Add(value);
               UtilHelper.loadDataToDataGridView(this.dgridPFF2, values);

   Instead of databind :)

   bugotrep, 16 Августа 2009

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

9. C# / Говнокод #1598

   +139

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

   public bool IsEmpty()
           {
               int x = 0;
               x += EntityType.Length == 0 ? 1 : 0;
               x += EntityTypeName.Length == 0 ? 1 : 0;
               x += EntityDisplayName.Length == 0 ? 1 : 0;
               x += EntityId == Guid.Empty ? 1 : 0;
   
               return x == 0 ? false : true;
           }

   Аритметичный if :)

   bugotrep, 16 Августа 2009

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

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

    +137

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

    if (messages[i].description != "")
                                if (messages[i].description.StartsWith("<![CDATA["))
                                {
                                    messages[i].description = messages[i].description.Substring(9);
                                    messages[i].description = messages[i].description.Substring(0, messages[i].description.Length - 3);
                                }

    для строк типа <![CDATA[******]!>
    Возникает устойчивое ощущение говнокода. Как нормально написать? :)

    ikovalyov, 16 Августа 2009

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

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

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