User
public void sendMessage(String message) { long startTime = LoggerUtil.getMeasureStartTime(); try { /** * business logic - send message */ LoggerUtil.operationSuccess(LogFactory.getLog({log's appender name}), startTime, "messageService", "sendMessage"); } catch (Exception e) { LoggerUtil.operationFailure(LogFactory.getLog({log's appender name}), startTime, "messageService", "sendMessage"); } }
должно быть так
убогое решение:
1. Вы глотаете исключения — никто и никогда не узнает в чем же была ошибка.
2. название LoggerUtil никак не ассоциируется с сбором статистики
3. Поддерживать и не дай бог делать рефакторинг для этого кода крайне не удобно
4. Более удобные и гибкие решения уже написаны до вас, поэтому мне кажется, что вы изобрели тругольное колесо
1. Dы глотаете исключения — никто и никогда не узнает в чем же была ошибка.
2. название LoggerUtil никак не ассоциируется с сбором статистики
поддерживать и не дай бог делать рефакторинг для этого кода крайне не удобно, более удобныеб гибки
вот молодцы, как константы хорошо пределяют. Кстати, в чем крутость этого кода?
это как?