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

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# / Говнокод #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

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

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

+145

1
2
3
4

public partial class TaskViewSearchParameter
{
    public int Top5 { get; set; }
}

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

ZlobnyiSerg, 25 Ноября 2010

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

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

+126

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

private int VedOrderBy(int N_VED)
    {
        switch (N_VED)
        {
            case 2:
                return 3;
                break;
            case 3:
                return 5;
                break;
            case 4:
                return 1;
                break;
            case 5:
                return 0;
                break;
            case 6:
                return 4;
                break;
            case 7:
                return 6;
                break;
            case 8:
                return 2;
                break;
        }
        return 10;
    }

foondook, 25 Ноября 2010

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

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

+122

01
02
03
04
05
06
07
08
09
10
11
12
13
14

public enum SqlCallType {
		Text = 1,
		StoredProcedure = 2
	}

		private SqlCallType ConvertToSqlCallType(int sqlCallType) {
			SqlCallType type = (SqlCallType)0;
			if (sqlCallType == (int)SqlCallType.Text) {
				type = SqlCallType.Text;
			}
			else if (sqlCallType == (int)SqlCallType.StoredProcedure) {
				type = SqlCallType.StoredProcedure;
			}
			return type;

приведение enum-->int по индусски для enum'а с двумя значениями....

Unknown_2, 25 Ноября 2010

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

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

+110

1
2
3
4
5
6
7

protected static string CreateSubIndexName(string prefix, int index) {
            return String.Format(CultureInfo.InvariantCulture, "{0}[{1}]", prefix, index);
        }

        protected static string CreateSubIndexName(string prefix, string index) {
            return String.Format(CultureInfo.InvariantCulture, "{0}[{1}]", prefix, index);
        }

Из исходников ASP.NET MVC 2. Класс DefaultModelBinder.
Рядышком друг за другом два метода с совершенно одинаковым содержимым.
Наш девиз: дублирование кода, умещающегося в одну строчку, за дублирование не считается.

Pauchok-Anaynckiy, 24 Ноября 2010

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

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