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

Ullr · Регистрация: 17.07.2015

Author24 — интернет-сервис помощи студентам

Пишу в Visual Studio 2015 Community

Объявляю двумерный динамический массив так:

C++

int **mas = new int* [sizeY];
for (int i = 0;i < sizeY;i++) 
{
    mas[i] = new int[sizeX];
}

После завершения работы освобождаю память так:

C++

1
2
3

for (int i = 0;i < sizeY;i++)
    delete [] mas[i];
delete[] mas;

В результате на экране вижу эту прекрасную картинку прикрепленную ниже. Эта ошибка происходит всегда именно при значении счетчика i когда он доходит до 2. Подскажите пожалуйста как избавиться от этой ошибки.

Ullr · 13.09.2015, 11:53 **[ТС]**

Не правильно написал в теме...

Объявляю динамический массив так (иначе пишет что массив неиницализирован)

C++

1 2	int *mas = new int[0]; mas[0] = new int[0];

Затем передаю его в функцию где задаю размерность

C++

int **mas = new int* [sizeY];
for (int i = 0;i < sizeY;i++) 
{
    mas[i] = new int[sizeX];
}

А в завершении уже потом освобождаю память

C++

1
2
3

for (int i = 0;i < sizeY;i++)
    delete [] mas[i];
delete[] mas;

@DrOffset · 14.09.2015, 02:22

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

Объявляю динамический массив так

Это неправильно. Такого быть не должно.
Больше без полного кода нельзя сказать.

Ullr · 14.09.2015, 11:12 **[ТС]**

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

Это неправильно. Такого быть не должно.
Больше без полного кода нельзя сказать.

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

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

@DrOffset · 14.09.2015, 14:14

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

После нескольких экспериментов пришел к выводу что освободить память динамического массива можно только в той функции в которой она была выделена

И это также неверный вывод.
Если все-таки есть желание, чтобы научили и подсказали, то лучше выложить проблемный код

Ullr · 14.09.2015, 18:38 **[ТС]**

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

И это также неверный вывод.
Если все-таки есть желание, чтобы научили и подсказали, то лучше выложить проблемный код

Буду очень благодарен если подскажите как исправить проблему

C++

#include "stdafx.h"
#include <stdio.h>
#include <conio.h>
#include <clocale>
#include <stdlib.h>
#include <ctime>
#include <fstream>
#include <io.h>
 
using namespace std;
 
int menu();
bool**  read(bool**, int, int);
bool** hand(bool**, int, int);
bool** random(bool**, int, int);
int writRez();
int analysis();
bool** deleteMas(bool**, int, int);
 
int main() {
    setlocale((LC_ALL), "Russiun");
    setlocale((LC_ALL), "rus");
    menu();
    bool **mass = new bool*[0];
    char p;
    int n = 0, m = 0;
    do
    {
        p = _getch();
        switch (p)
        {
        case 'a':
        case'A':
        {
            if (n > 0) {
                deleteMas(mass, n, m);
            }
            system("cls");
            menu();
            printf("A\nВведите количество строк (число больше 0): ");
 
            ///////////////////////////
            do
            {
                scanf_s("%d", &n);
                while (scanf_s("%c", &p))               // анализ нажатых клавиш после завершения ввода числа
                {                                       // 
                    if (p == '\n')                      //
                    {                                   //
                        break;                          //
                    }                                   //
                    else                                //
                    {                                   //
                        if (p != ' ')                   //
                        {                               //
                            n = 0;                      //
                        }                               //
                    }                                   //
                }
                if (n <= 0) printf("Введено недопустимое значение, повторите ввод: ");
            } while (n <= 0);
            //////////////////////////
            printf("Введите количество столбцов (число больше 0): ");
            do
            {
                scanf_s("%d", &m);
                while (scanf_s("%c", &p))               // анализ нажатых клавиш после завершения ввода числа
                {                                       //
                    if (p == '\n')                      //
                    {                                   //
                        break;                          //
                    }                                   //
                    else                                //
                    {                                   //
                        if (p != ' ')                   //
                        {                               //
                            m = 0;                      //
                        }                               //
                    }                                   //
                }
                if (m <= 0) printf("Введено недопустимое значение, повторите ввод: ");
            } while (m <= 0);
            ////////////////////
            read(mass,n, m);
            system("cls");
            menu();
            printf("A\nМассив успешно заполнен\n");
            printf("\nДля выбора пункта меню введите соответствующую букву: ");
            remove("analysis.txt");
            break;
        }
        case'b':
        case'B':
        {
            system("cls");
            menu();
            printf("B\n");
            analysis();
            break;
        }
        case'c':
        case'C':
        {
            writRez();
            break;
        }
        }
    } while (p != 'D' && p != 'd'&&p != 'В'&&p != 'в');
    return 0;
}
//
//
//
int menu() {
    printf("\t\t\tПрограмма для расчета количества проданных билетов на сеанс\n\t\t\t\t и количества билетов в заданном ряду\n");
    printf("\tA. Ввести данные о проданных билетах\n\tB. Обработать данные\n\tC. Вывести данные\n\tD. Выход\n\n");
    printf("Для выбора пункта меню введите соответствующую букву: ");
    return 0;
}
//
//
//
bool** read(bool** mass, int sizeY, int sizeX) {
    mass = new bool*[sizeY];
    for (int i = 0;i < sizeY;i++)
    {
        mass[i] = new bool[sizeX];
    }
    char p;
    printf("Ручной ввод массива (Y/N): ");
    do {
        p = _getch();
        switch (p) {
        case 'y':
        case 'Y': {
            printf("Y\n");
            printf("\n");
            mass = hand(mass, sizeY, sizeX);
            break;
        }
        case 'n':
        case 'N': {
            printf("N\n");
            mass = random(mass, sizeY, sizeX);
            break;
        }
        }
    } while (p != 'Y' && p != 'y' && p != 'n' && p != 'N');
    return mass;
}
//
//
//
bool** hand(bool** mas, int sizeY, int sizeX)
{
    ofstream out;
    out.open("data.txt");
    out << sizeY << " " << sizeX << endl;
    char p;
    for (int i = 0;i < sizeY;i++)
    {
        printf("Введите данные о %d-ом ряде: \n", (i + 1));
        for (int j = 0;j < sizeX;j++) {
            printf("_");
        }
        printf("\r");
        for (int j = 0;j < sizeX;j++)
        {
            do {
                p = _getch();
            } while (p != '1'&&p != '0');
            if (p == '1')
            {
                mas[i][j] = 1;
                out << "1 ";
            }
            else
            {
                out << "0 ";
                mas[i][j] = 0;
            }
            printf("%c", p);
        }
        out << endl;
        printf("\n");
    }
    out.close();
    return mas;
}
//
//
//
bool** random(bool** mas, int sizeY, int sizeX)
{
    srand(time(NULL));
    ofstream out;
    out.open("data.txt");
    out << sizeY << " " << sizeX << endl;
    for (int i = 0;i < sizeY;i++)
    {
        for (int j = 0;j < sizeX;j++)
        {
            mas[i][j] = rand() % 2;
            out << mas[i][j] << " ";
        }
        out << endl;
    }
    out.close();
    return mas;
}
//
//
//
int analysis()
{
    if (!_access("data.txt", 0)) {
        ifstream in;
        in.open("data.txt");
        ofstream out;
        out.open("analysis.txt");
        int sizeX, sizeY, bought, series = 0;
        bool x;
        in >> sizeY >> sizeX;
        out << sizeY*sizeX << endl << sizeY << " " << sizeX << endl;
        for (int i = 0;i < sizeY;i++)
        {
            bought = 0;
            for (int j = 0;j < sizeX;j++)
            {
                in >> x;
                bought += (int)x;
                series += (int)x;
            }
            out << bought << endl;
        }
        out << series;
        in.close();
        out.close();
        printf("Данные успешно проанализированы\n");
    }
    else
    {
        printf("Данные не введены\n");
    }
    printf("\nДля выбора пункта меню введите соответствующую букву: ");
    return 0;
}
//
//
//
int writRez()
{
    if (!_access("analysis.txt", 0)) {
        ifstream in;
        in.open("analysis.txt");
        int sizeY, sizeX, mest, series;
        in >> mest >> sizeY >> sizeX;
        int* mas = new int[sizeY];
        for (int i = 0;i < sizeY;i++)
        {
            in >> mas[i];
        }
        in >> series;
        int n = 0;
        char p;
        do {
            system("cls");
            menu();
            printf("C\n");
            printf("\tВсего мест: %d\n", mest);
            printf("\tВсего продано билетов: %d\n", series);
            printf("\tВсего рядов: %d\n", sizeY);
            if (n > 0) {
                printf("Введите номер ряда для подсчета занятых мест в этом ряду (для завершения введите 0): %d", n);
                printf("\nВ %d-ом ряду продано: %d\n", n, mas[n - 1]);
                printf("В %d-ом ряду осталось: %d\n", n, sizeX - mas[n - 1]);
            }
            printf("Введите номер ряда для подсчета занятых мест в этом ряду (для завершения введите 0): ");
            do
            {
                scanf_s("%d", &n);
                while (scanf_s("%c", &p))               // анализ нажатых клавиш после завершения ввода числа
                {                                       //
                    if (p == '\n')                      //
                    {                                   //
                        break;                          //
                    }                                   //
                    else                                //
                    {                                   //
                        if (p != ' ')                   //
                        {                               //
                            n = -1;                     //
                        }                               //
                    }                                   //
                }
                if (n < 0 || n>sizeY)
                {
                    printf("Введено недопустимое значение, повторите ввод: ");
                }
            } while (n < 0 || n>sizeY);
        } while (n != 0);
        in.close();
        system("cls");
        menu();
    }
    else
    {
        system("cls");
        menu();
        printf("C\n");
        printf("Данные не проанализированы\n");
        printf("\nДля выбора пункта меню введите соответствующую букву: ");
    }
    return 0;
}
 
