> Based on these findings, I went through the code and I have incorporated
> these rules and I rewrote it like this:
>
> 1) no wrapping in "if" if we are not logging more than 2 parameters.
> 2) rewritten log messages to not contain any string concatenation but
> moving it all to placeholders ({}).
> 3) wrap it in "if" if we need to execute a method(s) on parameter(s) which
> is resource-consuming.
>


+1


It's a shame slf4j botched it with lambdas, their 2.0 fluent api doesn't
impress me.

Reply via email to