Yeah in reality these should be based on something like the bounding box of the model.

The only problem there is what do when you switch models? Should the camera snap to a location outside of the model? If you don't do that and you have a huge model then go to a small one your movement will be so slow that it'll take forever to get close up.

Yeah they shouldn't be member variables.