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

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)

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