commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sebb (JIRA)" <>
Subject [jira] [Commented] (CSV-113) Check whether ISE/IAE are being used appropriately
Date Tue, 06 May 2014 00:54:15 GMT


Sebb commented on CSV-113:

CSVFormat.validate() should throw IAE not ISE
CSVParser.initializeHeader() throws ISE. The header probably ought to be validated in the
CSVFormat constructor.

Should null parameters throw IAE or NPE?
Needs discussion.

> Check whether ISE/IAE are being used appropriately
> --------------------------------------------------
>                 Key: CSV-113
>                 URL:
>             Project: Commons CSV
>          Issue Type: Task
>            Reporter: Sebb
> The code throws a lot of IllegalStateExceptions (ISE).
> It also throws some IllegalArgumentExceptions (IAE).
> These need reviewing to check that ISE really does apply to the current state, and IAE
is used for reporting an incorrect argument.
> Generally if ISE applies, no argument values will be valid.
> IAE should be used when a specific argument or set of arguments is invalid.

This message was sent by Atlassian JIRA

View raw message