Estaba haciendo un pequeño programa para calcular la factorización de un número y quería que la entrada fuese exactamente un número entero, y que el usuario no pudiera meter otro tipo de dato. La cosa es que cuando se introduce un "char
" el programa se vuelve loco y entra en un bucle infinito. Parece que lo he solucionado un poco usando "scanf(tipo_dato,&variable)
" pero aún así el código sigue fallando en determinadas ocasiones.
Estoy estudiando y puede que el código no sea lo mejor (incluso que os dé dolor de ojos) por eso si alguno tiene alguna sugerencia sobre como podría depurar el código, se lo agradecería.
#include <iostream>
using namespace std;
bool correcto(int n){
bool h = true;
h = n <= 999999 && n > 0 ;
if (h == false ){
cout << "Introduzca un número correcto porfavor\n";
}
return h;
}
int nextPrimo (int num){
bool esPrimo = false;
int numero = num+1;
int divisores = 2;
while(!esPrimo){
for(int i = 2;i<=numero/2;i++){
if(numero % i == 0){
divisores++;
}
}
if (divisores == 2 ) {
esPrimo = true;
}else {
numero++;
divisores = 2;
}
}
return numero;
}
void factorizacion (int n){
int primo = 2;
cout << "La factorización de "<<n<<" es:\n\n";
while (n > 1){
if (n % primo == 0){
if(n >= 100000){
cout << n<<"| "<<primo<<"\n";
}
else if(n >= 10000 && n <= 99999){
cout << n<<" | "<<primo<<"\n";
}else if(n >= 1000 && n <= 9999){
cout << n<<" | "<<primo<<"\n";
}else if(n >= 100 && n <= 999){
cout << n<<" | "<<primo<<"\n";
}else if(n >= 10 && n <= 99){
cout << n<<" | "<<primo<<"\n";
}else if ( n != 0){
cout << n<<" | "<<primo<<"\n";
}
n = n/primo;
}else{
primo = nextPrimo(primo);
}
}
cout<<"1 | "<<"\n\n";
}
int main() {
int numero;
bool cont = true;
char sig;
do{
do{
cout << "\nIntroduzca el número a factorizar: ";
if (scanf("%i",&numero) != 1){
cout<<"Entrada no númerica\n\n";
numero = 1;
}
}while(!correcto(numero));
if (numero != 1) {
factorizacion(numero);
}
cout << "¿Quiere seguir usando el programa? (s/n) ";
cin>>sig;
if(sig == 'n') cont = false;
}while(cont);
return 0;
}
El código no es lo más eficiente del mundo porque pretendo que la salida sea similar a como se hace cuando uno lo aprenda.
scanf
. Primero porque estas en C++,scanf
es de C. Y segundo,scanf
es probablemente la peor forma de leer entrada de el usuario que existe.