do while false

[loCk636]

Estoy intentado que haga x cosa mientras que lo introducido no sea numero

Código C++:

do {
    .................
    } while (n>='0' || n<='0');
 

Ahora mismo se realiza al reves, cuando es numero se repite, he probado con while !(n>='0' || n<='0') pero falla

[leosansan]

Iniciado por loCk636

Estoy intentado que haga x cosa mientras que lo introducido no sea numero

Código C++:

do{
    .......................
}while ( n < '0' || n > '9' ) ;
 

¡¡¡Saluditos!!!

[loCk636]

Me sigue fallando y no entiendo por que

Código C++:

int n;
do{
    .......................
}while ( n < '0' || n > '9' ) ;
 

es por que utilizo int?

[leosansan]

Iniciado por loCk636

Me sigue fallando y no entiendo por que

Código C++:

int n;
do{
    .......................
}while ( n < '0' || n > '9' ) ;
 

es por que utilizo int?

Claro, tal como está es para un char. Si lo que quieres es para un int:

Código C++:

int n;
do{
    .......................
}while ( n < 0 || n > 9) ;
 

¡¡¡Saluditos!!!

[loCk636]

Iniciado por leosansan

Claro, tal como está es para un char. Si lo que quieres es para un int:

Código C++:

int n;
do{
    .......................
}while ( n < 0 || n > 9) ;
 

¡¡¡Saluditos!!!

Con esta solucion valida los numeros positivos, pero los negativos tambien son numeros

[leosansan]

Iniciado por loCk636

Con esta solucion valida los numeros positivos, pero los negativos tambien son numeros

Cambia el cero y el nueve por el rango de valores admisibles....y si no, porfi, explica mejor lo que pretendes.

¡¡¡Saluditos!!!

[loCk636]

Iniciado por leosansan

Cambia el cero y el nueve por el rango de valores admisibles....y si no, porfi, explica mejor lo que pretendes.

¡¡¡Saluditos!!!

Yo introduzco un numero que lo meto en la variable int "n"

si n no es numero (tanto <0 o >0) que vuelva a introducir por teclado

[Madh]

Una variable de tipo int siempre almacenará un número entero (0,1,2,-1,994,-374,...). Sabiendo esto, debes darte cuenta de que no puedes encontrar una condición en la que una variable int sea diferente de un número, no tiene sentido.

Ahora bien, una variable de tipo char almacenará carácteres de la tabla ASCII (256 carácteres diferentes en total). Aunque todos ellos son variables de tipo char, representan tanto letras, como números, símbolos, etc.

Si utilizas un char en vez de un int para guardar la respuesta del usuario, puedes verificar si el cáracter introducido es un número o no con la condición que ya te han comentado arriba. Esta sería la implementación completa:

Código C++:

#include <iostream>
using namespace std;
 
int main(void){
 
    char n;
 
    cout << "Introduce algo: ";
    cin >> n;
 
    do{
        cout << "Repetir si no soy un numero" << endl;
    }while ( n < '0' || n > '9') ;
 
    return 0;
}
 

[kutcher]

Iniciado por Madh

Una variable de tipo int siempre almacenará un número entero (0,1,2,-1,994,-374,...). Sabiendo esto, debes darte cuenta de que no puedes encontrar una condición en la que una variable int sea diferente de un número, no tiene sentido.

Ahora bien, una variable de tipo char almacenará carácteres de la tabla ASCII (256 carácteres diferentes en total). Aunque todos ellos son variables de tipo char, representan tanto letras, como números, símbolos, etc.

Si utilizas un char en vez de un int para guardar la respuesta del usuario, puedes verificar si el cáracter introducido es un número o no con la condición que ya te han comentado arriba. Esta sería la implementación completa:

Concuerdo totalmente con Madh dejo otra opción con la condición de do while diferente:

Código C++:

char n;
do{
    printf("Repetir si no soy un numero\n");
    scanf(" %c", &n);
}while( !(n >= '0' && n <= '9') );
 

Saludos

[leosansan]

Iniciado por kutcher

