I guess 'else' is not required here since the if block has a return... personally, I'd find this easier to read:

```
MessageHandler handler = createDirectHandlerIfPossible(targetObject, targetMethodName);
if (handler == null) {
    ServiceActivatingHandler handler = (StringUtils.hasText(targetMethodName))
        ? new ServiceActivatingHandler(targetObject, targetMethodName)
        : new ServiceActivatingHandler(targetObject);
    handler = this.configureHandler(handler);
}
return handler;
```

...but feel free to ignore that :)