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

    +75.3

    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
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    public static ru.project.subpackage.PersonDTO convertOshPersonDtoToPersonDto(OshPersonDTO person){
        ru.project.subpackage.PersonDTO dto = new ru.project.subpackage.PersonDTO();
        dto.setPersonid(person.getPersonid().longValue());
        dto.setNamelast(person.getNamelast());
        dto.setNamefirst            (person.getNamefirst            ());
        dto.setNamesec              (person.getNamesec              ());
        dto.setInitials             (person.getInitials             ());
        dto.setSex                  (convertSkVocValue(person.getSex()));
        dto.setReason               (person.getReason               ());
        dto.setWorkphone            (person.getWorkphone            ());
        dto.setWorkphonedigit       (person.getWorkphonedigit       ());
        dto.setLocalphone           (person.getLocalphone           ());
        dto.setLocalphonedigit      (person.getLocalphonedigit      ());
        dto.setHomephone            (person.getHomephone            ());
        dto.setHomephonedigit       (person.getHomephonedigit       ());
        dto.setMobilephone          (person.getMobilephone          ());
        dto.setMobilephonedigit     (person.getMobilephonedigit     ());
        dto.setFax                  (person.getFax                  ());
        dto.setFaxdigit             (person.getFaxdigit             ());
        dto.setPager                (person.getPager                ());
        dto.setEmail                (person.getEmail                ());
        dto.setWeb                  (person.getWeb                  ());
        dto.setNamelastdative       (person.getNamelastdative       ());
        dto.setNamefirstdative      (person.getNamefirstdative      ());
        dto.setNamesecdative        (person.getNamesecdative        ());
        dto.setNamelastaccusative   (person.getNamelastaccusative   ());
        dto.setNamefirstaccusative  (person.getNamefirstaccusative  ());
        dto.setNamesecaccusative    (person.getNamesecaccusative    ());
        dto.setNamelastgenitive     (person.getNamelastgenitive     ());
        dto.setNamefirstgenitive    (person.getNamefirstgenitive    ());
        dto.setNamesecgenitive      (person.getNamesecgenitive      ());
        dto.setNamelastinstrumental (person.getNamelastinstrumental ());
        dto.setNamefirstinstrumental(person.getNamefirstinstrumental());
        dto.setNamesecinstrumental  (person.getNamesecinstrumental  ());
        dto.setNamelastprepositional(person.getNamelastprepositional());
        dto.setNamefirstprepositional(person.getNamefirstprepositional());
        dto.setNamesecprepositional (person.getNamesecprepositional ());
        return dto;
    }

    И так далее еще несколько сотен строк. А главное переупаковка из одних объектов в другие и обратно бессмысленна, т.к. можно использовать исходные (они доступны в приложении)

    Запостил: johnsoft, 07 Сентября 2009

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

    • ava EfremoffPaul 08.09.2009 13:27 # +1

      Зато повышается Чувство Собственной Важности. Теперь я архитектор!
      Ответить

    • ava guest 08.09.2009 16:59 # 0

      На самом деле нормальный код когда вебсервису на выход нужно подать законченную нециклическую иерархию объектов (DTO) а исходные объекты имеют бесконечно циклическую вложенность (Hibernate Persistent object).

      Например Person имеет набор из Certificate. А Certificate имеет ссылко на Person. В случае с хибером можно итерироваться по кругу бесконечно....
      Ответить

      • ava johnsoft 08.09.2009 17:52 # 0

        Да согласен, но на самом деле, у объектов не было больше 1-го уровня потомков и все они использовались в приложении. Т.е., то что я привел фактически является вариантом клонирования. При этом используются другие объекты с другими именами
        Ответить

        • ava guest 09.09.2009 12:49 # 0

          Стоило адаптер использовать, а не функцию преобразования одного в другой.
          public class PersonDtoOshAdapter implements PersonDTO...
          или
          public class PersonDtoOshAdapter extends PersonDTO...
          Тогда будет вполне приличный код.
          Ответить

      • ava johnsoft 10.09.2009 09:49 # 0

        На самом деле, если PersonDTO implement Clonable, то можно просто делать поверхностное копирование и пользоваться!
        Ответить

    • ava guest 10.09.2009 00:26 # 0

      посмотрел на имя функции - convertOshPersonDtoToPersonDto

      представил, какая там ООП-каша

      сблеванул
      Ответить

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