При вызове деструктора в строке delete[] data выбрасывается исключение

@NNNNNNNNN · Регистрация: 27.02.2016

Author24 — интернет-сервис помощи студентам

вот как раз вопрос по этой теме. программа отрабатывает все команды и в конце при вызове деструктора в строке delete[] data выбрасывает исключение. Где искать? Что делать? maxsize делала и в классе и вне класса статической переменной, ничего не помогает.

C++

Set(int n) {
        cout << "вызвался конструктор Set с параметром" << endl;
        maxsize = n;
        data = new char[maxsize];//выделить динамическую память для массива элементов
        cursize = 0;
    }
    Set(const Set& ob) {
        cout << "вызвался конструктор копирования Set" << endl;       
        data = new char[maxsize];
        for (int i = 0; i < cursize; i++)
            data[i] = ob.data[i];
    }
    ~Set() {
        cout << "вызвался деструктор Set" << endl;
        delete[] data;
    }

@zayats80888 · 13.12.2019, 21:07

Увидеть бы выражение, в котором выбрасывается исключение, а по этому обрывку можно сказать, что не хватает конструктора по умолчанию и оператора присваивания

Verevkin · 13.12.2019, 21:08

покаж весь класс, не стесняйся.

@zayats80888 · 13.12.2019, 21:08

Также в конструкторе копирования почему то не копируются maxsize и cursize

@wolf1396 · 13.12.2019, 21:18

я думаю тут cursize и maxsize поля класса
да и к тому же maxsize присваивается значение входного параметра n

Добавлено через 2 минуты
если я правильно помню, сначала надо очистить элементы (пройтись по всем и вызвать у каждого delete), а затем выполнить удаление непосредственно у переменной

ну т.е что-то вроде:

C++

for (int i = 0; i < maxsize; ++i)
{
delete data[i];
}
 
delete [] data;

@NNNNNNNNN · 14.12.2019, 16:05 **[ТС]**

весь код: делаю класс, конструктор копирования есть.

C++

static int maxsize = 0;
class Set {
private:
    char* data;    
    int   cursize;
public:
    Set() {
        cout << "вызвался конструктор Set" << endl;
    };
    Set(int n, char*D) {
        cout << "вызвался конструктор Set с параметром" << endl;
        maxsize = n;
        data = new char[maxsize];//выделить динамическую память для массива элементов
        data = new char[maxsize]; // под массив выделяется память - нужен деструктор для ее удаления
        for (int i(0); i < maxsize; i++)
        {
            data[i] = D[i];
        }
        cursize = 0;
    }
    Set(const Set& ob) {
        cout << "вызвался конструктор копирования Set" << endl;       
        data = new char[maxsize];
        for (int i = 0; i < cursize; i++)
            data[i] = ob.data[i];
    }
    ~Set() {
        cout << "вызвался деструктор Set" << endl;
        delete[] data;
    }
 
    // перегрузить оператор присваивания
    Set& operator =(Set& b) {
        // Выполнить копирование значений      
        cursize = b.cursize;
        data = b.data;
        // Возвратить текущий объект, чтобы иметь возможность связать в цепочку выполнение нескольких операций присваивания
        return *this;
 
    };
 
    //вставить по принципу дерева
    //(упорядоченная вставка при помощи двоичного поиска)
    Set& operator + (const char c) {
        if (cursize >= maxsize)
            return *this;
        else if (!cursize || (c > data[cursize - 1])) {
            data[cursize++] = c;
            return *this;
        }
        int m, f = 0, l = cursize;
        while (f < l) {
            m = f + (l - f) / 2;
            if (c == data[m])
                return *this;
            else if (c < data[m])
                l = m;
            else
                f = m + 1;
        }
 
        for (int i = cursize; i > f; --i)
            data[i] = data[i - 1];
 
        data[f] = c;
        ++cursize;
        return *this;
    }
 
    //
    Set& operator - (const char d) {
        unsigned int number, i, j;
        for (i = 0; i < cursize; i++)
        {
            if (d == data[i])
            {
                for (j = i; j < cursize - 1; j++)
                    data[j] = data[j + 1];
                cursize--;
            }
 
        }
        return *this;
 
    };
    // перегрузить оператор равенства
    bool operator == (const Set &c) {
        if (this->cursize != c.cursize)
        {
            cout << "размеры неравны!" << endl;
            return false;
        }
        cout << "размеры равны!" << endl;
 
        for (int i = 0; i < c.cursize; i++)
        {
            if (this->data[i] != c.data[i])
            {
                cout << "но значения не неравны!" << endl;
                return false;
            }
            else
                cout << "значения равны!" << endl;
 
        }
        return true;
    };
 
