Be careful, you are heavingly doing copy of your objects. You should pass T& when you add the object and return T& when you retrieve them. I would also suggest to have a const version of your accessor, returning 'const T&'.