Рефакторинга, вложенный метод вызывает

Question

Я рефакторинг программы, которая имеет много вложенных вызовы, такие как

ServiceManagement.getGlobalizationService() 
        .createExportConfiguration(
          exportConfiguration, 
          getAdminWebClientSession().getUser().getToken() 
            .getTokenValue()); 

Я хотел бы несколько советов относительно того, какого рода рефакторинга был бы уместно здесь, а также лучшим и самое простое место для начала рефакторинга, прямо сейчас я считаю, что это слишком сложный и неясный блок.

Answer 1

Первое: вы полностью уверены, что ни один из этих вызовов не может (и я имею в виду EVER) вернуть null? Если вы не полностью уверены, то вы должны делать:

GlobalizationService gs = ServiceManagement.getGlobalizationService(); 
if (gs!=null) { 
    .... 
} 

Или вы могли бы поймать NullPointerException.

Если вы полностью уверены, что не является необходимым, то есть полезная небольшая refactorization для читаемости:

TokenValue tv = getAdminWebClientSession().getUser().getToken().getTokenValue(); 
ServiceManagement.getGlobalizationService() 
       .createExportConfiguration(exportConfiguration,tv); 

Вы могли бы рассмотреть вопрос о применении закона Деметры получить TokenValue непосредственно из AdminClientSession, но в в общем случае, что рефакторизация не всегда полезна. Альтернативное применение Закона Деметры означало бы, что TokenValue или Token передаются методу, в котором этот код лежит, но опять же это не обязательно хорошая идея.