Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

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

+133

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

public string GenerateWinCode(String PrefixWinCode, String Name, String LastName, String NameCompany, bool IsCompany = false)
        {
            string _NormalWinCode = "";
            if (!IsCompany)
                _NormalWinCode = (PrefixWinCode + Name[0] + LastName[0]).ToUpper();
            else
            {
                var i = 1;
                try
                {
                    while (String.IsNullOrWhiteSpace(NameCompany[i].ToString()))
                    {
                        i++;
                    }
                    _NormalWinCode = (PrefixWinCode + NameCompany[0] + NameCompany[i]).ToUpper();
                }
                catch (Exception)
                {
                    _NormalWinCode = (PrefixWinCode + NameCompany[0] + NameCompany[0]).ToUpper();
                }
            }
            if (PrefixWinCode == "IN")
                throw new RuleException("ErrorWincode", Resources.Accounts.Account.WincodeInvalid);
            try
            {
                using (var context = db)
                {
                    var _WincodesSim =
                        (from q in context.UserPartners.Where(m => m.WinCode.ToUpper().StartsWith(_NormalWinCode))
                         where q.WinCode.Length > 4
                         select q.WinCode).ToList();
                    var _Sufix = _WincodesSim.Select(m => Convert.ToInt32(m.Substring(4))).Max();
                    return _NormalWinCode + (_Sufix + 1);
                }
            }
            catch (InvalidOperationException)
            {
                return _NormalWinCode + 1;
            }
        }

Генерация уникальных ключей

AndrewTakker, 24 Февраля 2015

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

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

+137

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

public bool StartListener()
		{
			bool flag;
			try
			{
				flag = (this.m_Listener.BeginAccept() ? true : false);
			}
			catch (Exception exception)
			{
				CAssert.ReportAssert(exception);
				flag = false;
			}
			return flag;
		}

Из реального корпоративного проекта.

Danmer, 23 Февраля 2015

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

Python / Говнокод #17675

−115

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

def checkio(res):
    #bruteforce LOL
    if res[0][0] == res[1][0] == res[2][0] == "X":
        return "X"

    if res[0][1] == res[1][1] == res[2][1] == "X":
        return "X"

    if res[0][2] == res[1][2] == res[2][2] == "X":
        return "X"

    if res[0][0] == res[0][1] == res[0][2] == "X":
        return "X"

    if res[1][0] == res[1][1] == res[1][2] == "X":
        return "X"  

    if res[2][0] == res[2][1] == res[2][2] == "X":
        return "X"

    if res[0][0] == res[1][1] == res[2][2] == "X":
        return "X" 

    if res[0][2] == res[1][1] == res[2][0] == "X":
        return "X"

    if res[0][0] == res[1][0] == res[2][0] == "O":
        return "O"    

    if res[0][1] == res[1][1] == res[2][1] == "O":
        return "O"

    if res[0][2] == res[1][2] == res[2][2] == "O":
        return "O"

    if res[0][0] == res[0][1] == res[0][2] == "O":
        return "O"

    if res[1][0] == res[1][1] == res[1][2] == "O":
        return "O"
        
    if res[2][0] == res[2][1] == res[2][2] == "O":
        return "O"

    if res[0][0] == res[1][1] == res[2][2] == "O":
        return "O"        

    if res[0][2] == res[1][1] == res[2][0] == "O":
        return "O"

    return "D"

Определить результат игры в крестики нолики.

gost, 22 Февраля 2015

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

PHP / Говнокод #17665

+161

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

public function set_debug($fparam) {
     $this->set_debug_mode($fparam);
}

protected function set_debug_mode($fparam) {
      $this->set_localization_debug_mode($fparam);
}

private function set_localization_debug_mode($fparam) {
      $this->set_localization_debug_mode_log_service($fparam);
}

private function set_localization_debug_mode_log_service($fparam) {
      $conf = //...тут уже что-то делает, наконец-то
}

Я понимаю, если бы они просто были алиасами... Но почему protected/private?

Lowezar, 18 Февраля 2015

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

Java / Говнокод #17649

+74

1
2
3
4

/**
 * singletone staff:
 */
private static LogisticsAppContext local;

Однотонный посох?

someone, 16 Февраля 2015

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

PHP / Говнокод #17613

+157

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

/**
	 * @return DevblocksSearchField[]
	 */
	static function getFields() {
		$translate = DevblocksPlatform::getTranslationService();
		
		$columns = array(
			SearchFields_Ticket::TICKET_ID => new DevblocksSearchField(SearchFields_Ticket::TICKET_ID, 't', 'id', $translate->_('common.id'), Model_CustomField::TYPE_NUMBER),
			SearchFields_Ticket::TICKET_MASK => new DevblocksSearchField(SearchFields_Ticket::TICKET_MASK, 't', 'mask', $translate->_('ticket.mask'), Model_CustomField::TYPE_SINGLE_LINE),
			SearchFields_Ticket::TICKET_SUBJECT => new DevblocksSearchField(SearchFields_Ticket::TICKET_SUBJECT, 't', 'subject', $translate->_('ticket.subject'), Model_CustomField::TYPE_SINGLE_LINE),
			
			SearchFields_Ticket::TICKET_FIRST_MESSAGE_ID => new DevblocksSearchField(SearchFields_Ticket::TICKET_FIRST_MESSAGE_ID, 't', 'first_message_id'),
			SearchFields_Ticket::TICKET_FIRST_OUTGOING_MESSAGE_ID => new DevblocksSearchField(SearchFields_Ticket::TICKET_FIRST_OUTGOING_MESSAGE_ID, 't', 'first_outgoing_message_id'),
			SearchFields_Ticket::TICKET_LAST_MESSAGE_ID => new DevblocksSearchField(SearchFields_Ticket::TICKET_LAST_MESSAGE_ID, 't', 'last_message_id'),
			
			SearchFields_Ticket::TICKET_FIRST_WROTE_ID => new DevblocksSearchField(SearchFields_Ticket::TICKET_FIRST_WROTE_ID, 't', 'first_wrote_address_id'),
			SearchFields_Ticket::TICKET_FIRST_WROTE => new DevblocksSearchField(SearchFields_Ticket::TICKET_FIRST_WROTE, 'a1', 'email',$translate->_('ticket.first_wrote'), Model_CustomField::TYPE_SINGLE_LINE),
			SearchFields_Ticket::TICKET_FIRST_WROTE_FIRST_NAME => new DevblocksSearchField(SearchFields_Ticket::TICKET_FIRST_WROTE_FIRST_NAME, 'a1', 'first_name', $translate->_('ticket.first_wrote') . ' First Name'),
			SearchFields_Ticket::TICKET_FIRST_WROTE_LAST_NAME => new DevblocksSearchField(SearchFields_Ticket::TICKET_FIRST_WROTE_LAST_NAME, 'a1', 'last_name', $translate->_('ticket.first_wrote') . ' Last Name'),
			SearchFields_Ticket::TICKET_FIRST_WROTE_SPAM => new DevblocksSearchField(SearchFields_Ticket::TICKET_FIRST_WROTE_SPAM, 'a1', 'num_spam',$translate->_('address.num_spam'), Model_CustomField::TYPE_NUMBER),
			SearchFields_Ticket::TICKET_FIRST_WROTE_NONSPAM => new DevblocksSearchField(SearchFields_Ticket::TICKET_FIRST_WROTE_NONSPAM, 'a1', 'num_nonspam',$translate->_('address.num_nonspam'), Model_CustomField::TYPE_NUMBER),
				
			SearchFields_Ticket::TICKET_LAST_WROTE_ID => new DevblocksSearchField(SearchFields_Ticket::TICKET_LAST_WROTE_ID, 't', 'last_wrote_address_id'),
			SearchFields_Ticket::TICKET_LAST_WROTE => new DevblocksSearchField(SearchFields_Ticket::TICKET_LAST_WROTE, 'a2', 'email',$translate->_('ticket.last_wrote'), Model_CustomField::TYPE_SINGLE_LINE),
			SearchFields_Ticket::TICKET_LAST_WROTE_FIRST_NAME => new DevblocksSearchField(SearchFields_Ticket::TICKET_LAST_WROTE_FIRST_NAME, 'a2', 'first_name', $translate->_('ticket.last_wrote') . ' First Name'),
			SearchFields_Ticket::TICKET_LAST_WROTE_LAST_NAME => new DevblocksSearchField(SearchFields_Ticket::TICKET_LAST_WROTE_LAST_NAME, 'a2', 'last_name', $translate->_('ticket.last_wrote') . ' Last Name'),
				
			SearchFields_Ticket::ORG_NAME => new DevblocksSearchField(SearchFields_Ticket::ORG_NAME, 'o', 'name', $translate->_('contact_org.name'), Model_CustomField::TYPE_SINGLE_LINE),
			SearchFields_Ticket::REQUESTER_ADDRESS => new DevblocksSearchField(SearchFields_Ticket::REQUESTER_ADDRESS, 'ra', 'email',$translate->_('ticket.requester'), Model_CustomField::TYPE_SINGLE_LINE),
			
			SearchFields_Ticket::TICKET_ORG_ID => new DevblocksSearchField(SearchFields_Ticket::TICKET_ORG_ID, 't','org_id',$translate->_('contact_org.id')),
			SearchFields_Ticket::TICKET_OWNER_ID => new DevblocksSearchField(SearchFields_Ticket::TICKET_OWNER_ID,'t','owner_id',$translate->_('common.owner'), Model_CustomField::TYPE_WORKER),
			SearchFields_Ticket::TICKET_GROUP_ID => new DevblocksSearchField(SearchFields_Ticket::TICKET_GROUP_ID,'t','group_id',$translate->_('common.group')),
			SearchFields_Ticket::TICKET_BUCKET_ID => new DevblocksSearchField(SearchFields_Ticket::TICKET_BUCKET_ID, 't', 'bucket_id',$translate->_('common.bucket')),
			SearchFields_Ticket::TICKET_CREATED_DATE => new DevblocksSearchField(SearchFields_Ticket::TICKET_CREATED_DATE, 't', 'created_date',$translate->_('common.created'), Model_CustomField::TYPE_DATE),
			SearchFields_Ticket::TICKET_UPDATED_DATE => new DevblocksSearchField(SearchFields_Ticket::TICKET_UPDATED_DATE, 't', 'updated_date',$translate->_('common.updated'), Model_CustomField::TYPE_DATE),
			SearchFields_Ticket::TICKET_CLOSED_AT => new DevblocksSearchField(SearchFields_Ticket::TICKET_CLOSED_AT, 't', 'closed_at',$translate->_('ticket.closed_at'), Model_CustomField::TYPE_DATE),
			SearchFields_Ticket::TICKET_WAITING => new DevblocksSearchField(SearchFields_Ticket::TICKET_WAITING, 't', 'is_waiting',$translate->_('status.waiting'), Model_CustomField::TYPE_CHECKBOX),
			SearchFields_Ticket::TICKET_CLOSED => new DevblocksSearchField(SearchFields_Ticket::TICKET_CLOSED, 't', 'is_closed',$translate->_('status.closed'), Model_CustomField::TYPE_CHECKBOX),
			SearchFields_Ticket::TICKET_DELETED => new DevblocksSearchField(SearchFields_Ticket::TICKET_DELETED, 't', 'is_deleted',$translate->_('status.deleted'), Model_CustomField::TYPE_CHECKBOX),

https://github.com/jstanden/cerb6/blob/6.9.1/features/cerberusweb.core/api/dao/ticket.php#L1979

lav45, 10 Февраля 2015

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

Java / Говнокод #17589

+78

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

for (int i = 0; i < 100 && components.isEmpty(); i++) {
            try {
                Thread.sleep(1000); 
            } catch (InterruptedException ie) {
                    /* shouldn't happen */
            }
            components = parent.getChildrenByObjType(type);
            LOG.debug("Iteration " + i +"components = " + components);
        }
        if (components.isEmpty()) {
            throw new RuntimeException(COMPONENT_NOT_FOUND);
        }

зуб, что через 100 итераций со слипом в 1 секунду, коллекция components точно будет заполнена!

edhex, 06 Февраля 2015

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

Куча / Говнокод #17571

+132

1

.gen-tabs .acctab:hover, .gen-tabs .acctab:focus, .gen-tabs .acctab.current { Carat up! }

Говняшечка

elvis, 04 Февраля 2015

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

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

+132

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

private double readDouble(string name, string wholeFile)
{
	try
	{
		int ind = -1;
		if ((ind = wholeFile.IndexOf(name)) != -1)
		{
			var restofstr = wholeFile.Substring(ind + name.Length);
			int lineendind = -1;
			lineendind = restofstr.IndexOfAny(new char[] { '\n', '\r', (char)13, (char)10 });
			if (lineendind == -1 && restofstr.Length > 1)
			{
				lineendind = restofstr.Length;
			}
			if (lineendind != -1)
			{
				int eqind = -1;
				string valueString = restofstr.Substring(0, lineendind);
				if ((eqind = valueString.IndexOf("=")) != -1)
				{
					double res = 0.0;
					if (Double.TryParse(valueString.Substring(eqind + 1).Trim(), out res))
					{
						return res;
					}
				}
			}
		}
	}
	catch (Exception) { }

	return 0.0;
}

А как бы вы написали это?

tbolk, 03 Февраля 2015

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

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

+137

1
2
3

try { UserInfoProvider.DeleteUser(u.ID); }
                        catch { }
                        return "Ваш аккаунт успешно активирован";

alexscrat, 27 Января 2015

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

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