[SOLUCIONADO] Error de buffer al usar scanf

[TriN]

Buenos dias comunidad forosweb, hoy vengo por aqui debido a que tengo un problema en una funcion de un programa de registros que estoy elaborando, busque bastante información sin embargo no he conseguido solución, el problema que tengo es el siguiente, en la funcion agregar registros de mi programa de registros, tengo declarada una variable que guardara el numero de registro, hasta aqui todo bien el problema reside si se coloca alguna letra o una cadena de caracteres el programa se bugea y se queda como en un ciclo infinito en vez de imprimir el mensaje de error, he buscando diversas formas de arreglar este error sin embargo no he conseguido solución dejo el codigo esperando sus buenos consejos saludos y gracias de antemano

Código C:

int agregar(void)
{
FILE *arc;
int r,temp,i=0;
arc=fopen("lista.dat","ab");
    if(arc==NULL)
    {
        
    puts("Error");
    return 1;
        
    }   
    else
    {
 
    puts("Numero del PC que desea agregar");
    scanf("%d",&temp);
    
            
    
                if( (temp<1) || (temp>100)) 
                {
                puts("Numero de registro no valido");
                getch();
                system("cls");
                fclose(arc);
                return 1;
                }
        
        
    fseek(arc,0,SEEK_END);
    if(ftell(arc)!=0)
    {
        fclose(arc);
        arc=fopen("lista.dat","rb");
            if(arc==NULL)
            {
            puts("Error");
            getch(); 
            return 1;
            }
    
        fread(&dat,sizeof(datos),1,arc);
        while(!feof(arc))
        {
                if(dat.n==temp)
                {
                puts("No  se pueden repetir los elementos del registro");
                getch();
                fclose(arc);
                return 1;
                }
            fread(&dat,sizeof(datos),1,arc);
        }
    
    }
        fclose(arc);
            
    arc=fopen("lista.dat","ab");
    if(arc==NULL)
    {
    puts("Error al abrir fichero");
    exit(1);
    }   
 
    dat.n=temp;
    
    puts("Marca ");
    scanf("%s",&dat.marca);
        puts("Modelo");
    scanf("%s",&dat.modelo);
        puts("Serial del case");
    scanf("%s",&dat.cas);
        puts("Serial del procesador");
    scanf("%s",&dat.proc);
        puts("Serial del monitor");
    scanf("%s",&dat.mon);
        puts("Serial del teclado");
    scanf("%s",&dat.tecl);
        puts("Serial del mouse");
    scanf("%s",&dat.mou);
        puts("Serial del camara");
    scanf("%s",&dat.cam);
        puts("Serial del regleta");
    scanf("%s",&dat.reg);
        puts("Serial de las cornetas");
    scanf("%s",&dat.cor);
        printf("Esta seguro de agregar esta PC al registro?");
        puts("1.- SI    2.-NO");
        scanf("%d",&r);
            if(r==1)
            {
            fwrite(&dat,sizeof(datos),1,arc);
            }
            
    }
    
    fclose(arc);
    return 1;
}
 

[Principe_Azul]

Hola TriN, el error está acá:

Código C:

puts("Numero del PC que desea agregar");
    scanf("%d",&temp);
 

el mismo programa no va a funcionar bien, porque está esperando un valor numérico y vos en algún momento le ingresas una letra, cosa que no debería ser así, porque la variable temp es un int, no una string ni un char, por lo tanto no pueden llevar letras dentro de él, sólo números.
Lo que debes hacer es guardar el dato en una variable tipo string o char, luego hacer una comparación si la variable por temp es toda numérica, de ser así, el programa seguiría con normalidad y ahí hacés el cambio de tipo de variables, por ejemplo si la variable temp es un char, entónces pasa de char a int, si la variable temp es una string, entónces pasá de string a int, luego de hacer la comparación si la variable temp es toda numérica, osea si tiene todo números dentro de ella, en el caso que se haya ingresado una letra, la condición no se cumplirá y el programa no dará error, simplemente le informará al usuario que solo ingrese un número de 1 a 100.

