title: Enclosing calls to debug() in if isDebugEnabled(): a good policy?
body: <p>Our team has the policy of doing logging like</p>
  <pre><code>if (LOGGER.isDebugEnabled()) {  
    LOGGER.debug(&quot;model[&quot; + model + &quot;]&quot;);
  }
  </code></pre>
  <p>instead of simply calling the logging method like this:</p>
  <pre><code>LOGGER.debug(&quot;model[&quot; + model + &quot;]&quot;);
  </code></pre>
  <p>This practice is capable of leading to some <a href="https://stackoverflow.com/questions/545844/biggest-performance-improvement-youve-had-with-the-smallest-change/#546409">performance improvement</a>, but on the other hand it makes the codebase more complex.
  Our application has no performance problems, probably never will, the argument for the introduction of the policy was simply that it's a good practice, so it shall be used every time we are doing logging.</p>
  <p>Do you think it's a good policy?</p>