Question about the PlayerData class in monkeyzone

I’m learning to use spidermonkey, and I’ve been looking at the code for monkey zone as a reference. I noticed that to store the player data you use a HashMap object (eg HashMap and then use synchronized class methods to make changes to the HashMap.

Wouldn’t it be just as effective (or atleast easier to read/deal with) to use a ConcurrentHashMap object instead, and not have synchronized class methods?

Is there a reason monkey zone uses HashMaps over ConcurrentHashMaps?

Theres no reason for that, no.

The only reason would be if other changes (outside the hashmap) were also being made or might potentially be made in the future…

thats actually a pretty good point. I might follow the same kind of pattern. Thanks.