yep you're right, definitely should only have getters