My take on it: a private member with public getters and setters that do nothing other than get and set = a waste of time and effort, especially if you are the only one to use the code.Back on topic - we can not really diagnose without seeing the declaration for itemMap.I think get and set methods try and fail to address a real problem.