С Новым годом! Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.77/13: Рейтинг темы: голосов - 13, средняя оценка - 4.77
1 / 1 / 0
Регистрация: 07.01.2015
Сообщений: 48

Деструктор вызывает исключение Access violation

02.02.2016, 18:17. Показов 2573. Ответов 12
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Создаю велосипед - класс с динамическим массивом. При вызове деструктора срабатывает исключение Access violation. Без деструктора программа работает нормально. Как решить проблему?
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
class MyVector
{        // класс с динамическим массивом
  private:
 
  public:
    int size;
    int* Element;
    MyVector(int n,int *m)  // или MyVector(int n,int m[])
    {
      size=n;
      Element = new int[size];
      for (int i=0;i<n;i++)
        Element[i]=m[i];
    }
    ~MyVector()
    {
      delete[] (Element);   //<-  !!Вот здесь возникает исключение
    }
/*
    int* operator[](int ind)
    {
        return Element+ind;
    }   */
 
};
 
MyVector operator+ (MyVector a, MyVector b)
{
   if (a.size!=b.size) throw;
   int *m= new int[a.size];
   MyVector c(a.size,m);
   for(int i=0;i<a.size;i++)
     c.Element[i]=a.Element[i]+b.Element[i];
   return(c);
}
Работа с классом:
C++
1
2
3
4
5
  int *m= new int[4];
  m[0]=1;m[1]=2;m[2]=3;m[3]=4; MyVector vect(4,m);
  m[0]=2;m[1]=1;m[2]=2;m[3]=4; MyVector vect1(4,m);
  m[0]=0;m[1]=0;m[2]=0;m[3]=0; MyVector vect2(4,m);
  vect2=vect+vect1;
0
Лучшие ответы (1)
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
02.02.2016, 18:17
Ответы с готовыми решениями:

Исключение Access violation writing location 0x0582D000
При попытке запустить программу на других ПК возникает исключение &quot;Access violation writing location 0x0582D000.&quot;, но не на всех ПК на...

Необработанное исключение по адресу 0x4050D302 в ConsoleApplication17.exe: 0xC0000005: Access violation (параметры: 0x00000008)
Добрый вечер. Я в сети нашел программу градиентному методу дробления шага и изменил в ней функцию на свою и градиенты на свои, но почему то...

Исключение "unwanted exception at 0x775c15ee Access violation" при считывание из файла
Добрый вечер! Подскажите пожалуйста. У меня в файле записана следующая строка b6589fc6abdc8ac. Я её считываю следующим образом char...

12
 Аватар для avgoor
1550 / 877 / 179
Регистрация: 05.12.2015
Сообщений: 2,555
02.02.2016, 18:27
Лучший ответ Сообщение было отмечено BOEH как решение

Решение

Цитата Сообщение от BOEH Посмотреть сообщение
MyVector operator+ (MyVector a, MyVector b)
Здесь аргументы передаются по значению, а т.к. конструктор копирования не определен вызывается сгенерированный, который копирует MyVector побитово. В конце operator+ вызываются деструкторы для аргументов (которые удаляют память) в конце main вызываются деструкторы vect и vect1 (удаляющие ту же самую память)

Резюме:
Реализовать MyVector(const MyVector&);
MyVector operator+(MyVector& a, MyVector& b) здесь надо передавать по ссылке.

Добавлено через 2 минуты
Есть правило трех - Если нужно хоть что-то из: деструктор, копирующий конструктор и оператор присваивания, реализуйте все вышеперечисленное.
2
Эксперт С++
 Аватар для hoggy
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
02.02.2016, 18:32
Цитата Сообщение от BOEH Посмотреть сообщение
Без деструктора программа работает нормально.
нарушение правила "трёх".

Распространенные ошибки
2
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
02.02.2016, 18:38
Кроме этого, в operator+, у тебя утечка памяти: int *m= new int[a.size];
Указатель локальный, исчезает при выходе из функции.
1
1 / 1 / 0
Регистрация: 07.01.2015
Сообщений: 48
03.02.2016, 08:31  [ТС]
Все спасибо за помощь. Сделал в соответствии с Распространенные ошибки. Получилось следующее:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
//---------------------------------------------------------------------------
 
