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

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

+119

1
2
3
4
5
6

if (this.bmp == null) {
				} else if (this.bmp.Size.Height < this.ViewSize.Height) {
					DisposeBmp();
				} else if (this.bmp.Size.Width < this.ListData.RowSize.Width) {
					DisposeBmp();
				}

Еще один Американский перл.... Присмотритесь к 1-му "if". И так по всему коду.

Dmitriy, 31 Мая 2010

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

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

+113

1

public static bool DefaultSuppressVersionCheck = true; //default is false

Как пишут программисты США.

Dmitriy, 31 Мая 2010

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

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

+126

01
02
03
04
05
06
07
08
09
10

public partial class Form1 : Form
    {
        string str;
        int количество;
        public Form1(string str, int количество)
        {
            InitializeComponent();
            str = this.str;
            количество = this.количество;
        }

Классный конструктор, ничего не скажешь.
А потом сидишь и удивляешься, почему же у тебя пустая строка и количество не прописалось...

alexanika, 28 Мая 2010

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

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

+124

1
2
3
4
5
6
7
8

char arr[] = ... // Здесь происходит чтение массива, неважно как
string result = "";
for(int i = 0; i < arr.Length; i++) 
{
    result += Char.ToString(arr[i]);
}

Console.WriteLine(result);

Что интересно, автор уже пол года на C# пишет, до этого 2 года на Java. Решение ко всему прочему еще и очевидно с квадратичной сложностью. А должно быть, естественно Console.WriteLine(new String(array));

theos, 27 Мая 2010

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

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

+131

1
2
3
4
5
6

object FirstElement = null;
foreach(var item in SomeCollection)
   {
       FirstElement = item;
       break;
   }

Ось так от береться перший елемент з колекції :)

O_O, 26 Мая 2010

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

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

+119

1
2
3
4
5

if (dataTank.UpdateSQLTable(addressTbl, (int)SharedConstants.FocusAdapterTables.Address, SharedConstants.FocusAdapterTables.Address.ToString()) > 0)
          {
            return true;
          }
          return false;

Yeah, baby!
Похоже, ребята специально создавали себе места для рефакторинга - набудущее, чтобы было чем заняться.

Progreso, 26 Мая 2010

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

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

+124

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81

switch ((int)row["State"])
          {
            case 0:
              /*** Get Contact Row for Company ***/
              if (contactTbl != null)
              {
                contactRow = null;
                DataRow[] contactRows = contactTbl.Select("Company_Id='" + row["Id"].ToString() + "'");
                if (contactRows != null && contactRows.Length > 0)
                {
                  contactRow = contactRows[0];
                  if ((bool)contactRow["fDontMailshot"] == false)
                  {
                    thisMailShot = true;
                  }
                  else
                  {
                    thisMailShot = false;
                  }
                }
              }

              if ((bool)row["fManagementCompany"] == true)
              {
                if (row["Id"].ToString() == "00-20181" || row["Id"].ToString() == "00-60838")
                {
                  myEntityID = Entity(2, row);
                  if (!SubContractor(myEntityID, row)) { return false; }
                }
                else
                {
                  myEntityID = Entity(10, row);
                }
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
              }
              else if ((bool)row["fSubcontractor"] == true || ((bool)row["fSupplier"] == true && (bool)row["fCustomer"] == false))
              {
                myEntityID = Entity(2, row);
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
                if (!SubContractor(myEntityID, row)) { return false; }
              }
              else if ((bool)row["fHousingAssociation"] == true)
              {
                myEntityID = Entity(7, row);
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
                if (!Customer(myEntityID, row, thisMailShot)) { return false; }
              }
              else if ((bool)row["f2ndOwnerTenant"] == true && (bool)row["fCustomer"] == false)
              {
                myEntityID = Entity(8, row);
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
                if (!Customer(myEntityID, row, thisMailShot)) { return false; }
              }
              else if ((bool)row["fCustomer"] == true && (bool)row["f2ndOwnerTenant"] == false)
              {
                myEntityID = Entity(7, row);
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
                if (!Customer(myEntityID, row, thisMailShot)) { return false; }
              }
              else if ((bool)row["fCustomer"] == true && (bool)row["f2ndOwnerTenant"] == true)
              {
                myEntityID = Entity(7, row);
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
                if (!Customer(myEntityID, row, thisMailShot)) { return false; }
              }
              break;
            default:
              break;
          }

Отличный пример использования конструкций ветвления. Имеем switch, в нём один case, а в кейсе много-много иф-элсов. И что очень характерно для этого проекта с тремя классами по 9000 строк, никакого повторного использования кода. Реализация принципа "зачем писать меньше?".

Progreso, 26 Мая 2010

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

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

+120

1
2
3
4

public static bool GetUpperBit(long number)
{
   return number < 0;
}

Да это не ГК, но остроумный способ получить верхний бит числа.

FMB, 25 Мая 2010

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

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

+120

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
34
35
36
37
38
39
40
41
42
43
44
45

///<summary>
/// Перемещение страницы в/из корзины
///</summary>
///<param name="PageID">Идентификатор страницы</param>
///<param name="Direction">Определяет в корзину или из нее перемещать страницу</param>
///<param name="Page">Экземпляр страницы</param>
///<returns></returns>
///<exception cref="Exception"></exception>
public static Exception PageToRecicle(int PageID, int Direction, Page Page)
{
    SiteData data = new SiteData("MsSqlConnectionString");
    SiteXml xml = new SiteXml();
    Exception ee = new Exception();

    SitePage page = new SitePage(Page);
    if (page.StartPageID == PageID)
    {
        try
        {
            throw new Exception(xml.GetXmlResource("adminmenu.ascx", "RemoveStart"));
        }
        catch (Exception exc)
        {
            ee = exc;
        }
        return ee;
    }
    if (Direction == 0 || !SitePage.HasPageChilds(PageID, false))
    {
        data.ExecuteStoredProcedure("sPageToRecicle", ref ee, "@PageID", PageID, "@Direction", Direction,
                                    "@UserName", HttpContext.Current.User.Identity.Name);
    }
    else
    {
        try
        {
            throw new Exception(xml.GetXmlResource("adminmenu.ascx", "RemoveError"));
        }
        catch (Exception exc)
        {
            ee = exc;
        }
    }
    return ee;
}

Это фрагмент коммерческой CMS между прочим. Если бы автор родился в Спарте, его бы незамедлительно ё*нули об скалы.

sombre-hombre, 25 Мая 2010

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

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

+106

1
2
3
4
5
6

bool tos = true;
               susid:
                   while (tos)
                    {
                          goto susid;
                }

Энто был я=(

BlincAttack, 24 Мая 2010

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

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