    // перегрузить оператор неравенства
    bool operator != (const Set &c) {
        if (this->cursize != c.cursize)
 
        {
            cout << "размеры неравны!" << endl;
            return true;
        }
        cout << "размеры равны!" << endl;
        for (int i = 0; i < c.cursize; i++)
        {
            if (this->data[i] != c.data[i])
            {
                cout << "значения  не равны!" << endl;
                return true;
            }
            else
                cout << "значения равны!" << endl;
        }
        return false;
    };
 
    // перегрузить оператор проверки на подмножество
    bool operator > (const Set &b) {
        if (b.cursize > this->cursize)
        {
            cout << "размеры некоректны для проверки условия!" << endl;
            return false;
        }
        //cout << "размеры сопоставимы!" << endl;
        for (int i = 0; i < b.cursize; i++) // идем по элементам b
        {
            bool notfound = true; // считаем пока , что b[i] нет
            for (int j = 0; j < this->cursize; j++) // сравниваем со всеми a[j] по очереди
            {
                if (b.data[i] == this->data[j]) // нашли
                {
                    cout << b.data[i] << " == " << this->data[j] << " этот элемент состоит в множестве!" << endl;
                    notfound = false;
                    break;
                }
            }
            if (notfound)
 
                //cout << "не является!" << endl;
                return false;
            // если не нашли возвращаем false
        }
        return true; // все найдены
    };
 
    //ввести элемент множества с клавиатуры
    Set& Input() {
        char c;
        cout << "введи новый символ в множество Дерево" << endl;
        cin >> c;
        if (cursize >= maxsize)
            return *this;
        else if (!cursize || (c > data[cursize - 1])) {
            data[cursize++] = c;
            return *this;
        }
 
        int m, f = 0, l = cursize;
        while (f < l) {
            m = f + (l - f) / 2;
            if (c == data[m])
                return *this;
            else if (c < data[m])
                l = m;
            else
                f = m + 1;
        }
 
        for (int i = cursize; i > f; --i)
            data[i] = data[i - 1];
 
        data[f] = c;
        ++cursize;
    };
 
    // удалить элемент множества
    void extract()
    {
        unsigned int number, n, i, j;
        n = cursize;
        std::cout << "Введите номер элемента от 0 до " << cursize - 1 << std::endl;
        std::cin >> number;
 
        if (number < 0 || number > cursize)
            std::cout << "Ошибка " << std::endl;
        else
        {
            for (i = 0; i < n; i++)
            {
                if (i == number)
                {
                    for (j = i; j < n - 1; j++)
                        data[j] = data[j + 1];
                    n--;
                }
 
            }
            cursize--;
        }
    };
 
    //вывести все элементы множества на экран
    void Print(void) {
        cout << "размер множества = " << cursize << " элементов" << endl;
        cout << "множество состоит из  " << endl;
        for (int i = 0; i < cursize; ++i)
            cout << data[i] << ' ';
        cout << endl;
    };
 
};

Добавлено через 4 минуты
а если пытаюсь удалять поэлементно - не дает, подчеркивает data красным

C++

for (int i = 0; i < maxsize; ++i)
{
delete data[i];
}
 
delete [] data;

@zayats80888 · 14.12.2019, 17:18

NNNNNNNNN,

C++

class Set {
private:
    static const int maxsize = 100; //<-- какую роль это играет в вашем коде я не понял, поэтому сделал так
    int cursize;
    char* data;
public:
 
    Set():cursize(0), data(nullptr)
    {
        cout << "вызвался конструктор Set" << endl;
    }
 
    Set(int n, char*D):cursize(n), data(new char[n])
    {
        cout << "вызвался конструктор Set с параметром" << endl;
        for (int i(0); i < cursize; i++)
        {
            data[i] = D[i];
        }
    }
 
    Set(const Set& ob):Set(ob.cursize, ob.data)
    {
        cout << "вызвался конструктор копирования Set" << endl;
    }
 
    ~Set() {
        cout << "вызвался деструктор Set" << endl;
        delete[] data;
    }
 
    // перегрузить оператор присваивания
    Set& operator =(const Set& b) {
        // Выполнить копирование значений
        if (this != &b)
        {
            delete[] data;
            cursize = b.cursize;
            data = new char[cursize];
            for (int i(0); i < cursize; i++)
            {
                data[i] = b.data[i];
            }
        }       
        // Возвратить текущий объект, чтобы иметь возможность связать в цепочку выполнение нескольких операций присваивания
        return *this;
    }
// ...
// в остальное не вникал
// ...
};

@NNNNNNNNN · 14.12.2019, 20:06 **[ТС]**

исключение на строчку data[i] = D[i];

@zayats80888 · 14.12.2019, 20:13

Сообщение от NNNNNNNNN

исключение на строчку data[i] = D[i];

покажи 16 строку функции main

Добавлено через 5 минут
Хотя лучше всю функцию main(только выдели комментарием строку, которая в твоем проекте №16)

@TheCalligrapher · 14.12.2019, 20:40

Сообщение от NNNNNNNNN

конструктор копирования есть.

Конструктор копирования есть, но в нем написана чушь. Оператор присваивания есть, но в нем написана чушь... Дальше можно не смотреть.

Вам уже показали пальцем на чушь в конструкторе копирования. Зачес вы снова приводите этот код?

Добавлено через 2 минуты

Сообщение от NNNNNNNNN

C++

а если пытаюсь удалять поэлементно - не дает, подчеркивает data красным
for (int i = 0; i < maxsize; ++i)
{
delete data[i];
}

Что такое "удалять поэлементно"?

@zayats80888 · 14.12.2019, 20:47

Не по теме:

Сообщение от TheCalligrapher

Что такое "удалять поэлементно"?

скорее всего это попытка последовать совету из непонятно к чему относящемуся поста №5

@NNNNNNNNN · 14.12.2019, 21:33 **[ТС]**

вот весь main
работает, пока закоментировано //delete[] data; в деструкторе
(при php.h которое в начале этой темы)

int Max_size = 20;//

int main() {
setlocale(LC_ALL, "ru");
std::cout << "Hello World!\n";
char *d=nullptr;

Set a = Set(Max_size, d);
a = a + 'm' + 'p' + 'a' + 'd' + 'e' + 'x' + 'w';
a = a - 'M';
a = a - 'k';
a.Print();

Set b = Set(Max_size, d);
b = b + 'a' + 't' + 'e' + 'm' + 'i';
b.Print();

Set c = Set(Max_size, d);
c = c + 'm' + 'p' + 'a' + 'd' + 'e' + 'x' + 'w' + 'r';
c.Print();
c.extract();
c.Print();

if (a == b) cout << "yes, a = b" << endl;
else cout << "a != b" << endl;
if (a == c) cout << "yes, a = c" << endl;
else cout << "a != c" << endl;

if (a != b) cout << "yes, a != b" << endl;
else cout << "a == b" << endl;
if (a != c) cout << "yes, a != c" << endl;
else cout << "a == c" << endl;

Set Q = Set(Max_size, d);
Set q2 = Set(Max_size, d);
Q = Q + 'A' + 'B' + 'O' + 'R' + 'A' + 'B' + 'W' + 'I';
Q.Print();

Set q = Set(Max_size, d);
q = q + 'O' + 'A' + 'R' + 'I' + 'W';
q.Print();
q2 = Q;
q2.Print();

if (Q > q)
cout << "подмножество q принадлежит Q" << endl;
else
cout << "НЕТ! подмножество q не принадлежит Q" << endl;
Q.extract();
Q.Print();
if (Q > q) cout << "подмножество q принадлежит Q " << endl;//вызвать оператор проверки на подмножество
else cout << "НЕТ! подмножество q не принадлежит Q" << endl;
Q.extract();

Q.Print();
q.Print();

if (Q > q) cout << "подмножество q принадлежит Q " << endl;//вызвать оператор проверки на подмножество
else cout << "НЕТ! подмножество q не принадлежит Q!" << endl;
return 0;

}

Добавлено через 2 минуты

да, "это попытка последовать совету из поста №5"

@zayats80888 · 14.12.2019, 21:44

Сообщение от NNNNNNNNN

Set a = Set(Max_size, d);

ну ясен пень, в чем смысл такого вызова?
Set a;

Сообщение от NNNNNNNNN

char *d=nullptr;

удалить

Добавлено через 6 минут
Смысл maxsize я понял, только зачем тут динамический массив? Можно статическим обойтись

@NNNNNNNNN · 14.12.2019, 21:51 **[ТС]**

а конструктор? туда что вставлять вместо char *D?

C++

 Set(int n, char*D):cursize(n), data(new char[n])
    {
        cout << "вызвался конструктор Set с параметром" << endl;
        for (int i(0); i < cursize; i++)
        {
            data[i] = D[i];
        }
    }

@zayats80888 · 14.12.2019, 21:53

исправил конструкторы под вашу задачу

C++

class Set {
private:
    static const int maxsize = 20;
    int cursize;
    char* data;
public:
 
    Set() :cursize(0), data(new char[maxsize])
    {
        cout << "вызвался конструктор Set" << endl;
    }
 
    Set(int n, char* D) :cursize(n), data(new char[maxsize])
    {
        cout << "вызвался конструктор Set с параметром" << endl;
        for (int i(0); i < cursize; i++)
        {
            data[i] = D[i];
        }
    }
 
    Set(const Set& ob) :Set(ob.cursize, ob.data)
    {
        cout << "вызвался конструктор копирования Set" << endl;
    }
 
    ~Set() {
        cout << "вызвался деструктор Set" << endl;
        delete[] data;
    }
 
    // перегрузить оператор присваивания
    Set& operator =(const Set& b) {
        // Выполнить копирование значений
        if (this != &b)
        {
            cursize = b.cursize;
            for (int i(0); i < cursize; i++)
            {
                data[i] = b.data[i];
            }
        }
        // Возвратить текущий объект, чтобы иметь возможность связать в цепочку выполнение нескольких операций присваивания
        return *this;
    }
    // ...
    // в остальное не вникал
    // ...

и функцию main()

C++

int main() {
    setlocale(LC_ALL, "ru");
    std::cout << "Hello World!\n";
 
    Set a;
    a = a + 'm' + 'p' + 'a' + 'd' + 'e' + 'x' + 'w';
    a = a - 'M';
    a = a - 'k';
    a.Print();
 
    Set b;
    b = b + 'a' + 't' + 'e' + 'm' + 'i';
    b.Print();
 
    Set c;
    c = c + 'm' + 'p' + 'a' + 'd' + 'e' + 'x' + 'w' + 'r';
    c.Print();
    c.extract();
    c.Print();
 
    if (a == b) cout << "yes, a = b" << endl;
    else cout << "a != b" << endl;
    if (a == c) cout << "yes, a = c" << endl;
    else cout << "a != c" << endl;
 
    if (a != b) cout << "yes, a != b" << endl;
    else cout << "a == b" << endl;
    if (a != c) cout << "yes, a != c" << endl;
    else cout << "a == c" << endl;
 
    Set Q;
    Set q2;
    Q = Q + 'A' + 'B' + 'O' + 'R' + 'A' + 'B' + 'W' + 'I';
    Q.Print();
 
    Set q;
    q = q + 'O' + 'A' + 'R' + 'I' + 'W';
    q.Print();
    q2 = Q;
    q2.Print();
 
    if (Q > q)
        cout << "подмножество q принадлежит Q" << endl;
    else
        cout << "НЕТ! подмножество q не принадлежит Q" << endl;
    Q.extract();
    Q.Print();
    if (Q > q) cout << "подмножество q принадлежит Q " << endl;//вызвать оператор проверки на подмножество
    else cout << "НЕТ! подмножество q не принадлежит Q" << endl;
    Q.extract();
 
    Q.Print();
    q.Print();
 
    if (Q > q) cout << "подмножество q принадлежит Q " << endl;//вызвать оператор проверки на подмножество
    else cout << "НЕТ! подмножество q не принадлежит Q!" << endl;
    return 0;
 
}

@NNNNNNNNN · 14.12.2019, 23:05 **[ТС]**

) смысл? задание - сделать массив динамическим. преподаватель так придумал.

Добавлено через 23 минуты
спасибо! теперь работает, пытаюсь понять где была проблема - можете намекнуть? если есть время, конечно. Но спасибо в любом случае!

Добавлено через 46 минут
стала искать определение двоеточия
оно имеет название? где про это прочитать?
это оно, нет? понять дико сложно, конечно...

Stack overflow-
"Область кода за двоеточием и до начала тела конструктора называется инициализатором конструктора.

Используется как для инициализации членов класса, так и для вызова конструктора базового/базовых классов, т.е. по сути, инициализации базовой составляющей. Также здесь может быть вызов другого конструктора текущего класса (делегирование конструкторов, начиная с c++11)."

При вызове деструктора в строке delete[] data выбрасывается исключение

Решение

Решение

@zayats80888 6340 / 3511 / 1427 Регистрация: 07.02.2019 Сообщений: 8,977
	13.12.2019, 21:07	2
	Увидеть бы выражение, в котором выбрасывается исключение, а по этому обрывку можно сказать, что не хватает конструктора по умолчанию и оператора присваивания 0

Verevkin Злостный нарушитель 9492 / 5139 / 1172 Регистрация: 12.03.2015 Сообщений: 24,221
	13.12.2019, 21:08	3
	покаж весь класс, не стесняйся. 0

@NNNNNNNNN 0 / 0 / 0 Регистрация: 27.02.2016 Сообщений: 68
	14.12.2019, 20:06 [ТС]	8
	исключение на строчку data[i] = D[i]; Миниатюры 0

@zayats80888 6340 / 3511 / 1427 Регистрация: 07.02.2019 Сообщений: 8,977
	14.12.2019, 20:13	9
	Сообщение от NNNNNNNNN исключение на строчку data[i] = D[i]; покажи 16 строку функции main Добавлено через 5 минут Хотя лучше всю функцию main(только выдели комментарием строку, которая в твоем проекте №16) 0

@zayats80888
	14.12.2019, 20:47 #11
	Не по теме: Сообщение от TheCalligrapher Что такое "удалять поэлементно"? скорее всего это попытка последовать совету из непонятно к чему относящемуся поста №5 0

@NNNNNNNNN 0 / 0 / 0 Регистрация: 27.02.2016 Сообщений: 68
	14.12.2019, 21:33 [ТС]	12
	вот весь main работает, пока закоментировано //delete[] data; в деструкторе (при php.h которое в начале этой темы) int Max_size = 20;// int main() { setlocale(LC_ALL, "ru"); std::cout << "Hello World!\n"; char d=nullptr; Set a = Set(Max_size, d); a = a + 'm' + 'p' + 'a' + 'd' + 'e' + 'x' + 'w'; a = a - 'M'; a = a - 'k'; a.Print(); Set b = Set(Max_size, d); b = b + 'a' + 't' + 'e' + 'm' + 'i'; b.Print(); Set c = Set(Max_size, d); c = c + 'm' + 'p' + 'a' + 'd' + 'e' + 'x' + 'w' + 'r'; c.Print(); c.extract(); c.Print(); if (a == b) cout << "yes, a = b" << endl; else cout << "a != b" << endl; if (a == c) cout << "yes, a = c" << endl; else cout << "a != c" << endl; if (a != b) cout << "yes, a != b" << endl; else cout << "a == b" << endl; if (a != c) cout << "yes, a != c" << endl; else cout << "a == c" << endl; Set Q = Set(Max_size, d); Set q2 = Set(Max_size, d); Q = Q + 'A' + 'B' + 'O' + 'R' + 'A' + 'B' + 'W' + 'I'; Q.Print(); Set q = Set(Max_size, d); q = q + 'O' + 'A' + 'R' + 'I' + 'W'; q.Print(); q2 = Q; q2.Print(); if (Q > q) cout << "подмножество q принадлежит Q" << endl; else cout << "НЕТ! подмножество q не принадлежит Q" << endl; Q.extract(); Q.Print(); if (Q > q) cout << "подмножество q принадлежит Q " << endl;//вызвать оператор проверки на подмножество else cout << "НЕТ! подмножество q не принадлежит Q" << endl; Q.extract(); Q.Print(); q.Print(); if (Q > q) cout << "подмножество q принадлежит Q " << endl;//вызвать оператор проверки на подмножество else cout << "НЕТ! подмножество q не принадлежит Q!" << endl; return 0; } Добавлено через 2 минуты* да, "это попытка последовать совету из поста №5" 0

@zayats80888 6340 / 3511 / 1427 Регистрация: 07.02.2019 Сообщений: 8,977
	14.12.2019, 21:44	13
	Сообщение от NNNNNNNNN Set a = Set(Max_size, d); ну ясен пень, в чем смысл такого вызова? Set a; Сообщение от NNNNNNNNN char d=nullptr; удалить Добавлено через 6 минут* Смысл maxsize я понял, только зачем тут динамический массив? Можно статическим обойтись 0

@NNNNNNNNN 0 / 0 / 0 Регистрация: 27.02.2016 Сообщений: 68
	14.12.2019, 23:05 [ТС]	16
	) смысл? задание - сделать массив динамическим. преподаватель так придумал. Добавлено через 23 минуты спасибо! теперь работает, пытаюсь понять где была проблема - можете намекнуть? если есть время, конечно. Но спасибо в любом случае! Добавлено через 46 минут стала искать определение двоеточия оно имеет название? где про это прочитать? это оно, нет? понять дико сложно, конечно... Stack overflow- "Область кода за двоеточием и до начала тела конструктора называется инициализатором конструктора. Используется как для инициализации членов класса, так и для вызова конструктора базового/базовых классов, т.е. по сути, инициализации базовой составляющей. Также здесь может быть вызов другого конструктора текущего класса (делегирование конструкторов, начиная с c++11)." 0