[API Review] Property and expression conversion methods
steve.x.northover at oracle.com
steve.x.northover at oracle.com
Tue Feb 19 14:30:58 PST 2013
I understand. Since they are all a kind of expression, perhaps
asObjectExpression() is the name we should use.
Another possibility is to create a new instance of the adapter class
explicitly (ie. new ReadOnlyObjectProperyAdapter(zzzz)) rather than
zzzzx.asXXX()) method. Not sure if this is better or worse.
Steve
On 19/02/2013 3:26 PM, Martin Sladecek wrote:
> As properties extend read only properties, which extend expressions,
> using asObject() everywhere would mean we have only one method, which
> return covariant return type in subclasses.
>
> Renaming these method will result in 3 method in property classes.
> E.g. DoubleProperty would have:
> ObjectExpression<Double> asObjectExpression();
> ObjectReadOnlyProperty<Double> asReadOnlyObjectProperty();
> ObjectProperty<Double> asObjectProperty();
>
> instead of just having
>
> ObjectProperty<Double> asObject();
>
> which overrides ObjectExpression<Double> asObject() from
> DoubleExpression.
>
> -Martin
>
> On 02/19/2013 06:44 PM, steve.x.northover at oracle.com wrote:
>> I agree. asObject seems a bit to general.
>>
>> Steve
>>
>> On 19/02/2013 12:23 PM, Richard Bair wrote:
>>>> public ObjectExpression<Double> asObject();
>>>> public ReadOnlyObjectProperty<Double> asObject();
>>>> ObjectProperty<Double> asObject();
>>>>
>>>> public static<T extends Number> DoubleExpression
>>>> doubleExpression(ObservableValue<T>)
>>>> public static<T extends Number> ReadOnlyDoubleProperty
>>>> readOnlyDoubleProperty(ReadOnlyProperty<T>);
>>>> public static<T extends Number> DoubleProperty
>>>> doubleProperty(Property<T>);
>>> I wonder if we ought not use a consistent naming here, such as
>>> asObjectProperty and asDoubleExpression? Also should it be asFoo or
>>> toFoo? Is there a precedent?
>>>
>>> Richard
>
More information about the openjfx-dev
mailing list