Estás silenciando excepciones

[fedemgp]

TP3-9508/client_src/client_main.cpp

Lines 15 to 23 in cad058c

	try {
	worker.start();
	} catch (std::invalid_argument& e) {
	std::cerr << e.what() << std::endl;
	} catch (...) {
	std::cerr << GENERIC_ERROR << std::endl;
	}
	return 0;
	}

Si se lanzara una execepción de la STL como por ejemplo std::runtime_error
en vez de printear el mensaje correspondiente, imprimirías un GENERIC_ERROR.
como si estuvieras haciendo un throw "error". El primer catch debe ser mas
genérico, capturando todo lo que herede de std::exception

[Jonathan-R0]

Es decir que debería tener primero un catch para el std::exception ¿y luego para cualquier otra cosa dejo que explote? ¿o es preferible hacer lo de imprimir el GENERIC_ERROR? 🤔

[fedemgp]

si vos catcheas std::exception vas a catchear todo lo que herede de esta clase (no te explota con nada), después ponés el catch de las elipsis para que todo lo que no herede de std::exception (un string, un char *, cualquier cosa que algún programador no avispado quiera lanzar). Con esos simples dos catch te asegurás de que tu aplicación no explote con nada.

[fedemgp]

Y además te asegurás de que todo lo que herede de std::exception puedas imprimir el mensaje que estas excepciones te informan.