manera correcta para inicializar un objeto con excepción constructor tirar

Question

Ésta parece ser una cuestión trivial, pero me quedó colgado en él durante unas cuantas horas (quizás demasiado Java mató a mi C++ braincells).

He creado una clase que tiene el siguiente constructor (es decir, ningún constructor por defecto)

VACaptureSource::VACaptureSource(std::string inputType, std::string inputLocation) { 
    if(type == "" || location == "") { 
    throw std::invalid_argument("Empty type or location in VACaptureSource()"); 
} 
type = inputType; 
location = inputLocation; 

// Open the given media source using the appropriate OpenCV function. 
if(type.compare("image")) { 
    frame = cvLoadImage(location.c_str()); 
    if(!frame) { 
     throw std::runtime_error("error opening file"); 
    } 
} 
else { 
    throw std::invalid_argument("Unknown input type in VACaptureSource()"); 
} 

}

Cuando quiero crear una instancia, utilizo

// Create input data object 
try { 
    VACaptureSource input = VACaptureSource("image", "/home/cuneyt/workspace/testmedia/face_images/jhumpa_1.jpg"); 
} 
catch(invalid_argument& ia) { 
    cerr << "FD Error: " << ia.what() << endl; 
    usage(argv[0]); 
} 
catch(runtime_error& re) { 
    cerr << "FD Error: " << re.what() << endl; 
    usage(argv[0]); 
} 

Sin embargo , en este caso, la instancia es local para este bloque y no puedo referirme a él en ningún otro lado. Por otra parte, no puedo decir

VACAptureSource input; 

al inicio del programa ya que no hay constructor por defecto.

Cuál es la forma correcta de hacer esto?

Gracias!

Answer 1

¿Qué pasa con el uso de un puntero (o alguna versión RAII del mismo)?

VACaptureSource* input = NULL; 

try { 
    input = new VACaptureSource(...); 
} catch(...) { 
    //error handling 
} 

//And, of course, at the end of the program 
delete input; 

Answer 2

Una variable local se asigna al bloque en el que está asignada (como Java) pero se destruirá tan pronto como finalice el bloque (a diferencia de Java) por lo que debe hacer todo lo que desee en el bloque try (que puede no ser deseable si solo desea manejar excepciones de constructor) o debe asignar el objeto a otro lugar (por ejemplo, montón) y usar un puntero en el bloque principal para acceder a él.

Answer 3

Es posible utilizar un puntero

VACaptureSource* input; 
// Create input data object 
try { 
    input = new VACaptureSource("image", "/home/cuneyt/workspace/testmedia/face_images/jhumpa_1.jpg"); 
} 
catch(invalid_argument& ia) { 
    cerr << "FD Error: " << ia.what() << endl; 
    usage(argv[0]); 
} 
catch(runtime_error& re) { 
    cerr << "FD Error: " << re.what() << endl; 
    usage(argv[0]); 
} 

y que necesita para liberar el objeto cuando termine de usarlo

delete input 

Answer 4

¿por qué necesita hacer referencia a ella fuera de la try ¿bloquear?

En lugar de

try { 
    VACaptureSource input = VACaptureSource("image", "/home/cuneyt/workspace/testmedia/face_images/jhumpa_1.jpg"); 
} 
//catch.... 

//do stuff with input 

se podía mover todo en el bloque try:

try { 
    VACaptureSource input = VACaptureSource("image", "/home/cuneyt/workspace/testmedia/face_images/jhumpa_1.jpg"); 
    //do stuff with input 
} 
//catch.... 

o se puede factorizar a cabo en una función separada, que se llama desde el bloque try:

void doStuff(VACaptureSource& input){ 
    //do stuff with input 
} 

try { 
    VACaptureSource input = VACaptureSource("image", "/home/cuneyt/workspace/testmedia/face_images/jhumpa_1.jpg"); 
    doStuff(input); 
} 
//catch.... 

el último de ellos incluso le da la buena ventaja de separar la construcción de uso, lo cual se ubica muy bien en pruebas unitarias en las que es posible que desee que la función funcione en un objeto simulado en su lugar.

Answer 5

¿Qué pasa con la adición de un constructor por defecto que deja el objeto en un estado no configurado especial? Luego tenga una función create() para realmente crearlo.

entonces usted puede hacer:

VACaptureSource input; 
try 
{ 
    input.create("image", "..."); 
} 
catch(...) 
{ 
    ... 
} 

Dependiendo de la situación esto podría ser mejor que jugar con los punteros. A pesar de que entonces también tiene que comprobar si el create() se llama en realidad antes de hacer algo ...

Answer 6

Yo en realidad no veo ningún probelms aquí:

Un par de cosas que actualizaría:

• Captura excepciones por referencia de referencia.
• El compilador puede optimizar la construcción de copia en su código
  Pero se ve mejor sin él. Simplemente declare la entrada con sus parámetros.
• Refactorizaría el constructor para tomar los parámetros de referencia const
  Y los inicializaría en la lista de inicializadores.
• También me aseguraría de que el miembro 'frame' sea realmente un puntero inteligente.

Así que haría esto (para mayor claridad).

VACaptureSource::VACaptureSource(std::string const& inputType, 
            std::string const& inputLocation) 
     :type(inputType) 
     ,location(inputLocation) 
{ 
    // Other Code that throws. 
} 
void playWithCode() 
{ 
    // Get input information from user. 
    VACaptureSource input("image", "/home/cuneyt/workspace/testmedia/face_images/jhumpa_1.jpg"); 

    // use the input object. 
    // Do not need to play with pointers here. 
} 
int main() 
{ 
    try 
    { 
     playWithCode(); 
    } 
    catch(invalid_argument const& ia) 
    { cerr << "FD Error: " << ia.what() << endl; 
     usage(argv[0]); 
    } 
    catch(runtime_error const& re) 
    { cerr << "FD Error: " << re.what() << endl; 
     usage(argv[0]); 
    } 
} 

Answer 7

¿Puedo observar que casi cualquier pero el constructor más trivial se puede esperar que una excepción. Por lo tanto, no debe considerar las excepciones como "especiales" en algún sentido, sino que debe escribir su código para que se ocupe de ellas de forma natural. Esto significa usar RAII y las otras técnicas que otras respuestas aquí han sugerido.

Answer 8

Simple. No arrojes excepciones dentro de un constructor. No solo tiene que envolver el constructor en el bloque try, no podrá manejar la memoria muy bien en el caso de que detecte una excepción (¿llama al destructor? ¿Cuánta memoria de la clase necesita eliminarse?)

UPDATE0: Aunque no estoy seguro de si la administración de la memoria es un problema si está utilizando una instancia.

Update1: Hmmm, tal vez estoy pensando acerca de las excepciones en destructores.

int 
main2(int argc, char* argv[]) 
{ 
    MyClass class; 
    class.doSomething(); 
} 

int 
main(int argc, char* argv[]) 
{ 
    int result = 0; 
    try { 
     main2(argc, argv); 
    } catch (std::exception& se) { 
     // oh noes! 
     result = 1; 
    } 
    return result; 
} 

Answer 9

No puedo decir

VACaptureSource input; 

al inicio del programa ya que no hay constructor por defecto.

Hay una buena razón por la que no creó un constructor predeterminado: a saber, que un VACaptureSource solo tiene sentido cuando se asocia con un archivo. Por lo tanto, no cree un constructor predeterminado. En su lugar, simplemente reconozca que el alcance del objeto VACaptureSource es el bloque try, y úselo dentro de allí.