Nothing wrong with HashMap - not a weakness - its behaviour is implicit in the hashing algorithm. Its simply using the wrong tool for a job :-)
The problem with observation is you see what you want to see, Grasshopper :-) Modern programmers learn about frameworks - old fashioned ones with Zimmer frames like me learned algorithms and predicate logic. Each approach has advantages and weaknesses. All HashMap usage should definitely be reviewed. I fixed a few when I found them in code I was working with a few years , but this was probably on the ComplexFeatures branch. Rob On Tue, Jun 15, 2010 at 3:53 PM, Ben Caradoc-Davies <ben.caradoc-dav...@csiro.au> wrote: > On 15/06/10 13:25, Jody Garnett wrote: >> Can we arrange for maven to yell at us when HashMap is used? > > I do not think we should do this ,as there are legitimate occasions > where HashMap may be used for performance, particularly when we have > many insertions, one iteration, and iteration order is not exposed (e.g. > the contents are re-sorted in another container). I think being too > dogmatic, and in particular, allowing a simple rule-of-thumb enforced in > maven to override the judgement of an experienced developer might not be > the best approach. > >> Ben I have been known to use a couple of the other Map implementations; such >> as ConcurrentHashMap (and we also have our own WeakHashMap) - we may need to >> check if these implementations suffer the same weaknesses as HashMap. > > That is a good point. > > -- > Ben Caradoc-Davies <ben.caradoc-dav...@csiro.au> > Software Engineering Team Leader > CSIRO Earth Science and Resource Engineering > Australian Resources Research Centre > > ------------------------------------------------------------------------------ > ThinkGeek and WIRED's GeekDad team up for the Ultimate > GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the > lucky parental unit. See the prize list and enter to win: > http://p.sf.net/sfu/thinkgeek-promo > _______________________________________________ > Geotools-devel mailing list > Geotools-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/geotools-devel > ------------------------------------------------------------------------------ ThinkGeek and WIRED's GeekDad team up for the Ultimate GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the lucky parental unit. See the prize list and enter to win: http://p.sf.net/sfu/thinkgeek-promo _______________________________________________ Geotools-devel mailing list Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
