0
class A {

  QRegExp * r;
public:
  A():r(new QRegExp) { ... }
  ~A() { delete r; }
  ...
  A(const A & a)
  {
     *this = a;
  }

  A & operator=(const A & a)
  {
     if (this == &a)
         return *this;
     delete r; //остановка здесь
     r = new QRegExp();
     return *this;
   }
}

//main
A a;
A a1 = a;

Почему и как можно исправить?

11
  • Хм. Вроде всё нормально в коде. Попробуйте пройтись отладчиком. Или сделайте cout в начале каждого метода класса A. --- А какое именно исключение?
    – VladD
    6 июн 2014 в 16:20
  • @VladD c:\work\build\qt5_workdir\w\s\qtbase\src\corelib\tools\qregexp.cpp:3891: ошибка: Exception at 0x7770af26, code: 0xc0000005: read access violation at: 0x0, flags=0x0 (first chance) Прохожу отладчиком: вызов конструктора копирования -> вызов оператора -> проверка на равенство (false) -> остановка
    – andrey3
    6 июн 2014 в 16:30
  • @andrey3: О, отлично. А чему равны значения всех релевантных переменных в этой точке? this, a, this->r? Проверьте на всякий случай, действительно ли из конструктора копирования вызывается оператор присваивания (протрассируйте или добавьте отладочную печать).
    – VladD
    6 июн 2014 в 16:35
  • @VladD a = 0x17d3b0 this = 0x17d3a0. значения this-r отладчик не выводит.
    – andrey3
    6 июн 2014 в 16:54
  • 1
    Моя ошибка. Обнулите r перед вызовом this =a
    – AlekseyOk
    6 июн 2014 в 17:25

3 ответа 3

1

А, я понял.

Когда вызывается конструктор A():r(new QRegExp) { ... }, поле r инициализируется. Но когда вызывается конструктор A(const A & a), поле r НЕ инициализируется, то есть, имеет случайное значение. Поэтому в операторе присваивания строка delete r удаляет случайный объект, «на кого бог пошлёт»!

Простейший выход — переписать конструктор так:

A(const A & a) : r(nullptr)
{
    *this = a;
}

Кстати, у вас не вполне правильно реализовано копирование: вы не копируете r, а создаёте новый. Нужно так:

 A& operator=(const A& a)
 {
     if (this != &a)
     {
         delete r;
         r = new QRegExp(*(a.r)); // создаёт копию
     }
     return *this;
}

Обратите внимание, что код

r = a.r;

был бы неправильным, т. к. оба объекта ссылались бы на один и тот же regex, и в деструкторе было бы непонятно, кому из них надо этот regex удалять.

1

Если я все верно понял, то вам нужно выделить код из вашего конструктора в отдельный метод. Конструктор копирования переписать примерно так(код не проверяю):

A(const A & a)
{
     this->r = new QRegExp(*(a.r));
     //вызов общего метода
}

При необходимости добавить в QRegExp конструктор копирования. И при необхоимости вызывать выделеный метод в операторе присвоения.

UPD: поправил код.

0

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

П.с. реализован он неверно

9
  • @andrey3 а как верно?
    – andrey3
    6 июн 2014 в 16:06
  • Оопс, ошибочка вышла: r = new QRegExp(а);
    – AlekseyOk
    6 июн 2014 в 16:10
  • @AlekseyOk хм... действительно. А как же тогда записать конструктор копирования через оператор присваивая? Просто у меня много раз будут повторяться одни действия и так в нескольких классах подряд.
    – andrey3
    6 июн 2014 в 16:19
  • @andrey3, A a1; a1 = a;
    – nanotexnik
    6 июн 2014 в 17:12
  • Или я не верно понял вопрос и тогда вам подойдет, A a1(a);
    – nanotexnik
    6 июн 2014 в 17:14

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.