VPF::помогите с деструктором - Форум программистов

tilser

Дата 30.1.2007, 11:22 (ссылка)

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

Загрузка ...

Новичок

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

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

имеется класс

Код


class matrix 
   { 
      int MRows; 
      int MColumns; 
      double **M; 
   public: 
      int Rows() 
         { 
            return MRows; 
         } 
      int Columns() 
         { 
            return MColumns; 
         } 
      matrix(int r, int c) 
         { 
            MRows = r; 
            MColumns = c; 
            M = new double *[r]; 
            for (int i = 0; i < MColumns; i ++) 
               M[i] = new double[r]; 
         } 
      double & operator()(int r, int c) 
      { 
         return M[r - 1][c - 1]; 
      }

по логике вещей деструктор должен быть такой

Код


~matrix() 
{ 
for (int i = 0; i < MColumns; i ++) 
  delete [] M[i]; 
 delete [] M; 
}

но иногда при выходе из программы вылетает ошибка:
"Access violation reading location 0xfeeefee2."
отслеживая в дебагере стновится ясно, что это имненно из-за деструктора
кто может подсказать в чем проблема smile

заранее спасибо smile

MAKCim

Дата 30.1.2007, 11:29 (ссылка)

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

Загрузка ...

Воін дZэна

Профиль
Группа: Экс. модератор
Сообщений: 5644
Регистрация: 10.12.2005
Где: Менск, РБ

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

tilser,
определи
operator(const matrix&)
operator=(const matrix&)

--------------------

Ах, у елі, ах, у ёлкі, ах, у елі злыя волкі ©

Anikmar

Дата 30.1.2007, 11:56 (ссылка)

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

Загрузка ...

Эксперт

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

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

Цитата(tilser @ 30.1.2007, 11:22

)

for (int i = 0; i < MColumns; i ++)
M[i] = new double[r // <----- Вот здесь точно r а не с?,];

KelTron

Дата 30.1.2007, 13:57 (ссылка)

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

Загрузка ...

Опытный

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

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

По логике вещей конструктор должен быть такой:

Код


matrix(int r, int c) 
{ 
            MRows = r; 
            MColumns = c; 
            M = new double *[MRows]; 
            for (int i = 0; i < MRows; i++) 
               M[i] = new double[MColumns]; 
}

а деструктор соответственно:

Код


~matrix() 
{ 
   for (int i = 0; i < MRows; i++) 
       delete [] M[i]; 
   delete [] M; 
}

У тебя получается, что ты выделяешь память под r елементов для массива M, а заполняешь как для с еслементов.

Это сообщение отредактировал(а) KelTron - 30.1.2007, 14:06

--------------------

Тысячами незримых нитей обвивает тебя Закон. Разрубишь одну - преступник. Десять - смертник. Все - Бог.
Эвенгар Салладорский, основатель Школы Тьмы.

Xenon

Дата 30.1.2007, 20:40 (ссылка)

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

Загрузка ...

Эксперт

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

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

Вроде так

Код


class matrix 
{
    int MRows; 
    int MColumns; 
    double **M; 
public:
    int Rows() 
    {
        return MRows; 
    }
    int Columns() 
    { 
        return MColumns; 
    } 
    matrix(int r, int c) 
    {
        MRows = r; 
        MColumns = c; 
        M = new double *[r]; 
        for (int i = 0; i < MColumns; i++) 
        {
            M[i] = new double[MColumns]; 
        }
    }
     double& operator()(int r, int c) 
     { 
      return M[r - 1][c - 1]; 
     }
    // matrix():M(NULL) {}
     ~matrix() 
     {
         //if (M!=NULL)
         //{
           for (int i = 0; i < MRows; i++) 
           {
               delete [] M[i]; 
           }
           delete [] M; 
         //}
     }
};

--------------------

Rockie

Дата 30.1.2007, 21:44 (ссылка)

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

Загрузка ...

Эксперт

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

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

В твоем случае матрица квадратная:

Код


      matrix(int r, int c) 
         { 
            MRows = r; 
            MColumns = c; 
            M = new double *[r]; 
            for (int i = 0; i < MColumns; i ++) 
               M[i] = new double[r]; 
         }

корректно так:

Код


      matrix(int r, int c) 
         { 
            MRows = r; 
            MColumns = c; 
            M = new double *[MRows]; 
            for (int i = 0; i < MRows; i ++) 
               M[i] = new double[MColumns]; 
         }

в деструторе тоже ошибка, ты ведь сначала выделяешь память под строки:

Код


~matrix() 
{ 
for (int i = 0; i < MRows; i ++) 
  delete [] M[i];    
 delete [] M;        
}

--------------------

Чтобы иметь большой гардероб - надо иметь большой гардероб.

tilser

Дата 31.1.2007, 11:19 (ссылка)

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

Загрузка ...

Новичок

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

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

спасибо, понял, в чем ошибка.

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