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

    +134

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    85. 85
    86. 86
    87. 87
    88. 88
    89. 89
    90. 90
    91. 91
    92. 92
    93. 93
    94. 94
    95. 95
    96. 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) RSS

    • ava hack2root 10.11.2014 13:50 # 0

      И феерией этого говнокода я считаю возврат входного параметра opr метода Parse с только что иницилизированным публичным параметром Parameters, с только что созданным массивом элементов без одного указанного элемента!
      Ответить

    • ava roman-kashitsyn 10.11.2014 14:15 # +1

      функциональненько
      Ответить

    • ava Lokich 10.11.2014 15:43 # +1

      ебанный велосепедист
      Ответить

    • ava Анонимус 10.11.2014 16:25 # +1

      Скорее уж неизлечимая Java2.0 потому что человек не умеет работать с коллекциями совершенно, и работает так топорно словно на дворе 1998й год.
      Ответить

    • ava wvxvw 10.11.2014 18:16 # +2 

      ArrayList paParameters;
ArrayList maMarameters;
      Ответить

    • ava passiv 25.08.2021 19:31 # 0

      Он продолжал улыбаться, а у меня в голове пронеслась фраза "мои парни". И сколько же их было?
      Ответить

    Добавить комментарий