Generating custom exception in c#

Question

I have tried this code to raise a manual exception

 protected void test ()
    try
    {
        throw new Exception("HI");     //line22
    }
    catch (Exception ex) { lblerror.Text = ex.ToString(); }

but received exception below

System.ArgumentException: HI at Project_Test_M_Test.btnsubmit_Click(Object sender, EventArgs e) in D:\Project\Test\M_Test.aspx.cs:line 22

I want to see error message that I have send not this.

Why are you using exceptions in this way, Exceptions are for unexpected, unrecoverable errors. They should not be used to control program flow in a way that showing ex.Message to the user would be useful. Please read the MSDN guidelines on throwing exceptions — Scott Chamberlain
– Scott Chamberlain, Commented Oct 17, 2015 at 7:49
Also, that is not a custom exception, here is a link to the MSDN on how to create a actual custom exception. — Scott Chamberlain
– Scott Chamberlain, Commented Oct 17, 2015 at 7:52
Thanks for suggestion but I am a student and personally think It's a better way to learn generating exception and their handling. But I will remember your suggestions and will be back if stuck anywhere else. Thanks once again. — Vijay809
– Vijay809, Commented Oct 19, 2015 at 5:20

Kapoor · Accepted Answer · 2015-10-17 06:41:00Z

2

Please use ex.Message instead of ex.ToString(). btw, its not a good idea to throw the base class Exception. please use a more specific one.

answered Oct 17, 2015 at 6:41

Kapoor

1,42813 silver badges22 bronze badges

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

4 Comments

Dieter B Over a year ago

And possibly even the InnerException if you want more information for debugging

Scott Chamberlain Over a year ago

@DieterB ex.ToString() contains the details of InnerException. @Kapor Teaching people to use ex.Message is a bad habit to get people to start doing, you loose a lot of debugging info

Kapoor Over a year ago

@ScottChamberlain, I suspect there is some misunderstanding, my point is that in the string passed in the constructor ("HI" in sample code) is available in readonly property ex.Message. And that is what was asked in this question - "I want to see error message that I have send not this.".

Kapoor Over a year ago

@ScottChamberlain, As far as usage of Message is concerened, As per my understanding - It should contain helpful text indicating why the exception was thrown. Ideally end users should not see this but it should be logged. Therefor it can be technical so that the developers viewing the logs can use the information in the message to fix the code when producing a new version. I'm sorry, but I did not get your point. Do you mean that I should have also told the usage of message property ? (Which I have written in this comment)?

yue shi · Accepted Answer · 2015-10-17 07:48:53Z

1

This is what you need to do, use Message property to access the error message.

 protected void test ()
 {
     try
     {
         throw new Exception("HI");  // Exception message passed from constructor
     }
     catch (Exception ex) 
     { 
         lblerror.Text = ex.Message;
     }
}

edited Oct 17, 2015 at 7:48

yue shi

925 bronze badges

answered Oct 17, 2015 at 6:41

Don

6,9103 gold badges29 silver badges35 bronze badges

Collectives™ on Stack Overflow

Generating custom exception in c#

2 Answers 2

4 Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

Comments

Related