C中的优先级队列实现 - 将字符更改为整数

Question

我目前正在做一个需要 C 语言优先级队列的项目。我正在使用来自Rosettacode.org 的代码。

我正在尝试修改优先级队列，使其采用整数而不是字符。我尝试更改所有变量类型，但出现以下错误。

test.c:62:16: 警告：不兼容的整数到指针转换传递'int' 到 'int *' 类型的参数 [-Wint-conversion]

当它是一个 char 时这很好用，但当它是一个 int 时它会突然停止。为什么会这样？这是我的代码：

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

typedef struct {
    int priority;
    int *data;
} node_t;

typedef struct {
    node_t *nodes;
    int len;
    int size;
} heap_t;

void push (heap_t *h, int priority, int *data) {
    if (h->len + 1 >= h->size) {
        h->size = h->size ? h->size * 2 : 4;
        h->nodes = (node_t *)realloc(h->nodes, h->size * sizeof (node_t));
    }
    int i = h->len + 1;
    int j = i / 2;
    while (i > 1 && h->nodes[j].priority > priority) {
        h->nodes[i] = h->nodes[j];
        i = j;
        j = j / 2;
    }
    h->nodes[i].priority = priority;
    h->nodes[i].data = data;
    h->len++;
}

int *pop (heap_t *h) {
    int i, j, k;
    if (!h->len) {
        return NULL;
    }
    int *data = h->nodes[1].data;
    h->nodes[1] = h->nodes[h->len];
    h->len--;
    i = 1;
    while (1) {
        k = i;
        j = 2 * i;
        if (j <= h->len && h->nodes[j].priority < h->nodes[k].priority) {
            k = j;
        }
        if (j + 1 <= h->len && h->nodes[j + 1].priority < h->nodes[k].priority) {
            k = j + 1;
        }
        if (k == i) {
            break;
        }
        h->nodes[i] = h->nodes[k];
        i = k;
    }
    h->nodes[i] = h->nodes[h->len + 1];
    return data;
}

int main () {
    heap_t *h = (heap_t *)calloc(1, sizeof (heap_t));
    push(h, 3, 3);
    push(h, 4, 4);
    push(h, 5, 5);
    push(h, 1, 1);
    push(h, 2, 2);
    int i;
    for (i = 0; i < 5; i++) {
        printf("%d\n", pop(h));
    }
    return 0;
}

Answer 1

在您的push() 函数签名中，第三个参数的类型为int *，但您在调用它时发送了int。指向整数转换的指针是一种特定于实现的行为，很有可能导致undefined behavior。

在我看来，你不需要 data 是一个指针，一个简单的 int 在任何地方都应该可以完成这项工作。

Answer 2

所以以前你的函数原型（大概）是void push (heap_t *h, char priority, char *data)，所以作为第三个参数传递的单个字符串（"c"，而不是'c'）参数可以转换为char *，编译器不会不要抱怨。

但是，既然您已将所有内容都切换到 int，编译器就有话要说。

另一个注意事项：特别是，如果您传递单个字符串，例如"c"，编译器将保持沉默。 'c' 会给出与您现在的错误类似的错误。

您似乎误解了原始代码是使用字符作为数据，而实际上它使用的是字符串。您将所有chars 切换到ints，现在您拥有int *，您应该拥有int。