添加到动态数组

时间:2015-10-10 22:40:47

标签: c++ class templates dynamic-arrays

免责声明:是的,我知道std :: vector。我为了学习而这样做。

我正致力于制作动态数组课程,并且我正在尝试添加工作。

template <class T>
void Array<T>::add(T value)
{
    T * tmp = new T[mCount];

    for (int i = 0; i < mCount; i++)
    {
        tmp[i] = mData[i];
    }

    mCount++;

    delete[] mData;
    mData = tmp;

    mData[mCount - 1] = value;
}

它有效......有点儿。该函数用于添加元素,但程序在退出时崩溃。没有错误,没有任何错误。它只是冻结,我必须使用(Shift + F5)关闭它。

那么,这有什么不对?

这是全班同学。如果我没有包含某个功能,则表示其中没有代码。

#ifndef ARRAY_H
#define ARRAY_H

using namespace std;

template <class T>
class Array
{
private:
    T * mData;
    int mCount;

public:
    Array();
    ~Array();

    void add(T value);
    void insert(T value, int index);
    bool isEmpty();
    void display();
    bool remove(T value);
    bool removeAt(int index);
    int size();

    T & operator[](const int index);
};

// Constructors / Destructors
// --------------------------------------------------------

template <class T>
Array<T>::Array()
{
    mCount = 0;
    mData = new T[mCount];
    for (int i = 0; i < mCount; i++)
        mData[i] = 0;
}

template <class T>
Array<T>::~Array()
{
    delete[] mData;
}

// General Operations
// --------------------------------------------------------

template <class T>
void Array<T>::add(T value)
{
    T * tmp = new T[mCount];

    for (int i = 0; i < mCount; i++)
    {
        tmp[i] = mData[i];
    }

    mCount++;

    delete[] mData;
    mData = tmp;

    mData[mCount - 1] = value;
}

template <class T>
void Array<T>::display()
{
    if (isEmpty())
    {
        cout 
            << "The array is empty."
            << "\n\n";
        return;
    }

    cout << "(";

    for (int i = 0; i < mCount; i++)
    {

        cout << mData[i];

        if (i < mCount - 1)
            cout << ", ";
    }

    cout << ")" << "\n\n";
}

template <class T>
bool Array<T>::isEmpty()
{
    return mCount == 0;
}

template <class T>
int Array<T>::size()
{
    return mCount;
}

// Operator Overloads
// --------------------------------------------------------

template <class T>
T & Array<T>::operator[](const int index)
{
    return mData[index];
}

#endif

如果您需要了解任何其他信息,我可以发布。

1 个答案:

答案 0 :(得分:1)

假设mCount保留了数组中元素的数量,那么在添加新元素时,您必须至少分配mCount + 1元素(假设您当然希望保留所有旧元素和新的)通过:

T * tmp = new T[mCount + 1];

而不是:

T * tmp = new T[mCount];

如果是教育用途以外的任何其他内容,请改用std::vector。例如,您的add函数不是异常安全的。