Утечка памяти. Как ее побороть?

@Lemm_Leto · Регистрация: 07.01.2013

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
void Player::add()
{
    Ship *bombers;
    bombers = new Bomber[num_bombers];
 
    for(int i=0; i<num_bombers; i++)
    {
        Bomber temp(7, 40, 1, 1);
        bombers[i] = temp;
    };
    for(int i=0 ; i<num_bombers; i++)
        cout << bombers[i] << endl;
    getch();
}

Есть вот такой вот метод класса, как сделать чтобы память не утекала, то есть не вызывался деструктор для объекта temp, ну или какие-то другие способы обойти эту утечку. Если что скину остальные куски кода. Помогите, люди добрые

@m1Rr0r · 02.02.2013, 23:36

Как насчет такого ?

C++
1
bombers[i](7, 40, 1, 1);

скобки перегрузить, хотя хз)

Jupiter · 02.02.2013, 23:50

Не по теме:

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

Утечка памяти. Как ее побороть?

смените с++ на язык с автоматическим управленим памятью

@Lemm_Leto · 03.02.2013, 00:13 **[ТС]**

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

Не по теме:

смените с++ на язык с автоматическим управленим памятью

А что-то более дельное посоветовать?

Jupiter · 03.02.2013, 00:28

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

А что-то более дельное посоветовать?

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

ValeryS · 03.02.2013, 00:29

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

то есть не вызывался деструктор для объекта temp,

а это здесь причем ???
объект создается в стеке он к утечке не имеет никакого отношения, если конечно внутри Bomber с памятью не балуешься
вот у тебя утечка

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

Ship *bombers;
* * bombers = new Bomber[num_bombers];

bombers локальный указатель при выходе из функции пропадает, значит память удалить никто не может
или делай членом класса
или в конце функции

C++
1
delete []bombers;

MrGluck · 03.02.2013, 02:27

C++
1
Bomber temp(7, 40, 1, 1);

magic number

@vbloodv · 03.02.2013, 03:39

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

magic number

Антипатерн

, но 7:40 тут не спроста ZOG заговор

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

как сделать чтобы память не утекала,

может сделай:

C++
1
std::tr1::shared_ptr<Ship> bombers(new Bomber[num_bombers]);

Croessmah · 03.02.2013, 06:49

Не по теме:

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

C++
1
std::tr1::shared_ptr<Ship> bombers(new Bomber[num_bombers]);

Зачем tr1? Все уже в стандарте =)

iifat · 03.02.2013, 08:51

Утечка памяти никак с вызовом дструктор не связана, это тебе уже сказали.
Утечка памяти связана как раз с вещами типа

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

C++
4
bombers = new Bomber[num_bombers];

без вызова соответствующего деструктора.
Решение, вообще говоря -- не писать такого кода. Такой код должен быть обёрнут в класс. Переменная класса заводится на стеке либо глобально, вызов генерится методом, вызов деструктора гарантируется, а в нём уж проще предусмотреть освобождение памяти.
Есть и паттерн "разумный указатель" или что-то типа того, и соответствующие классы, но я как-то предпочитаю вручную ему следовать.

@Van111 · 03.02.2013, 09:19

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

Bomber temp(7, 40, 1, 1);

а можно реализацию конструктора?

@Lemm_Leto · 03.02.2013, 12:36 **[ТС]**

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

а можно реализацию конструктора?

C++
1
2
3
4
5
6
7
8
9
10
11
12
Bomber::Bomber(float temp_att, float temp_hp, float multiply_attack, float multiply_hp) :Squadron(temp_hp, multiply_hp)
{
    attack_beam = temp_att * multiply_attack;
};
 
Squadron::Squadron(float temp_hp, float multiply_hp)
{
    hp = temp_hp * multiply_hp;
};
 
Ship::Ship()
{};

Добавлено через 2 минуты

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

а это здесь причем ???
объект создается в стеке он к утечке не имеет никакого отношения, если конечно внутри Bomber с памятью не балуешься
вот у тебя утечка

bombers локальный указатель при выходе из функции пропадает, значит память удалить никто не может
или делай членом класса
или в конце функции

C++
1
delete []bombers;

У меня утечка еще на этапе cout происходит почему-то. Вернее даже не утечка, а ошибка нарушения прав доступа

@Van111 · 03.02.2013, 14:46

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

bombers[i] = temp;

как у вас реализована функция переопределения оператора присваивания?

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

cout << bombers[i]

как у вас реализована функция переопределения оператора []?

Новые блоги и статьи Все статьи Все блоги /
моя боль iceja 24.01.2026 Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .	Модель сукцессии микоризы anaschu 24.01.2026 Решили писать научную статью с неким РОманом	http://iceja.net/ математические сервисы iceja 20.01.2026 Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .	http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .
Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь(не выше 3-го порядка) постоянного тока с элементами R, L, C, k(ключ), U, E, J. Программа находит переходные токи и напряжения на элементах схемы классическим методом(1 и 2 з-ны. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .	Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .