It's unnecessary declare as const, because it calls only in ctor. And then max and min have excess 'mutable'