Justin Deoliveira ha scritto: > I came up with this "useless" class because I wanted to isolate the > logic of converting values in one place. Since there wasn't a ton of > feedback with it at the time I added I figured that people would > probably have issues with it at some point, so isolating the logic > seemed like a good idea. I guess I could have perhaps done it with > another method in a super class or something... Regardless I am fine > with getting rid of it if its not useful.
The clean solution would be Converter.convert(source, target) handle it directly, the drawback being that Class.isInstance is such a heavy hitter performance wise that we may be sorry later. As an alternative, we can have a Converter.checkAndConvert(source, target) method that does the isInstance check before calling convert. Bleah, ugly, I know, since it exposes a performance issue. Any ideas? Cheers Andrea ------------------------------------------------------------------------- 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
