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

    +147

    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
    public class Person {
    	
    	private final int id;
    	private final String name;
    	private final int age;
    	
    	public Person(int i, String s, int a)
    	{
    		id = i;
    		name = s;
    		age = a;
    	}
    	
    	public int getId()
    	{
    		return id;
    	}
    	
    	public String getName()
    	{
    		return name;
    	}
    	
    	public int getAge()
    	{
    		return age;
    	}
    }

    Примерно такой код был найден мною совсем недавно, сразу же скажу это не точный код, это то, как выглядел тот класс.

    Запостил: redenemy, 05 Августа 2011

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

    • И что? Обычный Transfer object
      Ответить
      • А смысл делать private id, name, age и делать им геттеры если они final ?
        Ответить
    • Топим!!!!111111
      Ответить
    • id = i; name = s (???); age = a; где логика, блеать, я вас спрашиваю???
      Ответить
      • Раз iнтеджер, значит i.
        Раз sтринг, значит s.

        Вы не учились в школе?
        Ответить
        • но на age = a логика обрывается
          по вашей логике здесь должно быть что то типа age = i2, раз age - iнтеджер.
          Ответить
          • Ну i уже занято, а i2 - это неинформативненько.
            Поэтому a от age.
            Ответить
            • очень даже информативно
              в a - не содержится имя типа
              а в i2 - содержится его имя

              что у вас с логикой?
              Вы не учились в школе?


              ps: автора говнокода я никоим образом не поддерживаю
              Ответить
              • Видимо, мне все-таки стоит ставить теги [irony].
                Ответить
                • Обвинять zooz13 в том что у него проблемы с логикой и он не учился в школе - это не скроет ни какой тег [irony].
                  Ответить
            • а должно бы обозначать символьный тип, если бы пхпешники разбирались в программировании
              Ответить
    • @redenemy, ты ещё не дорос до immutable data. минус тебе, раз ты не способен оценить профессиональный код.
      Ответить
      • Возможно, может объясните зачем так делать ? Ведь если они final изменить их уже нельзя ?
        Ответить
        • Потому что обращение к методу и обращение к полю - это две большие разницы. Методы можно переопределять, на обращение к методу можно повесить аспекты, методы можно профилировать, на методы можно вешать брэйк-поинты, методы можно генерить на лету.
          А главное - методы создают абстракцию. Почитайте Блоха на досуге.
          Ответить
    • показать все, что скрытоvanished
      Ответить

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