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

    +119

    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
    public void SendCommand(string command, bool fake)
    {
        ...
        for (var tries = 0; tries < 2; tries++)
        {
            for (var i = 0; i < 2; i++)
                try
                {
                    sended = m_Socket.Send(sendBytes);
                    break;
                }
                catch (Exception e)
                {
                    Log.Error(LogSys, "SendCommand: Xception on send command to server: {0}", e.Message);
                    Thread.Sleep(m_Socket.SendTimeout);
                    if (i == 10)
                    {
                        Log.Error(LogSys, "SendCommand: cant send command in 10 tries.");
                        return;
                    }
                    continue;
                }
            ...
        }
        ...
    }

    Метод "отправил и забыл" ака "Почта России" казался ведущему разработчику способным решить задачу взаимодействия двух сетевых приложений. Возможно, в будущем он планировал написать код, который бы отслеживал изменения в логах и сообщал о найденных в них ошибках, но этот метод был обнаружен быстрее

    banderror, 04 Декабря 2010

    Комментарии (13)
  2. C# / Говнокод #4827

    +108

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    if (m_Level != value)
                    {
                        if (m_Level != null)
                        {
                            Logger.Unsubscribe(m_Level.Value, Logger_LogEvent);
                        }
                        m_Level = value;
                        if (m_Level != null)
                        {
                            Logger.Subscribe(m_Level.Value, Logger_LogEvent);
                        }
                    }

    Бывает и так. . .

    walash, 03 Декабря 2010

    Комментарии (33)
  3. C# / Говнокод #4824

    +126

    1. 1
    2. 2
    3. 3
    public static void Authorize(int id)
    {
    if (id != null)

    вдруг откуда нивозьмись

    ursus, 03 Декабря 2010

    Комментарии (6)
  4. C# / Говнокод #4819

    +115

    1. 1
    2. 2
    if(!string.IsNullOrEmpty(((AccessLevelObject)cbAccessLevel.SelectedItem).Description))
            txtAccessLevelDescription.Text = ((AccessLevelObject)cbAccessLevel.SelectedItem).Description;

    догадайтесь что здесь лишнее )))

    Maxim546, 03 Декабря 2010

    Комментарии (14)
  5. C# / Говнокод #4814

    +117

    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
    private void DBGridCategories_RowPostPaint(object sender, DataGridViewRowPostPaintEventArgs e)
            {
                int rowsCount = (sender as DataGridView).RowCount;
                if ((rowsCount >= 0) && (rowsCount < 100))
                {
                    (sender as DataGridView).RowHeadersWidth = 30;
                }
                else
                {
                    if ((rowsCount >= 100) && (rowsCount < 1000))
                    {
                        (sender as DataGridView).RowHeadersWidth = 40;
                    }
                    else
                    {
                        if ((rowsCount >= 1000) && (rowsCount < 10000))
                        {
                            (sender as DataGridView).RowHeadersWidth = 50;
                        }
                        else
                        {
                            if ((rowsCount >= 10000) && (rowsCount < 100000))
                            {
                                (sender as DataGridView).RowHeadersWidth = 65;
                            }
                        }
                    }
                }
                using (SolidBrush brush = new SolidBrush(Color.Red))
                {
                    e.Graphics.DrawString((e.RowIndex + 1).ToString(System.Globalization.CultureInfo.CurrentUICulture) + ".", e.InheritedRowStyle.Font, brush, e.RowBounds.Location.X + 10, e.RowBounds.Location.Y + 1);
                }
            }

    школьный профессионал ))

    Maxim546, 02 Декабря 2010

    Комментарии (5)
  6. C# / Говнокод #4810

    +118

    1. 1
    if (string.IsNullOrEmpty(email) != false)

    zheka, 02 Декабря 2010

    Комментарии (80)
  7. C# / Говнокод #4808

    +121

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    if (OnCreated != null)
                {foreach (EventHandler<CreatedEventArgs> handler in OnCreated.GetInvocationList())
                    {
                        handler(this, args);
                    }
    }

    человек слышал про GetInvocationList но когда его надо применять не понял - зато можно написать 4 строки вместо одной.
    В данном случае аналогом будет простой вызов OnCreated().

    eval_2009, 02 Декабря 2010

    Комментарии (2)
  8. C# / Говнокод #4807

    +113

    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
    [Localizable(false)]
        internal abstract class ClassAIndex : IDisposable
        {
            public static ClassAIndex Run()
            {
                ClassBIndex index = new ClassBIndex();
                index.initialize();
                return index;
            }
         
          // еще методы - абстрактные
       
        }
    
         internal class ClassBIndex : ClassAIndex
        {
             // тут имплементация
        }

    я заменил названия у ClassA &B. Надо ли упомянать что другого наследника от ClassA нет и не предвидится?
    Ну а localizable атрибут со значением по умолчанию у простого класса (не Component) это уже на сладкое.

    eval_2009, 02 Декабря 2010

    Комментарии (0)
  9. C# / Говнокод #4764

    +116

    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
    private void axVPortSDK_OnDIOStatus(object sender, AxVPORTSDKLib._DVPortSDKEvents_OnDIOStatusEvent e)
    {
        int i, nDIStatus;
        String szDIStatus = "";
    
        for (i = 3; i >= 0; i--)
        {
            if ((e.lDIStatus & (long)Math.Pow(2, i)) == ((long)Math.Pow(2, i)))
                nDIStatus = 1;
            else
                nDIStatus = 0;
            szDIStatus = szDIStatus + System.Convert.ToString(nDIStatus);
        }
        label_di.Text = szDIStatus;
        for (i = 0; i < 2; i++)
        {
            if (System.Convert.ToBoolean(e.lDOStatus & (long)Math.Pow(2, i)))
            {
                if (i == 0)
                    label_do1.Text = "Close";
                else
                    label_do2.Text = "Close";
            }
            else
            {
                if (i == 0)
                    label_do1.Text = "Open";
                else
                    label_do2.Text = "Open";
            }
        }
    }

    VPort ActiveX SDK PLUS от Moxa

    absolut, 27 Ноября 2010

    Комментарии (0)
  10. C# / Говнокод #4752

    +145

    1. 1
    2. 2
    3. 3
    4. 4
    public partial class TaskViewSearchParameter
    {
        public int Top5 { get; set; }
    }

    Часть класса одного из наших проектов

    ZlobnyiSerg, 25 Ноября 2010

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