Menu

A matter of style on your exceptions

2002-12-18
2002-12-20
  • Eduard Ralph

    Eduard Ralph - 2002-12-18

    Hi there again,

    this is really a small issue you might want to look at (or not). The way you use the "catch( Exception e)" leads to problems as soon as you have a Namespace called the same. As part of the Integration into our software I integrated your exceptions into our exception sheme (which is why we have a namespace called that way). If you try to compile the result you get a lot of errors, because the compiler believes that you are referencing the Namespace insted of the System.Exception. You might want to change that.
    Another thing is that the compiler shows some warnings because you define "catch(Exception e)" and the proceed to not use the e in the catch-clause. If you just remove the e to "catch(System.Exception) {}" the whole problem is gone.
    (It doesn't really matter all that much, it takes all of a few moments to change, but you might hit similar problems yourself and I've gotten paranoic about warnings over time)

    Greetings,

    Eduard Ralph

    P.S.: in that connection.... you hit a index if you click on "Home Page" in sourceforge.....

     
    • Ian Stallings

      Ian Stallings - 2002-12-20

      Those suggestions make sense so I'll implement them in the next build, wont be hard to change.

      I still gotta build a home page, heh. Soon.

       

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.