警告：从不兼容的指针类型返回 [-Wincompatible-pointer-types]|

Question

我正在尝试从 function 指针返回一个数组，代码工作但在 C 中显示“指针类型不兼容”的警告。 我想返回一个数组，它已经是动态分配的。 有人可以告诉我问题和解决方案吗

#include <stdlib.h>
#include<stdio.h>
unsigned short *reverse_seq(unsigned short num)
{
    if(num==0) return NULL;
    int size=num+1;
    int* numbers=(int *) malloc(size*sizeof(int));
    for(int i=0;i<num;i++){
        numbers[i]=num-i;
    }
    for(int i=0;i<num;i++){
        printf("%d ",numbers[i]);
    }
    return numbers;
}
int main(void)
{
    int num=5;
    reverse_seq(num);
    return 0;
}

有人可以给我这个警告的解决方案吗？

Answer 1

• 您的 function 被声明为返回unsigned short *但您为int分配空间并尝试返回int* 。 我假设您想将unsigned short存储在分配的 memory 中。
• 当您返回指向动态分配的 memory 的指针时，您应该始终将该指针分配给一个变量，以便您可以free分配的 memory。

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

unsigned short *reverse_seq(unsigned short num) {
    if (num == 0) return NULL;

    // corrected allocation (there's no need for num + 1 elements either):
    unsigned short *numbers = malloc(num * sizeof *numbers);

    if(numbers) {                       // check that allocation worked
        for (int i = 0; i < num; i++) {
            numbers[i] = num - i;
        }
    }

    // printing moved to `main` to make use of the data there

    return numbers;
}

int main(void) {
    unsigned short num = 5; // same type as `reverse_seq` wants
    
    unsigned short *numbers = reverse_seq(num);
    if(numbers) {                        // again, check that allocation worked
        for (int i = 0; i < num; i++) {
            printf("%d ", numbers[i]);
        }
        
        free(numbers);                   // free the memory
    }
}

Answer 2

尝试

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

unsigned short *reverse_seq(unsigned short num) {
    if (num == 0) return NULL;
    int size = num + 1;
    unsigned short *numbers = malloc((size_t) size * sizeof(unsigned short));  // THIS IS THE IMPORTANT CHANGE
    for (int i = 0; i < num; i++) {
        numbers[i] = (unsigned short) (num - i);
    }
    for (int i = 0; i < num; i++) {
        printf("%d ", numbers[i]);
    }
    return numbers;
}

int main(void) {
    int num = 5;
    unsigned short *res = reverse_seq((unsigned short) num);
    free(res);
    return 0;
}

这将解决原始警告的主要问题。

重要的是分配您返回的正确类型的 memory。 问题是当请求指向unsigned short*的指针时，您正在返回一个int* 。 不这样做会对稍后访问 memory 产生影响，因为从返回的指针访问的 memory 布局与您分配 memory 序列的方式不匹配。

Answer 3

如果你想返回一个unsigned short数组，你应该分配一个unsigned short数组，而不是int s：

unsigned short* reverse_seq(unsigned short num)
{
    if (num == 0)
        return NULL;

    unsigned short* numbers = malloc(num * sizeof(unsigned short));
    if (numbers) {
        for (unsigned short i = 0; i < num; i++) {
            numbers[i] = num - i;
        }
    }
    return numbers;
}

Answer 4

您正在分配一个整数数组，但您正在返回一个短裤数组（通常是整数大小的一半）。 只需返回一个指向int的指针，如下所示：

int *reverse_seq(unsigned short num) {
    /* ... */

或分配一个短裤数组，如：

    unsigned short *numbers = malloc(num * sizeof *numbers);  /* and please, ***never*** cast the result of malloc() */