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

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

+145

1
2
3
4
5
6
7
8

if (users.ToList().Count == 0)
{
          return true;
}
else
{
          return false;
}

Mikita, 23 Ноября 2010

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

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

+114

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

Ссылка ниже, там много всего, наиболее яркие моменты:
1. 
String Conn = (String)((NameValueCollection)Context.GetConfig("system.web/dsnstore"))["sql_site"];

2. 
qGetSum = new SqlCommand("select abs(ISNULL(sum(doc_bonus),0)) from vdoc where card_id = " + Session["card_id"] + " and dbo.Get_storno(doc_id) = 1", cGetSum);

3. 
SqlConnection cDiscount2 = new SqlConnection(Conn);
SqlConnection cDiscount3 = new SqlConnection(Conn);

4.
 log.Text = "Вы определены как:<br><b>" + Session["login"] + "<br><br></b>Бонус:<b>" + Session["bonus"] + "<br><br><a href=\"default.aspx?page=cardedit\">Личная карточка</a></b><br><br>";

5. 
Имена переменных: card_prefix, error_login_count, logoCMD, 

6. 
Session["access"] = 1;
Session["access"] = -1;
Session["access"] = -2;

http://www.cyberforum.ru/asp-net/thread195915.html

HIMen, 23 Ноября 2010

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

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

+121

01
02
03
04
05
06
07
08
09
10
11

txtContacts.Text = "";
bool first = true;

foreach (string contact in contacts)
{
	if (first != true)
		txtContacts.Text += ";";
	first = false;

	txtContacts.Text += contact;
}

Думаю, кто-то хотя бы раз сталкивался с самопальным String.Join'ом

zheka, 22 Ноября 2010

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

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

+110

1
2
3
4
5
6
7

void someMethod(Object obj)
{
	if(!obj.Equals(null))
	{
		...
	}
}

а это я сам когда-то очень-очень давно наклал :))))
до их пор с теплотой вспоминаю, как сам потом ржал, когда заметил :)

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

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

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

+127

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

