GUI в С++. Boots и Qt. Что использовать?

[rrrFer]

- в конструкторе создаётся двумерный динамический массив matrix[][] c размером 2x2 (nrows x ncols)

- при нажатии на кнопку:

1) удаляется массив matrix[][] размера nrows x ncols

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

[8Observer8]

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

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

Простой пример. Я запустил приложение. Но не загрузил значения в матрицу. Завершил приложение.

Я хотел бы чтобы массив matrix[][] был доступен всегда. Для данного приложения достаточно выделить память перед загрузкой в QTableWidget и освободить после.

Просто логично если matrix будет оставаться неосвобождённым до конца работы приложения. Чтобы можно было бы с ним работать. То есть загрузил один раз из файла. А потом меняешь, например, значения вручную в матрице (QTableWidget) и все изменения и в matrix отражаются.

В данном случае можно согласен. Можно и без выделения в конструктроре и освобождения в деструкторе обойтись.

[rrrFer]

Простой пример. Я запустил приложение. Но не загрузил значения в матрицу. Завершил приложение.

В этом случае тебе вообще ненадо выделять память, и освобождать, соответственно тоже. Если бы ты внимательно читал ответы не пришлось бы повторять. Я хотел предложить тебе читать внимательнее еще вчера )

Сообщение об ошибке (которые ты выше привел) получаешь, наверное, когда метод loadMatrixFromXmlFile не вызывается ниразу (при этом память не выделяется вообще) и при попытки освободить ее ты можешь такое получить.

остальную часть сообщения и пути решения смотри выше.

[rrrFer]

кстати, если очень хочется сделать именно так как ты делаешь - то вместо выделения памяти лишний раз можешь присвоить указателю nullptr. В деструкторе проверяешь были ли выделена память (указатель не равен nullptr) и освобождаешь.
-----
но нехорошо это в твоем случае ИМХО. У тебя есть данные в массиве и есть данные в таблице.

меняешь, например, значения вручную в матрице (QTableWidget) и все изменения и в matrix отражаются.

ты дублируешь данные и выполняешь лишние операции, это плохо.

То есть загрузил один раз из файла.

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

[8Observer8]

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

кстати, если очень хочется сделать именно так как ты делаешь - то вместо выделения памяти лишний раз можешь присвоить указателю nullptr. В деструкторе проверяешь были ли выделена память (указатель не равен nullptr) и освобождаешь.

Это я попробую.

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

У тебя есть данные в массиве и есть данные в таблице.

ты дублируешь данные и выполняешь лишние операции, это плохо.

Я не пойму, почему плохо. Загрузил я данные из файла в matrix. Отобразил их в QTableWidget. Потом вручную изменил в таблице. Нажал кнопку - расчёт какой-нибудь. После нажатия у меня данные считались из таблицы в matrix и произошёл расчёт. То есть во время работы приложения matrix всегда актуален (синхронизирован с тем, что я вижу на экране)

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

я писал что если он загрузит его несколько раз - у тебя утечет память, опять же, читай внимательнее)

Я подробно описал в сообщении № 30 подробно. Там нет утечки памяти, так как память освобождается каждый раз после отрытия файла.

Сообщение от 8Observer8

Вот как ситуация выглядит на данный момент:

- в конструкторе создаётся двумерный динамический массив matrix[][] c размером 2x2 (nrows x ncols)

- при нажатии на кнопку:

1) удаляется массив matrix[][] размера nrows x ncols

2) из файла читаются размеры матрицы: nrows и ncols. Выделяется память для массива matrix[][]. Элементы матрицы инициализируются из файла

- когда мы опять нажимаем кнопку - случаются события 1) и 2)

- когда мы закрываем приложение, то срабатывает деструктор, который освобождает память от массива matrix[][]