Te ayudaría con código, pero no sé las funciones que tiene C, debería haber algo como "isalnum" o "isnum", algo así, o también lo podés hacer creando una función y llamando a ella para que verifique caracter por caracter, para saber si son todos números.

Buena suerte.

[Drewermerc]

hola TriN.
tiene razon principe_azul nesesitas hacer un comprobacion aqui te dejo un ejemplo de como usar isdigit para hacer esto en caso de que es valor sea entero devuelve 1 y sino de devuelve 0 nadamas aque aqui tendias que usar fgets en vez de escanf y arrays, aqui te dejo un ejemplo espero que te sirva.

Código C:

#include <stdio.h>
    #include <stdlib.h>
    #include <string.h>
    #include <ctype.h>  
int main ()
{
    char c[50];
    int num, i, f;
    printf("ingresa un valor\n");
    fgets(c, 50,  stdin);
    num = strlen(c);
    for (i = 0; i < num-1 ; i++)
    {
       if (isdigit(c[i]) !=0)
            printf("numero %c\n", c[i]);
        else
            printf("no valido\n");  
    }
 
    return 0;
}
 

bueno espero que te sirva.
saludos.
Drewermerc.

[Malenko]

Y porque no usar atoi?

[TriN]

Muchas gracias por sus respuestas ya investigare sobre eso y posteo el codigo cuando haya solucionado ! gracias nuevamente.

[TriN]

Hola amigos, ya he solucionado el problema, se me hizo mas entendible aplicar la funcion atoi, igualmente averiguare sobre otras funciones para convertir una variable char en int posteo el codigo y nuevamente muchas gracias.

Código C:

int agregar(void)
{
FILE *arc;
int r,i=0;
char ctemp[20];
int ntemp;
arc=fopen("lista.dat","ab");
    if(arc==NULL)
    {
        
    puts("Error");
    return 1;
        
    }   
    else
    {
 
    puts("Numero del PC que desea agregar");
    scanf("%s",&ctemp);
    
        if(atoi(ctemp)==0)
        {
        puts("Debe colocar solo numeros en el registro");
        getch();
        return 1;
        }
    
    ntemp=atoi(ctemp);
    
                if( (ntemp<1) || (ntemp>100)) 
                {
                puts("El numero de registro debe estar del 1 al 100");
                getch();
                system("cls");
                fclose(arc);
                return 1;
                }
        
        
    fseek(arc,0,SEEK_END);
    if(ftell(arc)!=0)
    {
        fclose(arc);
        arc=fopen("lista.dat","rb");
            if(arc==NULL)
            {
            puts("Error");
            getch(); 
            return 1;
            }
    
        fread(&dat,sizeof(datos),1,arc);
        while(!feof(arc))
        {
                if(dat.n==ntemp)
                {
                puts("No  se pueden repetir los elementos del registro");
                getch();
                fclose(arc);
                return 1;
                }
            fread(&dat,sizeof(datos),1,arc);
        }
    
    }
        fclose(arc);
            
    arc=fopen("lista.dat","ab");
    if(arc==NULL)
    {
    puts("Error al abrir fichero");
    exit(1);
    }   
 
    dat.n=ntemp;
    
    puts("Marca ");
    scanf("%s",&dat.marca);
        puts("Modelo");
    scanf("%s",&dat.modelo);
        puts("Serial del case");
    scanf("%s",&dat.cas);
        puts("Serial del procesador");
    scanf("%s",&dat.proc);
        puts("Serial del monitor");
    scanf("%s",&dat.mon);
        puts("Serial del teclado");
    scanf("%s",&dat.tecl);
        puts("Serial del mouse");
    scanf("%s",&dat.mou);
        puts("Serial del camara");
    scanf("%s",&dat.cam);
        puts("Serial del regleta");
    scanf("%s",&dat.reg);
        puts("Serial de las cornetas");
    scanf("%s",&dat.cor);
        printf("Esta seguro de agregar esta PC al registro?");
        puts("1.- SI    2.-NO");
        scanf("%d",&r);
            if(r==1)
            {
            fwrite(&dat,sizeof(datos),1,arc);
            }
            
    }
    
    fclose(arc);
    return 1;
}
 

