在 C 上避免此错误的正确方法是什么

Question

我有这个代码，我想知道为什么我的代码会跳过扫描...

#include <stdio.h>

char nombre[20];
int tipo;
int edad;
int dias;

int repetir=0;

int main()
{

   while (repetir==0){
        int error=0;
        do{


            printf("Diga su Nombre: ");
            scanf("%19[^\n]", nombre);
            printf("Diga el tipo de Sala: ");
            scanf("%d", & tipo);
            printf("Diga la edad del Paciente: ");
            scanf("%d", & edad);
            printf("Dias de Hospitalizacion: ");
            scanf("%d", & dias);

            if ((tipo>4) || (tipo<0) || (edad<0) || (dias <0) ){
                printf("Eror al ingresar los datos");
            } else {
                error = 1;
            }
        }
        while(error==0);

        printf("¿Algun otro paciente? SI=0 NO=1");
        scanf("%d", &repetir);
    }

    return(0);
}

这只是一个简单的代码，但是当我运行代码时会出现这样的情况：

 Diga su Nombre: (Name with spaces)
 Diga el Tipo de Sala (Integer Number)
 Diga la edad del Paciente (Integer Number)
 Dias de Hospitalizacion (Integer Number)

 //Check For errors

 ¿Algun otro paciente? SI=0 NO=1 (Pressing 0 To repeat)

 //Repeat

 Diga su Nombre: Diga el Tipo de Sala
 ....
 ....

代码不再向我询问字符串数据，我真的不知道在这种情况下我必须做什么......非常感谢您的帮助！

更新

这是我尝试过的......但只是冻结

            // .....
            scanf("%d", & dias);
            //Dont really know what to do here :S
            int c; while ( (c = getchar()) == '\n' && c != EOF ) { }

            if ((tipo>4) || (tipo<0) || (edad<0) || (dias <0) ){

Answer 1

在main() 内部，char nombre[]= {0}; 行声明了一个大小为1 的数组。然后你使用scanf 读取它，这会溢出缓冲区导致未定义的行为。

数组在 C 中不会自动增长，它们有一个固定的大小，你必须在定义它们时指定它。 （由于您将大小留空，它通过计算初始化器的数量来计算大小）。

修复示例：

char nombre[50];
// ...
scanf("%49[^\n]%*c", nombre);

注意。全局变量 char nombre[]; 是与 main 的 nombre 不同的变量，您目前不使用它，所以我建议删除两者中的一个（并给剩余的一个大小，例如 @ 987654329@)。

---

还有一个问题，如果你多次执行这个循环。当您使用 scanf("%d"... 时，它会提取整数的字符，但不会提取换行符。所以下次当你来到 scanf("%[... 时，它会在那里读取换行符。

要解决这个问题，您需要在循环结束时手动使用换行符，例如

scanf("%d", &repetir);

while ( getchar() != '\n' ) { }

也可能在scanf("%d", & dias); 之后执行此操作。从技术上讲，它应该是int c; while ( (c = getchar()) != '\n' && c != EOF ) { }，但在交互式程序中这不是问题。

另外，%*c 是多余的，你可以直接改成scanf("%49[^\n]"。正确地说，此时您也可以丢弃该行的其余部分。

Answer 2

要将nombre 作为字符串（char 数组）获取，请将 scanf 语句替换为：

scanf("%s", nombre);