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

    +71

    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
    @XmlRootElement(name = "bar")
public class Bar implements IBarObject{

    @XmlElement(name = "return")
    private SuperFoo barObj;

    @Override
    public void setBarObject(Foo obj) {
        //TODO: implement method
    }

    @Override
    public void setBarObject(URI obj) {
        //TODO: implement method
    }

    @Override
    public void setBarObject(SuperFoo obj) {
        barObj = obj;
    }

    @Override
    public void setBarObject(List<URI> obj) {
        //TODO: implement method
    }

    @Override
	@XmlTransient
	public Object getBarObject() {
		return barObj;
	}
}

    Ну хотя бы NotImplementedException, но нет...

    PS Код немного по-переименовывал дабы не спалиться

    Запостил: myzone, 26 Марта 2013

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

    • ava Lure Of Chaos 26.03.2013 00:44 # +1

      убил бы за паттерн setFooObject(Bar bar)
      Ответить

      • ava myzone 26.03.2013 00:47 # 0

        Пустые сеттеры хуже, не?
        Хотя их там вообще быть не должно.
        Ответить

      • ava guest 26.03.2013 17:32 # 0

        объясните пожалуйста что конкретно вам здесь не нравится? я правильно понимаю, что название метода? и, если нет, то о чем вы тогда?
        Ответить

        • ava Lure Of Chaos 26.03.2013 23:31 # 0

          то, что название не соответствует имени(типу) аргумента.
          Ответить

          • ava myzone 26.03.2013 23:46 # 0

            это частично моя вина, я заменил некоторые специфичные слова на Foo и Bar, дабы не спалили
            Ответить

      • ava bot 26.03.2013 20:59 # 0

        Я ебусь в глаза, или методом там и не пахнет?
        Ответить

    • ava someone 26.03.2013 07:43 # 0

      Я так понимаю, по этим аннотациям бины сериализуются в XML?

      Тогда лучше вообще сделать их неизменяемыми.

      А ещё можно сделать, чтобы getBarObject возвращал SuperFoo, а не Object. Covariant return types же. Тогда, подозреваю, можно будет прилепить @XmlElement на геттер и забыть про @XmlTransient.
      Ответить

    • ava SSSandman 26.03.2013 12:51 # +1

      java.lang.UnsupportedOperationException
      Ответить

      • ava myzone 26.03.2013 15:02 # 0

        Ну как вариант, но это все равно было бы костылем.
        Ответить

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