Using System Exception

Question

I have a project I am creating and I am adding a reference to a class that generates an email where the exception happened and lots of good stuff.

My question is. If I was to put this in a method and call it in the Catch{} how would i do this for the whole project so i could just name the method in the catch and it would know on every page of the project.

var exceptionUtility = new genericExceptions();
exceptionUtility.genericSystemException(
ex,
Server.MachineName,
Page.TemplateSourceDirectory.Remove(0, 1) + Page.AppRelativeVirtualPath.Remove(0, 1),
ConfigurationManager.AppSettings["emailSupport"],
ConfigurationManager.AppSettings["emailFrom"],
string.Empty);

If you mean adding Try/Catch at every method calls or property calls, then i would say its a bad coding style. Exception handling at every level is costly. — Zenwalker
– Zenwalker, Commented Feb 25, 2013 at 10:03
I wrap my methods in try/Catch and add the above in every catch{} I want to make this better. — StudentRik
– StudentRik, Commented Feb 25, 2013 at 10:04
Jeffery Ritcher suggests that no where in your code base is wise enough to add try catch. Rather allow all exceptions to bubble up. Collect them at the top (App start perhaps Main() ). This way youll know every corner issues in your app plus quality increases and performance also no problem much. Try/catch at every level is kinda noise. — Zenwalker
– Zenwalker, Commented Feb 25, 2013 at 10:07
@zenwalker The performance impact of a try{}catch{} is low as long as you don't actually throw exceptions. — CodesInChaos
– CodesInChaos, Commented Feb 25, 2013 at 10:09
The code works great for problems in the projects and sends the problem and what line/page. I just wondered if I could just use it in a method and put the method in the Catch{} and reference it some how in the project and just keep calling the method in the Catch{} — StudentRik
– StudentRik, Commented Feb 25, 2013 at 10:12

शेखर · Accepted Answer · 2013-02-25 10:13:57Z

1

on applicationError method you can do it.

Throw error in each catch block

private void ThrowingMethod()
{
    try
    {
        throw new InvalidOperationException("some exception");
    }
    catch
    {
        throw;
    }
    finally
    {

    }
}

and get it on the following event of Global.asax

void Application_Error(object sender, EventArgs e)
{

}

Here is a msdn link
http://www.asp.net/web-forms/tutorials/aspnet-45/getting-started-with-aspnet-45-web-forms/aspnet-error-handling

http://msdn.microsoft.com/en-us/library/24395wz3%28v=vs.100%29.aspx

edited Feb 25, 2013 at 10:13

answered Feb 25, 2013 at 10:02

शेखर

17.6k14 gold badges64 silver badges123 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Using System Exception

1 Answer 1

Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Related