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

    +112

    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
    private void RefreshNewsList()
            {
                List<NewsObject> tempNewsList = new List<NewsObject>();
                tempNewsList.AddRange(lstOldNews.FindAll(delegate (NewsObject newsOld) 
                {
                    if (lstDeletedNewsIDs.Contains(newsOld.ID) || -1 == lstUpdatedNews.FindIndex(delegate(NewsObject newsUpdate)
                    {
                        if (newsUpdate.ID == newsOld.ID)
                            return true;
                        else
                            return false;
                    }))
                        return false;
                    else
                        return true;
                }));
                tempNewsList.AddRange(lstNewNews);
                tempNewsList.AddRange(lstUpdatedNews);
    
                lstNewsList.Items.Clear();
                lstNewsList.Items.AddRange(tempNewsList.ToArray());
            }

    не охота было лишние переменные использовать... ))) только что написал )) чудо родилось в 07,12,2010 16:44 ))) видно что пора домой..

    Maxim546, 07 Декабря 2010

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

    +101

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (CoursesString[CoursesString.Length - 1] == '\r')
                    {
                        sb.Remove(CoursesString.Length - 1, 1);
                        CoursesString = sb.ToString();
                    }

    а потом я понял...

    Golovastick, 05 Декабря 2010

    Комментарии (6)
  3. 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)
  4. 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)
  5. C# / Говнокод #4824

    +126

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

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

    ursus, 03 Декабря 2010

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

    +115

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

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

    Maxim546, 03 Декабря 2010

    Комментарии (14)
  7. 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)
  8. C# / Говнокод #4810

    +118

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

    zheka, 02 Декабря 2010

    Комментарии (80)
  9. 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)
  10. 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)