C# — Говнокод.ру

C# / Говнокод #4831

+101

1
2
3
4
5

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

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

Golovastick, 05 Декабря 2010

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

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

+119

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
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)

C# / Говнокод #4827

+108

01
02
03
04
05
06
07
08
09
10
11
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)

C# / Говнокод #4824

+126

1
2
3

public static void Authorize(int id)
{
if (id != null)

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

ursus, 03 Декабря 2010

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

C# / Говнокод #4819

+115

1
2

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

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

Maxim546, 03 Декабря 2010

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

C# / Говнокод #4814

+117

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
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)

C# / Говнокод #4810

+118

1

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

zheka, 02 Декабря 2010

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

C# / Говнокод #4808

+121

1
2
3
4
5
6

if (OnCreated != null)
            {foreach (EventHandler<CreatedEventArgs> handler in OnCreated.GetInvocationList())
                {
                    handler(this, args);
                }
}

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

eval_2009, 02 Декабря 2010

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

C# / Говнокод #4807

+113

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
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)

C# / Говнокод #4764

+116

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
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