I have a class which has many small functions. By small functions, I mean functions that doesn’t do any processing but just return a literal value. Something like:
string Foo::method() const{ return 'A'; } I have created a header file ‘Foo.h’ and source file ‘Foo.cpp’. But since the function is very small, I am thinking about putting it in the header file itself. I have the following questions:
- Is there any performance or other issues if I put these function definition in header file? I will have many functions like this.
- My understanding is when the compilation is done, compiler will expand the header file and place it where it is included. Is that correct?
If the function is small (the chance you would change it often is low), and if the function can be put into the header without including myriads of other headers (because your function depends on them), it is perfectly valid to do so. If you declare them extern inline, then the compiler is required to give it the same address for every compilation unit:
headera.h:
Member functions are implicit inline provided they are defined inside their class. The same stuff is true for them true: If they can be put into the header without hassle, you can indeed do so.
Because the code of the function is put into the header and visible, the compiler is able to inline calls to them, that is, putting code of the function directly at the call site (not so much because you put inline before it, but more because the compiler decides that way, though. Putting inline only is a hint to the compiler regarding that). That can result in a performance improvement, because the compiler now sees where arguments match variables local to the function, and where argument doesn’t alias each other – and last but not least, function frame allocation isn’t needed anymore.
Yes, that is correct. The function will be defined in every place where you include its header. The compiler will care about putting only one instance of it into the resulting program, by eliminating the others.