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

    +63

    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
    public class ExtendFSDirectoryProvider {
        // skipped
        @Override
        public boolean equals(Object obj) {
            if (obj == this) {
                return true;
            }
            if (obj == null || !(obj instanceof FSDirectoryProvider)) {
                return false;
            }
    
            return indexName.equals(((ExtendFSDirectoryProvider) obj).indexName);
        }
    }

    Даже неважно, что автор делает лишнюю проверку на null - в instanceof указан другой класс! Копипаста детектед.

    Запостил: victor-homyakov, 17 Апреля 2012

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

    • А где лишняя проверка на null? А то, что копипаста, тут ничего не поделаешь - беда самой джавы. Другое дело, что надо было не копипастить, а генерить эклипсом.
      Ответить
    • показать все, что скрытоvanished
      Ответить
    • Очень трудно простить измену любимого человека, но ещё труднее жить без него. Знаете чувство, когда смотришь на человека, и мурашки бегут по коже. Когда хочется обнять и сжать так сильно, насколько только можно. Когда ты считаешь секунды до вашей встречи. Когда целуешь его и всегда как в первый раз. Когда он говорит: "Нам надо поговорить", - и у тебя сердце замирает, и, ещё ничего не услышав, ты начинаешь молить Бога о том, чтобы это не было что-то плохое. Вот это - любовь! Да - любовь. Чистая, искренняя, от всего сердца.
      Ответить

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