数据管理错误 C++

Question

我的代码有问题。问题是即使我的程序一直运行，它仍然在最后崩溃。我的 IDE 没有给我任何反对意见，也没有任何迹象表明我的程序为什么最终会失败，所以我被引导相信有什么东西越界或什么东西弄乱了我的堆栈。我查看了我的代码，似乎没有什么超出范围，所以我对这件事完全感到困惑。

我正在使用我创建的头文件、包含头文件中的函数的文件和主方法文件。该错误是由包含我的函数的文件引起的。

我班级的领域是 大小 - 整数

低 - 整数

高 - 整数

windowSize - 整数

*filteredArray - int

数组[] - 整数

这是头文件：

#ifndef FILTER_HPP_
#define FILTER_HPP_
#include<iostream>


class Filter {
    int size;
    int high;
    int low;
    int windowSize;
    int *filteredArray;
    int array[];
    public:
        // Constructor
        Filter(int windowSize);
        // Methods
        void randArrayGen();
        int* randArrayGenRecurHelper(int arrayRecur[], int count);
        void randArrayGenRecur();
        void printArrays();
        int hanning(int ar[]);
        void hanningFilter();
        void graphicArrays();
        int getSize();
        int getHigh();
        int getLow();

};





#endif /* FILTER_HPP_ */

代码如下：

#include "Filter.hpp"
#include <iostream>
#include <time.h>
#include <stdlib.h>
#include <cmath>
#include <array>
#include <string>
#include <cstring>
using namespace std;

// Constructor
Filter::Filter(int num){
    size = ((rand() % 25) + 25);
    high = ((rand() % 6) + 5);
    low = ((rand() % 6) + 5) * -1;
    windowSize = num;
    randArrayGen();
    hanningFilter();

}


void Filter::randArrayGen(){
    for(int i = 0; i < size;i++){
        array[i] = (rand() % (high + low * -1)) + low;
    }
}




int Filter::hanning(int ar[]){
    int weightAvg;

    if(windowSize == 3){
        weightAvg = (ar[0] + ar[1] * 2 +ar[2]) / 4;
    }
    else if(windowSize == 5){
        weightAvg = (ar[0] + ar[1] * 2 + ar[2] * 3 + ar[3] * 2 + ar[4]) / 9;
    }
    else if(windowSize == 7){
        weightAvg = (ar[0] + ar[1] * 2 + ar[2] * 3 + ar[3] * 4 + ar[4] *3 + ar[5] * 2 + ar[6]) / 16;
    }
    else if(windowSize == 9){
            weightAvg = (ar[0] + ar[1] * 2 +ar[2] * 3 + ar[3] * 4 + ar[4] * 5 + ar[5] * 4 + ar[6] * 3 + ar[7] * 2 + ar[8]) / 25;
        }
    else{
        weightAvg = 0;
    }
    return weightAvg;
}



void Filter::hanningFilter(){
    filteredArray = new int[size];
    for(int i = 0; i < size; i++){
        if(i - (windowSize/2) < 0 or i + (windowSize/2) > size - 1){
            filteredArray[i] = 0;
        }
        else{
            filteredArray[i] = hanning(&array[i-(windowSize/2)]);
        }
    }
}



int Filter::getHigh(){
    return high;
}

int Filter::getLow(){
    return low;
}

int Filter::getSize(){
    return size;
}

Answer 1

你没有为你的数组分配内存：

int array[];

当你像这样索引它时：

for(int i = 0; i < size;i++){
    array[i] = (rand() % (high + low * -1)) + low;
}

您调用未定义的行为，因为您超出了界限（例如，您请求第一个元素，但来自一个未知大小的数组）。

您可以声明具有固定大小的数组，例如：

int array[100];

---

但是，由于这是 C++，请考虑使用 std::vector。一个直接的优势是您不必为静态分配的数组硬编码大小，但您可以依靠向量在数据插入/推入时调整大小的能力。