if ((textBox2.Text != "") && (textBox3.Text != "") && (Convert.ToInt32(textBox2.Text) < Convert.ToInt32(textBox3.Text)))
{
    int наименование = 0;
    string несозданы = null;
    int количество = 0;
    bool накладнаясоздана = false;
    int накладная = 0;
    наименование = Convert.ToInt32(comboBox1.SelectedValue);
    string наименованиест = наименование.ToString();
    for (int i = Convert.ToInt32(textBox2.Text); i < Convert.ToInt32(textBox3.Text) + 1; i++)
    {
        bool одинаково = false;
        foreach (СкладDataSet.ПоступлениеRow row in складDataSet.Поступление.Rows)
        {
            if ((i == row.__) && (наименованиест == row.Наименование))
            {
                одинаково = true;
                несозданы += " " + i.ToString();
                yes = 2;
            }
        }
        if (одинаково == false)
        {
            складDataSet.Состояние.AddСостояниеRow(наименование.ToString(), i.ToString(), "2", "1", "");
            if (накладнаясоздана == false)
            {
                поступлениесписокTableAdapter.Insert(Convert.ToDateTime(dateTimePicker1.Text).ToString(), наименование, "000", richTextBox1.Text);
                поступлениесписокTableAdapter.Fill(складDataSet.Поступлениесписок);
                накладнаясоздана = true;
                foreach (СкладDataSet.ПоступлениесписокRow row in складDataSet.Поступлениесписок.Rows)
                { if (накладная <= row.Код)  накладная = row.Код; }
            }
            поступлениеTableAdapter.Insert(наименование, i, накладная.ToString());
            количество = количество + 1;
        }
    }
    поступлениесписокTableAdapter.Update(Convert.ToDateTime(dateTimePicker1.Text).ToString(), наименование, количество.ToString(), richTextBox1.Text, накладная, Convert.ToDateTime(dateTimePicker1.Text).ToString(), наименование, "000", richTextBox1.Text);
    if (yes == 2)
        MessageBox.Show(comboBox1.Text + " - " + несозданы + " не созданы т.к. существуют в базе", "Не созданы...", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
    Close();
}
else
    MessageBox.Show("Поле пустое, введите номер прибора!!!", "Внимательнее!", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);

на готдотнете видел один чел вопрос задавал: http://www.gotdotnet.ru/forums/3/133079/
автору там намекали, что хорошо было бы, чтоб он переделал код, но автор глубоко забил

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

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

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

+118

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

if(wordApp != null)
                {
                    try
                    {
                        if(isNewApp && wordApp.Documents.Count == 0)
                        {
                            object arg1 = Word.WdSaveOptions.
                                            wdDoNotSaveChanges;
                            object arg2 = null;
                            object arg3 = null;
                            wordApp.Quit(ref arg1, ref arg2, ref arg3);

                            // Wait until Word shuts down.

                            for(;;)
                            {
                                Thread.Sleep(100);
                                try
                                {
                                    // When word shuts down this call 

                                    // throws an exception.

                                    string dummy = wordApp.Version;
                                }
                                catch
                                {
                                    break;
                                }
                            }
                        }
                    }
                    catch {}

                    wordApp = null;
                }

Вот такой вот Dispose для ворда
http://www.codeproject.com/KB/cs/winwordloader.aspx

Lennis, 21 Ноября 2010

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

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

+118

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

static Func<object, object> get;

static void Add(object id, object data)
{
    if (get == null)
    {
        get = i =>
        {
            if (i.Equals(id)) return data;
            else return null;
        };
    }
    else
    {
        var _ = get;
        get = i =>
        {
            if (i.Equals(id)) return data;
            else return _(i);
        };
    }
}

psina-from-ua, 21 Ноября 2010

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

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

+145

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

using System;

//using System.Collections.Generic; Нафега ани???
//using System.Linq;
//using System.Text;

namespace ConsoleApplication5
{
    class Program
    {
        //Эт типа метад сёрч:)
        public static int[] Search(int[] arr)
        {
            int a = 0; //первае самае бальшое чесло
            int b = 0; //втарое самае бальшое чесло (Бирём n... Мала, нада m)
            for (int i = 0; i < arr.Length; i++) //Фигачем мосив, пака не дафигачим
            {
                if (arr[i] > a && arr[i] > b) //если нашлося бальшущее чесло, вписуем
                {
                    if (a < b) //либа в а
                    {
                        a = arr[i];
                    }
                    else if (b < a) //либа в b
                    {
                        b = arr[i];
                    }
                    else //либа тожэ в a
                    {
                        a = arr[i];
                    }
                }
            }
            int[] ret = { a, b }; //выплёвуем мосивом бальших и ни очинь чисил
            return ret; //да, таки выплёвуем
        }

        static void Main(/*string[] args*/) //нах.. вы мне эту фегню впехнуле. Уроды мелкомягкее
        {
            int[] m = { 1, 2, 4, 5, 6, 7, 8, 9 }; //Зодоём мосив
            int[] ret = Search(m); //ищим два самэх бальшых чесла

            for (int i = 0; i < 2; i++)
            {
                Console.WriteLine(ret[i]); //выводем их в кансоль
            }
            Console.ReadKey();
        }
    }
}

Таким я вижу говнокод. Или почти таким...:)

opentornado, 19 Ноября 2010

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

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

+114

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 static DataTable GetMainMenu()
{
    DataTable dt = null;
    if (DataManager.IsClientServer)
    {
        dt = new DataTable();
        using (OracleCommand qGetMainMenu = new OracleCommand())
        {
            qGetMainMenu.Connection = DataManager.OracleSession;
            qGetMainMenu.CommandText = String.Format("SELECT * FROM {0}.V_MENU", DataManager.DataSchema);
            using (OracleDataReader odr = qGetMainMenu.ExecuteReader())
            {
                dt.Load(odr);
                odr.Close();
            }
        }
    }
    else
    {
        using (Service.AbvServiceClient client = Utils.CreateServiceClient())
        {
            dt = client.GetMainFormMenu();
        }
    }
    return dt;
}

2-х или 3-х звенка? прямой вызов или Wcf-служба?
з.ы. повсеместно во всем проекте

coyote, 18 Ноября 2010

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

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