VPF::проблема с new и delete в классе - Форум программистов

Revares

Дата 20.9.2009, 13:38 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 22
Регистрация: 18.4.2008

Репутация: нет
Всего: нет

Совершенно непонятная ошибка вылезает, на ровном месте:

Код


#include <iostream>
using namespace std;

class Array {

public:
        
    Array(int sz = 12) {
        int i = 0;
        _size = sz;
        this->pia = new int(_size);
        for(i = 0; i < _size; i++) {
            this->pia[i] = i;
        }
    }

    ~Array() {
        delete[] this->pia;
    }

    void print() {
        int i = 0;
        for(i = 0; i < _size; i++) {
            cout << this->pia[i] << " ";
        }
    }

private:

    int *pia;
    int _size;
    

};

int main() {
    Array a(15);
    cin.get();
}

Компилируется без проблем, запускаю:

Код


ОС Windows инициировала точку останова в test.exe.

Это может быть вызвано повреждением кучи и указывает на ошибку в test.exe или в одной из загруженных им DLL.

Возможной причиной так же может быть нажатие пользователем клавиши F12, когда фокус принадлежит test.exe

Выведенное на экран окно содержит дополнительные данные для диагностики ошибки

Ну и программа вылетает естественно. Я предполагаю что ошибка связана с выделением памяти. Но где тут может быть ошибка?! Ведь все элементарно...

Можете объяснить что к чему?

Это сообщение отредактировал(а) Revares - 20.9.2009, 13:42

Daevaorn

Дата 20.9.2009, 13:52 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2155
Регистрация: 29.11.2004
Где: Москва

Репутация: 51
Всего: 70

Цитата(Revares @ 20.9.2009, 14:38

)

this->pia = new int(_size);

Цитата(Revares @ 20.9.2009, 14:38

)

delete[] this->pia;

Почему вы память выделяете под 1 int, а освобождаете множество? Читайте умные книги.

ИванМ

Дата 20.9.2009, 14:04 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1260
Регистрация: 19.6.2006
Где: СПб

Репутация: 1
Всего: 23

Код


this->pia = new int(_size);

заменить на

Код


this->pia = new int[_size];

В вашем случае вы создаете одну переменную int и присваиваете ей значение _size

Revares

Дата 20.9.2009, 15:31 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 22
Регистрация: 18.4.2008

Репутация: нет
Всего: нет

Спасибо Иван, действительно в этом и была моя ошибка. Теперь заработало!

1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »