Foro de elhacker.net

Hola, estoy desarrollando una tarea y en esta me pide  ingresar un numero (todo bien hasta ahi) pero digamos la persona que esta usando el programa ingresa una letra le deberia decir: ingrese un numero pero en vez de eso cuando ingreso una letra me manda un error y se cierra, no se como poner que el valor a ingresar tiene que ser un numero y no una letra y si es una letra que lo devuelva al paso anterior hasta que ingrese un numero

Hola, meaf75. -
No tengo todavía mucha autoridad para responderte pero fíjate si este código te soluciona en parte tu problema, el único error es que si ingresas 12a te toma como válido el 12, tú dirás. -

#include <stdio.h>
#include <limits.h>
 
int main( void ){
	int numero = 0, ch, ok;
 
	do{
		printf("\n Ingresa un numero entero (maximo %u)....: ", UINT_MAX);
		ok = scanf("%d", &numero) == 1 && numero >= 1 && numero <= UINT_MAX;
 
		while ((ch = getchar()) != EOF && ch != '\n');
	}while(!ok);
 
	printf( "\n Ingreso...: %d", numero );
 
	return 0;
}

Saludos.
Daniel

---

(http://i68.tinypic.com/rt42vb.png)

Huy me explicas jaja voy en primer semestre y no entiendo mucho lo que hay ahi

El código que ha subido NOB2014 está en C. Tiene que estar en c o en c++?
Si es en C, ya te explico el código.

Pues yo estoy trabajado en pseint pero aprendiendo en c++ lo puedo pasar a pseint

#include <iostream>
using namespace std;
 
int main(){
	string n;
	bool notnum = false;
	cout << "Ingresa un numero"  << endl;
	while(cin >> n){
		for(int i = 0; i < n.size() && !notnum; ++i){
			notnum = (n[i]< 48 || n[i]> 57);
		}
		if(notnum) cout << "Por favor, ingresa un numero correcto" << endl;
		else cout << "Numero correcto" << endl;
	}
}

Mirate esto a ver si lo entiendes, si no enviame MP y te explico.

El ultimo codigo que se posteó (de @LaiaxanIV) esta escencialmente bien creo que por un boolean mal te queda en bucle infinito:


Lo he modificado:

#include <iostream>
#include <stdlib.h> 
using namespace std;
 
/*
	Lee un integer por teclado (C++)
*/
int readint(string msg){
	string str_num;	
	bool notnum = true;
 
	cout << msg;
	cin >> str_num;	
 
	while(notnum){
		for(int i = 0; i < str_num.size();  i++){
			notnum = (str_num[i]< 48 || str_num[i]> 57);
 
			if(notnum){
				cout << msg;
				cin >> str_num;				
				break;		
			}
		}	
	}	
 
	return atoi(str_num.c_str());
}
 
 
int main(){
	int n;
	n = readint("Ingresa un numero: ");	
	printf("Perfecto! el numero ingresado fue: %d",n); 
 
	return 0;
}
 

Ahora funciona bien y te aclaro que si el numero fuera un numero decimal (float) deberia aceptarse tambien el punto(.)

He intentado traducir el codigo del compañero (que he modificado ligeramente) de C++ a C pero me explota!

Alguien podria hacerlo por mi para ver en que fallo ?  en C++ me funciona pero en C tengo un problema al pasar un string y querer utilizarlo como const char *

Ojala me iluminen  ;D

Hola mirate este ejemplo, en vez de que la funcion readInt  lea, valide y regrese la conversion de string a entero

Mejor me cree una funcion para validar si un array de char en C seria valido para poder convertilo en un entero

Mi funcion lo unico que hace es recorrer la cadena y usando las funciones isdigit que viene en la bibilioteca ctype de C comprobamos que cada caracter es un entero valido

Si encontramos algo raro la funcion retorna un cero(false) sino regresa un uno(True)

ejemplo:

#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
 
 
int isInteger(char possibleNumber[]){
 
    while(*possibleNumber){
        if(!isdigit(*possibleNumber))return 0;
        possibleNumber++;
    }
    return 1;
}
 
int main()
{
    //Primer ejemplo
    char number[]="352353";
 
    if(isInteger(number))
        printf("%s","Es numero valido");
    else
        printf("%s","no es un numero valido");
 
    //Segundo ejemplo
    char number2[20];
    printf("%s","\n\nIngrese un valor valido\n");
    scanf("%s",number2);
 
 
    if(isInteger(number2))
       printf("%s","Es numero valido");
    else
       printf("%s","no es un numero valido");
 
    return 0;
}

no te olvides de incluir <limits>
#include <iostream>
using namespace std;

@crack81 : me gustó mucho tu implementación  ;-)

