> - getters should all be const, unless there's a strong reason against > that. Perhaps it's worth mentioning that if it's not const because you cache the results in a member variable, then consider making it const anyway and make the caching class member mutable.