The following part of code raises a major bug at SonarQube : "Invoke method(s) only conditionally."
How am I supposed to fix this?
if(us != null){ logger.info("Log this: {}", us.toString()); }
The call to us.toString()
is redundant, toString()
method will be called regardless the configured log level. You should pass only us
as an argument to info
without an if
statement.
logger.info("Log this: {}", us);
As stated at the comments of the question, another working answer is:
if(logger.isInfoEnabled() && us != null){ logger.info("Log this: {}", us.toString()); }
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With