Jawa / Говнокод #7348 Ссылка на оригинал

0

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

Запостил: anatew anatew, (Updated )

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

    • ava хуита # Ссылка на оригинал 0 ⇈

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

      • ava auf1r2 # Ссылка на оригинал 0 ⇈

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

          • ava auf1r2 # Ссылка на оригинал 0 ⇈

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

            • ava tir # Ссылка на оригинал 0 ⇈

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

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

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

  • ava auf1r2 # Ссылка на оригинал 0

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

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

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