Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.67/6: Рейтинг темы: голосов - 6, средняя оценка - 4.67
22 / 1 / 0
Регистрация: 25.10.2016
Сообщений: 36

Ошибка при работе с памятью

01.05.2017, 17:20. Показов 1264. Ответов 8
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Есть класс, в котором я предпринимаю безрезультатные попытки работать с динамической памятью из-за того, что экземпляров класса может быть очень много а они нужны будут лишь некоторое время , а потом они будут просто храниться и тратить память. Вот сам класс:
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
class Block
{
public:
    int xpos, ypos, xsize, ysize;
    sf::Texture *block_texture;
    sf::Sprite *block_sprite;
    Block(int x, int y, string fname) {
        xpos = x;  // Выделяем место в памяти под координату по оси ox
        ypos = y;  // Выделяем место в памяти под координату по оси ou
        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 & operator = (Block &a) {
        if (this == &a)
            return *this; // присвоение самому себе, ничего делать не надо
 
        delete block_sprite;
        delete block_texture;
        
        xpos = a.xpos;
        ypos = a.ypos;
        xsize = a.xsize;
        ysize = a.ysize;
        block_sprite = new sf::Sprite(*a.block_sprite);
        block_texture = new sf::Texture(*a.block_texture);
 
        return *this;
    }
 
    ~Block() {
        delete block_sprite;
        delete block_texture;
    }
 
 
    Block(const Block &a) {
        xpos = a.xpos;
        ypos = a.ypos; 
        xsize = a.xsize;
        ysize = a.ysize;
        block_sprite = a.block_sprite;
        block_texture = a.block_texture;
    }
    
 
    void update(float dx, float 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:
 
};
Проблема в том, что программа, ещё не вызывая деструктор, выводит ошибку о нарушении прав доступа при чтении по адресу 0xXXX. Мне говорили, что нужно конструктор кописования и оператор присвоения добавить. Я добавил, но либо я скорее всего криво всё сделал. Что не так и как исправить ошибку?
0
Лучшие ответы (1)
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
01.05.2017, 17:20
Ответы с готовыми решениями:

Ошибка при работе с памятью
Если ввести например 6, 0, чтобы выбросить исключение. То по завершение программы выскакивает ошибка. В чём может быть дело? Так что то про...

Ошибка при работе с памятью
Написан следующий код: #include &quot;stdafx.h&quot; #include &lt;iostream&gt; #include &lt;stdio.h&gt; #include &lt;math.h&gt; #include &lt;cstdlib&gt; ...

Ошибка в коде при работе с динамической памятью
Не могу понят что не так ? При вводе в динамический массив А последовательность &quot;61 3 2 0 -2 -25 -61 -70&quot; программа крашится...

8
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
01.05.2017, 17:44
Конструктор копирования должен создавать копии, а не просто присваивать.
0
22 / 1 / 0
Регистрация: 25.10.2016
Сообщений: 36
01.05.2017, 17:48  [ТС]
А как создать именно копию? Просто в примере было вот так.
0
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
01.05.2017, 17:50
Лучший ответ Сообщение было отмечено Frankenween как решение

Решение

В операторе присваивания же у тебя правильно. Вот так и делай.
0
22 / 1 / 0
Регистрация: 25.10.2016
Сообщений: 36
01.05.2017, 17:50  [ТС]
Просто скопировать код? И всё?
0
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
01.05.2017, 17:51
Нужное скопировать. Перенос данных из a.
1
22 / 1 / 0
Регистрация: 25.10.2016
Сообщений: 36
01.05.2017, 18:04  [ТС]
Вроде бы заработало и даже правильно, но не все текстуры отображаются

Добавлено через 9 минут
Я сейчас добавил удаление xpos, ypos, xsize и ysize и на этапе создания экземпляра класса вылетает исключение std::bad_array_new_length. Новый код:
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
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;  // Выделяем место в памяти под координату по оси ox
        ysize = new int;  // Выделяем место в памяти под координату по оси ou
        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 & operator = (Block &a) {
        if (this == &a)
            return *this; // присвоение самому себе, ничего делать не надо
 
        delete xpos;
        delete ypos;
        delete xsize;
        delete ysize;
        delete block_sprite;
        delete block_texture;
        
        xpos = new int[*a.xpos];
        ypos = new int[*a.ypos];
        xsize = new int[*a.xsize];
        ysize = new int[*a.ysize];
        block_sprite = new sf::Sprite(*a.block_sprite);
        block_texture = new sf::Texture(*a.block_texture);
 
        return *this;
    }
 
    ~Block() {
        delete xpos;
        delete ypos;
        delete xsize;
        delete ysize;
        delete block_sprite;
        delete block_texture;
    }
 
 
    Block(const Block &a) {
        xpos = new int[*a.xpos];
        ypos = new int[*a.ypos];
        xsize = new int[*a.xsize];
        ysize = new int[*a.ysize];
        block_sprite = new sf::Sprite(*a.block_sprite);
        block_texture = new sf::Texture(*a.block_texture);
    }
    
 
    void update(float dx, float 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:
 
};
0
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
01.05.2017, 18:47
Ну а к этим-то полям зачем указатели? Было же нормально.
Тебе и для Texture и Sprite они, возможно, тоже не нужны.
0
22 / 1 / 0
Регистрация: 25.10.2016
Сообщений: 36
01.05.2017, 18:51  [ТС]
Ну кстати памяти не сильно освобождает и у меня другая проблема возникла. Но всё равно спасибо, я в общем просто хотел разобраться.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
01.05.2017, 18:51
Помогаю со студенческими работами здесь

Ошибка с памятью при работе класса и map
В общем есть класс тест class test { public: test(void); ~test(void); test(const int&amp;); private:

Ошибка с памятью при работе с двумерными массивами.
{ int i, j, c, kol; n=StrToInt(Edit1-&gt;Text); m=StrToInt(Edit2-&gt;Text); for(i=0; i&lt;n; i++) { for(j=0; j&lt;m; j++)...

Ошибка с памятью при работе с динамическим двумерным массивом
Вроде всё правильно, но в ходе работы программа выдаёт ошибку(на скрине). Дело в двумерном динамическом массиве. Ниже код с функцией, куда...

Ошибка в работе с памятью
После использования программы и последующего выхода из нее возникает ошибка HEAP CORRUPTION DETECTED. Хотелось бы узнать, в чем причина и...

Управление динамической памятью при работе с классами
Всем привет! Есть очень сложный класс, внутри которого есть куча указателей, в его конструкторе все эти указатели получают память при...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
9
Ответ Создать тему
Новые блоги и статьи
Символьное дифференцирование
igorrr37 13.02.2026
/ * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .
Камера 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. Пошагово создадим проект для загрузки изображения. . .
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, то после закрытия окошка. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru