1

我的C代码有问题，希望你能帮助我。该计划是关于制作一本基础书籍“数据库”。当我运行以下代码（在 Xcode 中）时，我不知道为什么会跳过以下句子：

得到（名词[i]）；

如果我从菜单中选择选项 1，它会在终端上直接打印以下内容：

Bienvenido al catalogo de libros。

Catalogo de tarjetas: 1. Introducir 2. Buscar por autor 3. Buscar por titulo 4. Salir

Elija opcion:1 警告：此程序使用gets()，这是不安全的。

Introduzca el nombre del libro:Introduzca el autor del libro:

好的，所以我测试了我的 scanf("%d", &opcion); 使用 printf("%d", opcion); 之后证明 scanf 正确读取了我的输入。令人惊讶的是，它读取了我正确介绍的选项。此外，我尝试在任何部分都没有“\n”的情况下运行该程序，以查看gets(nombre[i]) 是否有效，但它仍然会被跳转...

有任何想法吗？

这是完整的代码（不长）：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <math.h>
#include <ctype.h>

#define MAX 100

char nombre[MAX][20];
char autor[MAX][20];
char edit[MAX][20];
char buscar[20];
char buscar_t[20];
char buscar_a[20];


int opcion,i,j,k,l;

void menu(void);
void intro(void);
void buscar_autor(void);
void buscar_tit(void);
void salir(void);

void main(void)
{
    printf("Bienvenido al catalogo de libros. \n");
    menu();
}




void menu(void)
{
    printf("\n Catalogo de tarjetas:");
    printf("\n 1. Introducir");
    printf("\n 2. Buscar por autor");
    printf("\n 3. Buscar por titulo");
    printf("\n 4. Salir");

    printf("\n Elija opcion:");
    scanf("%d", &opcion);

    switch (opcion) {
        case 1:
            intro();
            break;
        case 2:
            buscar_autor();
            break;
        case 3:
            buscar_tit();
            break;
        case 4:
            salir();
            break;

    }


}

void intro(void)
{
        for (i=0; i<MAX; i++) 
        {
            printf("Introduzca el nombre del libro:");
            gets(nombre[i]);

            if (!strcmp(nombre[i],"salir")) 
            {
                break;
            }

            printf("Introduzca el autor del libro:");
            gets(autor[i]);
            printf("Introduzca la editorial del libro:");
            gets(edit[i]);
        }

    menu();

}

void buscar_tit(void)
{
    printf("Introduzca el titulo del libro que quiera buscar:");
    gets(buscar_t);

    for (j=0; j<MAX+1; j++) 
    {
        if (!strcmp(nombre[j],buscar_t)) 
        {
            printf("El libro se ha encontrado, el titulo es %s. ", nombre[j]);
            break;
        }
        if (j=MAX) 
        {
            printf("El libro no se ha encontrado.");
            break;
        }

    }

}

void buscar_autor(void)
{
    printf("Introduzca el autor del libro que quiera buscar:");
    gets(buscar_a);

    for (k=0; k<MAX+1; k++) 
    {
        if (!strcmp(autor[k],buscar_a)) 
        {
            printf("El libro se ha encontrado, el titulo es %s. ", nombre[k]);
            break;
        }
        if (k=MAX) 
        {
            printf("El autor no se ha encontrado.");
            break;
        }
    }
}

void salir(void)
{
    printf("Muchisimas gracias por usar el catalogo de libros. \n");
}

希望你能帮我找出错误。

多谢你们。

4

1 回答 1

1

请避免使用gets。使用fgets将标准输入指定为流。 

 char *fgets(char *s, int size, FILE *stream);

在您的代码中修改此功能： 

void intro(void)
{
        getchar(); //added this to avoid escaping at early stage of loop
        for (i=0; i<MAX; i++)
        {
            printf("Introduzca el nombre del libro %d : ",i+1); //modified this statement to know which iteration is going on by printing the loop counter value
            fgets(nombre[i],sizeof(nombre[i]),stdin); //  replaced  gets(nombre[i]);

            if (!strcmp(nombre[i],"salir"))
            {
                break;
            }

            printf("Introduzca el autor del libro: %d : ",i+1);
            gets(autor[i]);  //here also use fgets
            printf("Introduzca la editorial del libro %d : ",i+1);
            gets(edit[i]);   //here also use fgets 
        }

    menu();

}

小建议：

为了使调试容易，更改#define MAX 100为#define MAX 5

看到这个获取的安全替代方案

在您的情况下，如果问题与 fgets 重复，只需添加getchar();
在 fgets 之前，也可以看到这个如何在 fgets 溢出后清除输入缓冲区？

来自gets的手册页

BUGS 永远不要使用gets()。因为如果事先不知道数据，就不可能知道gets() 将读取多少个字符，并且因为gets() 将继续存储超过缓冲区末尾的字符，所以使用它是极其危险的。它已被用来破坏计算机安全。请改用 fgets()。

于 2013-08-28T12:00:38.377 回答