Гость
Войти | Профиль | Очистить
Целевая тема:
Создать новую тему:
Автор:
Действия ...
Форумы / Java [игнор отключен] [закрыт для гостей] / Как отрефакторить код / 7 сообщений из 7, страница 1 из 1
29.06.2016, 14:38
    #39264712
JulT
JulT
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как отрефакторить код
Всем привет. Есть такой кусок кода:
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
Optional<Region> region = resolveRegionByIP(contextRequest);
            if(!region.isPresent()){
                region = resolveRegionByRegionId(httpRequest.getParameter(REGION_ID), language);
                if(!region.isPresent()){
                    region = resolveRegionByParameter(httpRequest.getParameter(REGION_NAME), language);
                    if(!region.isPresent()){
                        region = resolveRegionByCookie(cookie, language);
                    }
                }
            }


Подскажите пожалуйста, как можно его отрефакторить? Стоит ли такой кусок выделять в отдельный класс?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
29.06.2016, 14:46
    #39264727
Blazkowicz
Blazkowicz
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как отрефакторить код
http://stackoverflow.com/a/28514834
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
29.06.2016, 15:01
    #39264742
JulT
JulT
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как отрефакторить код
Blazkowicz http://stackoverflow.com/a/28514834
Мерси:)
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
29.06.2016, 18:51
    #39264972
no56892
no56892
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как отрефакторить код
Как по мне, так первый вариант куда более понятнее.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
29.06.2016, 22:05
    #39265039
mayton
mayton
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как отрефакторить код
Надо глянуть как дебаггер степает по варианту с лямбдами.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
29.06.2016, 23:53
    #39265083
Mad_Head
Mad_Head
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как отрефакторить код
mayton,

не знаю как в плохих IDE, но в идее все не плохо продумано. Можно отдельно поставить точку на лямбду, можно на обрамляющий метод.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
30.06.2016, 00:30
    #39265104
rfq
rfq
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Как отрефакторить код
Я бы так написал:
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
Optional<Region> region = resolveRegionByIP(contextRequest);
resolve:
if(!region.isPresent()){
     region = resolveRegionByRegionId(httpRequest.getParameter(REGION_ID), language);
     if(region.isPresent()){
         break resolve;
     }
     region = resolveRegionByParameter(httpRequest.getParameter(REGION_NAME), language);
     if(region.isPresent()){
         break resolve;
     }
     region = resolveRegionByCookie(cookie, language);
}
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
Форумы / Java [игнор отключен] [закрыт для гостей] / Как отрефакторить код / 7 сообщений из 7, страница 1 из 1
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение  
Полит. конфиден.  
созд. / загр.: 384ms
Закрыть
start [/forum/topic.php?fid=59&mobile=1&tid=2123939]:
 0ms
get settings:
 5ms
get forum list:
 8ms
check forum access:
 2ms
check topic access:
 2ms
track hit:
 131ms
get topic data:
 8ms
get forum data:
 2ms
get page messages:
 29ms
get tp. blocked users:
 1ms
others: 196ms
total:  384ms
x
x
Закрыть


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