Si es una opción yo plantearía sustituir la unión anónima por boost::variant
, disponible desde tiempos inmemoriales... anteriores a la versión 1.35:
#include <iostream>
#include <string>
#include <boost/variant.hpp>
int main()
{
boost::variant<int,std::string> test;
test = 5;
std::cout << test << '\n';
test = "abcdef";
std::cout << test << '\n';
}
Este cambio sería casi equivalente a darle un nombre a la unión... la ventaja es que esta clase evita lecturas erróneas:
int main()
{
boost::variant<int,std::string> test;
test = 5;
std::cout << boost::get<int>(test); // ok
std::cout << boost::get<std::string>(test); // error en tiempo de ejecucion
}
Además también permite saber qué tipo es el que tiene actualmente valor:
int main()
{
auto lambda = [](boost::variant<int,std::string> const& var)
{
if( var.type() == typeid(int) )
{
int valor = boost::get<int>(var);
std::cout << valor << '\n';
}
else if( var.type() == typeid(std::string) )
{
std::string valor = boost::get<std::string>(var);
std::cout << valor << '\n';
}
};
boost::variant<int,std::string> test;
test = 5;
lambda(test);
test = "abcdef";
lambda(test);
}
Otra ventaja de usar esta estructura es que, posteriormente, el código será facilmente portable a C++17 cambiando el espacio de nombres boost
por std
EDITO
Otra posibilidad sería darle un nombre a la estructura... de esta forma habilitas los constructores por defecto:
class MyClass {
private:
union {
::std::string string_;
int int_;
float float_;
} impl_;
public:
MyClass() = default;
MyClass( const MyClass &o ) noexcept = default;
~MyClass() = default;
};
La parte mala, por decir algo, es que el acceso a las variables miembro ahora debe pasar por impl_
... pero tampoco creo que el cambio sea traumático.