错误代码 C6262 考虑将数据移动到堆

Question

此代码 C6262 警告不断出现，导致程序出现问题，我尝试寻找可能的解决方案，但我很难理解。如果有人能帮我解决这个问题，如果你能指出我可以改进的代码中的任何错误部分，我将不胜感激。

#include <iostream>
#include <cstdlib>
#include <algorithm>
using namespace std;

class sorting {
private:
int size, elements;
int arr[5000], x;


public:
void sort() {
    cout << "Enter number of desired elements for the 1st set" << ">"; cin >> elements;
    arr[elements];
    half(); cout << endl;
    bubble();

    
    for (int i = 0; i < elements; i++) {
        cout << arr[i] << " ";

    }

    

}
void half() {
    for (int i = 0; i < elements / 2; i++) {
        arr[i] = i + 1;
    }
    for (int i = elements / 2; i < elements; i++) {
        arr[i] = rand();
    }
    cout << "This is the elements of the 1st set: ";
    for (int i = 0; i < elements; i++) {
        cout << arr[i] << " ";
    }
}
void random() {
    for (int i = 0; i < elements; i++) {
        arr[i] = i + 1;

    }

    random_shuffle(&arr[0], &arr[elements]);

    cout << "This is the elements of the 2nd set: ";

    for (int i = 0; i < elements; i++) {
        cout << arr[i] << " ";

    }

}
void ascend_descend() {
    int x = elements / 2;
    arr[0] = x;
    for (int i = 0; i < elements / 2; i++) {
        arr[i + 1] = x - 1;
        x--;
    }
    for (int i = elements / 2; i < elements; i++) {
        arr[i] = i + 1;
    }
    cout << "This is the elements of the 3rd set: ";
    for (int i = 0; i < elements; i++) {
        cout << arr[i] << " ";
    }



};
void bubble() {
    for (int i = 0; i < elements; i++) {
        int temp;
        for (int j = i + 1; j < elements; i++) {
            if (arr[j] < arr[i]) {
                temp = arr[i];
                arr[i] = arr[j];
                arr[j] = temp;
            }

        }

    };
}
  };

int main()
{   
    sorting sortObject;

    sortObject.sort();
    return 0;
}

Answer 1

在类声明的顶部有int arr[5000], x;，它将在堆栈上分配整数数组。您的编译器警告您该数组可能不适合堆栈，您应该在堆上分配它。有关堆栈与堆的解释，请参阅this。假设您仍想使用带有指针的 c 样式数组（请参阅我的第二个解决方案以了解更惯用的方法），您将在构造函数中分配数组，并在析构函数中释放它。

class sorting
{
int  size;
int  elements;
int* arr;
int  x;

public:
    sorting()
    {
        arr = new int[5000];
    }

    ~sorting()
    {
        delete arr;
    }
/*the rest of your code...*/
};

此解决方案更多地采用手动内存管理方式，除非您有充分的理由这样做，否则我建议您使用std::vector 而不是手动分配您的数组。这将跟踪元素和大小。因此，您的成员变量将如下所示：

class sorting
{
private:
    int x;
    int elements;
    vector<int> arr;
}

然后您可以使用arr.push_back(int) 向向量添加元素。 arr[i] 访问和设置它们，arr.size() 获取数组中有多少元素。