#include <vcl.h>
#include <conio.h>
#pragma hdrstop
 
//---------------------------------------------------------------------------
 
#pragma argsused
 
class vect
{
private:
public:
    int *Element; // указатель на массив
    int Size;
    vect() :Element(0),Size(0) {}
    vect(int N, int *m) : Size(N)
    {
        Element = new int [N]; // под массив выделяется память - нужен деструктор для ее удаления
        for (int i(0);i<N;i++)
        {
          Element[i]=m[i];
        }
    }
 
    ~vect() // Создаем деструктор
    {
        delete []Element;
    }
 
    vect(const vect &a ):Size(a.Size) //  Обязательный копиконструктор (раз есть деструктор)
    {
        Element = new int [a.Size];
        for ( int i = 0; i < a.Size; i++ )
            Element[i] = a.Element[i];
    }
 
    vect &operator = ( const vect &a ) // Обязательный оператор присвоения (раз есть деструктор)
    {
        if ( this == &a ) 
            return *this; // присвоение самому себе, ничего делать не надо
 
        if ( Element != 0 )
            delete []Element;
 
        Element = new int [a.Size];
        for ( int i = 0; i < a.Size; i++ )
            Element[i] = a.Element[i];
    }
 
    vect operator+ ( vect &a ) // сложение векторов
    {
        if (this->Size!=a.Size) throw;
        vect c;
        c.Size = a.Size;
        c.Element = new int[c.Size];
        for (int i = 0; i < this->Size; i++ )
            c.Element[i] = this->Element[i]+a.Element[i];
        return c; // для передачи по значению используется копиконструктор
    }
};
 
int main()
{
    int m[5]={0,1,2,3,4}; vect a1(5,m);
    int m1[5]={5,6,7,8,9}; vect a2(5,m1);
    vect a3 = a1 + a2; // используется оператор присвоения
    getch();
    return 0;
}
Программа работает правильно, но все ли я сделал как надо? И к каким последствиям могло привести отсутсвтие деструктора в моем первоначальном варианте (результат верный, серьезной утечки памяти не обнаружил).
На сколько оправдано с точки зрения скорости применение перегрузок операторов и копиконструктора?

Добавлено через 1 минуту
Цитата Сообщение от nd2 Посмотреть сообщение
Кроме этого, в operator+, у тебя утечка памяти: int *m= new int[a.size];
Указатель локальный, исчезает при выходе из функции.
Подробнее можете объяснить, чем это вызвано?
0
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
03.02.2016, 15:05
Цитата Сообщение от BOEH Посмотреть сообщение
Подробнее можете объяснить, чем это вызвано?
В operator+ выделяешь память под указатель m, передаёшь его копию в конструктор, там выделяется новая память (под Element), куда копируется содержимое памяти по m. После выхода из operator+, доступ к памяти через m теряется (m исчезает, т.к. локальный).
В новом коде (5 пост) этого уже нет.

Добавлено через 10 минут
В operator = нет return *this(если первый if() не сработает), и if() тут лишний (просто delete []Element;):
Цитата Сообщение от BOEH Посмотреть сообщение
C++
1
2
if ( Element != 0 )
   delete []Element;
