> I just found the following code in ActionLoader.java:
>
>   /**
>       Attempts to load and execute the external action.
>
>       @exception ServletException if there is a servlet problem
>       @exception IOException if there is an output problem
>       @exception SQLException if there is a database problem
>       @exception DataSetException if there is a database problem
>   */
>   public void exec ( RunData data, String name ) throws Exception
>   {
>     // blah
>   }
>
> Is the intent to document that the Exception thrown could be
> one of ServletException, IOException, SQLException or DataSetException?
> If that is the case, shouldn't the throws clause explicitly list
> those four exceptions?

-1, This list is not exhaustive, nor can a complete list be known.

Otherwise, should all those @exception entries
> be replaced with one like this?
>
>       @exception Exception a generic exception
>

+1
the description could be something like: Any exception in the requested
action is passed up to be handled elsewhere.  Otherwise

 @exception Exception

The addition of "a generic exception" does not seem very informative.

John McNally




------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?:           [EMAIL PROTECTED]

Reply via email to