使用C中的函数进行输入验证

Question

我正在通过函数验证用户输入。

这是我的代码：

#include <stdio.h>

void get_income(double *pschool, double *pincome);
double get_double(void);

main()
{
    get_income(&pschool, pincome);
}

void get_income(double *pschool, double *pincome) {
    int tuition, supplies;

    printf("Enter tuition:");
    scanf("%d", &tuition);
    printf("Enter supplies:")
    scanf("%d", &supplies);

    *pSchool = tuition + supplies;
    return;
}

double get_double(void) {
    // validation code should go in here 
}

我需要验证用户提供的学费和用品的值，以确保它们都是0或更大且不为空。不应接受负值或除数字以外的任何内容。

验证代码应该在get\u double函数中，我似乎不知道如何验证驻留在单独函数中的内容。请寻求指导。

Answer 1

该代码存在许多问题，包括：

• 不检查scanf（）函数查找数据
• main（）的声明不正确
• 将未使用的指针传递到get\u income（）
• 不从检查功能返回状态
• 未报告输入的数据
• 未在main（）中声明变量
• 函数名称与目的不匹配
• 读取浮点数存储的整数
• 等等

“读取一行并用sscanf（）解析它”可能比“让scanf（）读取数据”做得更好；更容易处理错误报告。

这些观察结果导致：

#include <stdio.h>

int get_tuition_and_supplies(double *ptuition, double *psupplies);
int get_double(double *pvalue);

int main(void)
{
    double tuition, supplies;
    if (get_tuition_and_supplies(&tuition, &supplies) == 0)
        printf("Tuition: %.2f; supplies %.2f\n", tuition, supplies);
    return(0);
}

int get_tuition_and_supplies(double *ptuition, double *psupplies)
{
    int rc = -1;    // Failure
    printf("Enter tuition: ");
    if (get_double(ptuition) == 0)
    {
        printf("Enter supplies: ");
        if (get_double(psupplies) == 0)
            rc = 0;
    }
    return rc;
}

int get_double(double *pvalue)
{
    char buffer[4096];
    int rc = -1;
    if (fgets(buffer, sizeof(buffer), stdin) == 0)
        fprintf(stderr, "EOF (or error) reading data\n");
    else if (sscanf(buffer, "%lf", pvalue) != 1)
        fprintf(stderr, "Did not find a number in input (%.32s)\n", buffer);
    /* Could check for no extracharacters (blanks allowed) up to newline */
    else if (*pvalue < 0.0)
        fprintf(stderr, "Value (%g) may not be negative\n", *pvalue);
    else
        rc = 0;
    return rc;
}

“将返回代码（rc）设置为失败，直到证明成功”的习惯用法是处理验证的一种有效技术。最好在get_double（）函数中包含提示，这样它的签名就变成了：

int get_double(const char *prompt, double *pvalue);

这允许您将get_tuition_and_supplies（）中的代码简化为：

int rc = -1;    // Failure
if (get_double("Enter tuition: ", ptuition) == 0 &&
    get_double("Enter supplies: ", psupplies) == 0)
    rc = 0;
return rc;

这可以灵活地扩展到处理多个值，而不仅仅是两个值。

Answer 2

对我来说，我真的不喜欢使用scanf，因为这样。以下是我将如何解决您的问题：

1. 分配字符数组。充当将写入输入的缓冲区。大小一般需要反映用途。也就是说，如果我只接收一个不大于1000（整数类型）的数字，那么我只会分配6个字符。最后两个是'\n'和'\0'。
2. 将数组中最后一个元素之前的一个初始化为'\n'以检测溢出。
3. 使用fegts（）函数接收输入。
4. 通过检查我初始化为'\n'的最后一个元素之前的一个是否已更改来检查溢出。如果是这样，我报告溢出错误，否则继续执行。
5. 使用atof（）或atoi（）函数分别转换为双精度或整数值。

我想知道我的方法是否包含任何类型的bug，但据我所知，它工作得很好。

Answer 3

在我看来，get\u double（）的目的似乎是处理用户输入。这就是我要做的：

#include <stdio.h>

int get_double(double *in)
{
    scanf("%d", in);
    if (*in < 0) {
        printf("error message");
        return 1;
    }
    return 0;
}

void get_income(double ptuition, double psupplies, double *pschool)
{
    *pschool = ptuition + psupplies;
}

int main(void)
{
    double ptuition, psupplies, pschool;

    printf("Enter tuition: ");
    if (get_double(&ptuition)) {
        return 1;
    }

    printf("\nEnter supplies: ");
    if (get_double(&psupplies)) {
        return 1;
    }

    get_income(ptuition, psupplies, &pschool);
    printf("Total school cost: %f", pschool);
    return 0;
}