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

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

+136

1
2
3
4
5
6
7

while (f != null && !string.IsNullOrEmpty(f.FileName) && f.ContentLength != 0)
{
   if (f != null && !string.IsNullOrEmpty(f.FileName) && f.ContentLength != 0)
   {
      // ...
   }
}

Проверка на всякий случай

ReallyBugMeNot, 02 Февраля 2011

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

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

+128

1
2
3
4
5
6
7
8

//Hint: We have added one more overload to the method Load/LoadBinary/LoadSoap to achieve your requirement. Please refer the below code snippet.

Exception ex = null;
diagram1.LoadBinary(@"..\\..\\Basic Shapes.edp",out ex);
if (ex != null)
{
//Do your customization here
}

индусский обработчик исключений.
поддержка исключений в их компонент была добавлена по нашей просьбе.

kjuby2, 01 Февраля 2011

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

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

+121

1
2
3
4
5
6

public static T FirstOrDefault<T>(IEnumerable<T> it)
{
   foreach (T v in it)
     return v;
   return default(T);
}

Самодельный FirstOrDefault.

Боюсь даже представить, как у автора будет выглядеть Single

bober_maniac, 31 Января 2011

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

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

+118

1
2
3
4
5

public static void Clear(string[] array)
        {
            int n = default(int);
            Array.ForEach(array, element => array[n++] = String.Empty);
        }

Смешались в кучу кони, люди...

bober_maniac, 31 Января 2011

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

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

+121

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

public SqlTransaction GetSqlTransaction(string pMd5)
{
	if (_connection.State == ConnectionState.Closed)
	{
		try
		{
			_connection.Open();
		}
		catch (SqlException ex)
		{
			throw new ApplicationException("Unable to connect to database (" + _connection.DataSource + "/" + _connection.Database + "). Please contact your local IT administrator.", ex);
		}
	}
	else
	{
		try
		{
			throw new ApplicationException("COUCOU");
		}
		catch (ApplicationException ex)
		{
			System.Diagnostics.Trace.WriteLine(ex.StackTrace);
		}
		sqlTransaction = _connection.BeginTransaction();
	}
	return sqlTransaction;
}

Код из очередного проекта. А надежда то на коннект все-равно остается! :)

_Ru55_, 31 Января 2011

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

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

+146

1
2
3
4
5
6
7

//Мега-изобретательный, сцуко, флаг!
public static bool DONT_UPDATE_INPUTMANAGER = FACEPALM;

...

//Флаг выпилил, быстра блджад!!1
 if (DONT_UPDATE_INPUTMANAGER) DONT_UPDATE_INPUTMANAGER = false;

Достаточно недавний мой высер, сделанный на обезумевшую от непоняток голову. Трабла была в том, что при перехода из одного в меню в другое второе меню так же воспринимало эту кнопку и шагало дальше, хотя цикл вроде прошёл и InputManager уже сбросился. Пришлось сделать вот таким вот флагом (правда до сих пор не могу понять КАК я умудрился такой if написать...).

P.S: Похожая фигня и у MS, см. CurveEditor (create.msdn.com -> education catalog -> tools -> CurveEditor -> переменная disableUIEvents (причём int!!!)).

RaZeR, 29 Января 2011

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

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

+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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51

var sEmailRecipient = string.Empty;

                    string m_sPhysicalPath = "";
                    if (SaveType == "both" || SaveType == "email")
                    {
                        
                        List<User> recipients = null;
                        if (RecipentSelectMode == "auto")
                        {
                            if (!string.IsNullOrEmpty(AutoRecipient))
                                recipients = Notification.ConvertToUsers(AutoRecipient, MethodologyId, CurrentObjectId, CurrentUserId, CurrentEntityName);
                        }
                        if(RecipentSelectMode == "manual"){
                            if (Recipient != null)
                                sEmailRecipient = EvaluateExpression(CurrentUserId, MethodologyId, MainEntityName, MainObjectId, Recipient);
                        }
                        
                         m_sPhysicalPath = Document.AbsoluteApplicationPath + "/" + p_sReportPath.Substring(p_sReportPath.LastIndexOf("storage"));

                         if (RecipentSelectMode == "auto")
                         {
                             foreach (User user in recipients)
                             {
                                 if (Regex.IsMatch(user.Email, @"\w+([-+.']\w+)*@\w+([-.]\w+)*\.\w+([-.]\w+)*", RegexOptions.None))
                                 {
                                     SendReportByEmail(m_sPhysicalPath, user);
                                 }
                             }
                         }
                         if (RecipentSelectMode == "manual")
                         {
                             if (Regex.IsMatch(Recipient, @"\w+([-+.']\w+)*@\w+([-.]\w+)*\.\w+([-.]\w+)*", RegexOptions.None))
                             {
                                 SendReportByEmail(m_sPhysicalPath, new User { Email = sEmailRecipient });

                             }
                             else
                             {
                                 return CreateResult(GetClientData(), "Email message sending failed - incorrect email address - " + Recipient, (int)ReportRenderingFailType.Success);

                             }
                    
                         }

                    }
                    if (SaveType == "email")
                    {
                        FileInfo file = new FileInfo(m_sPhysicalPath);
                        if (file.Exists)
                            file.Delete();
                    }

