c++ - segmentation fault错误如何解决C++

Question

我正在编写一个获取整数数组及其逻辑大小的程序。调用时，它会创建一个新数组，其中仅包含数组中的正数。

现在，为了做到这一点，我需要编写一个带有以下参数的 void 类型函数:

(int* arr, int arrSize, int** outPosArrPtr, int* outPosArrSizePTR)

我应该使用指针 int** outPosArrPtr 来更新包含正数的数组的基地址，并使用指针 outPosArrSizePtr 来更新数组的基址逻辑大小。

现在，当我在 xcode 编译器上运行我的代码时，逻辑大小会更新为一个非常大的数字。因此，当我尝试使用在线 gdb 编译器运行该程序时，我收到了错误“段错误”。

通过阅读段错误的含义，我了解到这意味着我正在尝试访问“不属于我”的内存或不在调用堆栈或程序的堆部分中的内存。

我试图通过查看我是否在引用任何空指针或查看我是否在引用任何悬垂指针来调试我的代码，但似乎问题出在另一个问题上。

我的代码:

#include <iostream>

typedef int* IntArrPtr;
using namespace std;


int main() {
    int arrSize;
    int *ptrSize;
    ptrSize = &arrSize;

    cout << "How many integers will this array hold:\n ";
    cin >> arrSize;

    IntArrPtr a;
    a = new int[arrSize];

    fillArr(a, arrSize);

    getPosNums4(a, arrSize,&a, ptrSize);
    cout << "The new size in main is: " << arrSize << endl;

    cout <<"The new array with positive integers is:\n";
    /*for(int i =0; i<arrSize;i++) // this runs for a large size of arrSize
        cout<< a[i] << " ";
    cout<<endl; */
    return 0;
}

void fillArr(int a[], int size){
    cout << "Please enter " << size << " Integers separated by spaces\n";
    cout << "Press enter when finished >\n";
    int i;

    for (i=0;i<size;i++)
        cin >> a[i];

}

void getPosNums4(int* arr, int arrSize, int** outPosArrPtr,int* outPosArrSizePtr){
    IntArrPtr newArr;
    newArr = new int[arrSize];
    int i;
    int newIndx = 0;
    outPosArrSizePtr = &newIndx;//initiliaze the pointer.
    for(i=0;i<arrSize;i++){
        if(arr[i] > 0){
            newArr[newIndx] =arr[i];
            newIndx++;
        }
    }
    arrSize = newIndx;
    *outPosArrSizePtr = arrSize;
    cout << "The new size is of *outPosArrSizeptr is: " << *outPosArrSizePtr << endl;

    for(int j=0;j<newIndx;j++)
        outPosArrPtr[j] = &newArr[j];
    delete []newArr;
    newArr = NULL;
    for(int i=0;i<newIndx;i++)
        arr[i] = *outPosArrPtr[i];

}

一个例子当我在 Xcode 上运行这个程序时:

How many integers will this array hold:
 6
Please enter 6 Integers separated by spaces
Press enter when finished >
3 -1 -3 0 6 4
The new size is of *outPosArrSizeptr is: 3
The new array with positive integers is:
The new size in main is: 7445512
The program ended with exit code: 0

Answer 1

那里有很多问题，但最关键的一个是给函数的参数赋值对你作为参数传递值的变量没有影响。
参数是指针并不重要——指针没有什么特别之处。

我认为正在发生的事情是函数中的“来回复制”循环(我不明白它应该做什么)正在输入数组之外写入，导致未定义的行为，在这种情况下，覆盖main 中的变量。

您的函数过于复杂了。应该

• 创建一个新数组
• 将正值复制到这个数组
• 用这个数组的地址及其(逻辑)大小更新输出参数

(将 out 参数视为返回值并最后处理它们。)

像这样:

void getPosNums4(int* arr, int arrSize, int** outPosArrPtr,int* outPosArrSizePtr){
    int* newArr = new int[arrSize];
    int newIndx = 0;
    for (int i = 0; i < arrSize; i++){
        if (arr[i] > 0){
            newArr[newIndx] = arr[i];
            newIndx++;
        }
    }
    *outPosArrPtr = newArr;
    *outPosArrSizePtr = newIndx;
}

你也不应该传递指向你的“原件”的指针来修改这个函数，你应该使用新变量。

int main() {
    int arrSize = 0;
    cout << "How many integers will this array hold:\n ";
    cin >> arrSize;
    int* a = new int[arrSize];
    fillArr(a, arrSize);
    int * positives = nullptr;
    int positiveSize = 0;
    getPosNums4(a, arrSize, &positives, &positiveSize);
    cout << "The new size in main is: " << positiveSize << endl;

    delete [] a;
    delete [] positives;
}