C 输入返回

Question

我正在尝试开发一个员工管理系统，该系统将员工信息作为输入并相应地打印 output。

结构：

struct employee
{
    char empId[10];
    char name[20];
    char empType[10];
    int dd, mm, yyyy;
};

检查字符串是否只包含整数的程序：

void checkValidId(char *id)
{
    int count = 0, i;

    for(i = 0; i < strlen(id); i++)
    {
        if(id[i] >= '0' && id[i] <= '9')
        {
            count++;
        }
    }
    if(count == strlen(id))
        // continue executing next statements of input() function
    else
        // take the empId input again
}

Function 接受输入：

struct employee input()
{
    struct employee e;

    printf("Enter Employee ID: ");
    scanf("%s", &e.empId); 
    checkValidId(e.empId);

    // next statements ....
}

在这里，我试图检查 empId 的字符串输入是否仅包含由 checkValidId() 完成的整数。 如果是，则程序继续执行下一条语句。 否则我想再次输入 empId。 怎么做。

请求帮助！

谢谢！

Answer 1

包括 header <ctype.h> ，并使用isdigit来测试字符是否在集合'0' ... '9'中。

包含 header <stdbool.h> ，并将checkValidId的签名更改为

bool checkValidId(const char *id)

为了向调用 function 指示结果。

在checkValidId中，遍历字符串的每个字符。 如果当前字符不是数字，则立即从 function 返回false 。如果循环结束，则所有字符都必须是数字，因此可以返回true 。

请注意，没有理由在这里调用strlen 。 简单地循环直到当前字符是空终止字节。

---

&e.empId的类型为char (*)[10] ，即指向 array-of-10-char的指针。 scanf说明符"%s"需要类型char *或pointer-to-char 。 当传递给 function 时，数组将衰减为指向其第一个元素的指针，因此此处调用scanf的“正确”方法是scanf("%s", e.empId); .

也就是说，您必须检查scanf的返回值是否为预期的转换次数，否则您将对不确定数据进行操作。

此外，未绑定"%s"与gets function 一样危险，因为它不知道何时停止读取数据，并且很容易溢出提供的缓冲区。 您必须提供最大字段宽度以防止scanf读取太多字符。 这最多应该是缓冲区的大小减 1，为空终止字节留出空间。

安全使用scanf的示例：

if (1 != scanf("%9s", e.empId)) {
    /* handle input stream error */
}

请注意，当scanf无法执行转换时，问题数据会留在 stream 中。使用scanf从错误输入中恢复非常困难，因此通常建议采用基于行的用户输入方法。 这可以通过fgets来完成。

如果缓冲区中有空间， fgets会在读取时包含换行符。 有关strcspn的示例用法，请参阅从fgets()输入中删除尾随换行符，它也可用作获取输入长度的方法。

---

要反复要求用户输入，请使用无限循环。 仅当用户正确输入请求的数据或发生严重错误时才从循环中break 。

这是一个示例程序，使用讨论的方法：

#include <ctype.h>
#include <stdbool.h>
#include <stdio.h>
#include <string.h>

struct employee {
    char empId[10];
    /* ... */
};

bool checkValidId(const char *id)
{
    for (size_t i = 0; id[i] != '\0'; i++)
        if (!isdigit((unsigned char) id[i]))
            return false;

    return true;
}

bool getEmployee(struct employee *e, FILE *stream)
{
    char buffer[256];

    while (1) {
        printf("Enter employee ID: ");

        if (!fgets(buffer, sizeof buffer, stream))
            return false;

        size_t length = strcspn(buffer, "\r\n");
        /* remove the newline */
        buffer[length] = '\0';

        if (!length)
            fprintf(stderr, "Invalid ID. Zero length.\n");
        else if (length >= sizeof e->empId)
            fprintf(stderr, "Invalid ID. Too long.\n");
        else if (!checkValidId(buffer))
            fprintf(stderr, "Invalid ID. Contains non-digit character(s).\n");
        else {
            strcpy(e->empId, buffer);
            break;
        }

        puts("Try again.");
    }

    return true;
}

int main(void)
{
    struct employee emp;

    if (!getEmployee(&emp, stdin))
        return 1;

    printf("ID: <%s>\n", emp.empId);
}

与这个程序交互：

Enter employee ID: foobar
Invalid ID. Contains non-digit character(s).
Try again.
Enter employee ID: 
Invalid ID. Zero length.
Try again.
Enter employee ID: 1234567890
Invalid ID. Too long.
Try again.
Enter employee ID: 42
ID: <42>

Answer 2

1. checkValidId应该返回int以便它可以告诉调用者该id是否有效[或无效]。
2. 如果checkValidId的返回值指示一个错误的 id，调用者（例如input ）应该循环。
3. 根本不需要使用strlen 。
4. 无需比较长度。 可以使用更简单的算法。
5. 如果检测到无效字符，我们可以停止循环

这是重构的代码：

struct employee {
    char empId[10];
    char name[20];
    char empType[10];
    int dd, mm, yyyy;
};

// Program to check if the string contains only integers:
// RETURNS: 1=input valid, 0=retry
int
checkValidId(char *id)
{
    int valid = 0;

    for (int i = 0; id[i] != 0; i++) {
        valid = (id[i] >= '0') && (id[i] <= '9');
        if (! valid)
            break;
    }

    return valid;
}

// Function to take input:
struct employee
input(void)
{
    struct employee e;

    while (1) {
        printf("Enter Employee ID: ");
        scanf("%s", &e.empId);
        if (checkValidId(e.empId))
            break;
    }

    // next statements ....
}