Concuerdo totalmente con Madh dejo otra opción con la condición de do while diferente:

Pues yo no concuerdo con ninguno de los dos. Para empezar se puede producir un bucle infinito si no entran un número y, lo más importante, no consideran el caso de un número negativo, es decir que el primer caracter sea un menos. Por no mencionar que aún aunque se introduzca un número correcto sale el mensajito de "Repetir si no soy un numero". Además el número no tiene porque limitarse a una cifra.

Una opción chapucera sería:

Código C++:

#include <stdio.h>
 
int main ( ) {
  int i , numero , flag = 0 , flag1 = 0 ;
  char num [20];
  do {
    flag = 0 , flag1 = 0 ;
    printf ( "\nIntroduzca un numero entero: " ) ;
    scanf ( "%s" , num ) ;
    for ( i = 0 ; num [ i ] ; i++ ) {
      if ( num [0] == '-' && flag1 == 0 ) 
        flag1 = 1 ;
      else if ( num [i] < '0' || num [i] > '9')
        flag = 1 ;
    }
  }while ( flag != 0 ) ;
  printf ( "\nnumero = %d\n" , atoi ( num ) ) ; 
  return 0 ;
}
 

¡¡¡Saluditos!!!

[kutcher]

Iniciado por loCk636

Yo introduzco un numero que lo meto en la variable int "n"
si n no es numero (tanto <0 o >0) que vuelva a introducir por teclado

loCk636 que quieres decir con esto, tu asumes que ingresas un numero pero después necesitas comprobar si es un numero , considero esto redundante y sin sentido podrías aclarar este punto.

Saludos

[leosansan]

Iniciado por loCk636

Yo introduzco un numero que lo meto en la variable int "n"

si n no es numero (tanto <0 o >0) que vuelva a introducir por teclado

Pues no. Para verificar si es o no un entro, sea positivo o negativo, lo normal es capturarlo con un array de char y luego verificar si lo es o no. Una vez comprobado que es un int lo pasas de char a int.

¡¡¡Saluditos!!!

[eferion]

Coincido con leosansan. Lo mejor suele ser primero verificar y después convertir.
Cuando te enfrentas a un usuario lo lógico es pensar mal. Si hay alguna posibilidad de que un usuario introduzca un texto en un campo numérico, ten fe que así lo hará... eso y que el mejor mantenimiento se consigue cuando se separan tareas, lo más factible a la larga es chequear los valores en una primera capa y después hacer las conversiones en una segunda.

Claro que puedes fusionar ambas capas y hacerlo todo a la vez... sin embargo ese código luego te va a resultar más complicado de depurar y de actualizar.

Mi propuesta:

* Detecta el prefijo +/-
* Un número compuesto solo por el prefijo se considera no válido
* Se admiten números sin prefijo.
* Se detectan caracteres no numéricos en cualquier parte de la secuencia.

Código C++:

int EsNumero( char* cadena )
{
  int to_return = 0;
 
  if ( *cadena == '-' || *cadena == '+' )
    ++cadena;
 
  do
  {
    to_return = isdigit( *cadena );
  } while( to_return && *++cadena );
 
  return to_return;
}
 
int main ( )
{
  char num [20];
  int numero;
  do
  {
    printf( "Introduzca un numero entero: " );
    scanf( "%s" , num ) ;
  } while( !EsNumero( num ) );
 
  numero = atoi( num );
  printf ( "numero = %d\n" , numero ) ;
  return EXIT_SUCCESS;
}
 

Un saludo.

[leosansan]

Iniciado por eferion

Coincido con leosansan. Lo mejor suele ser primero verificar y después convertir.
Cuando te enfrentas a un usuario lo lógico es pensar mal.............

Mi propuesta:

* Detecta el prefijo +/-
* Un número compuesto solo por el prefijo se considera no válido
* Se admiten números sin prefijo.
* Se detectan caracteres no numéricos en cualquier parte de la secuencia.

Coincido plenamente con eferion en todas sus consideraciones.

Y para que no se diga, otra propuesta sin tanto" flag" como en el anterior y sin sacar la artillería de punteros:

Código C++:

#include <stdio.h>
#include <ctype.h>
 
int main ( ) {
  int i = 0 , numero  ;
  char num [20] ;
  do {
    printf ( "\nIntroduzca un numero entero: " ) ;
    scanf ( "%s" , num ) ;
    for ( i = 0 ; num [ i ] ; i++ ) {
      if ( num [0] == '-' || num [0] == '+' )   ++i;
      if ( ! isdigit ( num [i] ) ) { i = -1 ; break ; }
    }
  }while( i == -1 ) ;
  printf ( "\nnumero = %d\n" , atoi ( num ) ) ; 
  return 0 ;
}
 

¡¡¡AGGG¡¡¡, perdón que es con función :

Código C++:

#include <stdio.h>
#include <ctype.h>
 
int EsNumero( char* cadena ) {
    int i ;
    for ( i = 0 ; cadena [ i ] ; i++ ) {
      if ( cadena [0] == '-' || cadena [0] == '+' )  ++i;
      if ( ! isdigit ( cadena [i] ) )  return 0 ;
    }
    return 1 ;
}
 
int main ( ) {
  int numero  ;
  char num [20] ;
  do {
    printf ( "\nIntroduzca un numero entero: " ) ;
    scanf ( "%s" , num ) ;
  }while( !EsNumero( num ) ) ;
  printf ( "\nnumero = %d\n" , atoi ( num ) ) ; 
  return 0 ;
}
 

¡¡¡Güeeeno, está bien, con punteros:

Código C++:

int EsNumero( char *cadena ) {
  int i ;
  if ( *cadena  == '-' || *cadena  == '+' )  ++cadena;
  while ( *cadena ) {
     if ( ! isdigit ( *cadena ) )  return 0 ;
     ++cadena;
  }
  return 1 ;
}
 

¡¡¡Saluditos!!!

[kutcher]

Iniciado por eferion

Lo mejor suele ser primero verificar y después convertir.
Cuando te enfrentas a un usuario lo lógico es pensar mal. Si hay alguna posibilidad de que un usuario introduzca un texto en un campo numérico, ten fe que así lo hará... eso y que el mejor mantenimiento se consigue cuando se separan tareas, lo más factible a la larga es chequear los valores en una primera capa y después hacer las conversiones en una segunda.

Pues si es en ese plan dejo mi propuesta haciendo uso de la función strtol para variar :

Código C++:

bool es_numero( char *s )
{
    while( *s )
    {
        strtol(s, &s, 10);
        if (*s != 0) return false;
    }
    return true;
}
 
int main(void)
{
    char num [20];
    int numero = 0;
 
    do{
        printf( "Introduzca un numero entero: " );
        scanf("%s", num);
    }while( !es_numero(num) );
 
    numero = atoi(num);
    printf("Numero = %d\n", numero);
 
    return 0;
}
 

leosansan tienes un problema es tus códigos en las lineas :

Código C++:

if ( cadena [0] == '-' || cadena [0] == '+' )  ++i;
 

Siempre verificas el primer indice si este contiene un '-' o '+' siempre incrementas i esto te causa problamas con las cifras pares, debe quedar asi :

Código C++:

if ( cadena [i] == '-' || cadena [i] == '+' )  ++i;
 

Saludos

[leosansan]

Iniciado por kutcher

Pues si es en ese plan dejo mi propuesta haciendo uso de la función strtol para variar :

Código C++:

......................
    numero = atoi(num);
    printf("Numero = %d\n", numero);
.................................
 

