Is it safe (and correct) in a C++ header file to use the using declaration within a namespace as follows:
#include <boost/numeric/ublas/vector.hpp>
namespace MyNamespace {
using boost::numeric::ublas::vector;
vector MyFunc(vector in);
}
I.e. is the "using boost::numeric::ublas::vector" properly contained within the MyNamespace block, or will this pollute the namespace of any file that includes this header?
Best Answer
No, it is not safe - it won't pollute another namespace, but it is dangerous for other reasons:
A
using
directive will import anything that is currently visible by the name you specify into the namespace where you use it. While yourusing
will only be visible to users ofMyNamespace
, other things from "outside" will be visible to yourusing
declaration.So how is this dangerous when used in a header? Because it will import things that are visible at the point of the declaration, the exact behavior will depend on the order of headers you include before the declaration (There might be different things visible from
boost::numeric::ublas::vector
). Since you cannot really control which headers are included before your header (nor should you be! headers should be self-sufficient!), this can lead to very strange problems where your function will find one thing in one compilation unit, and another in the next.As a rule of thumb,
using
declarations should only be used after all includes in a .cpp file. There's also an item on this exact issue in the book "C++ Coding Standards" by Sutter and Alexandrescu (Item 59). Here's a quote:Even when it's unlikely that the name you are
using
doesn't exist anywhere else (as is probably the case here), things can get ugly: In a header, all declarations should be fully qualified. This is pain, but otherwise, strange things can happen.Also see Migrating to Namespaces, Using-declarations and namespace aliases and Namespace Naming for examples and the problem described in-depth.