views:

133

answers:

6

Hi guys, i'm writing a dll which is a wrapper to a access database. and i'm pretty new to c# in general as my background is in web development LAMP with perl, i'm not sure what's a good way to return error to a calling app in case they pass the wrong parameters to my functions or what not.

I have no idea as of now except to probably do some msgbox or throw some exceptions but i don't know where to start looking. Any help or resources would be more than useful :)

thanks~

+3  A: 

Wrong parameters are usually handled by throwing a ArgumentException or one of its subclasses.

Brian Rasmussen
Ill go for that one...
ojblass
A: 

throw new exception?

Paulo
+2  A: 

You want to throw an exception.

See

http://msdn.microsoft.com/en-us/library/ms229007.aspx

for the most common framework exceptions, such as ArgumentException and InvalidOperationException. See also

http://msdn.microsoft.com/en-us/library/ms229030.aspx

Brian
+7  A: 

You probably don't want to display message dialogs from within your dll, that's the job of the client application, as part of the presentation layer.

.Net library assemblies typically bubble up exceptions to the host application, so that's the approach I'd look at.

public static class LibraryClass
{
    public static void DoSomething(int positiveInteger)
    {
        if (positiveInteger < 0)
        {
            throw new ArgumentException("Expected a positive number", "positiveInteger");
        }
    }
}

Then it's up to your host application to handle those exceptions, logging and displaying them as appropriate.

try
{
    LibraryClass.DoSomething(-3);
}
catch(ArgumentException argExc)
{
    MessageBox.Show("An Error occurred: " + argExc.ToString());
}
Scott Ferguson
I hate dialog boxes tied to libraries.
ojblass
@Scott, thanks i'll follow this best practice then :)
melaos
+1  A: 

Check out Design Guidelines for Class Library Developers: Error Raising and Handling Guidelines

Aaron Fischer
That link is for 2003, it is a little out-of-date in that it advocates for (rather than against) ApplicationException, but the rest of the advice still seems timely.
Brian
A: 

Dlls generally should not create any kind of UI element to report an error. You can Throw (same meaning as raise) many different kinds of exceptions, or create your own and the calling code (client) can catch and report to the user.

public void MyDLLFunction()
{
    try
    {
     //some interesting code that may
     //cause an error here
    }
    catch (Exception ex)
    {
     // do some logging, handle the error etc.
     // if you can't handle the error then throw to
     // the calling code
     throw;
     //not throw ex; - that resets the call stack
    }
}
Gary.Ray