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

С++ оператор присвоения

10.11.2013, 17:28. Показов 2948. Ответов 13
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Всем доброго времени суток.
Я написал класс в котором имеется приватный указатель или вернее 2 мерный массив.
Выглядит он вот так вот:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
    class Bitmap
    {
        protected:
            Pixel** pixels = nullptr;
            Vector2 Size;
 
        public:
            //= Конструкторы / Деструкторы =============================
            Bitmap();
            virtual ~Bitmap();
            Bitmap(Vector2 size);
 
            //= Возращающие ============================================
            Pixel   GetPixel(Vector2 position);
 
            //= Перегруженные операторы ================================
            //Bitmap operator=(const Bitmap right);
 
    };
Проблема в том, что при присвоении через оператор '=' я натыкаюсь на неизбежную ошибку во время деструкции.
Как то так например:
C++
1
2
3
Bitmap amp = Bitmap(Vector2(25,25));
Bitmap bmp = amp;
// При уничтожении amp, ресурсы на которые ссылается bmp так же прекратят своё существование
Я попытался переписать оператор присвоения, пока что тупо с помощью инициализации.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
    Bitmap Bitmap::operator=(const Bitmap right)
    {
        //проверка на самоприсваивание
        if (this == &right)
        {
            return *this;
        }
 
        Size = right.GetSize();
        pixels = new Pixel*[Size.x]();
        for(unsigned int x = 0; x < Size.x; x++)
            pixels[x] = new Pixel[Size.y]();
 
        return *this;
    }
Ошибка всё равно вылазит.
Помогите, что я делаю не так???
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
10.11.2013, 17:28
Ответы с готовыми решениями:

Шаблоны и оператор присвоения
что я не правильно делаю? у меня входит в бесконечную рекурсию на операторе присвоения template &lt;typename T&gt; class A { ...

Оператор присвоения для класса
Никак не получается написать оператор присвоения. Помогите пожалуйста! #include &lt;iostream&gt; #include &lt;conio.h&gt; using...

Оператор присвоения в бинарном дереве
Не смог разобраться. Прокомментируйте, будьте добры. Задание такое, определите стандартный конструктор и функции управления...

13
 Аватар для ninja2
979 / 196 / 33
Регистрация: 26.09.2012
Сообщений: 2,041
10.11.2013, 17:38
Shaman163, я твой говнокод не смотрел, но понял в чом причина, ты просто видимо при присваивании ссылку передаешь на ту же самую область памяти, а нужно заново новую область памяти выделять. В операторе присваивания новую область памяти выделяй и в нее копируй значение из объекта который ты хочешь присвоить, а то у тебя получается раз деструктор вызвался для ячейки допустим по адресу1 и ты снова в другом деструкторе уничтожаешь тот же самый адресс1, поэтому и ошибка.

Добавлено через 4 минуты
Shaman163, возможно ошибка из за того что ты тут копию объекта вернул Bitmap Bitmap::operator=(const Bitmap right), а нужно ссылку, возможно вызвался деструктор в автопамяти и удалил адрес который ты выделил.
0
Неэпический
 Аватар для Croessmah
18144 / 10728 / 2066
Регистрация: 27.09.2012
Сообщений: 27,026
Записей в блоге: 1
10.11.2013, 17:49
Цитата Сообщение от Shaman163 Посмотреть сообщение
Проблема в том, что при присвоении через оператор '='
Сделайте конструктор копирования еще!
1
6 / 6 / 1
Регистрация: 22.12.2011
Сообщений: 134
10.11.2013, 22:17  [ТС]
ninja2 Огромное вам спасибо за то что рассказали мне о том, что я и так знал. Я не спрашивал что за ошибка, я спрашивал как её избежать. С ссылками пробовал, первый раз вообще с хабра скопировал пример перегрузки, но так как ошибка всё равно была, решил попробовать так как есть сейчас..
Вот код, до моей самодеятельности:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
    Bitmap& Bitmap::operator=(const Bitmap& right)
    {
        if (this == &right)
        {
            return *this;
        }
 
        Size = right.GetSize();
        pixels = new Pixel*[Size.x]();
        for(unsigned int x = 0; x < Size.x; x++)
            pixels[x] = new Pixel[Size.y]();
 
        return *this;
    }
Но как я уже сказал, проблему это не решает. И да, попрошу без оскорблений.

Добавлено через 45 минут
Croessmah
Большое спасибо, до этого вообще не знал о конструкторе копирования
Буду признателен если кто нибудь на пальцах покажет где именно вызывается оператор присвоения..
0
 Аватар для ninja2
979 / 196 / 33
Регистрация: 26.09.2012
Сообщений: 2,041
10.11.2013, 23:28
Shaman163, Ты вообще походу не понимаешь почему у тебя ошибка происходит знаток, ты обращаешься к недействительному указателю, исправь и все будет работать.

Добавлено через 2 минуты
Цитата Сообщение от Shaman163 Посмотреть сообщение
// При уничтожении amp, ресурсы на которые ссылается bmp так же прекратят своё существование
Ну знаешь в чом ошибка выдели новый ресурс, не используй одинаковые ресурсы.
0
Неэпический
 Аватар для Croessmah
18144 / 10728 / 2066
Регистрация: 27.09.2012
Сообщений: 27,026
Записей в блоге: 1
10.11.2013, 23:47
Цитата Сообщение от Shaman163 Посмотреть сообщение
где именно вызывается оператор присвоения..
Там, где происходит присваивание, то есть
C++
1
объект1 = объект2 ;
при
C++
1
ТипОбъекта объект1 = объект2 ;
будет вызван конструктор копий, так это не присвоение, а создание объекта1 на основе объекта2
1
 Аватар для ninja2
979 / 196 / 33
Регистрация: 26.09.2012
Сообщений: 2,041
11.11.2013, 00:21
Цитата Сообщение от Croessmah Посмотреть сообщение
будет вызван конструктор копий, так это не присвоение, а создание объекта1 на основе объекта2
Чо сразу конструктор копий? а мб он просто ссылку создаст и увеличит счетчик ссылок используя методику "copy on write"?

Добавлено через 2 минуты
Там может и не вызваться конструктор копий, он просто код повторит конструктора копирования и вернет ссылку на объект. Можно б конструктор копирования и не создавать.
0
6 / 6 / 1
Регистрация: 22.12.2011
Сообщений: 134
11.11.2013, 17:56  [ТС]
Цитата Сообщение от ninja2 Посмотреть сообщение
Ты вообще походу не понимаешь почему у тебя ошибка происходит знаток
Ну так вы же сами привели строку где я и сообщаю о том, что понимаю где именно происходит ошибка и почему..
Но меня больше заинтересовало то что вы назвали мой класс гавнокодом, можете как то аргументировать? Ну помимо той проблемы с деструкцией из за которой собственно весь сыр-бор.
Хотелось бы услышать совет от Гуру C++.
Croessmah, и снова спасибо за разъяснение.
0
 Аватар для ninja2
979 / 196 / 33
Регистрация: 26.09.2012
Сообщений: 2,041
12.11.2013, 06:37
Цитата Сообщение от Shaman163 Посмотреть сообщение
Хотелось бы услышать совет от Гуру C++.
То я для красоты словечко добавил, на самом деле я считаю любой код рабочий нормальным, правда щас переписываю код из пхп на С++ ну я его полтора года назад писал, так там да такое понаписано - не оптимизировано например файл можно раз было считать и данные в цикл передавать, а я в цикле каждый раз считываю. Да после С++ пхп вообще детским языком кажется, я сайт который раньше на пхп месяц делал, щас за 2 дня сделал, что называется скил С++ )) . Да пхп после С++ вообще не интересный язык!!!
0
12.11.2013, 06:44

Не по теме:

Цитата Сообщение от ninja2 Посмотреть сообщение
Да после С++ пхп вообще детским языком кажется, я сайт который раньше на пхп месяц делал, щас за 2 дня сделал, что называется скил С++
На полке пирожок лежит :)
Цитата Сообщение от Shaman163 Посмотреть сообщение
Хотелось бы услышать совет от Гуру C++.
- Самозванец...
- От самозванца слышу!
(c) Иван Васильевич меняет профессию

0
 Аватар для ninja2
979 / 196 / 33
Регистрация: 26.09.2012
Сообщений: 2,041
12.11.2013, 06:50

Не по теме:

Блин Croessmah, уже ответил, а я хотел подредактировать сообщение выше, ну ладно...



Добавлено через 6 минут
Ладно скоро господа увидете бесплатную cms: "kselax cms" - легкий движок для сайта на текстовых файлах от гуру С++ и экс эксперта ПХП, выложу в папбик, за 3 дня писаный килобайт 30 примерно веса и скорость загрузки страницы 0.01 секунда, короче бешеный движок можно использовать для высоконагруженных проектов. Доки только оформлю и видео-инструкцию по установке и использованию.
0
12.11.2013, 08:17

Не по теме:

Цитата Сообщение от ninja2 Посмотреть сообщение
"kselax cms" - легкий движок для сайта на текстовых файлах от гуру С++ и экс эксперта ПХП, выложу в папбик, за 3 дня писаный килобайт 30 примерно веса и скорость загрузки страницы 0.01 секунда, короче бешеный движок можно использовать для высоконагруженных проектов.
Ну ждём релиз тогда)

0
6 / 6 / 1
Регистрация: 22.12.2011
Сообщений: 134
17.11.2013, 21:03  [ТС]
Кстати всегда мучил вопрос по поводу полей класса внутри операторов.
Если посмотреть на тот же хабровский код:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
class Integer
{
private:
    int value;
public:
    Integer(int i): value(i) 
    {}
 
    Integer& operator=(const Integer& right) {
        //проверка на самоприсваивание
        if (this == &right) {
            return *this;
        }
        value = right.value;
        return *this;
    }
};
То мы увидим обращение к приватной переменной класса..
А это просто ломает всю мою сложившуюся у меня о C++ парадигму. Правильно ли я понял, что у перегрузки операторов есть свои привилегии?
0
Неэпический
 Аватар для Croessmah
18144 / 10728 / 2066
Регистрация: 27.09.2012
Сообщений: 27,026
Записей в блоге: 1
17.11.2013, 21:18
Цитата Сообщение от Shaman163 Посмотреть сообщение
Правильно ли я понял, что у перегрузки операторов есть свои привилегии?
C++
1
Integer& operator=(const Integer& right)
это такой же член класса, как и другие и у него есть доступ к приватной области. Другое дело если будет описан "внешний" оператор присваивания
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
17.11.2013, 21:18
Помогаю со студенческими работами здесь

Конструктор перемещения и оператор присвоения для двусвязного списка
Добрый день. Помогите разобраться в вопросом: пишу реализацию двусвязного списка, хотел бы разобраться с конструктором перемещения и...

Оператор присвоения копии для класса, действующего как значения
Приветсвую. Увидел в книге Липпмана, &quot;Язык программирования С++. Базовый курс&quot;, определение оператора присвоения для класса,...

Создать класс Str (символьная строка). Переопределить оператор присвоения.
Помогите решить с обяснениями. Создать класс Str (символьная строка). Переопределить оператор присвоения.

E0349 отсутствует оператор "=", соответствующий этим операндам. В перегруженном операторе присвоения
Всем привет! Начал изучать С++ по книжке Герберта Шилдта. Дошёл до перегрузки оператора присвоения и столкнулся с такой ошибкой. И до этого...

Как правильно перегрузить оператор присвоения под параметр типа AnsiString
Вот делаю домашку, и проблема, моя строка ждет интов, а не самих строк AnsiString, Builder 6 стоит Вот сами класы: class CMas {...


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

Или воспользуйтесь поиском по форуму:
14
Ответ Создать тему
Новые блоги и статьи
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
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
Создание Single Page Application на фреймах
krapotkin 16.11.2025
Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru