Что не так в коде?

[Dr.Yevhenius]

Есть класс:

Код:

class Row {
public:
    Row(int, int);
    ~Row() { delete [] arr; }
    void print();
    void rewrite(int, int);
    void extend(int, int);
    int _size;
private:
    int * arr;
    int size;
};

с реализацией конструктора:

Код:

Row::Row(int x, int y) {
    _size = size = 0;
    extend(x, y);
}

и реализацией функции extend:

Код:

void Row::extend(int x, int y) {
    //Записуємо вміст масива arr в temparr
    int * temparr = new int(size);
    for(int i = 0; i < size; ++i)
        temparr[i] = arr[i];
    //Збільшуємо розмір масиива arr
    if (size != 0)
        delete [] arr;
    _size = size += 2;
    arr = new int(size);
    //Ініціалізуємо новий масив
    for(int i = 0; i < size - 2; ++i)
        arr[i] = temparr[i];
    delete [] temparr;//видаляємо тимчасовий масив з динамічної пам’яті
    arr[size - 2] = x;
    arr[size - 1] = y;
}

В функции main пишу следующее:

Код:

    Row r(0, 0);
    vector <Row> a;
    a.push_back(r);

вызывает ошибку:

Инструкция по адресу "0х7с910с27" обратилась к памяти по адресу "0х003сff9d", память не может быть "read".

Подскажите пожалуйста, что не так.

[Dr.Yevhenius]

Работаю в Dev-C++ v4.9.9.2.

[Вад]

Ну, самое главное "что не так" - это отсутствие реализации копирующего конструктора и оператора копирования, при том, что в классе есть динамический массив. При помещении объекта в вектор происходит копирование, при котором будет скопировано только значение указателя, что обычно ведёт к проблемам.

Второе "что не так" - это совершенно непонятная работа с памятью. Массив ещё не выделен, а функция extend пытается сделать что-то странное:

Код:

int * temparr = new int(size); 

должно быть, подразумевалось

Код:

int * temparr = new int[size]; 

а потом, если размер не нулевой, не проверяя, существует ли arr, читает оттуда.

[Dr.Yevhenius]

должно быть, подразумевалось

Код:

int * temparr = new int[size]; 

Да, действительно, описался...

а потом, если размер не нулевой, не проверяя, существует ли arr, читает оттуда.

Если размер не нулевой, то массив обязательно существует.

[Вад]

Да, действительно, описался...

Неслабо описался. Вместо массива выделял память на 1 int, инициализируя его значением size

Да, действительно, если размер не нулевой, то массив должен существовать - но это если в прошлый раз из конструктора была вызвана extend. Ненадёжно как-то.