just_developer ([info]just_developer) wrote,
@ 2009-02-06 12:57:00
Previous Entry  Add to memories!  Tell a Friend  Next Entry
Рабочее
Вообще компетентность "коллег" часто поражает....

            StringBuilder message = new StringBuilder("<Payload type=\"HomepageMarketUpdate\">");
            message.append("<WebSiteId>").append(locale.getWebSite()).append("</WebSiteId>");
            message.append("<Locale>").append(locale.getLocaleCode()).append("</Locale>");
            message.append("</Payload>");
            ...
            return message.toString()

Народ который считается технически одними из самых продвинутых, не понимает, что плохого в таком коде...
Но это только мелочи... Усилинно заставляю себя не писать на эту тему ;)

(13 comments) - (Post a new comment)


[info]ctpeko3a
2009-02-06 01:20 pm UTC (link)
Я тоже заставляю себя не писать об этом. :о)

(Reply to this)


[info]jdevelop
2009-02-06 08:21 pm UTC (link)
народ не знает про Formatter?

(Reply to this)


[info]reichp
2009-02-06 10:02 pm UTC (link)
Да засунуть бы все в строку и String.format её..

(Reply to this) (Thread)

[info]just_developer
2009-02-06 10:25 pm UTC (link)
Проще - проще к людЯм нужно быть ;)

(Reply to this) (Parent)

[info]grez_ua
2009-03-29 08:02 am UTC (link)
поясните товарищам быдлокодорам, в чом соль,перец, и сахар... код свиду вполне адекватен...

(Reply to this) (Thread)

[info]just_developer
2009-03-29 12:16 pm UTC (link)
KISS принцип..
Неужели быдлокодеры не видят как сделать это проще?
Тут же нет совсем ну никакой сложности?
Представляете как тогда выглядит их код когда есть реальная сложность?

(Reply to this) (Parent)(Thread)

[info]grez_ua
2009-03-29 11:52 pm UTC (link)
> Неужели быдлокодеры не видят как сделать это проще?
Как официальный представитель оных, я вам заявляю, что нет - не видим. Простого знания определения метода кисс тут недостаточно, вы бы всетаки поконкретней объяснили, мне например полезно будет.

>Представляете как тогда выглядит их код когда есть реальная сложность?
Представляю; Я каждый день вижу его, и дописываю новый ;-)

(Reply to this) (Parent)(Thread)

[info]just_developer
2009-03-30 08:24 pm UTC (link)
Тогда как официальный представитель объясните чем вас обычная конкатенация не устраивает?
Пример (на всякий случай, не уверен в знаниях быдлокодеров):

...
return 
    "<Payload type=\"HomepageMarketUpdate\">" +
        "<WebSiteId>" + locale.getWebSite() + "</WebSiteId>" + 
        "<Locale>" + locale.getLocaleCode() + "</Locale>" +
    "</Payload>";

(Reply to this) (Parent)(Thread)

[info]grez_ua
2009-03-31 09:01 am UTC (link)
Гдето на ранних этапах изучания явы в мозгах отложилось что "Коннектация стрингов это зло, юзать надо стрингбилдер". Хотя я кагбе в курсе что ваш пример компилятор автоматически преобразовывает в стрингбилдер, но на всякий случай, для подстраховки продолжаю делать это явно... вдруг в компиляторе гдето какаято галочка оптимизации будет выключена, и о ужас мы потеряем в производительности... :-)

(Reply to this) (Parent)(Thread)

[info]just_developer
2009-03-31 04:14 pm UTC (link)
Вот вот ;)
Я про это и говорил.. Народ почему-то усиленно верит в магию

(Reply to this) (Parent)

[info]d_m_
2009-07-03 10:28 pm UTC (link)
Гыг, буквально на прошлой неделе отрефакторил такой код (не свой, естественно, а индусский), а меня же ещё и поругали. Типа - и так ведь работало, зачем время тратить?

(Reply to this) (Thread)

[info]just_developer
2009-07-03 10:42 pm UTC (link)
;-))
вот вот
а потом такой типичный проект двигается от бага к багу.
И счёт багам - на тысячи..
Вместо нового функционала фиксят старые (и новые) баги.
А потому что - ну не понимаю они, и не могут принципиално понять чем это плохо.
На этом проекте босса уже сместили ;) Ибо - поделом. Скорость развития - нулевая

(Reply to this) (Parent)(Thread)

[info]d_m_
2009-07-04 10:57 am UTC (link)
А в моей ситуации начальство ещё и желает видеть фантастически быстрое развитие проекта с целью заменить им два других, сходных по функциям, но морально устаревших. И поэтому время на рефакторинг им жалко :))

(Reply to this) (Parent)

(13 comments) - (Post a new comment)

Create an Account
Forgot your login or password?
Login w/ OpenID
English • Español • Deutsch • Русский…