Ideally I want one class per file or, when I have non-member functions, one function per file. The reasons are:
- When I read the code I will always know in what file I should find a certain function or class.
- If it's one class or one non-member function per header file, then I won't include a whole mess when I
includea header file. - If I make a small change in a function then only that function will have to be recompiled.
However, splitting everything into many header and implementation files can slow down compilation. Most functions access a number of templated other library functions so that code will be compiled over and over, once for each implementation file. Compiling my whole project takes 45 minutes. There are about 50 object files; each uses the same expensive-to-compile headers.
Is it acceptable to have one class (or non-member function) per header file, but putting the implementations of many or all of these functions into one implementation file, like in the following example?
// foo.h
void foo(int n);
// bar.h
void bar(double d);
// foobar.cpp
#include <vector>
void foo(int n) { std::vector<int> v; ... }
void bar(double d) { std::vector<int> w; ... }
Again, the advantage would be that I can include just the foo function or just the bar function, and compilation of the project will be faster because foobar.cpp is one file, so the std::vector<int> (just an example for some other expensive-to-compile templated construction) has to be compiled only once, as opposed to twice if I compiled a foo.cpp and bar.cpp separately. Of course my reason (3) is not valid for this scenario: After just changing foo(){...} I have to recompile the whole file foobar.cpp.
I'm curious what your opinions are!