bool** deleteMas(bool** mas, int sizeY, int sizeX) {
    for (int i = 0;i < sizeY;i++)
        delete[] mas[i];
    delete[] mas;
    return mas;
}

Добавлено через 8 минут
Изначально задумывалось что массив будет один для всех функций но из за того что не получилось так сделать сделал через файлы, сейчас не стал все возвращать обратно, а просто вернул то как у меня было удаление массива и его объявление.

@DrOffset · 14.09.2015, 18:42

Ullr, давай начнем вот с этих исправлений:
Строка 14:

C++

1	bool read(bool &, int, int); // было bool read(bool, int, int);

Строка 24:

C++

1	bool mass = 0; // было bool mass = new bool*[0];

Строка 123:

C++

1	bool read(bool & mass, int sizeY, int sizeX) // было bool read(bool mass, int sizeY, int sizeX)

Проверяй.

Ullr · 14.09.2015, 18:47 **[ТС]**

Теперь ошибку не выдает

То есть надо было отправлять массив как бы "не копию", а именно "оригинал" (я про знак & перед названием массива как параметр в функции) ?

@DrOffset · 14.09.2015, 19:00

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

То есть надо было отправлять массив как бы "не копию", а именно "оригинал" (я про знак & перед названием массива как параметр в функции) ?

Ну конечно. Только не массив, а указатель. Здесь важно, чтобы указатель mass в main имел то самое значение, что мы получили в функции read. До этого, у тебя оставалось прежнее значение, а вот удалялся массив уже как будто бы значение новое.
Если по указателю не находятся те данные, которые ждет delete, то мы как раз будем наблюдать спецэффекты, наподобие твоих.

Ullr · 14.09.2015, 19:04 **[ТС]**

Спасибо огромное. Не знал что так можно, теперь буду знать

@DrOffset · 14.09.2015, 19:08

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

Не знал что так можно, теперь буду знать

Можно было бы сделать более классически. Функция read возвращала бы указатель на выделенную память. А в функции main просто было бы присвоение.

C++

1	mass = read(n, m);

Получилось бы тоже самое, но менее вычурно и более привычно.

Ullr 0 / 0 / 0 Регистрация: 17.07.2015 Сообщений: 11
	14.09.2015, 19:04 [ТС]	10
	Спасибо огромное. Не знал что так можно, теперь буду знать 0

@DrOffset 18902 / 9860 / 2410 Регистрация: 30.01.2014 Сообщений: 17,305
	14.09.2015, 02:22	3
	Сообщение от Ullr Объявляю динамический массив так Это неправильно. Такого быть не должно. Больше без полного кода нельзя сказать. 1

Ullr 0 / 0 / 0 Регистрация: 17.07.2015 Сообщений: 11
	14.09.2015, 11:12 [ТС]	4
	Сообщение от DrOffset Это неправильно. Такого быть не должно. Больше без полного кода нельзя сказать. Увы код уже сильно изменил. После нескольких экспериментов пришел к выводу что освободить память динамического массива можно только в той функции в которой она была выделена (хотя в интернете об этом не нашел упоминаний), переместил освобождение памяти из отдельной функции в конец функции в которой она была выделена и ошибка исчезла. P.s. хотел вынести освобождение в отдельную функцию потому что данные из массива понадобятся позже если пользователь выберет другой пункт меню, по этому пришлось использовать файлы для хранения данных. 0

@DrOffset 18902 / 9860 / 2410 Регистрация: 30.01.2014 Сообщений: 17,305
	14.09.2015, 14:14	5
	Сообщение от Ullr После нескольких экспериментов пришел к выводу что освободить память динамического массива можно только в той функции в которой она была выделена И это также неверный вывод. Если все-таки есть желание, чтобы научили и подсказали, то лучше выложить проблемный код 1

Ullr 0 / 0 / 0 Регистрация: 17.07.2015 Сообщений: 11
	14.09.2015, 18:47 [ТС]	8
	Теперь ошибку не выдает То есть надо было отправлять массив как бы "не копию", а именно "оригинал" (я про знак & перед названием массива как параметр в функции) ? 0

@DrOffset 18902 / 9860 / 2410 Регистрация: 30.01.2014 Сообщений: 17,305
	14.09.2015, 19:00	9
	Сообщение от Ullr То есть надо было отправлять массив как бы "не копию", а именно "оригинал" (я про знак & перед названием массива как параметр в функции) ? Ну конечно. Только не массив, а указатель. Здесь важно, чтобы указатель mass в main имел то самое значение, что мы получили в функции read. До этого, у тебя оставалось прежнее значение, а вот удалялся массив уже как будто бы значение новое. Если по указателю не находятся те данные, которые ждет delete, то мы как раз будем наблюдать спецэффекты, наподобие твоих. 1

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

Решение