Okay tried out your idea Gabriel.

There are actually two factory finders in play:
- BasicFactories - used by the geometry library implementations; never hooked 
up to gt-referencing (and should be used by gt-referencing when creating direct 
positions)
- CommonFactoryFinder

The superclass you mentioned, FactoryFinder, is actually abstract.

I did try a couple of things:
- moving to gt-metadata; failed as the code references ReferencingFactoryFinder 
(probably not that big a deal as we could just reverse the relationship)
- moving to gt-referencing; failed as CommonFactory finder references 
FeatureCollection  which is part of gt-api and not an gt-opengis interface

I am going to go away and think for a bit.
Jody

On 11/01/2011, at 3:05 PM, Jody Garnett wrote:

> A very good idea Gabriel,
> 
> I mentioned this on the other email thread but I have an alternative to 
> consider?
> 
> I would like to collect all the factory finder stuff into the GeoTools class; 
> resulting in more readable code. I think we can do this as the factory lookup 
> code does not depend on any specific implementation?
> 
> Cheers,
> Jody
> 
> On 11/01/2011, at 2:59 PM, Gabriel Roldán wrote:
> 
>> On Tue, 2011-01-11 at 13:30 +1000, Jody Garnett wrote:
>>> The namespace 2 discussion has been productive (introducing of gt-opengis, 
>>> creation of two proposals) but is now stuck...
>>> 
>>> The Query interface would like to work with PropertyName to capture a list 
>>> of xpath expressions to be obtained via the Query. To do a nice job of this 
>>> it would be good to have access to a FilterFactory (by way of 
>>> CommonFactoryFinder).
>>> 
>>> Why is this an issue?
>>> - Query is in gt-api
>>> - CommonFactoryFinder is in gt-main
>>> 
>>> Any suggestions?
>> not sure it'll work, but FactoryFinder is in gt-metadata, so Query could
>> just use that one provided the lookup method is refactored into
>> FactoryFinder:
>> 
>> public class FactoryFinder{
>> 
>> public static Object lookup(Class category, Hints hints, Hints.Key key)
>> {
>>       hints = mergeSystemHints(hints);
>> ....
>> }
>> 
>> 
>>> 
>>> Jody
>>> ------------------------------------------------------------------------------
>>> Gaining the trust of online customers is vital for the success of any 
>>> company
>>> that requires sensitive data to be transmitted over the Web.   Learn how to 
>>> best implement a security strategy that keeps consumers' information secure 
>>> and instills the confidence they need to proceed with transactions.
>>> http://p.sf.net/sfu/oracle-sfdevnl 
>>> _______________________________________________
>>> Geotools-devel mailing list
>>> [email protected]
>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>> 
>> -- 
>> Gabriel Roldan
>> [email protected]
>> Expert service straight from the developers
>> 
> 


------------------------------------------------------------------------------
Gaining the trust of online customers is vital for the success of any company
that requires sensitive data to be transmitted over the Web.   Learn how to 
best implement a security strategy that keeps consumers' information secure 
and instills the confidence they need to proceed with transactions.
http://p.sf.net/sfu/oracle-sfdevnl 
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to