You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Michael Newcomb (JIRA)" <ji...@apache.org> on 2015/06/15 21:01:00 UTC

[jira] [Created] (CSV-152) CSVRecord.isSet() does not check if value is not null

Michael Newcomb created CSV-152:
-----------------------------------

             Summary: CSVRecord.isSet() does not check if value is not null
                 Key: CSV-152
                 URL: https://issues.apache.org/jira/browse/CSV-152
             Project: Commons CSV
          Issue Type: Bug
            Reporter: Michael Newcomb


Consider:
public boolean isSet(final String name)
{
  return isMapped(name) && mapping.get(name).intValue() < values.length;
}

The docs say 'and has a value', however, it does not check if values[mapping.get(name)] is null.

Propose:
public boolean isSet(final String name)
{
  return isMapped(name) && mapping.get(name) < values.length && values[mapping.get(name)] != null;
}

Perhaps a new method 'hasValue' if the above is not desirable?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)