Skip to main content
3 de 4
se añadieron 593 caracteres en el cuerpo
eferion
  • 52.6k
  • 5
  • 35
  • 75

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.

eferion
  • 52.6k
  • 5
  • 35
  • 75