来自返回数组最大值的函数的C分段错误

Question

我只有一个函数可以找出整数数组的最大值，但我得到了一个 分段错误，因为编译器没有显示该行，所以我找不到它的错误。

这是我的 C 代码：

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

//Funktion ermittelt den größten Wert eines Arrays
int groesstesElement(int **arrayPointer){
    int max = 0;
    for (int i = 0; i < 3; i++) {
        if (*arrayPointer[i]>max) {
            max = *arrayPointer[i];
        }
    }
    return max;
}


int main (int argc, char **argv) {
    int array[4]={1,2,3,4};
    int *ptr = array;
    int z = groesstesElement(&ptr);
    printf("%d\n", z);

    return EXIT_SUCCESS;
}

我使用 macOS 和 VSC。

Answer 1

在 C 中，数组索引 [] 的优先级高于指针取消引用 *：https://en.cppreference.com/w/c/language/operator_precedence

一些括号修复了段错误。

if ((*arrayPointer)[i]>max) {
    max = (*arrayPointer)[i];
}

Answer 2

由于运算符优先级，与

*arrayPointer[i]

你在告诉你的程序：

获取数组arrayPointer的第第i个元素并取消引用它。

但是arrayPointer 是一个指向int * 的指针，所以你得到的只是ptr 的地址（从main 定义的int ** 指针）带有一个偏移量。当您最终取消引用它时，您很可能访问了无效地址，从而导致分段错误。

正如主要答案已经建议的那样，修复是使用括号完成的，以便按照您想要的顺序应用运算符：

(*arrayPointer)[i]

代码中的次要问题

即使经过上述更正，程序不会再崩溃，但它不会工作。使用循环搜索数组

for (i = 0; i < 3; i++) 

您不会在搜索中包含最后一项（索引 3），而会精确跳过在您的示例中包含最大值的索引。您的意思可能是for (i = 0; i < 4; i++) 或for (i = 0; i <= 3; i++)。

无论如何，在代码中使用幻数（在本例中为数组的维度）被认为是不好的做法。更好的解决方案是使用#define，但更好的解决方案是将数组的大小传递给函数：

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

#define ARR_SIZE 4

//Funktion ermittelt den größten Wert eines Arrays
int groesstesElement(int *arrayPointer, unsigned int arrSize){
    int max = 0, i;
    for (i = 0; i < arrSize; i++) {
        if (arrayPointer[i]>max) {
            max = arrayPointer[i];
        }
    }
    return max;
}


int main (int argc, char **argv) {
    int array[ARR_SIZE]={1,2,3,4};
    int z = groesstesElement(array, ARR_SIZE);
    printf("%d\n", z);

    return EXIT_SUCCESS;
}

请注意如何不再需要使用双指针。