[Drewermerc]

bueno esa tambien esa una opcion.
pero ami no me gusta ya que si introduces numero y despues letras no te daria la comprobacion que le pides y esta se pasaria como si nada y bueno el otro tipo de comprobacion te analiza cada numero o letraque se haya introducido.
saludos.
Drewermerc.

[TriN]

Drewer tambien parece una buena opción pero como convertiria la cadena o el caracter ingresado a int ? ya que en tu codigo solo estas haciendo una comprobación mas no una asignación.

[Drewermerc]

bueno pues hay puedes seguir usando la funcion atoi.
mira aqui te dejo un ejemplo de como quedaria.

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>  
int validar(char c[50]);
int main ()
{
        char c[50];
        int num, validarr;
        printf("ingresa un valor\n");
        fgets(c, 50,  stdin);
        validarr = validar(c);
        if (validarr == 0)
        {
            num = atoi(c);
            printf("valor de ingresado: %d\n", num);
        }
        else
            printf("numero no valido\n");
     
        return 0;
}
int validar(char c[50])
{
    int numero, i;
    numero = strlen(c);
        for (i = 0; i < numero-1 ; i++)
        {
           if (isdigit(c[i]) ==0)
                return 1; 
        }
    return 0;
}
 

bueno espero que te sirva si tienes alguna duda puedes preguntar.
saludos.
Drewermerc.

[TriN]

Hola Drewermerc, intentare aplicarlo a mi codigo gracias por tu respuesta.

[Principe_Azul]

TriN puedes usar también este código de Drewermerc, que tan solo le hecho esta simple modificación:

Código C:

#include <stdio.h>
    #include <stdlib.h>
    #include <string.h>
    #include <ctype.h>  
int main ()
{
    char c[50];
    int num, i, f;
    printf("ingresa un valor\n");
    fgets(c, 50,  stdin);
    num = strlen(c);
    int temp;
    for (i = 0; i < num-1 ; i++)
    {
       if (isdigit(c[i]) !=0)
            printf("numero %c\n", c[i]);
            temp += c[i]            
        else
            temp = 0;
            break;
    }
 
    return 0;
}
 

ahí declaramos la variable temp de tipo int, luego la variable toma el valor de cada caracter en caso que sea numérico, luego comparas que la variable temp no sea igual a 0, en el caso que si sea 0, entónces es porque hay una letra u otro caracter en la cadena y el programa detendrá el bucle y le dará el valor 0 a la variable temp, al ser 0 su valor, no se podrá avanzar con el programa, ahí tienes que ver que es lo que debes hacer si no ingresan un número.
Con 0 podrás saber que lo que ingresó el usuario no es un número.

[leosansan]

Tan solo tienes que usar un bucle a la espera de que el valor introducido sea el correcto. Y para ello no es necesario el uso de librerías y/o funciones adicionales a stdio.h, las creas a tu medida:

Código C++:

#include <stdio.h>
 
int main (){
  char c[50];
  int num, i, flag, factor, lon;
  do{
    num=0, flag=0, factor=1, lon=0;
    printf("Ingresa un valor: ");
    scanf("%50s",c);
    for (lon = 0; c[lon] ; lon++);
    for (i = 0; c[i] ; i++)
      if (c[i]<'0' ||c[i]>'9'){
        flag=1;
        break;
      }
    if (flag==0)
      for (i = lon-1; i>=0 ; i--,factor*=10)
        num+=(c[i]-'0')*factor;
  }
  while(num>100 || flag!=0);
  printf("Valor ingresado: %d",num);
  return 0;
}
 

¡¡¡Saluditos!!!