Отправляет отчет по email

PetrLyapin, 27 Января 2011

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

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

+130

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

private int FindIndexOfItemWithValue(object value)
{
    for (int i = 0; i < base.Items.Count; i++)
    {
        object item = base.Items[i];
        if (InternalUtils.AreValuesEqual(this.SelectedValue, this.GetSelectedValue(item)))
        {
            return i;
        }
    }
    return -1;
}

Код комбобокса из System.Windows, Version=2.0.5.0 (Silverlight). Где тут ищется value - загадка природы.

Unseen, 27 Января 2011

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

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

+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
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
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98

static void processCmd(string command) {
	string[] c_args = command.Split(" ".ToCharArray());
	switch (c_args[0]) {
		case "beep":
			nbr.PlayTone(4096, 500);
			break;
		case "exit":
			exit_op();
			break;
		case "info":
			log_ca("Info:");
			log_ca(Application.ProductName + " " + Application.ProductVersion);
			log_ca("listener is " + ((services_running[0]) ? "running" : "down"));
			log_ca("updater is " + ((services_running[1]) ? "running" : "down"));
			break;
		case "start":
			try {
				switch (c_args[1]) {
					case "updater":
						break;
					case "listener":
						if (services_running[0])
							log_ca("listener is already running");
						else
							start_listener();
						break;
					default:
						throw new ArgumentException();
						break;
				}
			}
			catch {
				log_ca("Usage: start <service>. Available services: listener, updater.");
			}
			break;
		case "help":
			foreach (string hs in System.IO.File.ReadAllLines("help.txt")) {
				log_ca(hs);
			}
			break;
		default:
			log_ca("\"help\" will display all available commands");
			break;
		case "stop":
			try {
				nbr.MotorA.Brake();
				nbr.MotorB.Brake();
				nbr.MotorC.Brake();
			}
			catch { }
			break;
		case "run":
			try {
				switch(c_args[1]) {
					case "a":
						if(arr_motor[0]=="none")
							log_e("Motor not found or config error");
						else {
							if (c_args[4] == "false")
								nbr.MotorA = new NxtMotor(false);
							else
								nbr.MotorA = new NxtMotor(true);
							nbr.MotorA.Run(Convert.ToSByte(c_args[2]), Convert.ToUInt32(c_args[3]));
						}
						break;
					case "b":
						if(arr_motor[1]=="none")
							log_e("Motor not found or config error");
						else {
							if (c_args[4] == "false")
								nbr.MotorB = new NxtMotor(false);
							else
								nbr.MotorB = new NxtMotor(true);
							nbr.MotorB.Run(Convert.ToSByte(c_args[2]), Convert.ToUInt32(c_args[3]));
						}
						break;
					case "c":
						if(arr_motor[2]=="none")
							log_e("Motor not found or config error");
						else {
							if (c_args[4] == "false")
								nbr.MotorC = new NxtMotor(false);
							else
								nbr.MotorC = new NxtMotor(true);
							nbr.MotorC.Run(Convert.ToSByte(c_args[2]), Convert.ToUInt32(c_args[3]));
						}
						break;
					default:
						throw new Exception();
						break;
				}
			}
			catch (Exception ex) {
				log_ca("Usage: run <motor> <speed> <tacho> <reverse>. Example: run a 100 0 false.");
			}
		break;
	}
}

Мой код, написано 3 года назад.

A1mighty, 27 Января 2011

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

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