[udig-devel] Bleck; sorry that was more of a rant

Jody Garnett jgarnett at refractions.net
Tue Nov 13 14:16:11 PST 2007


I am afraid that last email was more of a rant; I am more upset that I 
failed to communicate one of the more interesting parts of the uDig design.

The ShapeMover code was brave and one of the only examples to even use 
the IResolveAdapterFactory facilities. I will endevour to improve the 
documentation and code base so it is easier to use the future.

Jody
> This class is both a factory( IResolveAdapterFactory) and a 
> destination (literally the ShapeMover class creates itself). This is 
> really really confusing when first looked at!
> - ShapeMover() no argument constructor is as a IResolveFactory in 
> order to check canAdapt
> - If canAdapt returns true the adapt method will end up calling new 
> ShapeMover( IResolve )
>
> While that is a little bit of fun; the kicker for me is the check in 
> canAdapt:
>> return adapter.isAssignableFrom(ShapeMover.class);
> Can I be reading this right? We can adapt if we want to? That is all 
> the time; we never check the provided handle...
>
> Please tell me this code is not used...
> Jody
>
>
> _______________________________________________
> User-friendly Desktop Internet GIS (uDig)
> http://udig.refractions.net
> http://lists.refractions.net/mailman/listinfo/udig-devel



More information about the udig-devel mailing list