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

ткчто спросил на всякий

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

если throw не обрабатывают, произойдёт утечка, я проверял, деструктор не вызовется

Вопрос непонятен. Уточни.

ну смотри, я разрабатываю класс матрицы. шаблонный. допустим произошла исключительная ситуация: пользователь по какой-то причиние умножил матрицы неподходящего размера. получить результат мы не можем, ибо операция алгебраически не определена. тогда я throw Exception();

что если пользователь умножал их не в блоке try{}catch()

эксепшн вздёрнет программу по SIGTERM

Тебя это не касается.

и память под объект утечёт (проверял валгриндом)

то есть, нет смысла делать delete перед throw?

Тебе нужно только обеспечить корректное освобождение ресурсов при уничтожении объекта.

хорошо, а где вообще прописана зона ответственности проектировщика классов? я не встречал. понятно, что все ситуации я не предусмотрю, но я хочу чтобы юзеры не плевались от моего класса и могли легко писать приложения, ага

Если твоя матрица представлена внутри как обычный динамический массив, то в конструкторе ты выделяешь под него память, а в деструкторе освобождаешь ее.

что мне надо для этого сделать?

Если твоя матрица представлена внутри как обычный динамический массив, то в конструкторе ты выделяешь под него память, а в деструкторе освобождаешь ее.

да, это я делаю. но при throw без catch уловил утечку. и думаю сижу...

Как создаешь экземпляры матриц?

template <class T> Matrix<T>::Matrix(const int _rows, const int _cols): rows(_rows), cols(_cols){ #ifdef DEBUG std::cout << "Constructor (int, int) called\n"; #endif if ((_rows < 1) || (_cols < 1)) throw Matrix<T>::ExceptionWrongInitialParams(); data = new T[rows*cols]; for (int i = 0; i < rows*cols; data[i++] = (T)0); } template <class T> Matrix<T>::Matrix(const Matrix<T> &copy): rows(copy.rows), cols(copy.cols){ #ifdef DEBUG std::cout << "Copy constuctor called\n"; #endif data = new T[rows*cols]; for (int i = 0; i < cols*rows; data[i] = copy.data[i], i++); }

других вариантов пока нет

Я спросил, как ты матрицы перемножаешь.

И где деструктор?

Как создаешь экземпляры матриц?

И где деструктор?

template <class T> Matrix<T>::~Matrix(){ #ifdef DEBUG std::cout << "Destructor called!\n"; #endif delete [] data; }

перемножая создаю первым конструктором матрицу, забиваю значениями, возвращаю

вот и всё

если размер неверный, throw

Ок. Я имел в виду, что непонятен сценарий использования класса, который вызывает утечку.

а, ну банально, смотри

template <class T> bool Matrix<T>::isSquare(){ return (cols == rows); } template <class T> T Matrix<T>::getTrack(){ if (this->isSquare()){ T track = (T)0; for (int i = 0; i < cols; track += (*this)[i][i], ++i); return track; } delete [] data; throw ExceptionSquareOperationForNonSquare(); } вот есть такие 2 функции

а я делаю где-то в программе так: Matrix <int> A(5,4); A.getTrack();

вот и всё, взлетает эксепшн "операция для неквадратной матрицы"

к ним же относится определитель, например

Конструкция в цикле это шедевр, конечно. :)

ты не поверишь, чего мне она стоила) раньше i++ стоял во вторых скобках, но это постоянно вызывало undefined behaviour)

А зачем освобождать память где-то еще, кроме деструктора?

так вот в чём дело. если за эксепшном никто не следит, то деструктор не вызывается

Эта функция возвращает сумму диагональных элементов?

и память не освобождается (если система не позаботится о том)

да, это след матрицы)

относится, что логично, только к квадратным

for (std::size_t i = 0; i < _rows; i++) track += data[i][i];

Так же гораздо понятнее?

ну теперь да, вообще говоря

не поверишь, я поменял, но откатился на старую ветку)

Только вот не помню, откуда компилятор берет информацию о размерности двумерного массива. Безопаснее писать data[i*_rows+j]

Только вот не помню, откуда компилятор берет информацию о размерности двумерного массива. Безопаснее писать data[i*_rows+j]

оооооооооооо. это извращение стоило мне трёх часов) там создаётся подобъект и перегружается два оператора []

оригинальный массив одномерный)

Т.е. ты перегрузил оператор двойные скобки?

дважды

А, читал вчера

наконец разобрался с константными функциями-членами и их комбинацией с this

там интересная лабуда

Вызов delete[] в функции расчета следа это бред полный. После этого объект переходит в неработоспособное состояние.

к слову, математический след по-английиски это trace а не track

Эта функция вообще должна быть const.

Вызов delete[] в функции расчета следа это бред полный. После этого объект переходит в неработоспособное состояние.

++

вообще говоря, я руководствовался той логикой, что раз объект вызвал исключение, он уже неработоспособен и надо стереть память пока не поздно

однако придумал-таки ситуацию, где этого делать не стоит и постирал дилиты

а потом откатился к старой ветке

Нет, это не так

так что спасибо, что внимательно читали и напомнили)

Эта функция вообще должна быть const.

спасибо!

к слову, математический след по-английиски это trace а не track

а на мехмате сказали трэк, прикинь) исправлю, спасибо

Твой класс предназначен для хранения матриц произвольных размерностей. Ты решил добавить функцию-член для подсчета следа, который определен только для квадратных. У тебя два выбора. Либо ты просто выбрасываешь исключение при вызове этой функции для неквадратной матрицы, либо создаешь подкласс SquareMatrix и уже для него реализуешь эту функцию. Но тогда его конструктор будет другой.

Второй вариант лучше не используй.

Твой класс предназначен для хранения матриц произвольных размерностей. Ты решил добавить функцию-член для подсчета следа, который определен только для квадратных. У тебя два выбора. Либо ты просто выбрасываешь исключение при вызове этой функции для неквадратной матрицы, либо создаешь подкласс SquareMatrix и уже для него реализуешь эту функцию. Но тогда его конструктор будет другой.

думал про square matrix. и не хочу. потому что если перемножить разноимённые прямоугольные, получим квадратную. надо постоянно проверять квадратность многих операций, а класс объекту уже не сменишь. это либо будет держаться на туче костылей, либо не моего уровня дело. так что я по старинке швырнусь эксепшном)

ERROR: S client not available

Хорошее решение.

есть тут кто-нить ещё?

м?

м?

о, американец, ты тут

слушай, я не понимаю, мы же можем вызывать константные методы для неконст объектов, так?

и никаких проблем быть не должно

однако я ловлю тучу проблем при вызове неконстантных методов для неконстантных объектов...

да и вообще тучу говна при вызовах

хз почму

главная проблема вот в чём. я не могу обратиться к this почему-то..

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

тут, да, чем еще заняться воскресным вечером?)

нахер понедельничную тоску потому что

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

да я всё ещё со своими двойными скобками, короче

показывай код)

не, я чуть позже, плиз

тут есть что поправить

ok

итак, есть константный объект

оператор [] от него возвращает другой константный объект

оператор [] от второго возвращает константный тип

так схуяяле нельзя передавать this в первом операторе второму объекту?

ща буит код

второй объект должен принимать его как const

http://ideone.com/srjnK3

вот. смотри

в строчке 173 обращаемся к константе

вызывается строчка 135

объект константный!! оператор тоже константный! генерит константный подобъект

хуле ему не нравится?!