Question

我正在尝试使用指针和模板在C ++中实现动态数组实现，以便我可以接受所有类型。该代码在int上可以正常工作，但是在string下使用会产生错误。我在网上尝试了其他SO问题，但未发现与我有关的情况。

代码：

#include <iostream>
#include <string>
using namespace std;

template <typename T>
class dynamicIntArray
{
private:
    T *arrPtr = new T[4]();
    int filledIndex = -1;
    int capacityIndex = 4;

public:
    // Get the size of array
    int size(void);

    // Insert a data to array
    bool insert(T n);

    // Show the array
    bool show(void);
};

template <typename T> 
int dynamicIntArray<T>::size(void)
{
    return capacityIndex + 1;
}

template <typename T> 
bool dynamicIntArray<T>::insert(T n)
{
    if (filledIndex < capacityIndex)
    {
        arrPtr[++filledIndex] = n;
        return true;
    }
    else if (filledIndex == capacityIndex)
    {
        // Create new array of double size
        capacityIndex *= 2;
        T *newarrPtr = new T[capacityIndex]();

        // Copy old array
        for (int i = 0; i < capacityIndex; i++)
        {
            newarrPtr[i] = arrPtr[i];
        }

        // Add new data
        newarrPtr[++filledIndex] = n;
        arrPtr = newarrPtr;

        return true;
    }
    else
    {
        cout << "ERROR";
    }
    return false;
}

template <typename T> 
bool dynamicIntArray<T>::show(void)
{
    cout << "Array elements are: ";
    for (int i = 0; i <= filledIndex; i++)
    {
        cout << arrPtr[i] << " ";
    }
    cout << endl;

    return true;
}

int main()
{
    dynamicIntArray<string> myarray;

    myarray.insert("A");
    myarray.insert("Z");
    myarray.insert("F");
    myarray.insert("B");
    myarray.insert("K");
    myarray.insert("C");

    cout << "Size of my array is: " << myarray.size() << endl;

    myarray.show();
}

错误：

segmentaion fault (core dumped)

Answer 1

经典Off-by-one error：

if (filledIndex < capacityIndex)
{
    arrPtr[++filledIndex] = n;

在插入第五项之前，filledIndex是3 <4（capacityIndex）。这将导致arrPtr[4]被访问（由于其范围当前为[0..3]，因此无法访问）。

首先将filledIndex设置为0，然后将arrPtr[++filledIndex] = n;更改为arrPtr[filledIndex++] = n;

，即可解决此问题。

您应该注意，您的代码虽然存在严重缺陷，例如：内存泄漏，名称和样式可疑等。您可能希望将其固定版本发布到https://codereview.stackexchange.com/。

C ++：分段错误（核心已转储）

1 个答案: