Question

我正在创建一个简单的应用程序，让我可以在集合上操作。需要它将设置数据保存在动态数组中。但是，我遇到的问题是我无法缩小范围。

Sets.cpp：

#include "stdafx.h"
#include "SetArray.h"


int main()


{

int a1[] = { 1,2,3,4,5 };
int a2[] = { 5,6,7,8 };

SetArray arr1 = SetArray(a1, (sizeof(a1) / sizeof(*a1)));
SetArray arr2 = SetArray(a2, (sizeof(a2) / sizeof(*a2)));
arr2.printSet();

SetArray arr3 = arr1 + arr2;
arr3.printSet();

return 0;
}

SetArray.h：

#pragma once

#include <string>
#include <iostream>
#include <algorithm>
#include <vector>
class SetArray
{
public:

SetArray(int * elems, int count);
~SetArray();

friend SetArray operator+(SetArray &set1, SetArray &set2);

int * getArray();
int getArraySize();
void printSet();
int * elemsArray;
private:

int count = 0;
};

SetArray.cpp：

#include "stdafx.h"
#include "SetArray.h"


SetArray::SetArray(int * elems, int count)
{
    elemsArray = elems;
    SetArray::count = count;

}


SetArray::~SetArray()
{
}

int * SetArray::getArray()
{
    return elemsArray;
}

int SetArray::getArraySize()
{
    return count;

}

void SetArray::printSet() 
{
    for (int i = 0; i < count; i++)
    {
        std::cout << elemsArray[i] << std::endl;
    }
}

SetArray operator+(SetArray & set1, SetArray & set2)
{
std::vector<int> vector1;
std::vector<int> vector2;

for (int x = 0; x < set1.count; x++)
{
    vector1.push_back(set1.elemsArray[x]);
}
for (int x = 0; x < set2.count; x++)
{
    vector2.push_back(set2.elemsArray[x]);
}

if (vector1.size() >= vector2.size())
{
    for (int i = 0; i < vector2.size(); i++) 
    {
        if (std::find(vector1.begin(), vector1.end(), vector2[i]) == vector1.end())
        {
            vector1.push_back(vector2[i]);
        }
    }

    int * dynamicArray;
    dynamicArray = new int(vector1.size());

    for (int j = 0; j < vector1.size(); j++)
    {
        dynamicArray[j] = vector1[j];
    }

    return SetArray(dynamicArray, vector1.size());



}
else
{
    for (int i = 0; i < vector1.size(); i++)
    {
        if (std::find(vector2.begin(), vector2.end(), vector1[i]) == vector2.end())
        {
            vector2.push_back(vector1[i]);
        }
    }

    int * dynamicArray;
    dynamicArray = new int(vector2.size());

    for (int j = 0; j < vector2.size(); j++)
    {
        dynamicArray[j] = vector2[j];
    }

    return SetArray(dynamicArray, vector2.size());

    }

}

因此，当我输入调试代码时，程序以这样的错误结束，尽管在控制台上显示正确的结果：

HEAP [Sets.exe]：为RtlValidateHeap指定的地址无效（00830000,00854A30） Sets.exe触发了一个断点。

然后我尝试通过逐行手动执行代码来缩小问题范围，没有任何反应，没有错误显示，程序正确执行。

然后，当我尝试在没有调试的情况下运行它时，再一次，在控制台中显示正确的数据，但程序以＆＃34;应用程序Sets.exe结束已停止工作＆＃34;

我相信必须有一些非定位指针导致所有问题，但我无法看到问题所在。你能帮我吗？

Answer 1

dynamicArray = new int(vector1.size());

非常小的错字。这应该是

dynamicArray = new int[vector1.size()];

同样适用于其他new分配。

此外，您的课程，如设计，将泄漏内存。但这将是一个不同的问题。

P.S。我也无法立即看到这个错误。乍一看，代码对我来说很好看。但是在我的调试器中找到它需要不到一分钟的时间。您应该花一些时间学习如何使用调试器。它将帮助您在代码中找到错误，而不必等待随机陌生人来帮助您，在stackoverflow.com上

为RtlValidateHeap

1 个答案: