You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Benedikt Ritter (JIRA)" <ji...@apache.org> on 2015/06/22 20:49:01 UTC
[jira] [Updated] (CSV-152) CSVRecord.isSet() does not check if
value is not null
[ https://issues.apache.org/jira/browse/CSV-152?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Benedikt Ritter updated CSV-152:
--------------------------------
Description:
Consider:
{code:java}
public boolean isSet(final String name)
{
return isMapped(name) && mapping.get(name).intValue() < values.length;
}
{code}
The docs say 'and has a value', however, it does not check if values[mapping.get(name)] is null.
Propose:
{code:java}
public boolean isSet(final String name)
{
return isMapped(name) && mapping.get(name) < values.length && values[mapping.get(name)] != null;
}
{code}
Perhaps a new method 'hasValue' if the above is not desirable?
was:
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?
> 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:
> {code:java}
> public boolean isSet(final String name)
> {
> return isMapped(name) && mapping.get(name).intValue() < values.length;
> }
> {code}
> The docs say 'and has a value', however, it does not check if values[mapping.get(name)] is null.
> Propose:
> {code:java}
> public boolean isSet(final String name)
> {
> return isMapped(name) && mapping.get(name) < values.length && values[mapping.get(name)] != null;
> }
> {code}
> Perhaps a new method 'hasValue' if the above is not desirable?
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)