I am working on that now (the official build is Java 1.4 based) - I can
commit code that compiles (but I have test failures).
For interest I found that the unsupported/geometry implementation is
built around some flaws ....
FeatGeomFactoryImpl - is a "container" that holds the following
- CoordianteReferenceSystem
- PrecisionModel - *new for geoapi* (implementation already existed)
- CoordinateFactoryImpl
- PrimitiveFactoryImpl
- PositionFactoryImpl - *new for geoapi* isolate position creation code
in one spot
- ComplexFactoryImpl
- CollectionFactory - own invention to isolate create list code
- dimensionModel - 2, 2.5, or 3 dimension (used as a strategy object by
some calculations)
It also looks like three singletons are available and used:
- singleton2D,singleton205D, singleton3D
I have two problems with this:
- dimension model should not be an alternative to checking the
coordinate reference system (should it?)
- all instances contain back pointers to FeatGeomFactoryImpl - and often
perform casts to get there (not safe)
- singletons do not mix with the idea of changing your coordinate
reference system
So right now this implementation appears to not support coordinate
reference systems (I hope I am reading the code wrong...debugger is
warmed up now).
Cheers,
Jody
> Hey all,
>
> No problems with the geoapi Precision update from my end.
>
> However I've got broken trunk in unsupported/geometry. Tests don't match the
> implementation:
>
> in WKTReader.java there's one constructor defined (with three args):
>
> public WKTReader(PrimitiveFactory aPrimitiveFactory,
> GeometryFactory aCoordinateFactory, PositionFactory
> aPositionFactory ) {
>
>
>
> Tests use a different constructor in a number of places:
>
> /home/ENV/sfarber/dev/geotools/geotools-trunk/modules/unsupported/geometry/src/test/java/org/geotools/geometry/iso/operations/SetOperationsTest.java:[209,38]
> cannot find symbol
> symbol : constructor
> WKTReader(org.geotools.geometry.iso.primitive.PrimitiveFactoryImpl,org.geotools.geometry.iso.coordinate.CoordinateFactoryImpl)
> location: class org.geotools.geometry.iso.io.wkt.WKTReader
>
>
>
> Ideas?
>
> --saul
>
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share your
> opinions on IT & business topics through brief surveys-and earn cash
> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> _______________________________________________
> Geotools-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
