Освобождение памяти

@Frankenween · Регистрация: 25.10.2016

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

Есть класс, в котором я выделяю память с помощью new. В деструкторе класса я с помощью delete освобождаю память, но у меня вылетает ошибка ещё задолго до вызова деструктора класса.

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
class Block
{
public:
    int *xpos, *ypos, *xsize, *ysize;
    sf::Texture *block_texture;
    sf::Sprite *block_sprite;
    Block(int x, int y, string fname) {
        xpos = new int;  // Выделяем место в памяти под координату по оси ox
        ypos = new int;  // Выделяем место в памяти под координату по оси ou
        xsize = new int;  // Выделяем место в памяти под размер спрайта по ширине
        ysize = new int;  // Выделяем место в памяти под размер спрайта по высоте
        block_texture = new sf::Texture;  // Выделяем место в памяти под текстуру
        block_sprite = new sf::Sprite;  // Выделяем место в памяти под текстуру
        *xpos = x;  // Записываем x
        *ypos = y;  // Записываем y
        block_texture->loadFromFile(fname);  // Загружаем изображение из файла
        block_sprite->setTexture(*block_texture);  // Добавляем текстуру в спрайт
        block_sprite->setPosition(*xpos, *ypos);  // Помешаем наш спрайт в координаты xpos;ypos
        *xsize = block_texture->getSize().x;  // Получаем ширину текстуры и записываем в xsize
        *ysize = block_texture->getSize().y;  // Получаем высоту текстуры и записываем в ysize
    }
    ~Block() {
        /*
        delete xpos;
        delete ypos;
        delete xsize;
        delete ysize;
        delete block_sprite;
        delete block_texture;*/
        
    }

все методы в классе работают правильно, программа компилируется, но при запуске меньше чем через секунду выдаёт ошибку. если код в деструкторе закомментировать, то никакой ошибки нет и программа правильно работает. вопрос: что сделать чтобы память освобождалась и где ошибка?

nd2 · 24.04.2017, 14:27

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

все методы в классе работают правильно

Какие есть методы? Наличие деструктора предполагает наличие конструктора копирования и оператора присваивания (правило трёх).
Распространенные ошибки

@Frankenween · 24.04.2017, 14:28 **[ТС]**

Вот весь класс:

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
class Block
{
public:
    int *xpos, *ypos, *xsize, *ysize;
    sf::Texture *block_texture;
    sf::Sprite *block_sprite;
    Block(int x, int y, string fname) {
        xpos = new int;  // Выделяем место в памяти под координату по оси ox
        ypos = new int;  // Выделяем место в памяти под координату по оси ou
        xsize = new int;  // Выделяем место в памяти под размер спрайта по ширине
        ysize = new int;  // Выделяем место в памяти под размер спрайта по высоте
        block_texture = new sf::Texture;  // Выделяем место в памяти под текстуру
        block_sprite = new sf::Sprite;  // Выделяем место в памяти под текстуру
        *xpos = x;  // Записываем x
        *ypos = y;  // Записываем y
        block_texture->loadFromFile(fname);  // Загружаем изображение из файла
        block_sprite->setTexture(*block_texture);  // Добавляем текстуру в спрайт
        block_sprite->setPosition(*xpos, *ypos);  // Помешаем наш спрайт в координаты xpos;ypos
        *xsize = block_texture->getSize().x;  // Получаем ширину текстуры и записываем в xsize
        *ysize = block_texture->getSize().y;  // Получаем высоту текстуры и записываем в ysize
    }
    ~Block() {
        delete xpos;
        delete ypos;
        delete xsize;
        delete ysize;
        delete block_sprite;
        delete block_texture;
        
    }
 
    void update(int dx, int dy) {
        /*
        Сдвигаем спрайт по оси x на dx(если dx < 0, то вправо, если dx > 0, то влево, если dx = 0, то не двигаем)
        Сдвигаем спрайт по оси y на dy(если dy < 0, то вверх, если dy > 0, то вниз, если dy = 0, то не двигаем)
        */
        block_sprite->move(dx, dy);
    }
 
 
    int reset_texture(string new_name) {
        if (block_texture->loadFromFile(new_name) == false) return -1;  // Загружаем новое изображение из файла, если не получилось, возвращаем -1
        block_sprite->setTexture(*block_texture);  // Добавляем обновлённую текстуру в спрайт
        return 0;
    }
 
 
    void draw(sf::RenderWindow &win) {
        win.draw(*block_sprite);
    }
 
    bool intersect(int x2, int y2) {
        if (((*xpos > x2 - *xsize) && (*xpos < x2 + *xsize)) && ((*ypos > y2 - *ysize) && (*ypos < y2 + *ysize))) {
            return true;
        }
        else {
            return false;
        }
    }
 
private:
 
};

@LightDays · 24.04.2017, 14:31

1. Строка 14 и 15. Вы там записываете x и y. Но что это собственно такое? Не объявления, не инициализации
2. Можно "ответ" компилятора на ваш код?

nd2 · 24.04.2017, 14:33

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

Вот весь класс:

Покажи использование.

@Frankenween · 24.04.2017, 14:33 **[ТС]**

Вызвано исключение по адресу 0x0FBD5F0C (ucrtbased.dll) в Проект1.exe: 0xC0000005: нарушение прав доступа при чтении по адресу 0x3F4B6B4D.

nd2 · 24.04.2017, 14:35

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

Вы там записываете x и y. Но что это собственно такое? Не объявления, не инициализации

Передаются в конструктор.

@Frankenween · 24.04.2017, 14:37 **[ТС]**

Примерно такое:

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
int main()
{
    // Создаём окно
    RenderWindow *App;
    App = new RenderWindow(VideoMode(800, 600, 32), "Hello World - SFML");
    App->setFramerateLimit(60);
    Block *bl1;
    bl1 = new Block(10, 20, "block.png");
    Block *bl2;
    bl2 = new Block(50, 17, "block.png");
    Block *bl3;
    bl3 = new Block(70, 2, "block.png");
    Block *bl4;
    bl4 = new Block(88, 111, "block.png");
    Block *bl5;
    bl5 = new Block(555, 65, "block.png");
    Block *bl6;
    bl6 = new Block(7, 88, "block.png");
 
    Chunk *new_chunk = new Chunk;
    new_chunk->AddBlock(*bl1);
    new_chunk->AddBlock(*bl2);
    new_chunk->AddBlock(*bl3);
    new_chunk->AddBlock(*bl4);
    new_chunk->AddBlock(*bl5);
    new_chunk->AddBlock(*bl6);
    
 
    // Основной цикл
    while (App->isOpen())
    {
        // Проверяем события (нажатие кнопки, закрытие окна и т.д.)
        Event Event;
        while (App->pollEvent(Event))
        {
            // если событие "закрытие окна":
            if (Event.type == Event::Closed)
                //закрываем окно 
                App->close();
        }
 
        // очищаем экран, и заливаем его синим цветом
        App->clear();
 
 
        // отображаем на экран
        new_chunk->DrawChunk(*App);
        App->display();
        bl1->update(1, 0);
        bl2->update(0, 1);
        bl3->update(1, 1);
        bl4->update(2, 0);
        bl5->update(5, 1);
        bl6->update(-1, 3);
    }
 
    
    delete new_chunk;
    delete bl1;
    delete App;
    system("pause");
    
 
    return 0;
}

@h3mbr0 · 24.04.2017, 14:39

Frankenween, стесняюсь спросить, а почему вообще у вас инты выделяются new?
Рискну предположить, что ошибка вне этого куска кода
Кстати, раз используете студию, встроенный отладчик в большинстве случаев покажет вам строку на момент ошибки, нажмите F5

nd2 · 24.04.2017, 14:40

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

C++
1
new_chunk->AddBlock(*bl1);

Аргумент по значению передаётся в AddBlock()?

@h3mbr0 · 24.04.2017, 14:41

А класс Chunk можно?

@Frankenween · 24.04.2017, 14:41 **[ТС]**

Строка 23

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
class Block
{
public:
    int *xpos, *ypos, *xsize, *ysize;
    sf::Texture *block_texture;
    sf::Sprite *block_sprite;
    Block(int x, int y, string fname) {
        xpos = new int;  // Выделяем место в памяти под координату по оси ox
        ypos = new int;  // Выделяем место в памяти под координату по оси ou
        xsize = new int;  // Выделяем место в памяти под размер спрайта по ширине
        ysize = new int;  // Выделяем место в памяти под размер спрайта по высоте
        block_texture = new sf::Texture;  // Выделяем место в памяти под текстуру
        block_sprite = new sf::Sprite;  // Выделяем место в памяти под текстуру
        *xpos = x;  // Записываем x
        *ypos = y;  // Записываем y
        block_texture->loadFromFile(fname);  // Загружаем изображение из файла
        block_sprite->setTexture(*block_texture);  // Добавляем текстуру в спрайт
        block_sprite->setPosition(*xpos, *ypos);  // Помешаем наш спрайт в координаты xpos;ypos
        *xsize = block_texture->getSize().x;  // Получаем ширину текстуры и записываем в xsize
        *ysize = block_texture->getSize().y;  // Получаем высоту текстуры и записываем в ysize
    }
    ~Block() {
        delete xpos;
        delete ypos;
        delete xsize;
        delete ysize;
        delete block_sprite;
        delete block_texture;
        
    }
 
    void update(int dx, int dy) {
        /*
        Сдвигаем спрайт по оси x на dx(если dx < 0, то вправо, если dx > 0, то влево, если dx = 0, то не двигаем)
        Сдвигаем спрайт по оси y на dy(если dy < 0, то вверх, если dy > 0, то вниз, если dy = 0, то не двигаем)
        */
        block_sprite->move(dx, dy);
    }
 
 
    int reset_texture(string new_name) {
        if (block_texture->loadFromFile(new_name) == false) return -1;  // Загружаем новое изображение из файла, если не получилось, возвращаем -1
        block_sprite->setTexture(*block_texture);  // Добавляем обновлённую текстуру в спрайт
        return 0;
    }
 
 
    void draw(sf::RenderWindow &win) {
        win.draw(*block_sprite);
    }
 
    bool intersect(int x2, int y2) {
        if (((*xpos > x2 - *xsize) && (*xpos < x2 + *xsize)) && ((*ypos > y2 - *ysize) && (*ypos < y2 + *ysize))) {
            return true;
        }
        else {
            return false;
        }
    }
 
private:
 
};

Добавлено через 26 секунд

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
class Chunk
{
public:
    vector<Block> *chunk_blocks;
    const int chunk_size = 64;
    Chunk() {
        chunk_blocks = new vector<Block>;
    }
    ~Chunk() {
 
    }
 
 
    void AddBlock(Block &new_block) {
        chunk_blocks->push_back(new_block);
    }
 
 
    void DrawChunk(sf::RenderWindow &win) {
        for (auto &block_to_draw : *chunk_blocks) {
            block_to_draw.draw(win);
        }
    }
 
private:
    
};

nd2 · 24.04.2017, 14:44

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

C++
1
chunk_blocks->push_back(new_block);

Здесь неявный конструктор копирования Block будет работать. Второй пост читай.

@Frankenween · 24.04.2017, 14:46 **[ТС]**

Просто мой основной язык - Python, а c++ я только начал изучать по этому ошибок много

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

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

Здесь конструктор копирования Block будет работать. Второй пост читай.

Тогда такой вопрос: что такое конструктор копирования?

@h3mbr0 · 24.04.2017, 14:46

Не используйте new там, где не нужно
Есть вероятность, что вы передали значения по ссылке тут:

C++
1
block_sprite->setPosition(*xpos, *ypos);

И освободили их в деструкторе Sprite (но это неточно

)

@LightDays · 24.04.2017, 14:49

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

Вы там записываете x и y. Но что это собственно такое? Не объявления, не инициализации
Передаются в конструктор.

Не заметил сразу конструктора вообще

@Frankenween · 24.04.2017, 14:50 **[ТС]**

Что-то я совсем ничего не понял. Я же не вызываю деструктор Sprite

nd2 · 24.04.2017, 14:51

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

Тогда такой вопрос: что такое конструктор копирования?

Тогда такой ответ: интернет есть?

@h3mbr0 · 24.04.2017, 14:52

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

Что-то я совсем ничего не понял. Я же не вызываю деструктор Sprite

Вообще-то вызываете, когда пишете

C++
1
delete block_sprite;

Но в данном случае эта строка ниже, так что дело видимо не в этом
Может быть скинете полный код?

@Frankenween · 24.04.2017, 14:56 **[ТС]**

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

Может быть скинете полный код?

Первый файл world.h

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
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
#pragma once
#include <string>
#include <vector>
#include <SFML/Graphics.hpp>
 
using namespace std;
 
class Block
{
public:
    int *xpos, *ypos, *xsize, *ysize;
    sf::Texture *block_texture;
    sf::Sprite *block_sprite;
    Block(int x, int y, string fname) {
        xpos = new int;  // Выделяем место в памяти под координату по оси ox
        ypos = new int;  // Выделяем место в памяти под координату по оси ou
        xsize = new int;  // Выделяем место в памяти под размер спрайта по ширине
        ysize = new int;  // Выделяем место в памяти под размер спрайта по высоте
        block_texture = new sf::Texture;  // Выделяем место в памяти под текстуру
        block_sprite = new sf::Sprite;  // Выделяем место в памяти под текстуру
        *xpos = x;  // Записываем x
        *ypos = y;  // Записываем y
        block_texture->loadFromFile(fname);  // Загружаем изображение из файла
        block_sprite->setTexture(*block_texture);  // Добавляем текстуру в спрайт
        block_sprite->setPosition(*xpos, *ypos);  // Помешаем наш спрайт в координаты xpos;ypos
        *xsize = block_texture->getSize().x;  // Получаем ширину текстуры и записываем в xsize
        *ysize = block_texture->getSize().y;  // Получаем высоту текстуры и записываем в ysize
    }
    ~Block() {
        delete xpos;
        delete ypos;
        delete xsize;
        delete ysize;
        delete block_sprite;
        delete block_texture;
        
    }
 
    void update(int dx, int dy) {
        /*
        Сдвигаем спрайт по оси x на dx(если dx < 0, то вправо, если dx > 0, то влево, если dx = 0, то не двигаем)
        Сдвигаем спрайт по оси y на dy(если dy < 0, то вверх, если dy > 0, то вниз, если dy = 0, то не двигаем)
        */
        block_sprite->move(dx, dy);
    }
 
 
    int reset_texture(string new_name) {
        if (block_texture->loadFromFile(new_name) == false) return -1;  // Загружаем новое изображение из файла, если не получилось, возвращаем -1
        block_sprite->setTexture(*block_texture);  // Добавляем обновлённую текстуру в спрайт
        return 0;
    }
 
 
    void draw(sf::RenderWindow &win) {
        win.draw(*block_sprite);
    }
 
    bool intersect(int x2, int y2) {
        if (((*xpos > x2 - *xsize) && (*xpos < x2 + *xsize)) && ((*ypos > y2 - *ysize) && (*ypos < y2 + *ysize))) {
            return true;
        }
        else {
            return false;
        }
    }
 
private:
 
};
 
 
class Chunk
{
public:
    vector<Block> *chunk_blocks;
    const int chunk_size = 64;
    Chunk() {
        chunk_blocks = new vector<Block>;
    }
    ~Chunk() {
 
    }
 
 
    void AddBlock(Block &new_block) {
        chunk_blocks->push_back(new_block);
    }
 
 
    void DrawChunk(sf::RenderWindow &win) {
        for (auto &block_to_draw : *chunk_blocks) {
            block_to_draw.draw(win);
        }
    }
 
private:
    
};

И source.cpp

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
74
75
#include <SFML/Graphics.hpp>
#include <SFML/Audio.hpp>
#include <string>
#include <vector>
#include "World.h"
using namespace sf;
 
#define si short int
 
 
int main()
{
    // Создаём окно
    RenderWindow *App;
    App = new RenderWindow(VideoMode(800, 600, 32), "Hello World - SFML");
    App->setFramerateLimit(60);
    Block *bl1;
    bl1 = new Block(10, 20, "block.png");
    Block *bl2;
    bl2 = new Block(50, 17, "block.png");
    Block *bl3;
    bl3 = new Block(70, 2, "block.png");
    Block *bl4;
    bl4 = new Block(88, 111, "block.png");
    Block *bl5;
    bl5 = new Block(555, 65, "block.png");
    Block *bl6;
    bl6 = new Block(7, 88, "block.png");
 
    Chunk *new_chunk = new Chunk;
    new_chunk->AddBlock(*bl1);
    new_chunk->AddBlock(*bl2);
    new_chunk->AddBlock(*bl3);
    new_chunk->AddBlock(*bl4);
    new_chunk->AddBlock(*bl5);
    new_chunk->AddBlock(*bl6);
    
 
    // Основной цикл
    while (App->isOpen())
    {
        // Проверяем события (нажатие кнопки, закрытие окна и т.д.)
        Event Event;
        while (App->pollEvent(Event))
        {
            // если событие "закрытие окна":
            if (Event.type == Event::Closed)
                //закрываем окно 
                App->close();
        }
 
        // очищаем экран, и заливаем его синим цветом
        App->clear();
 
 
        // отображаем на экран
        new_chunk->DrawChunk(*App);
        App->display();
        bl1->update(1, 0);
        bl2->update(0, 1);
        bl3->update(1, 1);
        bl4->update(2, 0);
        bl5->update(5, 1);
        bl6->update(-1, 3);
    }
 
    
    delete new_chunk;
    delete bl1;
    delete App;
    system("pause");
    
 
    return 0;
}

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .	SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer 8Observer8 08.02.2026 Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .
SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств 8Observer8 07.02.2026 Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .	SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .

@LightDays 12 / 12 / 8 Регистрация: 19.02.2017 Сообщений: 44
	24.04.2017, 14:31
	1. Строка 14 и 15. Вы там записываете x и y. Но что это собственно такое? Не объявления, не инициализации 2. Можно "ответ" компилятора на ваш код? 0

@Frankenween 22 / 1 / 0 Регистрация: 25.10.2016 Сообщений: 36
	24.04.2017, 14:33 [ТС]
	Вызвано исключение по адресу 0x0FBD5F0C (ucrtbased.dll) в Проект1.exe: 0xC0000005: нарушение прав доступа при чтении по адресу 0x3F4B6B4D. 0

@h3mbr0 298 / 107 / 31 Регистрация: 12.03.2012 Сообщений: 449
	24.04.2017, 14:39
	Frankenween, стесняюсь спросить, а почему вообще у вас инты выделяются new? Рискну предположить, что ошибка вне этого куска кода Кстати, раз используете студию, встроенный отладчик в большинстве случаев покажет вам строку на момент ошибки, нажмите F5 0

@h3mbr0 298 / 107 / 31 Регистрация: 12.03.2012 Сообщений: 449
	24.04.2017, 14:41
	А класс Chunk можно? 0

@Frankenween 22 / 1 / 0 Регистрация: 25.10.2016 Сообщений: 36
	24.04.2017, 14:50 [ТС]
	Что-то я совсем ничего не понял. Я же не вызываю деструктор Sprite 0