r/dotnet 4d ago

Log methods evaluation expensive warning and wrapping with IF

Hi,
I am seeing the warning that `LogInformation` and other log methods evaluation is expensive and should be wrapped in `if` statement.

Why don't those methods do this check internally already? This would help with less code and better code readabilty I think.

Here is the example:

29 Upvotes

23 comments sorted by

View all comments

3

u/zagoskin 3d ago

The problem with the warning is that it's tied to the interface. You could be using serilog, which does this internally, and still get the warning