Форум программистов, компьютерный форум CyberForum.ru

нужен ли свой деструктор - C++

Восстановить пароль Регистрация
 
red_88
6 / 6 / 0
Регистрация: 10.02.2009
Сообщений: 136
17.10.2010, 15:59     нужен ли свой деструктор #1
Есть класс
C++
1
2
3
4
5
6
7
8
9
10
class Vetv
{
public:
    Vetv(int a,int b, int *ves);
    Vetv();
    virtual ~Vetv(void);
    int nFrom;
    int nTo;
    int *Ves;
};
в нём виден динамический массив *Ves.
В одной из процедур программы выполняется такое действие:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
    while (!iFile.eof())
    {
        Vetv tmp;
        int from,to;
        from=to=0;
        iFile>>from;
        iFile>>to;
        for(int i=0;i<count;i++)
        {
            iFile>>params[i];
        }
        tmp=Vetv(from,to,params);
        Versh.push_back(tmp);
    }
 
    iFile.close();
где vector<Vetv> Versh;
Соответственно многократно вызывается деструктор ~Vetv(). Вопрос, надо ли в деструкторе явно писать
C++
1
2
3
4
Vetv::~Vetv(void)
{
    delete[] Ves;
}
или нет? Обращений предполагается порядка 10^8 поэтому память хотелось бы поэкономить.

Спасибо.
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
.::.DIMA.::.
142 / 142 / 4
Регистрация: 26.10.2008
Сообщений: 782
17.10.2010, 16:02     нужен ли свой деструктор #2
Надо.
red_88
6 / 6 / 0
Регистрация: 10.02.2009
Сообщений: 136
17.10.2010, 16:32  [ТС]     нужен ли свой деструктор #3
вот и я так подумал. а компилятор говорит:
Код
Необработанное исключение в "0x590e59da (msvcr100d.dll)"
при одном из первых (обычно 5-е) вхождений в деструктор. И выкидывает меня на строку
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
void operator delete(
        void *pUserData
        )
{
        _CrtMemBlockHeader * pHead;
 
        RTCCALLBACK(_RTC_Free_hook, (pUserData, 0));
 
        if (pUserData == NULL)
            return;
 
        _mlock(_HEAP_LOCK);  /* block other threads */
        __TRY
 
            /* get a pointer to memory block header */
            pHead = pHdr(pUserData);
 
             /* verify block type */
            _ASSERTE(_BLOCK_TYPE_IS_VALID(pHead->nBlockUse));<--вот эту строку-->
 
            _free_dbg( pUserData, pHead->nBlockUse );
 
        __FINALLY
            _munlock(_HEAP_LOCK);  /* release other threads */
        __END_TRY_FINALLY
 
        return;
}
в dbgdel.cpp

Добавлено через 26 минут
ПОлностью ошибка звучит так:
Необработанное исключение в "0x5a8159da (msvcr100d.dll)" в "MyFirst.exe": 0xC0000005: Нарушение прав доступа при чтении "0xcdcdcdc1".
.::.DIMA.::.
142 / 142 / 4
Регистрация: 26.10.2008
Сообщений: 782
17.10.2010, 17:34     нужен ли свой деструктор #4
Тогда проверяйте Vec на равенство с NULL.
red_88
6 / 6 / 0
Регистрация: 10.02.2009
Сообщений: 136
18.10.2010, 14:21  [ТС]     нужен ли свой деструктор #5
C++
1
2
3
4
Vetv::~Vetv(void)
{
    if (Ves!=NULL) delete[] Ves;
}
и та же ошибка...
.::.DIMA.::.
142 / 142 / 4
Регистрация: 26.10.2008
Сообщений: 782
19.10.2010, 00:21     нужен ли свой деструктор #6
Покажите код полностью.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
21.10.2010, 13:57     нужен ли свой деструктор
Еще ссылки по теме:

C++ Зачем нужен virtual деструктор?
C++ Виртуальный деструктор, для чего нужен?
Нужен ли в программе деструктор C++

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

Или воспользуйтесь поиском по форуму:
red_88
6 / 6 / 0
Регистрация: 10.02.2009
Сообщений: 136
21.10.2010, 13:57  [ТС]     нужен ли свой деструктор #7
UzelVC.h
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
#pragma once
 
#include <vector>
#include <iostream>
#include <fstream>
using namespace std;
 
class Vetv
{
public:
    Vetv(int a,int b, int *ves);
    Vetv();
    virtual ~Vetv(void);
    int nFrom;
    int nTo;
    int *Ves;
};
 
class UzelVC
{
private:
    vector<Vetv> Versh;
public:
    static int count;
    UzelVC(char *InpFile);
    virtual ~UzelVC(void);
    void PrintRes(int a,int b); 
};
UzelVC.cpp
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
#include "StdAfx.h"
#include "UzelVC.h"
 
UzelVC::UzelVC(char *InpFile)
{   
    char buf[10];
    ifstream  iFile(InpFile,ios::in);
    if (!iFile)  {
                cout << "Error of opening of a file.";  cin>>buf;
        }
    iFile>>count;
    int *params= new int[count];
    int i;
    while (!iFile.eof())
    {
        Vetv tmp;
        int from,to;
        from=to=0;
        iFile>>from;
        iFile>>to;
        for(int i=0;i<count;i++)
        {
            iFile>>params[i];
        }
        //iFile.getline(buf,10);
        tmp=Vetv(from,to,params);
        //memcpy(tmp.Ves, params, sizeof(params));
        Versh.push_back(tmp);
    }
 
    iFile.close();
}
int UzelVC::count;
void UzelVC::PrintRes(int a,int b)
{
    for each (Vetv V in Versh)
    {
        cout<<V.nFrom<<"  "<<V.nTo<<" ";
        for(int i=0;i<count;i++)
        {
            cout<<V.Ves[i]<<"  ";
        }
        cout<<'\n';
    }
    cin>>a;
}
 
UzelVC::~UzelVC(void)
{
}
 
Vetv::Vetv(int a,int b, int *ves)
{
    nFrom=a;nTo=b;
    Ves=new int[sizeof(ves)*2];
    memcpy(Ves, ves, sizeof(ves)*2);
}
 
Vetv::~Vetv(void)
{
    if (Ves!=NULL) delete[] Ves;
}
 
Vetv::Vetv(void)
{
    
}
Yandex
Объявления
21.10.2010, 13:57     нужен ли свой деструктор
Ответ Создать тему
Опции темы

Текущее время: 19:29. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru