Jawa / Говнокод #12242 Ссылка на оригинал

0

  1. 1
Long value = Long.valueOf(String.valueOf(avpValue));

не специалист по жабе, но пахнет: сначала по всему коду нагородить огородов с Byte/Short/Integer/Long что бы потом в самом важном месте вот так в лоб все в Long конвертить...

Запостил: Dummy00001 Dummy00001, (Updated )

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

  • ava Lure Of Chaos # Ссылка на оригинал 0

    нужно больше контекста, ибо подозреваю, что обертки там вообще не нужны, а особенно через String
    Ответить

  • ava Dummy00001 # Ссылка на оригинал 0

    нет там больше контекста. прога читает сообщения с финансовой информацией. из сообщения выкоревывается в класс (несколько классов для разных сообщений) около 90 полей различных типов, преимущестевенно Byte, Short, Integer и Long. потом делается извращенная херня что бы можно было еще к полям доступатся по номеру - аннотация + два Map<Integer,Method> (один мап для сеттеров, второй для геттеров). потом по всему коду - в центре всей бизнес логики - идет страдание с типами, Method.invoke(), и еще раз страдание с конвертацией из Object, потому что Method.invoke() возвращает Object. потом когда делается самая последня (и дорогая) операция перед посылкой ответа на транзакцию, вот так как сверху просто все в Ж конвертится в Long. вот как бы и вопрос: на кой хер кучи Г для страдания типами, когда можно все впихнуть в Long? что все равно и делается в конце? у меня такое впечатление что тот индус который это делал, к концу писания модуля просто устал с типами трахатся и по простой решил проблему.

    к слову, вопрос от лоха по жабе. а это как нибудь проще написать можно? без конвертации Byte/Short/Integer/Long в строку?
    Ответить

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