[?] Error fgets (Solucionado)

[leosansan]

No es eso, sino que se salta un caso. Y si le pongo aún más se salta más casos.

Es lo que te comenté. Al poner MAX=10 al noveno caracter se queda "fuera" de la cadena, es decir en el buffer. No he entendido lo de que se salta un caso.

Una forma simple de evitarlo es la ya indicada de sobredimensionar MAX a 100, por poner un número. Se supone que si al usuario se le piden cuatro caracteres tampoco va a ser tan bruto de ingresar tropecientos.

Claro que como siempre hay brutitos dispuestos a reventar un código a la fuerza, otra opción es comprobar si la cadena es mayor o menor de cuatro dígitos y si no pedir nueva introducción de datos, siem`re que no se "pase" de 100 y si no más tamaño a Cadena.

Lo cierto es que ahora funciona y sólo admite cuatro dígitos, ni más ni menos. Ya me comentarán que tal.

Y una muestra:

Introduce un numero de 4 digitos:
123

Introduce un numero de 4 digitos:
3215
Numero=3215     Cadena=3215
Introduce un numero de 4 digitos:
123456789456321

Introduce un numero de 4 digitos:
6541
Numero=6541     Cadena=6541
Introduce un numero de 4 digitos:
123

Introduce un numero de 4 digitos:
12

Introduce un numero de 4 digitos:
2589
Numero=2589     Cadena=2589
Introduce un numero de 4 digitos:
321654987654321654987

Introduce un numero de 4 digitos:
2561
Numero=2561     Cadena=2561

Código (cpp) [Seleccionar] Expandir


#include <stdio.h>
#include <string.h>
#define TONUM 48
#define MAX 100
#define N 5

int main()
{
char Cadena[MAX]={""};
int i,ch,Numero[N];

 while (1)
 {
   do{
     printf("\nIntroduce un numero de 4 digitos: \n");
     fgets(Cadena, MAX, stdin);
   }while (strlen(Cadena)!=N);
   Cadena[N-1] = '\0';
   // Ahora convertimos la cadena a numerico
   printf("Numero=");
   fflush(stdout);
   for(i=0;Cadena[i];i++)
   {
       Numero[i] = Cadena[i] - TONUM;
       printf("%d", Numero[i]);
   }
   printf("\tCadena=%s", Cadena);
 }
 return 0;
}


Por otro lado una pequeña observación:

.....................................................

int main()
{
   char Cadena[MAX+1]= {""};
   int i,Numero;
   char ch;

   while (1)
   {
       printf("\nIntroduce un numero de 4 digitos: \n");

       i = 0;

       //ch = getchar();

       while ((ch = getchar()) != '\n')
......................................................

la función getchar es de tipo int y por ello ch ha de ser int, no char.

¡¡¡¡ Saluditos! ..... !!!!

P.D: Se me olvidaba, el código como lo tienes, además de admitir menos de cuatro caracteres, admite hasta cinco.

Creo que es aquí donde está el error:

 while (1)
   {
       printf("\nIntroduce un numero de 4 digitos: \n");

       i = 0;

       //ch = getchar();

       while ((ch = getchar()) != '\n')
       {
           if (i < MAX) <== aquí va MAX-1
               Cadena[i++] = ch;
       }

[amchacon]

Tomo nota del getchar(), no sé porqué siempre pienso que devuelve un char (ese nombre puñetero  ).

Lo del MAX está perfecto, fijate que:

Código (cpp) [Seleccionar] Expandir

char Cadena[MAX+1]= {""};

La razón de ese cambio es que me parece más intuitivo y menos propenso a errores usar como unidad de longitud MAX que MAX - 1.

[leosansan]

Tal como tienes en el post esta es la salida:

Introduce un numero de 4 digitos:
12345
Cadena = 12345   Numero = 12345<==AQUI admite 5 caracteres

Introduce un numero de 4 digitos:
1234
Cadena = 1234    Numero = 1234

Con la corrección que te indico del -1 esta es la salida:

Introduce un numero de 4 digitos:
12345
Cadena = 1234    Numero = 1234<==AQUI está O.K

Introduce un numero de 4 digitos:
1234
Cadena = 1234    Numero = 1234

Introduce un numero de 4 digitos:
123
Cadena = 123     Numero = 123

¡¡¡¡ Saluditos! ..... !!!!

P.D: Pero conste que la gracia era solucionar lo del fgets, nada de scanf ni getchar.

[amchacon]

Ah leñe, lo que tengo que hacer ahí es cambiar este printf:

Código (cpp) [Seleccionar] Expandir

printf("\nIntroduce un numero de 4 digitos: \n");

Por este más genérico:

Código (cpp) [Seleccionar] Expandir

printf("\nIntroduce un numero de %d digitos: \n",MAX);

Ala, problema resuelto

[leosansan]

Ah leñe, lo que tengo que hacer ahí es cambiar este printf:

Código (cpp) [Seleccionar] Expandir

printf("\nIntroduce un numero de 4 digitos: \n");

Por este más genérico:

Código (cpp) [Seleccionar] Expandir

printf("\nIntroduce un numero de %d digitos: \n",MAX);

Ala, problema resuelto

¿Queeeeeeeeeeeeeeeeeeé?. ¿Qué es eso de problema resuelto?. Se trata de ingresar cuatro dígitos, no cinco pequeño saltamontes.

Es coña, la hora será.

Eh aquí tu solución adaptada a cuatro dígitos y sólo cuatro:

Introduce un numero de 4 digitos:
123
Numero menor cifras.

Introduce un numero de 4 digitos:
3265
Cadena = 3265    Numero = 3265

Introduce un numero de 4 digitos:
12
Numero menor cifras.

Introduce un numero de 4 digitos:
123456
Numero mayor cifras.

Introduce un numero de 4 digitos:
2541
Cadena = 2541    Numero = 2541

Introduce un numero de 4 digitos:
1
Numero menor cifras.
[/size]

Código (cpp) [Seleccionar] Expandir


#include <stdio.h>
#define MAX 5
#define TONUM 48

int main()
{
    char Cadena[MAX+1]= {""};
    int i,Numero;
    int ch;
    while (1){
        printf("\nIntroduce un numero de 4 digitos: \n");
        i = 0;
        while ((ch = getchar()) != '\n') {
            if (i<=MAX-1)
            Cadena[i++] = ch;
        }
        if (i==MAX-1){
            Cadena[i] = 0;
            printf("Cadena = %s \t Numero = ",Cadena);
          // Ahora convertimos la cadena a numerico
          for(i=0; Cadena[i]; i++) {
            Numero = Cadena[i] - TONUM;
            printf("%d", Numero);
        }
        printf("\n");
      }
      else if (i>MAX-1)
        puts ("Numero mayor cifras. ");
      else
        puts ("Numero menor cifras. ");
    }
    return 0;
}


¡¡¡¡ Saluditos! ..... !!!!

REEDITADO Y MEJORADO.

[amchacon]

Yo lo he interpetrado como n caraceteres 

Por cierto:

Código (cpp) [Seleccionar] Expandir

while ((ch = getchar()) != '\n') {
    Cadena[i++] = ch;
           //printf("\ni=%d\n",i);
}

¡Peligro de desbordamiento! Si le meto más caracteres de la cuenta (5) estaré desbordando el array.

[leosansan]

Yo lo he interpetrado como n caraceteres  

Por cierto:

Código (cpp) [Seleccionar] Expandir

while ((ch = getchar()) != '\n') {
   Cadena[i++] = ch;
          //printf("\ni=%d\n",i);
}

¡Peligro de desbordamiento! Si le meto más caracteres de la cuenta (5) estaré desbordando el array.

Se me paso un if en el código anterior que ya he arreglado:

Código (cpp) [Seleccionar] Expandir


.............................
while ((ch = getchar()) != '\n') {
           if (i<=MAX-1)
           Cadena[i++] = ch;
.....................................


¡¡¡¡ Saluditos! ..... !!!!

P.D: Sigo pensando que esto se aparta del objetivo del fgets, que me ratifico en lo que propuse, salvo nueva aportación, cosa que espero.

[xiruko]

Hola, bueno no me leí todo el hilo pero diría que se trata de problemas con el salto de línea y fgets(). Dejo aquí mi pequeño granito de arena:

Código (c) [Seleccionar] Expandir

#include <stdio.h>
#include <string.h>
#define MAX 5

int main()
{
    char buffer[MAX]={'\0'};
    int ch;
    char *p=NULL;

    fgets(buffer, MAX, stdin);
    if (p=strchr(buffer, '\n')) // si se encuentra el '\n' se cambia por null
        *p='\0';
    else                        // si no se encuentra es que hay que vaciar el buffer de entrada
while ((ch=getchar())!='\n' && ch!=EOF);

    printf("Introdujiste: %s\n", buffer);

    return 0;
}

Si no tiene nada que ver con el hilo pido disculpas, pero es que con tanto post la verdad que da un poco de pereza encontrar el problema.

Saludos.

[leosansan]

Hola, bueno no me leí todo el hilo pero diría que se trata de problemas con el salto de línea y fgets(). Dejo aquí mi pequeño granito de arena:
........................................
Si no tiene nada que ver con el hilo pido disculpas, pero es que con tanto post la verdad que da un poco de pereza encontrar el problema.

Buena aportación xiruko, da una nueva perspectiva.

Ahora sólo falta que diga que se rechaza por menor o mayor número de cifras.

¡¡¡¡ Saluditos! ..... !!!!

[xiruko]

Ahora sólo falta que diga que se rechaza por menor o mayor número de cifras.

Aquí lo dejo

Código (c) [Seleccionar] Expandir

#include <stdio.h>
#include <string.h>
#define MAX 5

int main()
{
    char buffer[MAX]={'\0'};
    int ch;
    char *p=NULL;
    int basura=0;

    printf("Introduce %d caracteres: ", MAX-1);
    fgets(buffer, MAX, stdin);

    if (p=strchr(buffer, '\n'))
printf("ERROR: Faltan %ld caracteres.\n", MAX-1-(p-buffer));
    else
    {
basura=1;
while ((ch=getchar())!='\n' && ch!=EOF) basura++;
if (basura > 1)
    printf("ERROR: Sobran %d caracteres.\n", basura-1);
else
    printf("BIEN! Introdujiste solo %d caracteres.\n", MAX-1);
    }

    return 0;
}

Saludos.