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

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

В номинации:

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

За время:

Pascal / Говнокод #17219

+80

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

procedure TInventory.removeItem(userid: integer; invid: integer;amount: integer = 1);
var
  MyRes: TInvItemArray;
  CurrItem: TInvItem;
  I,len: integer;
begin
  if (amount <= 0) then
    exit;
  if (amount = 1) then
    begin
     db.removeItemFromUserInventory(GetItems(GetIndexByInvId(invid)),userid);
     FInventoryList.Remove(GetIndexByInvId(invid));
    end else
    begin
      if (amount >= finventorylist.Size) then
        exit;
      MyRes:=GetResources(invid);
      len:=length(myres);
      if (len = 0) then
        exit;
      if (amount > len) then
        exit;
      for i:=0 to len - 1 do
       begin
         curritem:=MyRes[i];
         db.removeItemFromUserInventory(GetItems(GetIndexByInvId(curritem.InvId)),userid);
         FInventoryList.Remove(GetIndexByInvId(curritem.invid));
       end;
     Setlength(myres,0);
    end;

end;

function TInventory.GetResources(invid: integer): TInvItemArray;
function ToResult(var Return: TInvItemArray; const Item: TInvItem): Integer;
begin
  Result := Length(Return);
  SetLength(Return, Result + 1);
  Return[Result] := Item;
end;
var
  item,sameitem: TInvItem;
  Resource: TItem;
  i: integer;
begin
  SetLength(result,0);
  item:=GetItem(invid);
  if (item = nil) then
    exit;
  if (item.item <> nil) then
    begin
     // if (item.itemю = 30) then
     //   begin
          for i:=0 to FInventoryList.Size - 1 do
           begin
             sameitem:=TInvItem(FInventoryList.Items[i]);
              if (sameitem = nil) then
                continue;
             resource:=sameitem.item;
              if (resource = nil) then
                continue;
              if (item.item.itemid = resource.itemid) then
                ToResult(result,sameitem);
           end;
       // end;
    end;
end;

По сути - удалить объект из списка. OMG!

Cynicrus, 29 Ноября 2014

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

JavaScript / Говнокод #17218

+154

1

var coutryOption = commonSettings.parent().parent().parent().find("tr:contains('Country')");

очень поиск...

fess, 29 Ноября 2014

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

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

+56

1
2
3
4
5
6

std::uint8_t octets[4];
for(int i = 0; i < 4; ++i) {
    if(i != 0 && inp.get() != '.')
        { assert(false && "unexpected symbol"); }
    inp >> octets[i];
}

Два вида говна: в коде и в языке.

Soul_re@ver, 28 Ноября 2014

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

Си / Говнокод #17179

+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
41
42
43
44
45
46
47
48

if (*wpPat == L'\\')
    {
      wpCharStart=wpPat;
      if (++wpPat >= wpMaxPat) goto Error;

      if (*wpPat == L'x')
      {
        if (++wpPat >= wpMaxPat) goto Error;

        if (*wpPat == L'{')
        {
          wpStrTmp=++wpPat;
          for (;;)
          {
            if (wpPat >= wpMaxPat) goto Error;
            if (*wpPat++ == L'}') break;
          }
          if (lpREGroupItem->nGroupLen != -1 && !bClassOpen)
          {
            nPatChar=(int)hex2decW(wpStrTmp, (wpPat - 1) - wpStrTmp);
            if (nPatChar == -1)
            {
              wpPat=wpStrTmp;
              goto Error;
            }
            if (nPatChar <= MAXWORD)
              lpREGroupItem->nGroupLen+=1;
            else
              lpREGroupItem->nGroupLen+=2;
          }
        }
        else
        {
          if (wpPat + 2 > wpMaxPat)
            goto Error;
          wpPat+=2;
          if (lpREGroupItem->nGroupLen != -1 && !bClassOpen)
            ++lpREGroupItem->nGroupLen;
        }
      }
      else if (*wpPat == L'u')
      {
        if (wpPat + 5 > wpMaxPat)
          goto Error;
        wpPat+=5;
        if (lpREGroupItem->nGroupLen != -1 && !bClassOpen)
          ++lpREGroupItem->nGroupLen;
      }

Регулярные велосипеды.
Akelpad.

gost, 25 Ноября 2014

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

Си / Говнокод #17172

+134

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

//// AEM_* AkelEdit control messages

  if (uMsg >= WM_USER)
  {
    if (uMsg >= AEM_CANUNDO)
      goto CanUndo;

    //Text retrieval and modification
    if (uMsg == AEM_EXSETTEXTA)
    {
      AESETTEXTA *at=(AESETTEXTA *)lParam;

      return AE_SetTextAnsi(ae, at->nCodePage, at->pText, at->dwTextLen, at->nNewLine);
    }
    if (uMsg == AEM_EXSETTEXTW)
    {
      AESETTEXTW *at=(AESETTEXTW *)lParam;

      return AE_SetText(ae, at->pText, at->dwTextLen, at->nNewLine, FALSE);
    }
    if (uMsg == AEM_SETTEXTA)
    {
      return AE_SetTextAnsi(ae, CP_ACP, (char *)lParam, wParam, AELB_ASINPUT);
    }
    if (uMsg == AEM_SETTEXTW)
    {
      return AE_SetText(ae, (wchar_t *)lParam, wParam, AELB_ASINPUT, FALSE);
    }
    if (uMsg == AEM_APPENDTEXTA)
    {
      AEAPPENDTEXTA *at=(AEAPPENDTEXTA *)lParam;

      AE_AppendTextAnsi(ae, at->nCodePage, at->pText, at->dwTextLen, at->nNewLine);
      return 0;
    }
    if (uMsg == AEM_APPENDTEXTW)
    {
      AEAPPENDTEXTW *at=(AEAPPENDTEXTW *)lParam;

      AE_AppendText(ae, at->pText, at->dwTextLen, at->nNewLine);
      return 0;
    }
    if (uMsg == AEM_REPLACESELA)
    {
      AEREPLACESELA *rs=(AEREPLACESELA *)lParam;

      AE_ReplaceSelAnsi(ae, rs->nCodePage, rs->pText, rs->dwTextLen, rs->nNewLine, rs->dwFlags, rs->ciInsertStart, rs->ciInsertEnd);
      return 0;
    }
    if (uMsg == AEM_REPLACESELW)
    {
      AEREPLACESELW *rs=(AEREPLACESELW *)lParam;

      AE_ReplaceSel(ae, rs->pText, rs->dwTextLen, rs->nNewLine, rs->dwFlags, rs->ciInsertStart, rs->ciInsertEnd);
      return 0;
    }
    if (uMsg == AEM_GETTEXTRANGEA)
    {
      AETEXTRANGEA *tr=(AETEXTRANGEA *)lParam;

      return AE_GetTextRangeAnsi(ae, tr->nCodePage, tr->lpDefaultChar, tr->lpUsedDefChar, &tr->cr.ciMin, &tr->cr.ciMax, tr->pBuffer, tr->dwBufferMax, tr->nNewLine, tr->bColumnSel, tr->bFillSpaces);
    }

И далее около сотни if'ов в таком духе.
AkelPad

gost, 25 Ноября 2014

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

Pascal / Говнокод #17165

+88

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

function TCalc.getRewardItems(items : AnsiString; countItems : integer; userId : Integer; from:integer) : ISuperObject;
var tmpFirstOdds : integer;
    strLines, extrLines : TStringList;
    strItems, strOdds, strOdds2: TIntegerList;
    i, j, tmpCountItems : integer;
    tmpOddsItems, sum : Integer;
    str: ansistring;
begin
  Result := nil;

  // parse items line [id1,odd1],[id2,odd2],[id3,odd3]
  strLines := TStringList.Create;
  extrLines := TStringList.Create;
  strItems := TIntegerList.Create;
  strOdds := TIntegerList.Create;
  ExtractStrings(['[', ']'],[','], PChar(items), strLines);
  for i := 0 to strLines.Count - 1 do
  begin
    ExtractStrings([','],[' '], PChar(strLines[i]), extrLines);
    strItems.Add(StrToInt(extrLines[0]));
    strOdds.Add(StrToInt(extrLines[1]));
    extrLines.Clear;
  end;

  // calculate odds [a,b,c,d] as [a,a+b,a+b+c,a+b+c+d]
  sum := 0;
  strOdds2 := TIntegerList.Create;
  for i := 0 to strOdds.Count-1 do
  begin
    sum := sum + strOdds.Items[i];
    strOdds2.Add(sum);
  end;

  // select items
  str := '';
  for i := 0 to countItems - 1 do
  begin
    tmpFirstOdds := Random(101); // 0-100
    for j := 0 to strOdds2.Count-1 do
    begin
      if tmpFirstOdds < strOdds2.Items[j] then
      begin
        str := str + IntToStr(strItems[j]) + ',';
        break;
      end;
    end;
  end;
  str := '['+Copy(str,1,Length(str)-1)+']';
  // from monster
  if from=0 then Result := so('{"part":"battle","func":"itemsdropped","data":'+str+'}');
  // from chest
  if from=1 then Result := so('{"part":"battle","func":"itemsdroppedchest","data":'+str+'}');
  strLines.Free;
  extrLines.Free;
  strItems.Free;
  strOdds.Free;
  strOdds2.Free;
end;

У меня чуть глаза не вытекли от такого решения. Код по идее сообщает клиенту, что выпало из монстра после победы над ним.

Cynicrus, 24 Ноября 2014

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

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

+135

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

public void ValidateValue( ref object value ) {
            bool b;
            if ( value == null ) return;
            var typeName = BaseType.Name;
            var valueType = value.GetType();
            if ( valueType.Name == "String" ) {
                if ( BaseType == valueType ) return;
                if ( BaseType == Constants.Types.Boolean ) {
                    if ( string.Compare( "yes", (string) value, StringComparison.OrdinalIgnoreCase ) == 0 ) value = true;
                    else if ( string.Compare( "no", (string) value, StringComparison.OrdinalIgnoreCase ) == 0 ) value = false;
#if !CF
                    else if ( Boolean.TryParse( value.ToString(), out b ) ) value = b;
#else
          else if (TryParseUtility.TryParse(value.ToString(), out b)) value = b;
#endif
                    else throw new ArgumentException( String.Format( Resources.ValueNotCorrectType, value ) );
                    return;
                }
            }

#if !CF
            if ( typeName == "Boolean"
                 && Boolean.TryParse( value.ToString(), out b ) ) {
                value = b;
                return;
            }

            UInt64 uintVal;
            if ( typeName.StartsWith( "UInt64" )
                 && UInt64.TryParse( value.ToString(), out uintVal ) ) {
                value = uintVal;
                return;
            }

            UInt32 uintVal32;
            if ( typeName.StartsWith( "UInt32" )
                 && UInt32.TryParse( value.ToString(), out uintVal32 ) ) {
                value = uintVal32;
                return;
            }

            long intVal;
            if ( typeName.StartsWith( "long" )
                 && long.TryParse( value.ToString(), out intVal ) ) {
                value = intVal;
                return;
            }

            int intVal32;
            if ( typeName.StartsWith( "Int32" )
                 && Int32.TryParse( value.ToString(), out intVal32 ) ) {
                value = intVal32;
                return;
            }
#else
      if (typeName == "Boolean" && TryParseUtility.TryParse(value.ToString(), out b)) { value = b; return; }

      UInt64 uintVal;
      if (typeName.StartsWith("UInt64") && TryParseUtility.TryParse(value.ToString(), out uintVal)) { value = uintVal; return; }

      UInt32 uintVal32;
      if (typeName.StartsWith("UInt32") && TryParseUtility.TryParse(value.ToString(), out uintVal32)) { value = uintVal32; return; }

      long intVal;
      if (typeName.StartsWith("long") && TryParseUtility.TryParse(value.ToString(), out intVal)) { value = intVal; return; }

      Int32 intVal32;
      if (typeName.StartsWith("Int32") && TryParseUtility.TryParse(value.ToString(), out intVal32)) { value = intVal32; return; }
#endif

            object objValue;
#if RT
      Type baseType = BaseType.GetTypeInfo().BaseType;
#else
            var baseType = BaseType.BaseType;
#endif
            if ( baseType != null
                 && baseType.Name == "Enum"
                 && ParseEnum( value.ToString(), out objValue ) ) {
                value = objValue;
                return;
            }

            throw new ArgumentException( String.Format( Resources.ValueNotCorrectType, value ) );
        }

MySql.Data 6.9.5, MySqlConnectionStringBuilder.cs

kasthack, 24 Ноября 2014

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

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

+51

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

std::list<COptions *> COptions::m_InstanceList;
..........
COptions::COptions()
{
	for (int i = 0; i < OPTIONS_NUM; ++i)
		m_OptionsCache[i].bCached = FALSE;
	m_pOptionsHelperWindow = new COptionsHelperWindow(this);
	simple_lock lock(m_mutex);
#ifdef _DEBUG
	for (std::list<COptions *>::iterator iter=m_InstanceList.begin(); iter != m_InstanceList.end(); ++iter)
		ASSERT(*iter != this);
#endif _DEBUG
	m_InstanceList.push_back(this);
	m_SpeedLimits[0] = m_sSpeedLimits[0];
	m_SpeedLimits[1] = m_sSpeedLimits[1];
}

COptions::~COptions()
{
	{
		simple_lock lock(m_mutex);
		std::list<COptions *>::iterator iter;
		for (iter=m_InstanceList.begin(); iter != m_InstanceList.end(); ++iter) {
			if (*iter == this)
				break;
		}

		ASSERT(iter != m_InstanceList.end());
		if (iter != m_InstanceList.end())
			m_InstanceList.erase(iter);
	}

	if (m_pOptionsHelperWindow)
		delete m_pOptionsHelperWindow;
	m_pOptionsHelperWindow=0;
}
.......

В отладке включается тяжелый героиновый бред разработчиков FileZilla

fsmoke, 24 Ноября 2014

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

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

+49

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

void ZModulesDlg::OnDestroy(HWND hwnd)
{
	LVITEM lvi = { LVIF_PARAM };

	if (lvi.iItem = ListView_GetItemCount(hwnd))
	{
		do 
		{
			lvi.iItem--;
			if (ListView_GetItem(hwnd, &lvi))
			{
				((ZDll*)lvi.lParam)->Release();
			}
		} while (lvi.iItem);
	}
}

zhukas, 17 Ноября 2014

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

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

+156

01
02
03
04
05
06
07
08
09
10

// Real itemid is "contained" in public itemid in a bit tricky way. The formula is:

public_itemid = itemid * 256 + anum

// Quite easy, isn't it? Except where to get that anum. anum is some number assigned when you create a new post in LJ.

// But in fact, you don't need it. Because it is always less than 256, we can avoid searching for it, and just use simple Math to find itemid.

$anum = $public_itemid - floor($public_itemid / 256) * 256;
$itemid = ($public_itemid - $anum) / 256;

Прекрасный подход, то место, где вордпресс и ЖЖ сошлись воедино.

http://drumrock.skipitnow.org/livejournal-tools/itemid/

Fike, 10 Ноября 2014

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

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