Question

I have just begun working on a C# application (using log4net). I’m supposed to add logs to code written by someone else. Hence, it’s not possible for me to understand well the context each time (too much code, too less time 🙂 ).

I have been following a convention which seems quite crude. I display log level, datetime, class name, method name with every log. I print the log on entering and exiting every method (most of them, i try to exclude method within big loops), in constructors, some events and every catch statement.

I think I’m overdoing it at places but some degree of uniformity is required. Any suggestions on the right (or better) approach?

Answer 1

Logging the start and end of every method, and the constructors, is certainly overkill.

I’m afraid that if you don’t understand the method you won’t be able to log the appropriate issues.

At the very least log all exceptions, never catch an exception without doing something with it.