[

Bien por recordar el usa de strol, un poco rebuscado pero efectivo.

Iniciado por kutcher

leosansan tienes un problema es tus códigos en las lineas :

Código C++:

if ( cadena [0] == '-' || cadena [0] == '+' )  ++i;
 

¡¡¡¡Thanks¡¡¡, y sorry, se me escapo el flag:

Código C++:

int EsNumero( char* cadena ) {
    int i , flag = 0 ;
    for ( i = 0 ; cadena [ i ] ; i++ ) {
      if ( cadena [0] == '-' || cadena [0] == '+' && flag == 0 )  ++i , flag = 1 ;
      if ( ! isdigit ( cadena [i] ) )  return 0 ;
    }
    return 1 ;
}
 

Y puestos a experimentar :

Código C++:

int EsNumero( char *cadena ) {
    while ( *cadena  == '-' || *cadena  == '+' || isdigit ( *cadena ) && *cadena != 0 ) *cadena++ ;   
    return  ( strlen ( cadena ) != 0 )  ? 0 : 1 ;
}
 

Y ¡¡¡más fuerte¡¡¡ , sin usar ningún bucle. A ver quien supera la complejidad del que viene, je je :

Código C++:

int EsNumero ( char *num ) {
    int n = log10 ( abs ( atoi ( num ) ) ) + 1 , Strlen;
    return ( n == ( Strlen = ( num [0] == '-' ||  num [0] == '+')  ? strlen ( num ) - 1 : strlen ( num ) ) ) ? 1 : 0 ;
}
 

¡¡¡Saluditos!!!

[dehm]

No sé si esto se aleja de los que buscáis, pero para validar un número yo uso los streams.

Esto es una función que encontré una vez, que la manoseé para que admitiera la entrada de decimales con coma (vamos, que si se encuentra con una coma la transforma en punto)...pero la idea original no es mía

Código C++:

bool validar (std::string& entrada, float& valor)
{
    for (size_t i=0; i<entrada.length(); i++)
        if (entrada[i]==',')
        {
            entrada[i]='.';
        }
    std::stringstream mystream(entrada);
    if(mystream >> valor)
    {
        return true;
    }
    return false;
}
 

Y la llamamos así:

Código C++:

std::string entradanumerica;
        float Cantidad;        
        std::cout<<"Introduce cantidad: "<<std::endl;
        getline (std::cin,entradanumerica);
        while (!validar(entradanumerica,Cantidad))
        {
            getline (std::cin,entradanumerica);
        }
 

[kutcher]

Iniciado por leosansan

Y ¡¡¡más fuerte¡¡¡ , sin usar ningún bucle. A ver quien supera la complejidad del que viene, je je :

Si la cuestión es sin bucles se reduce a una sola linea

Código C++:

bool es_numero( char *s )
{
  return !(strtol(s, &s, 10)) ? false : true;
}
 

Saludos

[leosansan]

Iniciado por kutcher

Si la cuestión es sin bucles se reduce a una sola linea

Código C++:

bool es_numero( char *s )
{
  return !(strtol(s, &s, 10)) ? false : true;
}
 

Saludos

¡¡¡ Buen intento ¡¡¡, pero esta es la salida que obtengo:

Código C++:

Introduzca un numero entero: -123*
Numero = -123
 

A ver si pilláis el "pero" de mi último y matemático código .

¡¡¡Saluditos!!!

[Pantaláimon]

He creado una función siguiendo la idea de kutcher y comparando con la de leosansan veo los siguientes situaciones:

Código C:

#include <stdio.h>
#include <string.h>
#include <math.h>
 
int es_numero( char *s )
{
  char *p = s;
  strtol(p, &s, 10);
  return (s - p) == strlen(p);
}
 
int EsNumero ( char *num ) {
    int n = log10 ( abs ( atoi ( num ) ) ) + 1 , Strlen;
    return ( n == ( Strlen = ( num [0] == '-' ||  num [0] == '+')  ? strlen ( num ) - 1 : strlen ( num ) ) ) ? 1 : 0 ;
}
 
int main() {
    char a[] = " +123";
    char b[] = "+123 ";
    char c[] = "+123*";
    printf("%d %d\n", es_numero(a), EsNumero(a));
    printf("%d %d\n", es_numero(b), EsNumero(b));
    printf("%d %d\n", es_numero(c), EsNumero(c));
    return 0;
}
 

Ante esto me pregunto, ¿hasta que punto un string debe considerarse un entero?

Un saludo!

[leosansan]

Iniciado por Pantaláimon

Ante esto me pregunto, ¿hasta que punto un string debe considerarse un entero?

Y yo me pregunto, ¿ hasta qué punto, informáticamente hablando, son válidas como números enteros cosas como: " +123", "+123 4", 000123" y similares. Aquí no estamos escribiendo en un papel donde la separación entre los dígitos es indiferente, esto es informática donde un espacio es un caracter y si alguien me escribe "1 2 3", así como los casos mencionados antes, yo no lo acepto como válido y punto.

Es algo así como si tuviese como pin de la tarjeta de crédito "123" y yo intentase introducir " 123" ó "0123" pues nooooooo, no me dejaría.

Me recuerdas la discusión sobre si 500 era primo al tú considerar que sí tomándolo como equivalente a 00500, ¿ recuerdas ?.

Puntos de vista tan solo y eso me recuerda si se acepta "pulpo" como animal de compañía.....nooooooo........... ...pero resulta que, paradojas de la vida, tengo un antiguo alumno de Ciencias del Mar que, tiene, entre otros, un acuario marino y un par de pulpos, entre otros animalitos raritos, y él tiene muy claro que sí, el pulpo es un animal de compañía.

........Puntos de vista tan solo.................

¡¡¡Saluditos!!!



P.D: Y todo con el mejor humor del mundo. Sólo pretendo entretener con "puntos de vista", ¿ vale ?. Y conste que con mi remencionado punto de vista mis códigos validan correctamente.

[kutcher]

El inconveniente con strtol es que salta todos los espacios iniciales para localizar los caracteres +/- si es que existen, debido a esto valida la primera cadena como un numero

Saludos

[Pantaláimon]

Iniciado por leosansan

Me recuerdas la discusión sobre si 500 era primo al tú considerar que sí tomándolo como equivalente a 00500, ¿ recuerdas ?.

jajaja, que memoria tienes, pero para que la cosa tenga más lógica creo que el asunto iba de números capicúa o palíndromos, no de primos.

De todos modos yo exponía las dos situaciones y no me he decantado por ninguna. ¿Por qué? pues como bien dices para una contraseña hay que ser estrictos. Pero por otra parte, hay veces que hay que hacer programas en los cuales este sea amable con el usuario y no tocarle las pelotas por qué a puesto un espacio de más al escribir su edad o porque a puesto guioncitos para indicar las fechas en vez de slash(/). Lo que se llama una buena experiencia de usuario. Como aquí estamos jugando a hacer funciones de validación de enteros en base al enunciado más que insuficiente de loCk636, cualquier interpretación me parece válida.

Un saludo!

[leosansan]

Iniciado por Pantaláimon

jajaja, que memoria tienes, pero para que la cosa tenga más lógica creo que el asunto iba de números capicúa o palíndromos, no de primos.

Un saludo!

Sin duda alguna tengo una neurona en cortocircuito porque ya por entonces confundía primos, es que es un tema que me obsesiona y en el que sigo trabajando, con los capicúa y/o palíndromos.

Un fuerte abrazo Pantaláimon, da gusto verte con este "humor".

¡¡¡Saluditos!!!



EDITO:

Iniciado por kutcher

El inconveniente con strtol es que salta todos los espacios iniciales para localizar los caracteres +/- si es que existen, debido a esto valida la primera cadena como un numero

Sigue el problema de que acepta como válido -123*.

Aprovecho para corregir el que no admita espacios, tipo "123 4":

Código C++:

#include <stdio.h>
#include <string.h>
#include <ctype.h>
 
int EsNumero( char *cadena ) {
    while ( *cadena  == '-' || *cadena  == '+' || isdigit ( *cadena ) && *cadena != 0 ) *cadena++ ;   
    return  ( strlen ( cadena ) != 0 )  ? 0 : 1 ;
}
 
int main ( ) {
  int numero  ;
  char num [20] ;
  do {
    printf ( "\nIntroduzca un numero entero: " ) ;
    scanf ( "%[^\n]" , num ) ;
    while ( getchar ( ) != '\n' ) ;
  }while( !EsNumero( num ) ) ;
  printf ( "\nnumero = %d\n" , atoi ( num ) ) ; 
  return 0 ;
}