C：错误：“格式'％f'需要类型为'double'的参数”

Question

我回到C上，只是做了一些小练习，而我偶然发现了这一小题，而无法弄清为什么会这样？

该程序采用9字长的代码示例“ 011112222”。 第一个数字是操作代码（0表示+，1表示-，2表示*，3表示/），接下来的4个数字是第一个数字，其他4个是其他数字，因此对于“ 011112222”，结果应为1111 + 2222。

但是我得到一个恒定的错误：在函数“ main”中：

zadatak.c:54:14: warning: format ‘%f’ expects argument of type ‘double’, but argument 2 has type ‘float *’ [-Wformat=]
     printf("%f", result);

如果有人可以阐明这个问题，那就太好了，我将不胜感激。

#include<stdio.h>
    #include <unistd.h>
    #include <stdlib.h>
    #include <string.h>

    float* Calculate(char* buffer)
    {
        int number1, number2;

        char *firstPart = (char *)malloc(strlen(buffer));
        char *secondPart = (char *)malloc(strlen(buffer));

        strncpy(firstPart, buffer+1, 4);
        strncpy(secondPart, buffer+5, 8);

        number1 = atoi(firstPart);
        number2 = atoi(secondPart);

        float *result;

        if( buffer[0] == '0')
        {
            *result = (float)number1 + (float)number2;
        }
        else if(buffer[0] == '1')
        {
            *result = (float)number1 - (float)number2;
        }
        else if(buffer[0] == '2')
        {
            *result = (float)number1 * (float)number2;
        }
        else if(buffer[0] == '3')
        {
            *result = (float)number1 / (float)number2;
        }

        return result;
    }

    int main()
    {
        char buffer[20];
        printf("Insert code word: ");
        scanf("%s", buffer);

        float *result = Calculate(buffer);
        printf("%f", result);

        return 1;
    }

Answer 1

在您的printf调用中：

printf("%f", result);

%f格式说明符期望使用double （或会自动转换的float ），但是您正在向其传递指向 float的指针 。 您需要取消引用指针以获得float值：

printf("%f", *result);

但是 ，您还有另一个问题。

在Calculate内部，您定义了result （从函数返回的内容），它是指向float的指针，但是您从未将其设置为指向任何地方。 因此，无论何时尝试取消引用该指针，您都在读取未初始化的值并将其用作有效指针。 这样做会引起未定义的行为 。

没有理由使用指针float在你的代码，因此改变两者result在main和result在Calculate从float *到float ，并改变Calculate返回一个float 。

另外， strncpy函数并不总是以null终止目标字符串，并且malloc返回未初始化的内存，因此atoi在读取字符串时可以读取分配的内存末尾。 您可以使用calloc而不是malloc来解决此问题，后者将返回初始化为全0的内存。 另外，在使用完这些缓冲区后，请确保free它们。

float Calculate(char* buffer)
{
    int number1, number2;

    char *firstPart = calloc(strlen(buffer), 1);
    char *secondPart = calloc(strlen(buffer), 1);

    strncpy(firstPart, buffer+1, 4);
    strncpy(secondPart, buffer+5, 8);

    number1 = atoi(firstPart);
    number2 = atoi(secondPart);

    free(firstPart);
    free(secondPart);

    float result;

    if( buffer[0] == '0')
    {
        result = (float)number1 + (float)number2;
    }
    else if(buffer[0] == '1')
    {
        result = (float)number1 - (float)number2;
    }
    else if(buffer[0] == '2')
    {
        result = (float)number1 * (float)number2;
    }
    else if(buffer[0] == '3')
    {
        result = (float)number1 / (float)number2;
    }

    return result;
}

int main()
{
    char buffer[20];
    printf("Insert code word: ");
    scanf("%s", buffer);

    float result = Calculate(buffer);
    printf("%f", result);

    return 1;
}

Answer 2

您的代码需要进行许多验证和更改才能可靠，但是我进行了一些更改才能达到您的最初目的。

#include <stdio.h>
#include <unistd.h>
#include <stdlib.h>
#include <string.h>

float Calculate(char* buffer)
{
    int number1, number2;

    char *firstPart = (char *)malloc(strlen(buffer));
    char *secondPart = (char *)malloc(strlen(buffer));
    float result;

    strncpy(firstPart, buffer + 1, 4);
    strncpy(secondPart, buffer + 5, 4);
    number1 = atoi(firstPart);
    number2 = atoi(secondPart);


    if( buffer[0] == '0')
    {
        result = (float)number1 + (float)number2;
    }
    else if(buffer[0] == '1')
    {
        result = (float)number1 - (float)number2;
    }
    else if(buffer[0] == '2')
    {
        result = (float)number1 * (float)number2;
    }
    else if(buffer[0] == '3')
    {
        result = (float)number1 / (float)number2;
    }

    free(firstPart);
    free(secondPart);

    return result;
}

int main()
{
    char buffer[20];
    printf("Insert code word: ");
    scanf("%s", buffer);

    float result = Calculate(buffer);
    printf("%f", result);

    return 1;
}