You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Thomas Neidhart (Commented) (JIRA)" <ji...@apache.org> on 2012/01/23 23:31:40 UTC

[jira] [Commented] (MATH-575) Exceptions in genetics package or not consistent with the rest of [math]

    [ https://issues.apache.org/jira/browse/MATH-575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13191560#comment-13191560 ] 

Thomas Neidhart commented on MATH-575:
--------------------------------------

Phil started to work on this issue in r1135025.

In r1235038 additional cleanups have been performed:

 - add localized messages for all exceptions
 - add @throws to javadoc where appropriate
 - add final to method parameters

What is missing:

 - Phil mentioned that InvalidRepresentationException should be placed into [math], although I am not sure why, as it is not used outside the genetics package
 - add more custom exception classes specific to the genetics package (optional). By now mostly MathIllegalArgumentException or other appropriate ones have been used.
                
> Exceptions in genetics package or not consistent with the rest of [math]
> ------------------------------------------------------------------------
>
>                 Key: MATH-575
>                 URL: https://issues.apache.org/jira/browse/MATH-575
>             Project: Commons Math
>          Issue Type: Bug
>    Affects Versions: 2.0, 2.1, 2.2
>            Reporter: Phil Steitz
>            Priority: Minor
>             Fix For: 3.0
>
>
> InvalidRepresentationException is checked and non-localized.  This exception should be placed in the [math] hierarchy.  The AbstractListChromosome constructor also throws a non-localised IAE, which should be replaced by an appropriate [math] exception.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira