Por puntos:
- No puedes asumir que el nombre de una persona sólo contendrá letras ya que existen personas como Jennifer 8. Lee.
- No puedes limitar los datos que pueden entrar en un
std::string
, ya que cualquier dato es potencialmente convertible a texto.
- Se considera mala práctica lanzar como excepciones datos que no deriven de
std::exception
.
Suponiendo que el primer punto no existiese, para solventar el segundo punto podrías leer un número y si no falla su lectura lo consideras un error:
std::string nombre;
for (bool nombre_leido = false; !nombre_leido; )
{
int numero;
if (std::cin >> numero)
{
std::cout << numero << " no es un nombre\n";
}
else
{
std::cin.clear();
std::cin >> nombre;
nombre_leido = true;
}
}
Para solventar el tercer punto, cambia tus throw
por excepciones estándar.
Siguiendo los consejos anteriores, tu código podría parecerse a:
string pedir_solo_texto()
{
std::string texto;
for (bool nombre_leido = false; !nombre_leido; )
{
int numero;
if (std::cin >> numero)
{
std::cout << numero << " no es texto\n";
}
else
{
std::cin.clear();
std::cin >> texto;
nombre_leido = true;
}
}
return texto;
}
struct dni_incorrecto : public std::exception {};
struct sueldo_incorrecto : public std::exception {};
void Persona::pedir() try { // https://en.cppreference.com/w/cpp/language/function-try-block
std::cout << "Ingrese su DNI: ";
std::cin >> dni;
if (dni < 0) throw dni_incorrecto(std::format("el DNI {} no es valido", dni));
std::cout << "Ingrese su NOMBRE: ";
nombre = pedir_solo_texto();
std::cout << "Ingrese su MAIL: ";
std::cin >> mail;
std::cout << "Ingrese su EMPRESA: ";
std::cin >> empresa;
std::cout << "Ingrese su SUELDO: ";
std::cin >> sueldo;
if (sueldo < 1) throw sueldo_incorrecto(std::format("el sueldo {} no es valido", sueldo));
}
catch (const dni_incorrecto &e) {
// ...
}
catch (const sueldo_incorrecto &e) {
// ...
}