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

    +75

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    public class SomeClass
{
	@SuppressWarnings("unused")
	private Validator validator;

	... 
}

    Эта переменная реально не используется. Удалить бы - ан нет, автор почему-то считает, что лучше пометить её как неиспользуемую, чтобы компилятор и Эклипс не ругались.
    Чем только народ думает...

    Запостил: asolntsev, 21 Июля 2010

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

    • ava 3.14159265 21.07.2010 17:34 # +2

      >>>Чем только народ думает...
      ну так для некоторых это нормально заткнуть фонтан - ворнинги отключить итд... вместо устранения первопричин
      я об этом в соседней теме писал http://govnokod.ru/3752
      Ответить

    • ava Lure Of Chaos 21.07.2010 21:36 # 0

      ну как чем. а вдруг понадобится!
      Ответить

      • ava Nagg 21.07.2010 22:35 # +1

        так же отвечают бабушки, которых спрашивают "Нахрена вам холодильник, одна лыжа и прочее барахло на балконе?"
        Ответить

        • ava Lure Of Chaos 21.07.2010 22:40 # +1

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

          но аффтар, видать, не дошел еще до понимания этого
          Ответить

          • ava Анонимус 21.07.2010 23:19 # 0

            ээ
            а закомментить?
            Ответить

            • ava Lure Of Chaos 21.07.2010 23:20 # +1

              это же унижает достоинство, мол: как, мой код да закомменить???
              а так просто заткнуть глотку противной ёклипсе
              Ответить

              • ava Анонимус 21.07.2010 23:25 # +4

                компалеру, а не эклипсе.

                код надо комментить. Хуже того -- код надо удалять. Когда программист начнет получать удовольствие от удаления собственного кода, ставшего ненужным в результате рефакаторинга -- он познает дзен
                Ответить

                • ava Lure Of Chaos 21.07.2010 23:32 # +2

                  да и компалеру тоже.

                  код НЕ надо комментить. Код надо рефакторить и удалять. Безжалостно. Это не гениальный роман, который потом сколько угодно мартышек не смогут восстановить. наоборот, появляется шанс, что повторно код может быть написан еще лучше
                  нет дела более сладкого, чем рефакторить код. Эм, че я слышу? секс? ну ладно, секс. = ))
                  Ответить

                • ava 3.14159265 22.07.2010 08:44 # +2

                  >>>>Когда программист начнет получать удовольствие от удаления собственного кода, ставшего ненужным в результате рефакаторинга -- он познает дзен

                  это настолько метко подмечено, что надо высекать в камне
                  Ответить

                  • ava Lure Of Chaos 22.07.2010 09:34 # +1

                    только не надо высекать как "рефакаторинга". потомки не поймут
                    Ответить

                    • ava 3.14159265 22.07.2010 10:34 # 0

                      согласен,
                      - "в результате рефакаторинга"

                      >>>пусть восклицательный знак напоминает о недоделке (иначе забудешь и не сделаешь)
                      ага в эклипсе он еще и на все родительские пекеджи выставляется, так что оно постоянно глаз мозолит
                      Ответить

                      • ava Lure Of Chaos 22.07.2010 10:44 # 0

                        > на все родительские пекеджи выставляется
                        что бы типа не потерялся
                        Ответить

          • ava malleus 23.07.2010 16:03 # +2

            если тебе фишка не нужна СЕЙЧАС - ее лучше не писать. очень много фишек так и окажутся ненужными никогда.
            Ответить

    • ava ErmineMD 22.07.2010 11:31 # 0

      В порядке бреда: а, может, эта переменная делает что-то нужное? Например, увеличивает размер класса на сайзов Validator. Или чтобы в этом классе видеть, если вдруг отвалился исходный файл, где описал класс Validator. Короче, это нужно у автора спрашивать, почему он так сделал, а не нам тут гадать. Может, он познал Дцен, а мы его обзываем.
      Ответить

      • ava Анонимус 22.07.2010 13:59 # +1

        >>Например, увеличивает размер класса на сайзов Validator
        в джаве врядли это кто-то юзает
        Ответить

      • ava asolntsev 22.07.2010 14:37 # 0

        В данном случае я точно знаю, что это ни для чего не нужно. Я часто вижу кода этого автора, он просто любит такой фигней страдать. При том что он отнюдь на начинающий программист.
        Ответить

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