Освобождение памяти, выделенного под динамический массив

@f4 · Регистрация: 24.01.2017

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

Нужно освободить участок памяти, выделенный под динамический массив при помощи функции.
Однако вылазит ошибка segmentation fault. В чем ошибка?_{помогите}

C++
1
2
3
4
5
void del(Vector arr)
{
   delete [] arr.val;
   arr.n=0;    
}

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
68
69
70
71
72
#include <iostream>
#include <random>
#include <stdlib.h>
#include <locale.h>
using namespace std;
 
struct Vector
{
    int n;
    int *val;    
};
 
void init(int n)
{
    Vector arr;
    arr.n = n;
    arr.val = new int [n];
    for (int i=0; i<n; i++)
    {
        arr.val[i]=0;        
    }
    
}
 
void del(Vector arr)
{
   delete [] arr.val;
   arr.n=0;    
}
 
 
void print(Vector arr, int n)
{
    for(int i=0; i<n; i++)
    {
        cout<<arr.val[i]<<" ";
    }
}
 
 
void fill(Vector arr, int n)
{
    srand(time(NULL));
    for (int i=0; i<n; i++)
    {
        arr.val[i] = rand() % 100 + 1;
    }
}
 
int max(Vector arr, int n)
{
    int max;
    max=arr.val[0];
    for (int i=1; i<n; i++)
    {
        if (max<arr.val[i]){max=arr.val[i];}        
    }
    return max;
}
int main()
{   int n;
    Vector mas;
    cout<<"n = ";
    cin >> n;    
    
    init(n); 
    fill(mas,n);
    print(mas,n); 
    cout<<endl<<"Max = "<<max(mas,n)<<endl;
    del(mas); 
    return 0;
}

@L0M · 10.11.2019, 17:51

f4, в init() вы работаете с локальной переменной Vector arr, которая умирает при выходе из функции. При этом память, выделенная в init() утекает. Потом вы пытаетесь что-то делать с переменной Vector mas, определённой в main(). Но она не инициализирована, и память под массив не выделена. На этом и получаете segmentation fault.

Если уж вы пишите на С++, то инициализация должна быть в конструкторах структуры или оператре присваивания, а деинициализация - в деструкторе.

@zss · 10.11.2019, 18:09

Если категорически нельзя использовать инкапсуляцию, то можете написать так:

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
68
69
70
71
72
73
#include <iostream>
#include <cstdlib>
#include <ctime>
#include <clocale>
using namespace std;
 
struct Vector
{
    int n;
    int *val;
};
Vector* Init(int n)
{
    Vector* arr = new Vector;
    arr->n = n;
    arr->val = new int[n];
    for (int i = 0; i < n; i++)
    {
        arr->val[i] = 0;
    }
    return arr;
 
}
 
void free(Vector* arr)
{
    delete[] arr->val;
}
 
 
void print(Vector* arr)
{
    for (int i = 0; i < arr->n; i++)
    {
        cout << arr->val[i] << " ";
    }
}
 
 
void fill(Vector* arr)
{
    srand(time(NULL));
    for (int i = 0; i < arr->n; i++)
    {
        arr->val[i] = rand() % 100 + 1;
    }
}
 
int max(Vector* arr)
{
    int max;
    max = arr->val[0];
    for (int i = 1; i < arr->n; i++)
    {
        if (max < arr->val[i]) { max = arr->val[i]; }
    }
    return max;
}
 
//----------------------------------------
int main()
{
    int n;
    cout << "n = ";
    cin >> n;
    Vector* mas=Init(n);
    fill(mas);
    print(mas);
    cout << endl << "Max = " << max(mas) << endl;
    free(mas);
    delete mas;
    return 0;
}

Но, все же, настоятельно советую делать так:

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
68
69
70
#include <iostream>
#include <cstdlib>
#include <ctime>
//#include <clocale>
using namespace std;
 
class Vector
{
    int n;
    int *val;
public:
    Vector(int n)
    {
        //  Vector arr;
        this->n = n;
        val = new int[n];
        for (int i = 0; i < n; i++)
        {
            val[i] = 0;
        }
 
    }
 
    ~Vector()
    {
        delete[] val;
    }
 
 
    void print()
    {
        for (int i = 0; i < n; i++)
        {
            cout << val[i] << " ";
        }
    }
 
 
    void fill()
    {
        srand(time(NULL));
        for (int i = 0; i < n; i++)
        {
            val[i] = rand() % 100 + 1;
        }
    }
 
    int max()
    {
        int max;
        max = val[0];
        for (int i = 1; i < n; i++)
        {
            if (max < val[i]) { max = val[i]; }
        }
        return max;
    }
};
//----------------------------------------
int main()
{
    int n;
    cout << "n = ";
    cin >> n;
    Vector mas(n);
    mas.fill();
    mas.print();
    cout << endl << "Max = " << mas.max() << endl;
    return 0;
}

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

@L0M Мозгоправ 1745 / 1039 / 468 Регистрация: 01.10.2018 Сообщений: 2,138 Записей в блоге: 2
	10.11.2019, 17:51
	f4, в init() вы работаете с локальной переменной `Vector arr`, которая умирает при выходе из функции. При этом память, выделенная в init() утекает. Потом вы пытаетесь что-то делать с переменной `Vector mas`, определённой в main(). Но она не инициализирована, и память под массив не выделена. На этом и получаете segmentation fault. Если уж вы пишите на С++, то инициализация должна быть в конструкторах структуры или оператре присваивания, а деинициализация - в деструкторе. 1

Освобождение памяти, выделенного под динамический массив

Решение