On 1 August 2013 15:21, Emmanuel Bourg <[email protected]> wrote:
> Just like the bean mapping I prefer to keep the type conversion out of 1.0.
>
+1
It's definitely not essential to the initial release.
I'm not sure it belongs in CSV at all.
> Emmanuel Bourg
>
>
> Le 01/08/2013 16:00, Gary Gregory a écrit :
>> I would like to note this CSVRecord addition I am planning on:
>>
>> public Boolean getBoolean(String name) {
>> public boolean getBooleanPrimitive(String name)
>>
>> The method listings are at the end of this message.
>>
>> What I want to note here is that these are conversion methods and that the
>> record still stores the values internally as Strings. I do not want to
>> Javadoc the conversion in order to give us flexibility over representation
>> if we decide to change it in the future (caching or whatnot).
>>
>> I wanted to post here in CTR mode before I or others add APIs like
>> getLong() and getLongPrimitive(). Since this is a library, I do believe we
>> should end up providing such APIs at the record level for primitives.
>>
>> /**
>> * Returns a value by name.
>> *
>> * @param name
>> * the name of the column to be retrieved.
>> * @return the column value, or {@code null} if the column name is not
>> found
>> * @throws IllegalStateException
>> * if no header mapping was provided
>> * @throws IllegalArgumentException
>> * if the record is inconsistent
>> * @see #isConsistent()
>> */
>> public Boolean getBoolean(String name) {
>> String s = this.get(name);
>> return s != null ? Boolean.valueOf(s) : null;
>> }
>>
>> /**
>> * Returns a value by name.
>> *
>> * @param name
>> * the name of the column to be retrieved.
>> * @return the column value, or {@code false} if the column name is not
>> found
>> * @throws IllegalStateException
>> * if no header mapping was provided
>> * @throws IllegalArgumentException
>> * if the record is inconsistent
>> * @see #isConsistent()
>> */
>> public boolean getBooleanPrimitive(String name) {
>> return Boolean.parseBoolean(this.get(name));
>> }
>>
>> Gary
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
