ReSQL.ru
     
powered by simpleCommunicator - 2.0.61     © 2026 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Java [игнор отключен] [закрыт для гостей] / Как улучшить кусок кода
11 сообщений из 11, страница 1 из 1
Как улучшить кусок кода
    #39290361
JulT
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
JulT
Участник
Есть кусочек кода:
Код: java
1.
2.
3.
4.
5.
6.
private String buildMainMessage() {
        String result = buildMessageLocale(....);
        result = enrichMessage(....);
        result = buildAnotherMessage(result);
        return result;
    }


Как можно его отрефакторить?
Спасибо
...
Рейтинг: 0 / 0
11.08.2016, 14:52
| Ответить | Цитировать | Написать  
Как улучшить кусок кода
    #39290368
забыл ник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
забыл ник
Участник
JulT,

Код: java
1.
Message("str").toLocale().enrich().add(someth).build()
...
Рейтинг: 0 / 0
11.08.2016, 14:57
| Ответить | Цитировать | Написать  
Как улучшить кусок кода
    #39290369
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
JulT,

Как минимум вместо result нужны внятные переменные. Экономия на спичках будущую отладку вообще никак не упрощает.

Код: java
1.
2.
3.
4.
String local = buildMessageLocale(....);
String message = enrichMessage(....);
String mainMessage = buildAnotherMessage(message);
return mainMessage;



Опять же не понятно что у вас там в четвероточиях.
...
Рейтинг: 0 / 0
11.08.2016, 14:58
| Ответить | Цитировать | Написать  
Как улучшить кусок кода
    #39290401
JulT
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
JulT
Участник
Здорово, спасибо!
...
Рейтинг: 0 / 0
11.08.2016, 15:40
| Ответить | Цитировать | Написать  
Как улучшить кусок кода
    #39290405
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
забыл никJulT,

Код: java
1.
Message("str").toLocale().enrich().add(someth).build()



Билдер имеет смысл только если он массово используется. В противном случае он сам по себе усложняет реализацию и отладку.
...
Рейтинг: 0 / 0
11.08.2016, 15:45
| Ответить | Цитировать | Написать  
Как улучшить кусок кода
    #39290419
забыл ник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
забыл ник
Участник
Blazkowiczзабыл никJulT,

Код: java
1.
Message("str").toLocale().enrich().add(someth).build()



Билдер имеет смысл только если он массово используется. В противном случае он сам по себе усложняет реализацию и отладку.


Ну еще он имеет смысл, когда конструируемый объект не должен быть виден внешнему миру до момента окончательной инициализации, а также для проверки инвариантов во время конструирования, но вообще вы правы конечно, в данном случае код не настолько плох чтобы его вообще рефакторить
...
Рейтинг: 0 / 0
11.08.2016, 15:57
| Ответить | Цитировать | Написать  
Как улучшить кусок кода
    #39290423
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
JulTКак можно его отрефакторить?
если код обозрим на одном экране и читабелен, то ничего не надо делеать. Идёт как бы конвейер записи в строку.
...
Рейтинг: 0 / 0
11.08.2016, 16:00
| Ответить | Цитировать | Написать  
Как улучшить кусок кода
    #39290427
no56892
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
no56892
Участник
Вместо стринг использовать что-нибудь более подходящее (POJO, например). Я так понял оно там внутри еще и парсится...
...
Рейтинг: 0 / 0
11.08.2016, 16:02
| Ответить | Цитировать | Написать  
Как улучшить кусок кода
    #39290439
JulT
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
JulT
Участник
С сервера прилетают некие коды, по которым выводится определенный месседж пользователю. В одном случае нужно сразу отобразить пользователю сообщение, например, "Недостаточно средств на балансе", и выйти из метода, в другом, пойти по цепочке дальше. Более того, в проперти файле хранятся key-value, типа: change.service.balance=Изменить баланс, а есть и такие: current.balance=Ваш баланс %s. Т.е. в некоторых случаях вместо символа нужно еще и значение вставить. Хочется как-то сделать красиво, но не знаю как
...
Рейтинг: 0 / 0
11.08.2016, 16:14
| Ответить | Цитировать | Написать  
Как улучшить кусок кода
    #39290458
Basil A. Sidorov
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Basil A. Sidorov
Участник
Если шаблоны подходят для Text.Format - его и использовать.
ResourceBundle , опять-таки.
"По-моему, так" (ц) Винни-Пух.
...
Рейтинг: 0 / 0
11.08.2016, 16:21
| Ответить | Цитировать | Написать  
Как улучшить кусок кода
    #39290466
JulT
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
JulT
Участник
Basil A. SidorovЕсли шаблоны подходят для Text.Format - его и использовать.
ResourceBundle , опять-таки.
"По-моему, так" (ц) Винни-Пух.
я использую спринговый:
Код: java
1.
2.
MessageSource
messageSource.getMessage(
...
Рейтинг: 0 / 0
11.08.2016, 16:28
| Ответить | Цитировать | Написать  
11 сообщений из 11, страница 1 из 1
Форумы / Java [игнор отключен] [закрыт для гостей] / Как улучшить кусок кода
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение   Полит. конфиден.  
NoSQL.ru       Кролег: Проекты, Новости, Чат       РЕД ФОРУМ       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=59&msg=39290458&tid=2123827]:
 0ms
get settings:
 10ms
get forum list:
 19ms
check forum access:
 4ms
check topic access:
 4ms
track hit:
 41ms
get topic data:
 12ms
get forum data:
 3ms
get page messages:
 65ms
get tp. blocked users:
 2ms
others: 243ms
total:  403ms
созд. / загр.: 403ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]