Foro de elhacker.net

#Solucion del post (http://foro.elhacker.net/programacion_cc/solucionado_problema_con_lista_enlazada_en_c-t320045.0.html;msg1583532#msg1583532)

Bueno, estaba practicando con listas enlazadas en C, pero no se porque no me va esto :xD

El caso es que no me deja ingresar bien los datos, ¿uso mal fgets() (http://www.cplusplus.com/reference/clibrary/cstdio/fgets/)?

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
struct _contacto {
    char nombre[33];
    char telefono[13];
    struct _contacto *siguiente;
};
struct _contacto *primero, *ultimo;
 
void agregar_contacto()
{
     struct _contacto *nuevo;
     nuevo = (struct _contacto*)malloc(sizeof(struct _contacto));
     if(nuevo==NULL){
         printf("\nNo hay espacio suficiente.");
         return;
     }
     printf("\n\nAgregar contacto\n");
     printf("------------------");
     printf("\nNombre: ");
     fgets(nuevo->nombre,32,stdin);
     printf("\nTelefono: ");
     fgets(nuevo->telefono,12,stdin);
     nuevo->siguiente = NULL;
     if(primero==NULL){
         primero = nuevo;
         ultimo = nuevo;
     } else {
         ultimo->siguiente = nuevo;
         ultimo = nuevo;
     }
}
 
void buscar_contacto_tlfn()
{
     struct _contacto *busqueda, *actual;
     busqueda = (struct _contacto*)malloc(sizeof(struct _contacto));
     if(busqueda==NULL){
         printf("\nNo hay espacio suficiente.");
         return ;
     }
     printf("\n\nBuscar contacto por telefono\n");
     printf("----------------------------");
     printf("\nTelefono: ");
     fgets(busqueda->telefono,12,stdin);
     actual = primero;
     while(actual!=NULL){
         if(strcmp(actual->telefono,busqueda->telefono)==0){
             printf("+Nombre: %s\n", actual->nombre);
             return;
         }
         actual = actual->siguiente;
     }
     printf("No encontrado.");
}
 
 
int opcion = 0;
 
void mostrar_menu()
{
     opcion = 0;
     printf("\n\n\n MENU\n");
     printf("======\n");
     printf("1. Agregar contacto\n");
     printf("2. Buscar contacto por telefono\n");
     printf("3. Salir\n");
     scanf("%i", &opcion);
}
 
int main()
{
    while(opcion!=3){
        mostrar_menu();
        switch(opcion)
        {
            case 1:
                agregar_contacto();
                break;
            case 2:
                buscar_contacto_tlfn();
                break;
            default:
                break;
        }
    }
    return 0;
}
 

Saludos!

Hola, bueno lo primero que tienes que incluir la libreria <stdlib.h> para usar la malloc. El problema que sugieres al intentar meter los nombres, es porque el fgets suele dar problemas.Es porque se queda en el buffer del teclado el intro y por eso te salta una opcion del menu.Deberias poner antes de cada fgets, fflush(stdin) para limpiar el buffer y que no te salte.

Gracias, funciona, pero en este post fijado (http://foro.elhacker.net/programacion_cc/lo_que_no_hay_que_hacer_en_cc_nivel_basico-t277729.0.html) dicen que fflush(stdin); es mala costumbre y que lo del buffer debería darme problemas con scanf() pero no con fgets() :-X

Bueno pues entonces si no la quieres usar puedes probar lo del post, es una solucion mejor parece ser, es comprobar si se quedo almacenado un salto de linea en esa posicion, y si es asi mete un nulo. Puedes probar, y si funciona pues mejor. Sino siempre quedara fflush(stdin) xdd

Voi a comprobar yo si me funciona con lo de poner el nulo  :)

Puedes meter un '\0' directamente. Así es como yo lo hago.

Pero lo de cambiar el salto por línea por el caracter nulo sería en el momento de elegir la opción en el menú, ¿no?

En ese caso tengo que reemplazar el scanf() por fgets() y poner la variable opcion como un arreglo de caracteres :-X

Usa readline de GNU

Las variables globales son perjudiciales para la salud, trata de evitarlas siempre que sea posible.


 
Efectivamente. La forma mas segura de realizar un ingreso de datos por parte del usuario es leyendo como texto y luego parseando y validando lo necesario.

Saludos

Bueno, ya me deshice de variables globales y de los scanf(), pero curiosamente sigue pasando el mismo error :xD

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
struct _contacto {
    char nombre[33];
    char telefono[13];
    struct _contacto *siguiente;
};
struct _contacto *primero, *ultimo;
 
void agregar_contacto()
{
     struct _contacto *nuevo;
     nuevo = (struct _contacto*)malloc(sizeof(struct _contacto));
     if(nuevo==NULL){
         printf("\nNo hay espacio suficiente.");
         return;
     }
     printf("\n\nAgregar contacto\n");
     printf("------------------");
     printf("\nNombre: ");
     fgets(nuevo->nombre,32,stdin);
     printf("\nTelefono: ");
     fgets(nuevo->telefono,12,stdin);
     nuevo->siguiente = NULL;
     if(primero==NULL){
         primero = nuevo;
         ultimo = nuevo;
     } else {
         ultimo->siguiente = nuevo;
         ultimo = nuevo;
     }
}
 
void buscar_contacto_tlfn()
{
     struct _contacto *busqueda, *actual;
     busqueda = (struct _contacto*)malloc(sizeof(struct _contacto));
     if(busqueda==NULL){
         printf("\nNo hay espacio suficiente.");
         return ;
     }
     printf("\n\nBuscar contacto por telefono\n");
     printf("----------------------------");
     printf("\nTelefono: ");
     fgets(busqueda->telefono,12,stdin);
     actual = primero;
     while(actual!=NULL){
         if(strcmp(actual->telefono,busqueda->telefono)==0){
             printf("+Nombre: %s\n", actual->nombre);
             return;
         }
         actual = actual->siguiente;
     }
     printf("No encontrado.");
}
 
 
char mostrar_menu()
{
     char opcion[3];
     printf("\n\n\n MENU\n");
     printf("======\n");
     printf("1. Agregar contacto\n");
     printf("2. Buscar contacto por telefono\n");
     printf("3. Salir\n");
     fgets(opcion,2,stdin);
     return opcion[0];
}
 
int main()
{
    char opcion='0';
    while(opcion!='3'){
        opcion = mostrar_menu();
        if(opcion=='1')
            agregar_contacto();
        else if(opcion=='2')
            buscar_contacto_tlfn();
    }
    return 0;
}
 

Saludos!

Es que de la forma que haces la lectura, fgets nunca llega a leer el salto de linea por lo tanto nada puede hacer con el.

Si tenes una cadena de 3 posiciones, y a fgets le pasas como parámetro que lea solo 2, en realidad lee 2-1. Es decir, lee 1 solo carácter, luego agrega el carácter nulo a la cadena, y el salto de linea queda deambulando por el buffer de entrada.


Cuando dije leer como texto me referia a leer y analizar una expresion para verificar si el usuario ha ingresado la opción correctamente. Lo que estas haciendo ahora es utilizar una cadena como si fuese un solo carácter y aunque es legal no es del todo correcto.

Saludos!

Lo solucioné así:

char mostrar_menu()
{
     char opcion[2];
     printf("\n\n\n MENU\n");
     printf("======\n");
     printf("1. Agregar contacto\n");
     printf("2. Buscar contacto por telefono\n");
     printf("3. Salir\n");
     fgets(opcion,3,stdin);
     return opcion[0];
}

Pero me falla cuando elijo la opción "Salir" (3). No entiendo porqué funciona en las primeras y en esta no :-X

Dices que lo que hago no es del todo correcto... ¿tú como lo solucionarías?

Saludos!

Ahora tienes una cadena de 2 posiciones y a fgets le pasas 3 como tamaño por lo tanto lee 2 caracteres, por ende es posible que la cadena te quede sin el carácter nulo.

Si el problema es solo el contenido restante en el buffer de entrada, en Windows se puede tirar de la API para solucionarlo fácilmente, pero para no salir del estándar puedes hacer algo como esto:

void CleanStdin()
{
while(getchar() != '\n');
}
 

Aunque no debería hacer falta puesto que el salto de linea debería quedar perfectamente en la cadena pasada a fgets siempre que le dejes espacio. Luego puedes reemplazarlo fácilmente por un carácter nulo.

Con el tema de las lecturas, hay miles de formas de leer una opción. Usualmente cuando son opciones simples con fgets + sscanf basta y sobra:

if(fgets(Buff, sizeof Buff, stdin))
 sscanf(Buff, "%d", &Opt);
 

---

El problema del buffer de entrada en Windows esta muy tratado en el foro. Hoy ya vi 3 posts con la misma duda asi que mi recomendación es que utilicen el buscador del foro, ya que hay hilos enteros solo sobre este tema. Tambien con Google pueden buscar algo como:


o


Saludos

Bueno, gracias por todo, si al final era más simple de lo que parecía :rolleyes:

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
struct _contacto {
    char nombre[33];
    char telefono[13];
    struct _contacto *siguiente;
};
struct _contacto *primero, *ultimo;
 
void agregar_contacto()
{
     struct _contacto *nuevo;
     nuevo = (struct _contacto*)malloc(sizeof(struct _contacto));
     if(nuevo==NULL){
         printf("\nNo hay espacio suficiente.");
         return;
     }
     printf("\n\nAgregar contacto\n");
     printf("------------------");
     printf("\nNombre: ");
     fgets(nuevo->nombre,32,stdin);
     printf("\nTelefono: ");
     fgets(nuevo->telefono,12,stdin);
     nuevo->siguiente = NULL;
     if(primero==NULL){
         primero = nuevo;
         ultimo = nuevo;
     } else {
         ultimo->siguiente = nuevo;
         ultimo = nuevo;
     }
}
 
void buscar_contacto_tlfn()
{
     struct _contacto *busqueda, *actual;
     busqueda = (struct _contacto*)malloc(sizeof(struct _contacto));
     if(busqueda==NULL){
         printf("\nNo hay espacio suficiente.");
         return ;
     }
     printf("\n\nBuscar contacto por telefono\n");
     printf("----------------------------");
     printf("\nTelefono: ");
     fgets(busqueda->telefono,12,stdin);
     actual = primero;
     while(actual!=NULL){
         if(strcmp(actual->telefono,busqueda->telefono)==0){
             printf("+Nombre: %s\n", actual->nombre);
             return;
         }
         actual = actual->siguiente;
     }
     printf("No encontrado.");
}
 
 
void mostrar_menu()
{
     printf("\n\n\n MENU\n");
     printf("======\n");
     printf("1. Agregar contacto\n");
     printf("2. Buscar contacto por telefono\n");
     printf("3. Salir\n");
}
 
int main()
{
    int opcion = 0;
    char bufferOpcion[5];
    while(opcion!=3){
        mostrar_menu();
        fgets(bufferOpcion,sizeof(bufferOpcion),stdin);
        sscanf(bufferOpcion, "%d", &opcion);
        switch(opcion){
            case 1:
                agregar_contacto();
                break;
            case 2:
                buscar_contacto_tlfn();
                break;
            default:
                break;
        }
    }
    return 0;
}
 

Saludos!