logging exception in c#

Question

logging exception the code below allows to save the content of an exception in a text file. Here I'm getting only the decription of the error.

but it is not telling me where the exception occured, at which line. Can anyone tell me how can I achive that so I can get even the line number where the exception occured?

#region WriteLogError /// <summary> /// Write an error Log in File /// </summary> /// <param name="errorMessage"></param> public void WriteLogError(string errorMessage) {   try   {     string path = "~/Error/" + DateTime.Today.ToString("dd-mm-yy") + ".txt";     if (!File.Exists(System.Web.HttpContext.Current.Server.MapPath(path)))     {       File.Create(System.Web.HttpContext.Current.Server.MapPath(path))      .Close();     }     using (StreamWriter w = File.AppendText(System.Web.HttpContext.Current.Server.MapPath(path)))     {       w.WriteLine("\r\nLog Entry : ");       w.WriteLine("{0}", DateTime.Now.ToString(CultureInfo.InvariantCulture));       string err = "Error in: " + System.Web.HttpContext.Current.Request.Url.ToString()                   + ". Error Message:" + errorMessage;       w.WriteLine(err);       w.WriteLine("__________________________");       w.Flush();       w.Close();     }   }   catch (Exception ex)   {     WriteLogError(ex.Message);   }  }  #endregion 

Answer 1

I find that the easiest way to log exceptions in C# is to call the ToString() method:

try {  } catch (Exception ex) {     Console.WriteLine(ex.ToString()); } 

This usually gives you all the information you need such as the error message and the stack trace, plus any extra exception specific context information. (however note that the stack trace will only show you source files and line numbers if you have your application compiled with debug information)

It is worth noting however that seeing a full stack trace can be fairly offputting for the user and so wherever possible you should try to handle exceptions and print out a more friendly error message.

On another note - you should replace your method WriteLogError with a fully featured logging framework (like Serilog) instead of trying to write your own.

Your logging method is not thread safe (your log file will probably end up with log messages being intermingled with each other) and also should definitely not call itself if you catch an exception - this will mean that any exceptions that occur whilst logging errors will probably cause a difficult to diagnose StackOverflow exception.

I could suggest how to fix those things, however you would be much better served just using a proper logging framework.

Answer 2

Just log ToString(). Not only will it give you the stack trace, but it'll also include the inner exceptions.