The interface was just an idea I threw in there, because there were
concerns about ReferencedEnvelope3D not being a ReferencedEnvelope.
But the idea of not making separate classes and make the current ones
support 3D, I think is quite tricky too.
The algorithms aren't necessarily the same with 2d and 3d. Basically,
every method would need to have a check whether there is a 3rd dimension
or not.
On 07/17/2012 03:54 AM, Jody Garnett wrote:
So I kind of see where you are going with this. It gets a bit strange
around the ReferencedEnvelope / ReferencedEnvelope3D level.
-1
The FeatureCollections.getBounds() contract cannot be respected for
your ReferenceEnvelope or ReferenceEnvelope3D. I know you only want to
use this as part of a query, but I don't want our envelope story to
get even more strange if we can avoid it.
I don't want to impose a ReferenceEnvelope interface as that defeats
the purpose of making ReferenceEnvelope an easy to use middle ground
between JTS Envelope and the BoundingBox interface. If we had to go
this way we would end up going with BoundingBox / Envelope interface
and modifying them to be more method compatible with JTS Envelope.
Can you consider switching around your class hierarchy?
1) BBoxEnvelope extension of JTS Envelope
- Write this so it can work as a 2D or 3D envelope (depending on how
the constructor is called)
- the existing BoundingBox / Envelope interfaces should be sufficient?
2) Make ReferencedEnvelope extend BBoxEnvelope; and allow it to cover
the 2D or 3D use-case
Finally there is no need to make FilterFactory more crazy, treat your
ReferencedEnvelope as a prameter object:
BBOX3D bbox( Expression geometry, ReferencedEnvelope bbox,
MatchAction matchAction);
Waiting your reply, I am afraid we cannot make a ReferencedEnvelope
interface (as an implementation class lots of downstream code makes
direct reference to this one).
--
Jody Garnett
Forwarded message:
*From:* Niels Charlier <ni...@scitus.be>
*To:* geotools-devel@lists.sourceforge.net
*Date:* Tuesday, 17 July 2012 6:09:35 AM
*Subject:* [Geotools-devel] *proposal*: Support for three-dimensional
envelopes and bounding box filters
Hi Everyone,
Please have a look at my proposal:
http://docs.codehaus.org/display/GEOTOOLS/Support+for+three-dimensional+envelopes+and+bounding+box+filters
Please review.
Kind Regards
Niels Charlier
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond.
Discussions
will include endpoint security, mobile security and the latest in
malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
<mailto:GeoTools-Devel@lists.sourceforge.net>
https://lists.sourceforge.net/lists/listinfo/geotools-devel
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel