Cделать конструктор по умолчанию и копирования для класса

@ThreeS · Регистрация: 24.12.2014

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

Вот у меня класс строка:

C++

class strg  {
    char *str;
    int len;
public:
    strg();
    strg(const strg &); 
};

реализация:

C++

strg::strg()
{ 
    str = NULL;
    len = 0;
}
 
strg::strg(const strg &copy)
{ 
    cout << "Konstructor copy: ";
    len = copy.len;
    str = new char[len + 1];
    strcpy(str, copy.str);
}

А вот второй класс:

C++

class element
{
    strg str;
    element *next;
public:
    element();
    element(const element &);
};

Как мне сделать конструктор по умолчанию и копирования для класса element.
я вот так написал:

C++

element::element()
{
    str;
    next = NULL;
}
 
element::element(const element &copy)
{
    cout << "Konstructor copy: ";
    next = copy.next;
    str = copy.str;
}

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

@Gr1f0nn · 13.05.2015, 19:39

C++

element::element(const element &copy)
{
cout << "Konstructor copy: ";
next = copy.next;
str = copy.str; // У вас здесь вызов операции присваивания, а не копирования строки, как в конструкторе копирования
}

Перегрузите в вашем классе "строка" операцию присваивания =.
Она будет похожа на конструктор копирования, но необходимо будет также сделать проверку на присваивание элемента самому себе.

И перед присваиванием, я предполагаю, надо будет освободить память строки, в которую присваиваем, и выделить заново под новую строку

@Renji · 13.05.2015, 19:42

Что-то с памятью моей стало... strcpy разве переваривает NULL в качестве аргумента? Попробуйте написать что-то в духе:

C++

strg::strg():str(new char[1]),len(0)
{
    *str=0;
}

	13.05.2015, 19:42