Утечка памяти

@Abbat_90 · Регистрация: 11.10.2011

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

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
Array Array::Add(Array a, Array b)
        {
        Array *N = new Array;
 
         if (a.size>=b.size) {
        for (int i=0; i<b.size; i++) {N->mas[i]=a.mas[i]+b.mas[i]; }
                for(int i=b.size; i<a.size; i++) {N->mas[i]=a.mas[i];}
                N->size = a.size;        }
        else {for (int i=0; i<a.size; i++) {N->mas[i]=a.mas[i]+b.mas[i]; }
                for(int i=a.size; i<b.size; i++) {N->mas[i]=b.mas[i];} N->size = b.size;}
          return *N;
          delete N;
        }

В результате компилялии выдаёт предупреждение unreachable cod или что-то на подобе этого...
Заранее спасибо.

@fasked · 16.10.2011, 23:33

Abbat_90, 12-ая строка никогда не выполнится.

@Abbat_90 · 16.10.2011, 23:39 **[ТС]**

так вот как этого избежать???

@fasked · 16.10.2011, 23:46

Abbat_90, не выделять память динамически.

@Abbat_90 · 16.10.2011, 23:51 **[ТС]**

а если хочется???

@alkagolik · 17.10.2011, 00:11

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
#include <stdio.h>
#include <stdlib.h>
 
int *create_some(int size)
{
    int *some;
 
    some = (int*)malloc(size * sizeof(int));
 
    return some;
}
 
void delete_some(int *some)
{
    free(some);
}
 
int main()
{
    int *x, i;
    x = create_some(10);
    for (i = 0; i < 10; ++i)
        printf("%i\n", x[i]);
    delete_some(x);
    return 0;
}

@silentnuke · 17.10.2011, 00:13

как вариант, используйте умные указатели=)

accept · 17.10.2011, 04:28

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

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

если выделяешь память, нельзя терять указатель

C++
1
return N;

тип возвращаемого значения меняется на Array *

Евгений М. · 17.10.2011, 13:35

Abbat_90, если метод не статический, то поставьте delete в деструктор.

Добавлено через 2 минуты
Хотя неправильно сказал. Сработает, если только 1 раз вызвать функцию Add.

@fasked · 17.10.2011, 13:41

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

а если хочется???

В Вашем примере нет смысла в динамической памяти (если только показать утечку). Количество объектов известно на стадии компиляции - он один. Объект из метода возвращается по значению, а не указатель. Память выделяется и удаляется в одном месте.
"Хочется" - это не достаточный аргумент, чтобы использовать new.

@silentnuke · 17.10.2011, 16:15

Сообщение от Евгений М.

Abbat_90, если метод не статический, то поставьте delete в деструктор.

Добавлено через 2 минуты
Хотя неправильно сказал. Сработает, если только 1 раз вызвать функцию Add.

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

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@fasked 5045 / 2624 / 241 Регистрация: 07.10.2009 Сообщений: 4,310 Записей в блоге: 5
	16.10.2011, 23:33
	Abbat_90, 12-ая строка никогда не выполнится. 0

@Abbat_90 0 / 0 / 0 Регистрация: 11.10.2011 Сообщений: 26
	16.10.2011, 23:39 [ТС]
	так вот как этого избежать??? 0

@fasked 5045 / 2624 / 241 Регистрация: 07.10.2009 Сообщений: 4,310 Записей в блоге: 5
	16.10.2011, 23:46
	Abbat_90, не выделять память динамически. 0

@Abbat_90 0 / 0 / 0 Регистрация: 11.10.2011 Сообщений: 26
	16.10.2011, 23:51 [ТС]
	а если хочется??? 0

@silentnuke Android Programmer 141 / 142 / 10 Регистрация: 08.12.2010 Сообщений: 421
	17.10.2011, 00:13
	как вариант, используйте умные указатели=) 0

Евгений М. 1080 / 1007 / 107 Регистрация: 28.02.2010 Сообщений: 2,889
	17.10.2011, 13:35
	Abbat_90, если метод не статический, то поставьте delete в деструктор. Добавлено через 2 минуты Хотя неправильно сказал. Сработает, если только 1 раз вызвать функцию Add. 0