Soy demasiado novatillo y te quiero consultar por esto:

int isInteger(char possibleNumber[]){
 
    while(*possibleNumber){
        if(!isdigit(*possibleNumber))return 0;
        possibleNumber++;
    }
    return 1;
}
 

char possibleNumber[] es un array, cierto ? porque no lo declaras como puntero a char (char*) sino con []

Sin embargo,... luego usas possibleNumber como si fuera un puntero (*possibleNumber) y por ello termino confundido  :o


Si me lo explicas serias muy amable  :xD

Gracias desde ya!

Solución para Linux en compilador GCC, posiblemente funcione em MINGW para Windows ya que es un port de GCC.

El corazón del programa es la función getnum que sólo recibe números decimales (del 0 al 9) desde el teclado y devuelve un entero (int) con el valor igual al número introducido.

#include <stdio.h>
#include <ctype.h>
#include <termios.h>
 
static struct termios old, new;
 
void init_terminal() {
    tcgetattr(0, &old); /* old <- configuración actual del terminal
                         * el 0 en el primer argumento es el descriptor
                         * de archivo de stdin.
                         */
    new = old;          /* new copia el terminal original */
    new.c_lflag &= ~ICANON;      /* Modo no canónico: la entrada se acepta
                                  * caracter a caracter en vez de línea a
                                  * línea.
                                  */
    new.c_lflag &= ~ECHO;        /* No hay eco del carácter introducido */
    tcsetattr(0, TCSANOW, &new); /* Se guarda el nuevo comportamiento
                                  * del terminal para usarlo a partir
                                  * de ya.
                                  */
}
 
void reset_terminal() {
    tcsetattr(0, TCSANOW, &old); /* Se vuelve al comportamiento original */
}
 
int getnum() {
    int num = 0;
    int pos = 0;
    int negative = 0;
    char c;
 
    init_terminal(0);
    while((c = getchar()) != '\n') {
        if(pos == 0 && c == '-') {
            putchar('-');
            negative = 1;
            ++pos;
        }
        if(c == 127 && pos) { /* 127 es la tecla retroceso */
            num /= 10;
            putchar('\b');
            putchar(' ');
            putchar('\b');
            --pos;
        }
        if(isdigit(c)) {
            putchar(c);
            num *= 10;
            num += c - '0';
            ++pos;
        }
    }
    if(negative)
        num = -num;
    reset_terminal();
    putchar('\n');
    return num;
}
 
int main()
{
    int num;
    printf("Introduzca un número: ");
    num = getnum();
    printf("%i", num);
 
    return 0;
}

Hola boctulus, bueno me preguntas porque declare el parámetro como:

 char possibleNumber[]

y no por un puntero a char, bueno la respuesta es que son lo mismo.
Cuando tu le indicas que le vas a pasar un arreglo como paremetro realmente le estas indicando que le estas pasando un puntero no importa el tipo de dato, la cosas cambia cuando es mas un de una dimension ejemplo 

arreglo[][10]

aqui ya tiene un sentido un poco mas profundo.

En fin si tu declaras tu funcion asi

int isInteger(char *possibleNumber)

no afecta en nada, ahora con respecto a porque trato  a possibleNumber como un puntero es que realmente es un puntero.

Cuando hago:

 while(*possibleNumber)

estoy hanciendo varias operaciones una con el asterisco le estoy diciendo que me de el contenido de la direccion a donde esta apuntado la variable y al estar dentro de un while al mismo tiempo estoy haciendo una comparación booleana ya que en C o C++ el 1 significa true y el 0 False, asi que cuando accedo al contenido todo lo que sea diferente a 0 sera como decir

while(true)

.

Ahora hay que recordar que todas la cadenas en C o C++ "deberian" estar terminadas en nulo osea en CERO por ende el while vas a estar interando hasta que encuentre el final de la cadena osea hasta que encuentre el CERO

Pero es importante hacer esto

possibleNumber++

ya sino lo hacemos estariamos metidos en un bucle infinito ya que  possibleNumber++; lo que hace es apuntar a un siguiente caracter.

Bueno el tema es interesante si quieres saber mas detalladamente esto seria bueno hecharle una mirada a la artimetica de punteros y como manipular cadenas en C

Salutos...

Gracias crack, no sabia del azucar sintáctico que hace posible declarar un array como puntero pero como dices, tiene sentido  ;D

@LaiaxanIV : la idea está bien pero tiene una falla, creo es el valor por defecto del boolean