Question

我目前正坐在一个作业中，您应该从文件中读取球体。来自文件的typhical输入可能如下所示：“0.182361 -0.017057 0.129582 0.001816”。表示x，y和z坐标加上球体半径。在阅读文件时我正在使用我自己的方法：“AddSphere”，它将一个球体添加到一个数组中。

void Array::AddSphere(MySphere inp)
{
if (length == 10000 || length == 200000 || length == 30000)
{
    ResizeBuffer(); 
}
this->length++;
*arr = inp;
arr++;
//this->length++;
}

类“Array”应该像所有球体的持有者一样，并且包含指向当前元素的变量“arr”。

    class Array
    {
    public :

    int length;
    void AddSphere(MySphere inp);
    int arrLength();
    void RemoveAt(int pos);
    void AddFromFile();
    MySphere * Buffer;
    void CreatenewBuffer();
    private:
    MySphere * arr;

    public:Array()
       {
           Buffer = new MySphere[10000];
           arr = Buffer;
           length = 0;
       }
       ~Array()
       {
           delete[] arr;
           delete[] Buffer;
       }
};

它还包含指向“arr”中第一个元素的“Buffer”。那么现在我的问题是什么？问题是我希望能够在“length”等于指定值时动态增加Buffer的大小。假设我的文件包含15000个元素。然后我需要增加缓冲区大小才能在数组上有正确的长度。使用ResizeBuffer（）我尝试这样做。

    void Array::ResizeBuffer()
{

    MySphere * tempBuff = new MySphere[length+10000];
    memcpy(tempBuff, Buffer, sizeof((MySphere)*Buffer));
    this->Buffer = new MySphere[length+10000];
    arr = Buffer;
    memcpy(Buffer, tempBuff, sizeof((MySphere)*tempBuff));


    //int a = 0;

    }

但由于某种原因，我只得到输出中的最后5000个元素而不是所有15000个。我认为它与arr指针没有指向整个缓冲区有关，但我的尝试都没有工作。那为什么会这样呢？

感谢您的时间！

Answer 1

一些问题：

动态数据只分配一次（尽管你有两个指向arrea的指针）。

Array()
   {
       Buffer = new MySphere[10000];
       arr = Buffer;
       length = 0;
   }

所以你只能删除一次（一次分配一次破坏）。

   ~Array()
   {
       delete[] arr;
       delete[] Buffer;  // Double delete.
   }

复制对象时，不能使用memcpy（除非该对象属于一个非常特殊的子类别）。如果您要使用C ++方法及其所有其他功能，则不太可能出现这种情况。更喜欢使用std :: copy（）将内容从一个数组复制到另一个数组。

MySphere * tempBuff = new MySphere[length+10000];
memcpy(tempBuff, Buffer, sizeof((MySphere)*Buffer));
this->Buffer = new MySphere[length+10000];
arr = Buffer;
memcpy(Buffer, tempBuff, sizeof((MySphere)*tempBuff));

另外你为什么要复制两次？

注意除了使用std :: copy
调整大小应该分三个阶段进行。

1. Allocate and copy data       // Dangerous may throw
2. Reset object                 // Safe
3. Release old resources.       // Dangerous may throw

所以它应该是这样的。

// Phase 1: Allocate and copy.
std::unique_ptr<MySphere> tempBuffer = new MySphere[length+1000];   // Hold in a smart pointer
                                                                    // For exception safety
// Prefer to use std::copy. It will use the objects copy constructor
std::copy(Buffer, Buffer+length, MySphere);                         // Copies the objects correctly. 


// Everything worked so Phase 2. Reset the state of the object.
// We can now put the tempBuffer into the object
MySphere* toDelete = Buffer;                                        // keep old buffer for stage 3.
Buffer = tempBuffer.release();
length += 1000;

// Phase 3
// State of the new object is consistent.
// We can now delete the old array
delete [] toDelete;

如果您不允许使用智能指针。然后用以下代码替换第1阶段：

// Phase 1: Allocate and copy.
MySphere* tempBuffer = new MySphere[length+1000];

try                                                 // For exception safety
{
    // Prefer to use std::copy. It will use the objects copy constructor
    std::copy(Buffer, Buffer+length, MySphere);     // Copies the objects correctly. 
}
catch(...)
{
    delete [] tempBuffer;
    throw;
}

由于您的类已取得动态分配对象的所有权。你的班级也应该实施三级规则。这意味着您需要定义复制构造函数和赋值运算符。

动态数组真正需要的是三件事：

    1) A pointer to the buffer.
    2) The current size the user knows about (reported by size)
    3) The actual size. At which point you need to resize

以下是如何为数组实现三规则的示例：

https://stackoverflow.com/a/255744/14065

其他任何事情都是多余的。

在您的情况下不必要，因为这是一个项目。但是对于额外的标记，请查看如何使用新的贴图。这应该可以防止额外初始化数组中不存在的对象。

如何使用新地点的示例：

https://stackoverflow.com/a/13994853/14065

Answer 2

您没有在Array :: CreatenewBuffer（）函数中更新长度变量。

我假设你在迭代你可能有这样的东西的对象......

for (size_t i = 0; i < Array.length; ++i)
    printf("Item found");

现在我回顾过你使用长度的方式实际上相当混乱。您可能希望将其分为两个变量：大小和容量。容量将是当前分配的缓冲区可以容纳的对象数量，size是实际添加到缓冲区的对象数量。

Answer 3

这个怎么样？

void Array::CreatenewBuffer()
{
    MySphere * tempBuff = new MySphere[length+10000];
    std::copy ( Buffer, Buffer+length, tempBuff );
    delete[]Buffer;
    Buffer = tempBuff;
    delete[]tempBuff;
}

Answer 4

每个球体的值是否在单独的行中输入？如果是这样，您可以简单地计算行数，然后使用正确的值初始化缓冲区。

调整动态数组的大小c ++

4 个答案: