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

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

+139

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

// export in csv - part of the code
// ...
            foreach (users_view _item in _users_view)
            {
                _writer.Write(String.Format("{1}{0}{2}{0}{3}{0}{4}{0}{5}{0}{6}{0}{7}{0}{8}{0}{9}{0}{10}{0}{11}{0}{12}{0}{13}{0}{14}{0}{15}{0}{16}{0}{17}{0}{18}{0}{19}{0}{20}{0}{21}{0}{22}{0}{23}{0}{24}\n",
                    AppSettingsReader.GetValue("CSVFileSeparator"),
                     @"""" + _item.title_name + @"""", @"""" + _item.first_name + @"""",
                     @"""" + _item.last_name + @"""", @"""" + _item.job_title + @"""",
                    @"""" + _item.user_type_name + @"""",
                                            
                    @""""+_item.company_name+@"""",
                    (_item.telephone != null) ? (@"""" + _item.telephone + @"""") : (""),
                    (_item.fax != null) ? (@"""" + _item.fax + @"""") : (""),
                    @"""" + _item.email + @"""",
                    (_item.account_email != null) ? (@"""" + _item.account_email + @"""") : (""),
                    @"""" + _item.site_address + @"""",
                    @"""" + _item.advertisement_source_name+@"""",

                    @"""" + _item.address_1+@"""",
                    (_item.address_2 != null) ? (@"""" + _item.address_2+@"""") : (""),
                    @"""" + _item.country_name+@"""",
                   
                    //_item.email_format_name,

                    (_item.postcode != null) ? (@"""" + _item.postcode+@"""") : (""),
                    (_item.county != null) ? (@"""" + _item.county+@"""") : (""),
                    @"""" + _item.town + @"""",
                //    (_item.is_active == false) ? ("No") : ("Yes"),

                    @"""" + _item.username+@"""",
                    @"""" + _item.password+@"""",
                    @"""" + _item.account_type_name + @"""",

                    @"""" + _item.creation_date + @"""",

                    (_item.is_newsletter_subscriber == false) ? ("No") : ("Yes"),
                    (_item.is_marketing_subscriber == false) ? ("No") : ("Yes")
                    )
                );
            }

Вот что нашел :)

bugotrep, 14 Октября 2009

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

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

+132.9

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

//  Этот метод проверяет, создано ли окошло лога, если не создано, то создает его
        public void CheckLogWnd()
        {
            try
            {
                if (logWnd.IsAccessible)
                {
                }
            }
            catch
            {
                logWnd = new LogWnd();
            }
        }

Не помню как сделать это правильно :(

nolka4, 13 Октября 2009

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

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

+131.8

1
2
3
4
5

public string EncodeParameter_1251(string param) {
	var result = new StringBuilder();
	foreach (var bt in Encoding.GetEncoding("windows-1251").GetBytes(param)) result.AppendFormat("%{0}", bt.ToString("X02"));
	return result.ToString(); 
}

метод кодирующий значение параметра запроса в "windows-1251" в формате URL (%FF%FF%FF...).
у проекта кодировка запросов в "UTF-8", но у остальных проектов - "windows-1251", и в другой кодировке они некорректно обрабатывают кириллицу в строке запроса. знаю, наговнокодил, ну а шо поделаеш...

xeonix, 12 Октября 2009

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

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

+105.5

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

//одногрупнику надо было проверить, является ли "obj" - "А"
//наблюдал за процессом, и плакал
//----------------------------------------------------------------------------------
//1 версия
static bool IsA(object obj) {
if (obj.GetType().Name.Equals("A", StringComparison.InvariantCultureIgnoreCase))
    return true;
    else return false;
}
//----------------------------------------------------------------------------------
//2 версия
static bool IsA(object obj) {
    A a = new A();
    if (obj.GetType().Equals(a.GetType()))
        return true;
    else return false;
}
//----------------------------------------------------------------------------------
//3 версия
static bool IsA(object obj) {
    if (obj.GetType().Equals(typeof(A)))
        return true;
    else return false;
}
//----------------------------------------------------------------------------------
//потом он вспомнил, что от "A" могут наследоваться другие классы
static bool IsA(object obj) {
    Type typeObj = obj.GetType();
    do {
        if (typeObj.Equals(typeof(object)))
            return false;
        else if (typeObj.Equals(typeof(A)))
            return true;
        else typeObj = typeObj.BaseType;
    } while (true);
}

//плачу, смеюсь и плачу, а с виду одногрупник вроде не Индус...
//...и весь этот говнокод был написан, вместо простого:
static bool IsA(object obj) { return obj is A; }

via xeonix

striker, 12 Октября 2009

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

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

+134.4

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

private void TestWorksheetFunction() 
{
  ...
 
  Excel.WorksheetFunction wsf = ThisApplication.WorksheetFunction;
  ws.get_Range("Min", Type.Missing).Value2 = wsf.Min(rng, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing);
  ws.get_Range("Max", Type.Missing).Value2 = wsf.Max(rng, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing);
  ws.get_Range("Median", Type.Missing).Value2 = wsf.Median(rng,
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing);
  ws.get_Range("Average", Type.Missing).Value2 = wsf.Average(rng, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing);
  ws.get_Range("StDev", Type.Missing).Value2 = wsf.StDev(rng, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
    Type.Missing);
}

Если вы - разработчик на C#, вам придется привыкнуть к обилию значений Type.Missing в вызовах методов. Поскольку объектную модель Excel писали в расчете на VBA, многие ее методы принимают необязательные параметры - иногда до 30. Используйте либо многочисленные экземпляры значения Type.Missing или указывайте для каждого параметра определенное значение по умолчанию.
(c) http://www.gotdotnet.ru/LearnDotNet/NETFramework/22054.aspx

zerkms, 12 Октября 2009

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

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

+950

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19

public string OperatorName
{
    get
    {
        if (m_operator == null)
        {
            try
            {
                int idUser = 1;
                m_operator = (idUser > 0) ? "Василий" : string.Empty;
            }
            catch
            {
                m_operator = string.Empty;
            }
        }
        return m_operator;
    }
}

Василий решает )))

tonic, 05 Октября 2009

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

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

+139.6

1
2
3
4
5
6
7
8
9

var complete = new ManualResetEvent(false);

ThreadPool.QueueUserWorkItem(
	delegate
		{
			service.StartDownloadUpdatesProcess(complete);
		});

complete.WaitOne();

Запустим поток, а потом подождем пока он закончиться.

Mike Chaliy, 04 Октября 2009

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

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

+137.3

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 ArrayList GetSubscribers(string condition)
        {
           ArrayList subscribers = new ArrayList();
           ArrayList lst =  new user_category_notification().Factory.GetItems(condition, "user_category_notification.id_user");
            ArrayList distinc_lst = new ArrayList();
            ArrayList distinc_lst_ids = new ArrayList();
            ArrayList lst_ids = new ArrayList();
            for (int i = 0; i < lst.Count; i++)
            {
                lst_ids.Add(((user_category_notification) lst[i]).id_user);
            }
            for (int i = 0; i < lst_ids.Count;i++ )
            {
                if (distinc_lst_ids.Contains(lst_ids[i])) continue;
                else
                {
                    distinc_lst.Add(lst[i]);
                    distinc_lst_ids.Add(lst_ids[i]);
                }
            }
            foreach (user_category_notification _un in distinc_lst)
            {
                user _current = (user)new user().Factory.GetByID(_un.id_user);
                subscribers.Add(_current);
            }
            
            return subscribers;
        }

Филтрация :)

bugotrep, 02 Октября 2009

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

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

+100.4

1
2
3
4
5
6
7
8

public List<OrderGeneralVWJ> GetAll()
{
     var q = from og in Context.OrderGeneralVWJs
          select new { 
               og
          };
     return q.Select(r => r.og).ToList();
}

plsc_rover, 28 Сентября 2009

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

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

+94.5

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

/// Из aspx файла

//<asp:Repeater ID="Repeater1" runat="server">
//    <HeaderTemplate>
//        <table width="100%" cellspacing="5">
//    </HeaderTemplate>
//    <ItemTemplate>
//        <tr>
//            <td>
//                ...
//            </td>
//            <asp:Label runat="server" Visible='<%# DataBinder.Eval(Container.DataItem, "visibleCol2") %>'>
//                <td>
//                    ...
//                </td>
//            </asp:Label>
//            <asp:Label ID="LCol3" runat="server" Visible='<%# DataBinder.Eval(Container.DataItem, "visibleCol3") %>'>
//                <td>
//                    ...
//                </td>
//            </asp:Label>
//        </tr>
//    </ItemTemplate>
//    <FooterTemplate>
//        </table>
//    </FooterTemplate>
//</asp:Repeater>

// Из cs файла

DataTable dtres = // получаем товары
if (dtres != null && dtres.Rows.Count > 0)
{
    DataTable dt = new DataTable();
	/*...*/
    
    for (int i=0;i<dtres.Rows.Count;)
    {
        DataRow dr=dtres.Rows[i];
        /*...*/

        int col2Index = i + 1;
        if (col2Index < dtres.Rows.Count)
        {
            /*...*/
            i++;
            int col3Index = i + 1;
            if (col3Index < dtres.Rows.Count)
            {
                /*...*/
                i++;
            }
            else { /*...*/ }
        }
        else { /*...*/ }

        dt.Rows.Add(newRow);
        i++;
    }

    /*...*/
}

Человеку нужно было сделать товары в сетке 3x6.
От того что он сделал у меня пропал дар речи. (Чтобы тут очень много кода не бы большую (не нужную для понимания) часть заменил на "...")
Вобщем, в двух словах, он поместил вторую и третью ячейки таблицы в серверные контролы (причем в Label), а в коде берёт по три товара и засовывает в один RepeaterItem, ну а если количество товаров не делится нацело на три, то в последний RepeaterItem засовываются пустые данные, а Label делается невидимым.
Как-то так.
И самое главное, я не знаю, как человеку объяснить, что так нехорошо делать, т.к он главный программист...

Ordos, 27 Сентября 2009

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

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