if the constructor doesn't optimize this out then I'm all for putting it in the header, but otherwise I don't think I'll get behind it.

It's too easy to get to many many member variables and it bloats the header unnecessarily. The header should be quick and nice to look at, and a constructor initializing 15 member variables makes it gross.