1
1 / 1 / 0
Регистрация: 07.01.2015
Сообщений: 48
06.02.2016, 12:54  [ТС]
Цитата Сообщение от nd2 Посмотреть сообщение
и if() тут лишний (просто delete []Element:
Да он и здесь Распространенные ошибки кажется лишним, однако почему-то стоит
0
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
06.02.2016, 15:41
Цитата Сообщение от BOEH Посмотреть сообщение
Да он и здесь Распространенные ошибки кажется лишним, однако почему-то стоит
Где он там стоит?
Суть проста: освобождение памяти по нулевому указателю не приводит ни к каким действиям. Тогда смысл, перед delete, проверять указатель на равенство нулю?
0
1 / 1 / 0
Регистрация: 07.01.2015
Сообщений: 48
07.02.2016, 10:23  [ТС]
Цитата Сообщение от nd2 Посмотреть сообщение
Где он там стоит?
Суть проста: освобождение памяти по нулевому указателю не приводит ни к каким действиям. Тогда смысл, перед delete, проверять указатель на равенство нулю?
Распространенные ошибки #30 пост:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
class A
{
private:
    int * x; // указатель на массив
    int n;
 
public:
    A() : x( 0 ), n( 0 ) {}
    A( int N ) : n( N )
    {
        x = new int [N]; // под массив выделяется память - нужен деструктор для ее удаления
        for ( int i = 0; i < N; i++ )
            x[i] = i;
    }
 
    ~A() // Создаем деструктор
    {
        delete [] x;
    }
 
    A( const A & a ) : n( a.n ) //  Обязательный копиконструктор (раз есть деструктор)
    {
        x = new int [a.n];
        for ( int i = 0; i < a.n; i++ )
            x[i] = a.x[i];
    }
 
    A & operator = ( const A & a ) // Обязательный оператор присвоения (раз есть деструктор)
    {
        if ( this == &a ) 
            return *this; // присвоение самому себе, ничего делать не надо
 
        delete [] x;
 
        x = new int [a.n];
        for ( int i = 0; i < a.n; i++ )
            x[i] = a.x[i];
    }
 
    A operator + ( A & a ) // слияние массивов 
    {
        A c;
        c.n = this->n + a.n;
        c.x = new int [c.n];
 
        int i = 0;
        for ( ; i < this->n; i++ )
            c.x[i] = this->x[i];
 
        int j = 0;
        for ( ; i < c.n; i++, j++ )
            c.x[i] = a.x[j];
 
        return c; // для передачи по значению используется копиконструктор
    }
};
 
int main()
{
    A a1( 2 ), a2( 3 );
    A a3 = a1 + a2; // используется оператор присвоения
    return 0;
}
Я все делал по этому шаблону, только конструктор более удобный для себя сделал, и оператор+ уменя не массивы сцепляет, а элементы векторов складывает. Использование оператора= , мне кажется, должно быть аналогичным.
0
Эксперт С++
4986 / 3093 / 456
Регистрация: 10.11.2010
Сообщений: 11,170
Записей в блоге: 10
07.02.2016, 13:02
Цитата Сообщение от nd2 Посмотреть сообщение
Где он там стоит?
Я её убрал.
В примере ТС она была в 33-й строке.
0
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
07.02.2016, 16:33
BOEH,
Цитата Сообщение от castaway Посмотреть сообщение
Я её убрал.
Цитата Сообщение от nd2 Посмотреть сообщение
и if() тут лишний (просто delete []Element
0
1 / 1 / 0
Регистрация: 07.01.2015
Сообщений: 48
08.02.2016, 11:29  [ТС]
Чтобы никого не запутать, может и комментарий в код добавить, что там эта строчка была? И return в конце бы не помешал
1
08.02.2016, 11:33

Не по теме:

Цитата Сообщение от BOEH Посмотреть сообщение
Чтобы никого не запутать
Чтобы никого не запутать я написал 10-е сообщение.

Цитата Сообщение от BOEH Посмотреть сообщение
И return в конце бы не помешал
А вот тут вы правы.

0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
08.02.2016, 11:33
Помогаю со студенческими работами здесь

Шаблон класса деструктор вызывает ошибку
Не так чего после того как доходит до деструктора выдает такую ошибку #include &quot;stdafx.h&quot; #include &lt;stdio.h&gt; ...

Вызывает исключение
Не пойму почему при выполнении вызывает исключение char *word=&quot;&quot;; cout &lt;&lt; &quot;Введите добовляемое слово:&quot;; cin&gt;&gt;word; ...

Strcat вызывает исключение
Ребят, выручайте, не пойму ни черта. Пытаюсь соединить строки и всё время вылетают &quot;исключения&quot;, чтоб их. Вот имеющийся...

Массив элементов класса, деструктор которого кидает исключение
#include &lt;cstdio&gt; class SSS { int x; public: SSS() { } ~SSS() { fprintf (stderr, &quot;destr %p\n&quot;,...

Access violation
Помогите пожалуйста. Не могу разобраться. Нашел в каком месте ругается. Но не понял на что именно.void __fastcall...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
13
Ответ Создать тему
Новые блоги и статьи
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост.
Programma_Boinc 28.12.2025
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / **********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / **********/ gallery/ bKBkQFf Пост отсюда. . .
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США.
Programma_Boinc 26.12.2025
Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru