1. Java / Говнокод #7348

    +147

    1. 1
    2. 2
    3. 3
    public class RecordCount {
    public static int reccounter = 0;
}

    Запостил: anatew, 26 Июля 2011

    Комментарии (20) RSS

    • ava Lure Of Chaos 26.07.2011 19:04 # −2

      однозначно - говно говном!
      Ответить

    • ava guest 26.07.2011 20:15 # −1

      Где здесь Govno, anatew?!
      Ответить

    • ava guest 26.07.2011 20:47 # −4

      Глобальная переменная.
      Ответить

      • ava хуита 27.07.2011 10:20 # −2

        Вообще если класс создан как обёртка для трёх-четырёх переменных, то не много смысла в гет\сет методах.
        Подчёркиваю, именно как обёртка. Если он имеет методы для воздействия на переменные - то тогда говно.
        Ответить

        • ava auf1r2 27.07.2011 12:01 # −1

          Если класс создан исключительно как обёртка для трёх-четырёх переменных - то это уже по-умолчанию говно, и говорит о том что автор далек от понимания принципов ООП.
          Ответить

          • ava tir 27.07.2011 12:58 # 0

            А чем плоха обертка для переменных?
            Ответить

            • ava auf1r2 27.07.2011 13:27 # −2

              Если у автора возникла такая необходимость, значит он не смог найти места для этих полей в других объектах, а значит он не продумал объектную модель приложения. Т.е. у него нет четкого понимания какой класс за что отвечает и как взаимодействует с другими объектами. Более того, класс из примера выше в развивающемся проекте с > чем 1 разработчиком обязательно создаст грабли в будущем. Либо кто-то будет читать это значение, надеясь на определенную логику его генерации другим классом, либо другой класс изменит логику генерации значения и все кто его читал пойдут лесом.
              Ответить

              • ava tir 27.07.2011 13:40 # +1

                проблема в том, что ничего не известно про размер проекта, про его особенности, про размер команды разработчиков. если я работаю над проектом один, у меня 5 классов, никакой многопоточности и это обычный счетчик работающий по принципу count++, то я сделаю именно так. и не буду усложнять код только ради того что это "правильно". не стоит быть "разъяренной обезьяной" (кто не знает что это - гугл в помощь =)). также это может быть неоконченный рефакторинг.

                А насчет обертки для переменных - паттерн ParameterObject. Либо надо обработать 2-3 значения вместе, но на отдельную сущность они не тянут.

                так что однозначно говорить, что это говно я бы не взялся.
                Ответить

    • ava gegMOPO4 27.07.2011 09:21 # +4

      И это весь код?
      Ответить

    • ava auf1r2 27.07.2011 11:42 # −9

      показать все, что скрытоНе пойму, минусовали авторы говна?
      Говнокод 100% и безоговорочный. Кто минусует - убейтесь об стену.
      Ответить

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