В конструкторе копирования отцовского (_str) класса возникает некое "необработанное исключение"

@ilyhan2009 · Регистрация: 22.12.2013

Добрый день. Делаю курсовик в универе, иерархия классов и работа с ними.
В конструкторе копирования отцовского (_str) класса возникает некое "необработанное исключение" и строка (поле класса, str) почему-то имеет свойство "bad prt", соответсвенно, программа вываливается с ошибкой. В отнаследованных классах все в порядке.
Был бы очень признателен, если бы указали на ошибку.
Строка 60 - констр. копирования;
Строка 391 - его вызов

C++

#include <iostream>
#include <string>
#include <stdlib.h> //atoi\itoa
#include <conio.h> //getch
using namespace std;
 
 
 
class _str
{
protected:
    int len;
    char *str;
    
public:
    _str()
    {   cout<<"\n _str()";
        len = 1;
        str = new char;
        str[0] = '\0';
    }
 
 
    _str(char ch)
    {   cout<<"\n _str(char ch)";
        len = 2;
        str = new char[2];
        str[0] = ch;
        str[1] = '\0';
    }
 
 
    _str(char *temp_str)
    {   cout<<"\n _str(char *temp_str)";
        len = strlen(temp_str);
        str = new char[len+1];
        for (int i=0; i<len; i++)
            str[i] = temp_str[i];
        str[len] = '\0';
    }
 
 
    ~_str()
    {   cout<<"\n ~_str()";
        delete [] str;
    }
 
 
    int get_len()
    {
        return len;
    }
 
 
    char *get_str()
    {
        return str;
    }
 
    _str(_str &copy_str)
    {   cout<<"\n _str(_str &copy_str)";
        len = copy_str.get_len();
        if (str!=NULL)
        {
            delete [] str;
            str = new char[len+1];
        }
        strcpy(str,copy_str.get_str());
    }
 
    void clean_str()
    {
        delete [] str;
        len = 1;
        str = new char[len];
        str[0] = '\0';
    }
 
};
 
 
class _str_id : public _str
{
public:
 
    bool isid(char *cand) //cand=candidate
    {
        if (isdigit(cand[0])) return false;
        int candlen = 0;
        int AsciiValue = 0;
        for (int i = 0; i < strlen(cand); i++)
        {
            AsciiValue = (int)cand[i];
            if (((AsciiValue >= 65)&&(AsciiValue <=90))||((AsciiValue >= 97)&&(AsciiValue <=122))||(AsciiValue == 95)||((AsciiValue >= 48)&&(AsciiValue <=57)))
                candlen++;      
        }
        if (candlen == strlen(cand)) return true;
        return false;
    }
 
 
    _str_id()
    {   cout<<"\n _str_id()";
        len = 1;
        str = new char[1];
        str[0] = '\0';
    }
 
 
    _str_id(char ch)
    {   cout<<"\n _str_id(char ch)";
        if (isid(&ch)) 
        {
            len = 2;
            str = new char[2];
            str[0] = ch;
            str[1] = '\0';
        }
        else
        {
            len = 1;
            str = new char[1];
            str[0] = '\0';
        }
    }
 
 
    _str_id(char *temp_str)
    {   cout<<"\n _str_id(char *temp_str)";
        if (isid(temp_str))
        {
            len = strlen(temp_str);
            str = new char[len+1];  
            for (int i=0; i<len; i++)       
                str[i] = temp_str[i];   
            str[len] = '\0';
        }
        else
        {
            len = 1;
            str = new char[1];
            str[0] = '\0';
        }
 
    }
 
 
    ~_str_id()
    {   cout<<"\n ~_str_id()";
        //delete [] str;
    }
 
 
    _str_id(_str_id &copy_str)
    {   cout<<"\n _str_id(_str_id &copy_str)";
        len = copy_str.get_len();
        if (str!=NULL)
        {
            delete [] str;
            str = new char[len+1];
        }
        strcpy(str,copy_str.get_str());
    }
 
 
    int first_ins(char ch)
    {
        for (int i=0; i<this->get_len(); i++)
            if (ch == str[i]) return i;
        return -1;
    }
 
 
    const _str_id& operator = (const _str_id &M)
    {
        delete[] str;
        len=M.len;
        str = new char[len+1];
        strcpy(str, M.str);
        return *this;
    }
 
    _str_id& operator + (const _str_id &M)
    {
        int k = len-1;
        char *temp = new char[k+1];
        *temp = *str;
        temp[k] = '\0';
        len = k+M.len+1;
        delete[] str;
        str = new char[len];
        for (int i=0; i<strlen(temp);i++)
            str[i] = temp[i];
        int i = 0;
 
        while (M.str[i] != '\0')
        {
            str[k] = M.str[i];
            i++;
            k++;
        }
        delete [] temp;
 
        str[len-1] = '\0';
        return *this;
    }
 
 
    char operator [] (int i)
    {
        if (i<len)
            return *(str+i);
    }
 
};
 
 
class _str_complex : public _str
{
public:
 
    bool iscomplex(char *cand) //cand=candidate
    {
        int s = 0;  //number of - and +
        int i_ind = -1; //index of i
        int digsum = 0; //number of digits
        for(int i = 0; i<strlen(cand); i++)
        {
            if ((cand[i] == '-')||(cand[i] == '+')) 
                s++;
            if (cand[i] == 'i') i_ind = i;
            if (isdigit(cand[i])) digsum++;
        }
 
        if ((s == 2)&&((cand[0] == '-')||(cand[0] == '+'))&&((cand[i_ind+1] == '-')||(cand[i_ind+1] == '+'))&&(digsum == strlen(cand)-3))
            return true;
        if ((s == 1)&&((cand[0] == '-')||(cand[0] == '+')||(cand[i_ind+1] == '-')||(cand[i_ind+1] == '+'))&&(digsum == strlen(cand)-2))
            return true;
        if ((s == 0)&&(digsum == strlen(cand)-1))
            return true;
        return false;
    }
 
 
    _str_complex()
    {   cout<<"\n _str_complex()";
        len = 1;
        str = new char[1];
        str[0] = '\0';
    }
 
 
    _str_complex(char *temp_str)
    {   cout<<"\n _str_complex(char *temp_str)";
        if (iscomplex(temp_str))
        {
            len = strlen(temp_str);
            str = new char[len+1];  
            for (int i=0; i<len; i++)       
                str[i] = temp_str[i];   
            str[len] = '\0';
        }
        else
        {
            len = 1;
            str = new char[1];
            str[0] = '\0';
        }
 
    }
 
 
    _str_complex(_str_complex &copy_str)
    {   cout<<"\n _str_complex(_str_complex &copy_str)";
        len = copy_str.get_len();
        if (str!=NULL)
        {
            delete [] str;
            str = new char[len+1];
        }
        strcpy(str,copy_str.get_str());
    }
 
 
    ~_str_complex()
    {   cout<<"\n ~_str_complex()";
        //delete [] str;
    }
 
 
    const _str_complex& operator = (const _str_complex &M)
    {
        delete [] str;
        len=M.len;
        str = new char[len+1];
        for (int i = 0; i < len; i++)
            str[i] = M.str[i];
        str[len] = '\0';
        return *this;
    }
 
 
    _str_complex& operator + (_str_complex &M)
    {
        char *temp1 = new char[len];
        int i, re1, re2, im1, im2, k;
        re1 = 0;
        re2 = 0;
        im1 = 0;
        im2 = 0;
        for (i = 0; i<len; i++)
            if (str[i]!='i') temp1[i] = str[i];
            else 
            {
                temp1[i]='\0';
                break;
            }
        char *temp2 = str+i+1;
 
        re1 = atoi(temp1);  //auto adding of \0
        im1 = atoi(temp2);
 
        delete [] temp1;
 
        temp1 = new char[M.get_len()];
 
        for (i = 0; i<M.len; i++)
            if (M.str[i]!='i') temp1[i] = M.str[i];
            else 
            {
                temp1[i]='\0';
                break;
            }
        temp2 = M.str+i+1;
 
        re2 = atoi(temp1);  //auto adding of \0
        im2 = atoi(temp2);
 
        delete [] temp1;
        delete [] str;
 
        re1 = re1 + re2; //sum of re
        im1 = im1 + im2; //sum of im
 
        if (len > M.get_len()) len = len+1;
        else len = M.get_len()+1;
 
        temp1 = new char[len+1];
        temp2 = new char[len+1];
 
        itoa(re1, temp1, 10);
        itoa(im1, temp2, 10);
 
        str = new char[len+1];
 
        for (i=0; i<strlen(temp1);i++)
            str[i] = temp1[i];
        str[i] = 'i';
 
        k = i+1;
        i = 0;
 
        while (temp2[i] != '\0')
        {
            str[k] = temp2[i];
            i++;
            k++;
        }
        str[k] = '\0';
        len = strlen(str);
 
        delete [] temp1;
        delete [] temp2;
 
        return *this;
    }
    
 
 
};
 
 
int main()
{
    setlocale(LC_ALL, "rus");
 
    cout<<"\n                         WOW! This is successful build!\n";
    _str *a = new _str;
    _str *b = new _str('a');
    _str *c = new _str("privet");
    _str *c1 = new _str(*c);
    cout<<"\n";
    _str_id *d = new _str_id;
    _str_id *e = new _str_id('d');
    _str_id *f = new _str_id("this_is_sparta");
    _str_id *f1 = new _str_id(*f); //uses copy-constructor
    cout<<"\n";
    _str_complex *g = new _str_complex;
    _str_complex *k = new _str_complex("-4432i+54546");
    _str_complex *k1 = new _str_complex("13i-234");
    cout<<"\n________________________________________________________________________________\n";
 
    cout<<' '<<c1->get_str()<<' '<<c1->get_len()<<'\n'; //returns c1.str and c1.len
    cout<<' '<<f1->get_str()<<' '<<f1->get_len()<<'\n'; //returns f1.str and f1.len
    cout<<' '<<f->first_ins('o'); //returns -1, because 'o' not in "this_is_sparta"
    cout<<'\n'<<' '<<f->first_ins('p'); //returns 9, because first index is 0
 
    *d = *e+*f;
    cout<<'\n'<<' '<<d->get_str();
 
    *g = *k+*k1;
    cout<<'\n'<<' '<<g->get_str()<<' '<<g->get_len();
 
    char ch;
    ch = (*f)[8];
    cout<<'\n'<<' '<<ch;
 
 
    cout<<"\n________________________________________________________________________________";
    delete a;
    delete b;
    delete c;
    cout<<"\n";
    delete d;
    delete e;
    delete f;
    cout<<"\n";
    delete g;
    delete k;
    delete k1;
    cout<<"\n ";
    getch();
    return 0;
}

ValeryS · 25.03.2014, 20:17

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

if (str!=NULL)
{
delete [] str;

в str мусор
соответственно

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

delete [] str;

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

делай так

C++

 _str(_str &copy_str)
    {   cout<<"\n _str(_str &copy_str)";
        len = copy_str.get_len();
         str = new char[len+1];
       strcpy(str,copy_str.get_str());
    }

далее, объект того же класса, можно напрямую обращаться к закрытым членам
еще упрощаем

C++

 _str(_str &copy_str)
    {   cout<<"\n _str(_str &copy_str)";
         str = new char[copy_str.len+1];
            strcpy(str,copy_str.str);
    }

на остальное не смотрел

@ilyhan2009 · 25.03.2014, 20:23 **[ТС]**

ValeryS, Спасибо, сам изначально писал без delete, но препод начинала заливать про то, вдруг там уже что-то есть...
Но это ведь, кажется, неверно, ведь, я же правильно понимаю, что конструктор копирования служит для иницилизации нового экземпляра уже имеющимся, ведь нельзя в уже инициализированный им что-то скопировать? Ведь и в отнаследованных тоже delete удалять, да?

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

еще упрощаем

этого не нужно, т.к. мне нужно иметь поле len в экз. класса

@DrOffset · 25.03.2014, 20:23

Конструктор копирования нужен с константной ссылкой.

@ilyhan2009 · 25.03.2014, 20:24 **[ТС]**

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

Конструктор копирования нужен с константной ссылкой.

_str(const _str &copy_str) ??

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

Этот конструктор неверно реализован.

а как правильно?

@DrOffset · 25.03.2014, 20:26

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

C++

_str()
{  cout<<"\n _str()";
    len = 1;
    str = new char;
    str[0] = '\0';
}

Этот конструктор неверно реализован.

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

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

_str(const _str &copy_str) ??

ага

ValeryS · 25.03.2014, 20:32

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

пасибо, сам изначально писал без delete, но препод начинала заливать про то, вдруг там уже что-то есть...

ну два варианта
или он сам ничего не знает
или хотел тебя проверить
этак
при каждом выделении придется писать

C++

char *a;
if(a)
 {
   delete a;
   a=new char;
 }

смешно?

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

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

этого не нужно, т.к. мне нужно иметь поле len в экз. класса

пардон накосячил

C++

_str(const _str &copy_str)
    {   cout<<"\n _str(_str &copy_str)";
         len=copy_str.len;
       str = new char[len+1];
        strcpy(str,copy_str.str);
    }

@ilyhan2009 · 25.03.2014, 20:32 **[ТС]**

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

ага

у меня тогда copy_str в теле констр. копирования подчеркнут красным и написано:

В конструкторе копирования отцовского (_str) класса возникает некое "необработанное исключение"

@DrOffset · 25.03.2014, 20:39

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

а как правильно?

Сначала поясню почему вот это неправильно:

C++

_str()
{  
    len = 1;
    str = new char;
    str[0] = '\0';
}

Во-первых обычно семантика подсчета длины строки не подразумевает учет завершающего нуля. То есть строка в которой только '\0' имеет длину 0.
Во-вторых в случае, когда у тебя создалась пустая строка и больше в нее ничего не записывалось данный код приводит к неопределенному поведению при вызове деструктора. Т.к. delete [] предполагает, что удаляется массив, однако у тебя выделялся только один символ.
Поэтому лучше так:

C++

    _str()
    {  
        len = 0; // в строке нет значащих символов
        str = new char[1];
        str[0] = '\0';
    }
     _str(char ch)
    {   cout<<"\n _str(char ch)";
        len = 1; // надо 1, вместо 2
        str = new char[2];
        str[0] = ch;
        str[1] = '\0';
    }
    _str(char const *temp_str) // не забываем про const, мы же не меняем строку temp_str
    {   cout<<"\n _str(char *temp_str)";
        len = strlen(temp_str); // а вот здесь все правильно
        str = new char[len+1];
        strncpy(str, temp_str, len);
        str[len] = '\0';
    }

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

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

у меня тогда copy_str в теле констр. копирования подчеркнут красным и написано:

Потому что остальной код тоже надо в порядок приводить. Например функции get_len и get_str должны быть const:

C++

    int get_len() const
    {
        return len;
    }
  
    char *get_str() const
    {
        return str;
    }

@ilyhan2009 0 / 0 / 0 Регистрация: 22.12.2013 Сообщений: 17
	25.03.2014, 20:23 [ТС]	3
	ValeryS, Спасибо, сам изначально писал без delete, но препод начинала заливать про то, вдруг там уже что-то есть... Но это ведь, кажется, неверно, ведь, я же правильно понимаю, что конструктор копирования служит для иницилизации нового экземпляра уже имеющимся, ведь нельзя в уже инициализированный им что-то скопировать? Ведь и в отнаследованных тоже delete удалять, да? Сообщение от ValeryS еще упрощаем этого не нужно, т.к. мне нужно иметь поле len в экз. класса 0

@DrOffset 15801 / 8593 / 2103 Регистрация: 30.01.2014 Сообщений: 14,760
	25.03.2014, 20:23	4
	Конструктор копирования нужен с константной ссылкой. 1

@ilyhan2009 0 / 0 / 0 Регистрация: 22.12.2013 Сообщений: 17
	25.03.2014, 20:24 [ТС]	5
	Сообщение от DrOffset Конструктор копирования нужен с константной ссылкой. _str(const _str &copy_str) ?? Сообщение от DrOffset Этот конструктор неверно реализован. а как правильно? 0

@ilyhan2009 0 / 0 / 0 Регистрация: 22.12.2013 Сообщений: 17
	25.03.2014, 20:32 [ТС]	8
	Сообщение от DrOffset ага у меня тогда copy_str в теле констр. копирования подчеркнут красным и написано: 0