So I’m writing a library and I want to define a constant that will have the value of pi. I was thinking of defining a static const float pi = 3.14; in a pi.h file.
However, I’m almost sure that’s not the way to do it because I’ve read that you shouldn’t define variables in header files.
Another way I thought was to declare an inline function that returns the value of pi but that’s awkward to work with.
Another way (I think) is to put it in pi.cc compile it into the library and then use extern static const float pi; in whatever file you are using pi with and of course link those files with the library.
What’s the best way of doing this? The standard library would probably define a macro but I think a constant is better.
One reason to not put a constant in a header file is to avoid recompiling the code using the library (rather than just re-linking) if the constant changes. This reason doesn’t apply to constants such as π that never change as a program is maintained.
There is an additional issue if the constant is a floating point value. C++ requires that floating point constants have storage assigned to them and that they be initialized in a .cpp file, not a .h file. This can makes them the constants less efficient than the equivalent inlined functions. This is why the Boost library uses inline functions. See this answer.
So for your case, the best bet is probably to use the Boost library, or if you don’t want to take the dependency, to just make an inline function that does the same thing.