时间戳

首页 > 解决方案 > 在C中的循环中增加整数指针

c - 在C中的循环中增加整数指针

问题描述

我的原始代码按从低到大的顺序显示队列中的节点：

void display (queue *q) {
node *ptr = q->front;
int i = 0;
int size = q->size;
while (i <= size) {
    while (ptr->id != i) {
        ptr = ptr->next;
    }
    if (i == 0) {
        printf("%d ", ptr->id);
        printf("%d\n", ptr->running);
    }
    else {
        printf("%d ", ptr->id);
    }
    i++;
    ptr = q->front;
}

}

一直在产生分段错误（核心转储）错误。我有 malloc 正在比较的两个变量，并且此错误已得到修复。

void display (queue *q) {
node *ptr = malloc(10);
ptr = q->front;
int *i = NULL;
i = malloc(sizeof(int));
*i = 0;
int size = q->size;
while(*i <= size){
    while (ptr->id != *i) {
        ptr = ptr->next;
    }
    if (*i == 0) {
        printf("%d %d\n", ptr->id, ptr->running);
    }
    else {
        printf("%d %d %d %d\n", ptr->id, ptr->running, ptr->ready, ptr->blocked);
    }
    i = i + 1 * (sizeof(char));
    ptr = q->front;
}

}

但是现在这不会产生我想要的输出。我想增加 i 指针，以便它可以与我的原始代码相同。

这非常令人沮丧，任何帮助将不胜感激！

标签： cpointerssegmentation-faultmalloc

解决方案

如果我正确阅读了您的第一个代码清单，那么您至少需要考虑一件重要的事情。您似乎在这里有一个链接列表，并且您正在使用ptr = ptr->next. 这意味着您需要知道何时停止。建立链表的一种常见方式是链表中的最后一项的next值为NULL。然后一次处理一个节点，完成一个节点后，检查next值是否为NULL：如果不是，则可以继续下一个节点，如果是则NULL停止。

在这里，您没有进行这样的检查，因此您需要另一种方法来确保您知道何时停止。您似乎在做的是获取价值q->size并使用它来通知您链接列表中有多少项目。所以首先要考虑的是你对这个值是正确的有多大信心。例如，如果构建列表的代码只将两个项目放入列表中，赌注设置size为三个，那么您最终会掉出列表的末尾，并且不太可能出现分段错误。

但还有比这更重要的事情。即使您从 列表中获得正确数量的项目q->size，您也会将循环变量i与size以下内容进行比较：

int i = 0;
while (i <= size) {
  ⋮
}

这将使用i值 [0, 1, ... size] 进行循环，实际上是size+ 1 倍。如果你想准确地循环size，你需要一个测试，i < size而不是i <= size. 这是一个常见的错误，通常称为一个错误。

不幸的是，您的第二个列表使事情复杂化，而不是使它们变得更好。回到你的第一个，看看你能不能解决我在这里提到的问题。

推荐阅读