Список говнокодов пользователя _Ru55_

Всего: 8

C++ / Говнокод #12979

+16

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

try
{
   Application->Initialize();
}
catch (...)
 {
      try
     {
        throw Exception("");
       }
   catch (Exception &exception)
   {
      Application->ShowException(&exception);
   }
 }

Найдено в проекте написанном на Borland C++Builder :)

_Ru55_, 08 Мая 2013

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

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

+146

1
2
3
4
5
6
7
8
9

public bool IsUnplugged
{
    get { throw new NotImplementedException(); }
}

public bool IsPlugged
{
    get { throw new NotImplementedException(); }
}

_Ru55_, 13 Октября 2011

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

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

+109

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

[Test]
public void TryToParseFromIntToString()
{
            string str="12";
            int resultat;
            bool accept=int.TryParse(str, out resultat);
            Assert.AreEqual(resultat,12);
            Assert.IsTrue(accept==true);
}

[Test]
public void TestEqualsWhenThe2NumbersAreReallyEquals()
{
            Assert.IsTrue(AmountComparer.Equals(123.3456m,123.3456m));
}

Вот такие юнит-тесты мы сегодня вычищаем! Все знаки препинания и символы сохранены ;)
Зато всегда зеленые! ;)

_Ru55_, 14 Июля 2011

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

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

+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
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

public class IterateSQL
{
	private int _start = 0;
	private int _end = 0;
	private int number = 0;
	private string TOP_100_PERCENT = "100 percent ";
	private string SELECT_FROM = " SELECT*FROM (";
	private string SELECT_TOP_I = "SELECT TOP $I * FROM(";
	private string SELECT_TOP_F = "SELECT TOP $F * FROM(";
	private string JOINTURE = @" WHERE maTable.id NOT IN (SELECT maTable.id FROM(";
	private string CLOSE = ") maTable)";
	private string SELECT_COUNT = "SELECT COUNT(*) FROM ( XYZ ) maTable";
	private string _sql;
	private string _where;
	private string _parameter = string.Empty;
	private List<string> _tempParameterList = new List<string>();
	private Dictionary<string, object> _finalParameterList = new Dictionary<string, object>();

	public IterateSQL(string sql, string where, string parameter)
	{
		_sql = sql;
		_where = where;
		_parameter = parameter;
	}
	
	public string TransformAndReturnSQL(int start, int end)
	{
		_start = start;
		_end = end;
		if (_parameter == null)
			_parameter = string.Empty;
		string[] split = Utils.Splitter(_parameter);
		number = split.Length;
		StringBuilder sb = new StringBuilder(" XYZ ");
		StringBuilder st = new StringBuilder();

		for (int i = 0; i < number; i++)
		{
			if (split[i].ToString() != string.Empty)
				sb.Insert(0, SELECT_FROM);
		}

		for (int i = 0; i < number; i++)
		{
			if (split[i].ToString() != string.Empty)
			{
				sb.Insert(sb.Length, _where);
				sb.Replace("@param", split[i].ToString());
			}
		}
		sb.Replace("XYZ", _sql);
		st.Append(sb.ToString());
		sb.Insert(0, SELECT_TOP_I);
		sb.Append(JOINTURE);
		sb.Append(SELECT_TOP_F);
		sb.Append(st.ToString());
		sb.Append(CLOSE);
		sb.Replace("$I", _start.ToString());
		sb.Replace("$F", _end.ToString());

		return sb.ToString();
	}
}

Вот так надо запросы строить!

_Ru55_, 22 Апреля 2011

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

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

+115

1
2
3
4
5
6
7

RepaymentEvent e = DatabaseHelper.GetString("event_type", pReader) == "RBLE"
	                ? new BadLoanRepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
	                : (DatabaseHelper.GetString("event_type", pReader) == "RRLE"
	                       ? new RescheduledLoanRepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
	                       :(DatabaseHelper.GetString("event_type", pReader).StartsWith("P") 
                                ? new PendingRepaymentEvent (DatabaseHelper.GetString("event_type", pReader)) {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
                                : new RepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}));

Вот такую "элегантную" строчку нашел сегодня коллега в коде нашего проекта :)
Создаем событие пойди разбери какое :)

_Ru55_, 02 Марта 2011

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

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)

SQL / Говнокод #4841

−190

1
2
3
4
5
6
7
8
9

UPDATE [Accounts] 
SET [account_number] = @number, 
[local_account_number] = @localNumber, 
[label] = @label, 
[debit_plus] = @debitPlus, 
[type_code] = @typeCode, 
[description] = @description, 
[parent_account_id] = @parentAccountId
WHERE [account_number] = (SELECT TOP(1) [account_number] FROM [Accounts] WHERE id = @id)

Запрос, найденный в open source проекте.
Обновляем аккаунты...

_Ru55_, 07 Декабря 2010

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

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

+108

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

private void InitializeUser()
{
	textBoxFirstname.Text = _user.FirstName;
	textBoxLastname.Text = _user.LastName;
	textBoxPassword.Text = _user.Password;
	textBoxUsername.Text = _user.UserName;
	textBoxMail.Text = _user.Mail;
	comboBoxRoles.Text = _user.UserRole.Id>0 ? _user.UserRole.RoleName : "";
	
	comboBoxRoles.Enabled = true;

	if(User.CurrentUser.isLoanOfficer || User.CurrentUser.isVisitor || User.CurrentUser.isCashier)
	{
		comboBoxRoles.Enabled = false;
	}
	else if(User.CurrentUser.isAdmin)
	{
		if(_user.isAdmin || _user.isSuperAdmin) comboBoxRoles.Enabled = false;
	}
	else if(User.CurrentUser.isSuperAdmin)
	{
		if(_user.isSuperAdmin) comboBoxRoles.Enabled = false;
	}
}

Код взят из реального open source проекта. Реализация многоуровневой проверки прав текущего пользователя :).

_Ru55_, 22 Октября 2010

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

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