Skip to main content

[JSR-354] Re: Spec Updated

  • From: Stephen Colebourne <scolebourne@...>
  • To: jcurrency_mail@...
  • Subject: [JSR-354] Re: Spec Updated
  • Date: Sun, 13 Oct 2013 23:02:57 +0100

Adjuster and Query are a very open variation of the strategy design
pattern. They allow behaviour to be encapsulated outside of the main
money class. They are named and behave exactly as per JSR-310, which
users of the API will appreciate. They also fit well into a
functional/lambda world.

Stephen



On 13 October 2013 18:13, Werner Keil <werner.keil@...> wrote:
>
> So far MonetaryQuery.queryFrom() is used 3 times in the RI.
>
>
> The JavaDoc
>  /** * Queries this monetary amount for a value. * <p> * This queries this 
> amount using the specified query strategy object. * <p> * Implementations 
> must ensure that no observable state is altered when this * read-only 
> method is invoked. * * @param <R> * the type of the result * @param 
> adjuster * the query to invoke, not null * @return the query result, null 
> may be returned (defined by the query) */
>
> is a bit buggy to start with. "adjuster" should be called query, and the 
> whole notion of query(Query) seems a little odd.
> The text should be clarified, "value" is meaningless, especially since you 
> need to pass a query and all that query does so far is to route to its 
> internal (less conveniently named) proxy method. I think we should try to 
> untangle that, it feels a bit redundant.
>
> Werner
>
> On Sun, Oct 13, 2013 at 6:53 PM, Werner Keil <werner.keil@...> wrote:
>>
>> Anatole/all,
>>
>> I changed a few links to the Git repositories already as they were 
>> outdated.
>>
>> Just rephrased a method to match the actual code, but the JavaDoc in 
>> MonetaryQuery leaves more questions than we would like to see unanswered.
>>
>>
>>  * It is recommended to use the second approach, {@code 
>> query(MonetaryQuery)},
>>  * as it is a lot clearer to read in code.
>>
>>
>> If it is recommended to use a totally different method, then  can't we 
>> just call it query() and use that interface, or let's drop MonetaryQuery 
>> from the spec to avoid lose ends?
>>
>> These "we specify A,  but you should use B because A has a stupid name" 
>> makes no sense and confuses people.
>>
>> There's already enough ambiguity e.g. in the current JSPA or Process 
>> document, some of us in the EC hope to resolve, so couldn't we try to keep 
>> that out of specs please
>>
>> Cheers,
>> Werner
>>
>>
>>
>> On Sun, Oct 13, 2013 at 10:32 AM, Simon Martinelli <simon.martinelli@...> 
>> wrote:
>>>
>>> Hi Anatole,
>>>
>>> I was 3 weeks in vacation but now I am back and read the spec.
>>> Unfortunately I am not able to add comments. Can you please give me the 
>>> necessary authorization?
>>>
>>> Thanks,
>>> Simon
>>>
>>>
>>> On Sat, Oct 12, 2013 at 9:08 PM, Anatole Tresch <atsticks@...> wrote:
>>>>
>>>> Dear all
>>>>
>>>> I have updated the spec and accommodated the latest input, mainly from 
>>>> Stephen. I think we have a small, comprehensive and very elegant spec, 
>>>> so thanks to all (especially Stephen) for your time and contributions.
>>>> I have left some comments, so you may see easily, where there might be 
>>>> points to be discussed.
>>>>
>>>> So please revise the spec once more, and add your comments directly into 
>>>> the document (as Google Doc comment). If you feel OK, with the current 
>>>> state to progress, please add a comment at the end of the document, so I 
>>>> know we have now an agreement on the current document and I see who 
>>>> finally has read it!
>>>>
>>>> Here is the link:
>>>> https://docs.google.com/document/d/1FfihURoCYrbkDcSf1WXM6fHoU3d3tKooMnZLCpmpyV8/edit#heading=h.5wnzm49wl111
>>>>
>>>> I would be glad, if we can do that within the next 2-3 days, so we can 
>>>> then finally publish our Public Review. I will check now, if the API 
>>>> JavaDoc requirements are aligned with the current spec.
>>>>
>>>> Have a nice weekend!
>>>> Anatole
>>>>
>>>> --
>>>> Anatole Tresch
>>>> Java Lead Engineer, JSR Spec Lead
>>>> Gl√§rnischweg 10
>>>> CH - 8620 Wetzikon
>>>>
>>>> Switzerland, Europe Zurich, GMT+1
>>>> Twitter:  @atsticks
>>>> Blogs: http://javaremarkables.blogspot.ch/
>>>> Google: atsticks
>>>> Mobile  +41-76 344 62 79
>>>
>>>
>>
>


[JSR-354] Spec Updated

Anatole Tresch 10/12/2013

[JSR-354] Re: Spec Updated

Simon Martinelli 10/13/2013

[JSR-354] Re: Spec Updated

Werner Keil 10/13/2013

[JSR-354] Re: Spec Updated

Werner Keil 10/13/2013

[JSR-354] Re: Spec Updated

Anatole Tresch 10/13/2013

[JSR-354] Re: Spec Updated

Stephen Colebourne 10/13/2013

[JSR-354] Re: Spec Updated

Werner Keil 10/13/2013

[JSR-354] Re: Spec Updated

Anatole Tresch 10/14/2013

[JSR-354] Re: Spec Updated

Werner Keil 10/14/2013

[JSR-354] Re: Spec Updated

Anatole Tresch 10/14/2013

[JSR-354] Re: Spec Updated

Werner Keil 10/14/2013

[JSR-354] Re: Spec Updated

Stephen Colebourne 10/14/2013

[JSR-354] Re: Spec Updated

Werner Keil 10/14/2013

[JSR-354] Re: Spec Updated

Anatole Tresch 10/15/2013

[JSR-354] Re: Spec Updated

Werner Keil 10/15/2013

[JSR-354] Re: Spec Updated

Anatole Tresch 10/15/2013
 
 
Close
loading
Please Confirm
Close