Настраиваемый массив c ++ выходит из себя при присваивании значения индексу - PullRequest
0 голосов
/ 05 июня 2018

Мне нужна помощь с ошибкой в ​​моем собственном массиве c ++.Я хотел создать простой шаблон класса массива, намеренно игнорируя массив std.Но когда я пытаюсь присвоить значение через оператор индекса, по какой-то причине размер массива, кажется, меняется, и значение не сохраняется.

Это мой класс шаблонов массивов.

template<typename T>
class Array
{
       friend std::ostream &operator<<(std::ostream& output,const Array& a)
       {
            //output private ptr-based array
            for (size_t i=0; i < a.length; i++)
            {
                output << std::setw(12) << a.ptr[i];
                if ((i+1)%4 == 0)   //four numbers per row of output
                    output << std::endl;
            }
            if (a.length%4 != 0)      //end last line of output
                output << std::endl;
            return output;
        }

    public:
        Array(const int& arraySize = 0);
        Array(const Array&);
        ~Array();
        Int32 size();

        bool operator==(const Array&) const;
        bool operator!=(const Array&) const;

        T &operator[](Int32);
        const T operator[](Int32)const;

    private:
        size_t length;
        T* ptr;
};

template<typename T>
Array<T>::Array(const int& length):length(length > 0 ? length : throw std::invalid_argument("Array size must be greater than 0")), ptr(new T[length])
{
    for(size_t i = 0; i < length; ++i)
    {
        ptr[i] = NULL;
    }
}

template<typename T>
Array<T>::Array(const Array<T>& aryToCpy): length(aryToCpy.size()), ptr(new T[length])
{
    for(size_t i = 0; i < length; ++i)
    {
        ptr[i] = aryToCpy[i];
    }
}

template<typename T>
Array<T>::~Array<T>()
{
    delete[] ptr;
    ptr = nullptr;
}

template<typename T>
Int32 Array<T>::size()
{
    return length;
}

template<typename T>
T& Array<T>::operator[](int subscript)
{
        if (subscript < 0 || subscript >=0)
            throw std::out_of_range("Subscript out of range");

        return ptr[subscript];
}

template<typename T>
const T Array<T>::operator[](int subscript) const
{
        if (subscript < 0 || subscript >=0)
            throw std::out_of_range("Subscript out of range");

        return ptr[subscript];
}

template<typename T>
bool Array<T>::operator==(const Array<T>& right) const
{
    if(length != right.length)
        return false;

    for(size_t i = 0; i < length; i++)
    {
        if(ptr[i] != right.ptr[i])
            return false;
    }

    return true;
}

template<typename T>
bool Array<T>::operator!=(const Array<T>& right) const
{
    return !(this == right);
}

Это моя среда модульного тестирования catch2, где я хочу проверить, правильно ли работает мой массив.

SCENARIO("Arrays can be created empty.")
{
  GIVEN("An empty Array declaration and a defined length")
  {
    Array<Int16> *testArray;
    Int8 arrayLength = 4;

    WHEN("A new array gets created with empty items...")
    {
        testArray = new Array<Int16>(arrayLength);
        THEN("The size of the array is as defined")
        {
            REQUIRE(testArray->size() == 4);
        }

    }

    testArray = new Array<Int16>(arrayLength);
    WHEN("An empty array with size 4 gets filled up with values")
    {
        std::cout << "Array size start: " << testArray->size() << std::endl;
        std::cout << "Array index 0 (before): " << testArray[0] << std::endl;
        testArray[0] = 1;
        std::cout << "Array index 0 (after): " << testArray[0] << std::endl;
        testArray[1] = 2;
        testArray[2] = 3;
        testArray[3] = 4;


        THEN("The array is now filled with values")
        {
            std::cout << "Array size test: " << testArray->size() << std::endl;
            REQUIRE(testArray[0] == 1);
            REQUIRE(testArray[1] == 2);
            REQUIRE(testArray[2] == 3);
            REQUIRE(testArray[3] == 4);
        }
    }
}

1 Ответ

0 голосов
/ 05 июня 2018

попробуйте изменить "if (subscript <0 || subscript> = 0)" на "if (subscript <0 || subscript> = length)"

...