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

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

В номинации:

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

За время:

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

+134

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

string pars_param(string data) 
        {
            HtmlAgilityPack.HtmlDocument html = new HtmlAgilityPack.HtmlDocument();
            html.LoadHtml(data);
            HtmlNodeCollection qw = html.DocumentNode.SelectNodes(@"//input[@name=""k""]");
            if (qw != null)
            {
                foreach (HtmlNode n in qw)
                {
                    if (n.Attributes["value"] != null)
                    {
                        return n.Attributes["value"].Value;
                    }
                }
            }
        }

Error 1 'trcli.Form1.pars_param(string)': not all code paths return a value D:\zMyDoc\igl\Documents\Visual Studio 2013\Projects\trcli\trcli\Form1.cs 651 16 trcli

не магу разобраться в своем говнокоде, не хочет возвращать n.Attributes["value"].Value

igl, 04 Декабря 2014

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

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

+134

01
02
03
04
05
06
07
08
09
10

private bool CompareLvlCats(string[] cat,List<string[]> cats, int lvl)
{
    if (lvl == 1) return cats.Find(x => x[0] == cat[0] && (x[1] != cat[1] ||x[1]!="")) != null;
    if (lvl == 2) return cats.Find(x => x[0] == cat[0] && x[1] == cat[1] && (x[2] != cat[2] || x[2] != "")) != null;
    if (lvl == 3) return cats.Find(x => x[0] == cat[0] && x[1] == cat[1] && x[2] == cat[2] && (x[3] != cat[3] || x[3] != "")) != null;
    if (lvl == 4) return cats.Find(x => x[0] == cat[0] && x[1] == cat[1] && x[2] == cat[2] && x[3] == cat[3] && (x[4] != cat[4] || x[4] != "")) != null;
    if (lvl == 5) return cats.Find(x => x[0] == cat[0] && x[1] == cat[1] && x[2] == cat[2] && x[3] == cat[3] && x[4] == cat[4] && (x[5] != cat[5] || x[5] != "")) != null;
    if (lvl == 6) return false;
    return false;
}

Здесь мы идём снова.

pushistayapodmyshka, 29 Ноября 2014

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

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

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

public void OnKillEvent_ServerOnly(PhotonPlayer player)
    {
    case MatchTypeInfo.TDM:

                if (player != null)
                {
                    var hash = player.customProperties;

                    if (hash != null)
                    {
                        var team = (int) hash["team"];

                        var realScore = match.GetTeamScores();

                        var teamAScor = team == 0 ? realScore.x + TeamDeathScores : realScore.x;
                        var teamBScor = team == 1 ? realScore.y + TeamDeathScores : realScore.y;

                        //DebugModule("Kill Processed On Server completed: " + team + "// " + teamAScor + "/" + teamBScor);

                        match.ChangeTeamScores((int)teamAScor, (int)teamBScor);
                    }
                    else
                    {
                        Debug.LogError("OnKillEvent_ServerOnly ERROR Team hash");
                    }
                }

                break;
}  {
//через 500 строк кода
   private void LocalUpdate()
    { 
    if (Application.isEditor)
        {
            if (Input.GetKeyDown(KeyCode.Y))
            {
                switch (MatchType)
                {

                    case MatchTypeInfo.TDM:
                        ChangeTeamScores((int)(_teamScores.x - 1), (int)(_teamScores.y - 1), true);
                        break;   }

                SendTeamScoresToUI();
            }
        }

   switch (MatchType)
        {
            case MatchTypeInfo.TDM:
               // DebugModule("Load TDM type game");
                break;
}
}
//ещё 500 строк
 public void ChangeTeamScores(int a, int b, bool force = false)
    {

        if (PhotonNetwork.isMasterClient)
        {
            // check server rules
            var rules = GetGameRules();
            if (rules != null)
            {
                if (!rules.CheckPlayerCountRules() && !force)
                {
                    Debug.LogError("Game is not started, beacuse server has no players: " + rules.MinimumPlayerToPlay + "/" +
                                   PhotonNetwork.room.playerCount);
                    return;
                }
            }
            else
            {
                Debug.LogError("Rule error");
                return;
            }

            _teamScores.x = Mathf.Clamp(a, 0, float.MaxValue);
            _teamScores.y = Mathf.Clamp(b, 0, float.MaxValue);

            var matchIsCompleted = false;

            switch (MatchType)
            {
                case MatchTypeInfo.TDM:
                    matchIsCompleted = _teamScores.x <= 0 || _teamScores.y <= 0;
                    break;
            }
               if (matchIsCompleted)
            {
                Server_CallEndRound();
            }
            else
            {
                // Call Event??
              }
}
}

Чтобы я ещё раз полез помогать инди-разработчикам, мать их за ногу.

noshitleft, 26 Ноября 2014

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

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

+134

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

protected static DataTable[] ExecuteDataTablesReader(string ProcedureName, SqlParameter[] Params = null) {

            SqlConnection cnn = new SqlConnection(ConnectionString);
            SqlCommand cmd = new SqlCommand(ProcedureName, cnn);
            cmd.CommandType = System.Data.CommandType.StoredProcedure;
            if (Params != null && Params.Count() > 0) {
                cmd.Parameters.AddRange(Params);
            }
            cnn.Open();
            
            IDataReader rd = cmd.ExecuteReader();
            List<DataTable> tables = new List<DataTable>();
            do {
                DataTable dt = new DataTable();
                dt.Load(rd);
                tables.Add(dt);
            } while (rd.NextResult());

            return tables.ToArray();

        }

может я чего не понимаю, но какого хера rd.NextResult() кидает мне exception, что ридер закрыт?

Lokich, 26 Ноября 2014

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

Си / Говнокод #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)

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

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

class ParametersArray
{
        ...
        /// <summary>
        /// Копирующей конструктор (создает копию коллекции)
        /// </summary>
        /// <param name="prs">Коллекция на основе, которой будет создаваться копия</param>
        public ParametersArray(ParametersArray prs)
        {
            paParameters = new ArrayList();
            for (int i = 0; i < prs.Count; i++)
            {
                Parameter p = new Parameter(prs[i]);
                paParameters.Add(p);
            }
        }
        ...
        #region Внутренние состояния

        /// <summary>
        /// Внутреннее поле - массив параметров
        /// </summary>
        private ArrayList paParameters;

        /// <summary>
        /// Внутреннее поле – идентификатор операции, к которой относятся параметры
        /// </summary>
        private string paOperationId = "";

        #endregion
        /// <summary>
        /// Удаляет параметр из коллекции
        /// </summary>
        /// <param name="parameterId">Идентификатор параметра</param>
        public void Delete(string parameterId)
        {
            ArrayList list = new ArrayList();

            foreach (Parameter prt in paParameters)
            {
                if (prt.Id.Trim() != parameterId.Trim())
                {
                    list.Add(prt);
                }
            }
            paParameters = list;
        }
    }
}

...

  /// <summary>
  /// Класс для поиска и обработки шаблонов в параметрах операций
  /// </summary>
  public class Parser
  {
    ...
    #region Конструкторы

    /// <summary>
    /// Конструктор инициализирующей класс
    /// </summary>
    /// <param name="requestUserId">ID пользователя, который используется для обработки шаблона ##USER()</param>
    public Parser(string requestUserId)
    {
      pParameters = null;
      pUserId = requestUserId;
    }

    #endregion

    #region Внешние методы

    /// <summary>
    /// Метод перебирает все параметры операции, ищет там шаблоны и заменяет их значениями
    /// </summary>
    /// <param name="opr">Операция</param>
    /// <returns>Операция с обработанными шаблонами</returns>
    public Operation Parse(Operation opr)
    {
      pParameters = new ParametersArray(opr.Parameters);

      for (int i = 0; i < pParameters.Count; i++)
        if (FindTemplate(pParameters[i]))
        {
          ReplaceTemplate(pParameters[i]);

          //Удаляет параметр с ошибкой
          if (pParameters[i].Value == "parse err")
            pParameters.Delete(pParameters[i].Id);
        }

      opr.Parameters = new ParametersArray(pParameters);
      return opr;
    }

Внимание привлекает код в строках 81-95

В строке 81-82 мы создаём временную копию параметров операции opr во внутреннем поле класса Parser. Прекрасно. Ещё раз - в публичном в методе инициализируем внутреннее поле (Классный side effect - поле используется в других методах).

В строке 91 удаляет элемент из внутренней коллекции, пересоздавая коллекцию целиком, то есть требования к памяти временно удваиваются на выходе из метода Delete (до очистки памяти). В методе Delete. Да. Супер.
потом инициализируем вновь созданной коллекцией публичное поле Parameters во входящем параметре opr метода Delete.

P.S.: Диагноз - хроническая форма неизлечимой Java. Во первых, автор этих строк искренне думает, что в природе ничего, кроме коллекции ArrayList, не существует. Во вторых, для удаления элемента коллекции мы пересоздаём всю коллекцию целиком, то есть видимо искренне полагая, что коллекция ArrayList - immutable, как и все поля класса Parser (типа ParametersArray) равно как и поле класса ParametersArray (типа ArrayList). Занавес.

Говнокодище.

hack2root, 10 Ноября 2014

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

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

+134

1
2
3
4

{GENDER, select,
    male {He}
  female {She}
   other {They}

https://github.com/SlexAxton/messageformat.js

someone, 29 Октября 2014

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

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

+134

1
2
3
4

//#if UNITY_IPHONE && !(UNITY_3_0 || UNITY_3_1 || UNITY_3_2 || UNITY_3_3 || UNITY_3_4 || UNITY_3_5 || UNITY_3_6 || UNITY_3_7 || UNITY_3_8 || UNITY_3_9 || UNITY_4_0 || UNITY_4_1 || UNITY_4_2 || UNITY_4_3 || UNITY_4_4 || UNITY_4_5 || UNITY_4_6 || UNITY_4_7 || UNITY_4_8 || UNITY_4_9)
if (Selection.activeGameObject != null)
control = (IControl)Selection.activeGameObject.GetComponent("IControl");
//#endif

принял код от юнити юниора

sladkijBubaleh, 23 Октября 2014

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

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

+134

1
2
3
4
5

#ifdef __APPLE__
#define OPENGL_LIBRARY "/System/Library/Frameworks/OpenGL.framework/Libraries/libGL.dylib"
#else
#define OPENGL_LIBRARY "libGL.so"
#endif

А вообще, у мака есть переменные окружения?

Pythoner, 10 Октября 2014

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

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

+134

1
2
3
4
5
6
7

#define TIME_MATCH    500 /* 5 seconds */
#define TIMEOUT       2000 /* 20 seconds */
...
if ((pPoint->dwTimer >= TIME_MATCH) 
    || (pPoint->dwTimer >= TIMEOUT)) {
	    bNewDigit = TRUE;
      }

И вроде бы причем тут индусы со построчной оплатой кода.

qxoz, 07 Октября 2014

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

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