Сбой перегруженной функции оператора

Question

Сбой перегруженной функции оператора

0

Здесь я написал 2 перегруженных оператора:

stringb operator+(const stringb &a,const stringb &b)
{
    stringb temp;
    temp.len = a.len + b.len;
    temp.p=new char[(temp.len)+1];
    strcpy(temp.p, a.p);
    strcat(temp.p, b.p);
    return (temp);
}
stringb operator-(const stringb &a,const stringb &b)
{
    stringb temp;
    temp.len=a.len-b.len;
    temp.p=new char[temp.len+1];
    strcpy(temp.p,a.p);
    return (temp);
}

Однако, когда я компилирую фактический код, весь код работает, за исключением части, когда я вызываю этот оператор, и я получаю мусор из put. Что не так с моими функциями?

EDIT: Объявление класса stringb:

class stringb
{
    public:
        char *p;
int len;
public:
    stringb()
    {
    len=0;
    p=0;
    }
stringb(const char *s)
{
    len=strlen(s);
    p=new char[len+1];
    strcpy(p,s);

}
stringb(const stringb &s)
{
    len=s.len;//strlen(s);
    p=new char[len+1];
    strcpy(p,s.p);
}
~stringb()
{
    delete p;
}

friend int operator==(const stringb &a,const stringb &b);
friend stringb operator+(const stringb &a,const stringb &b);
friend stringb operator-(const stringb &a,const stringb &b);
friend void putstring(const stringb a);

};

Anurupa_Dey 23 нояб. 2013, в 17:37

Источник

3

Объявление stringb пожалуйста
P0W 23 нояб. 2013, в 16:09
0

WTH это stringb , не могли бы вы показать это, пожалуйста!
πάντα ῥεῖ 23 нояб. 2013, в 16:09
0

@P0W P0W Предоставленная декларация.
Anurupa_Dey 23 нояб. 2013, в 16:13
0

@g-makulik g-makulik Предоставлена декларация.
Anurupa_Dey 23 нояб. 2013, в 16:13
0

Только одна вещь, чтобы выбросить их, что происходит, когда вы вызываете strlen для указателя, который указывает на ноль? У вас есть подрядчик по умолчанию, который по умолчанию указывает на NULL. Однако, если вызывается конструктор копирования, он вызывает strlen для указателя, который указывает на ноль. Это также приведет к сбою вашего кода. Тем не менее, правильный ответ g-makulik ниже.
Freddy 23 нояб. 2013, в 17:18

Показать ещё 3 комментария

Теги:

c++

operator-overloading

2 ответа

Ещё вопросы

WTH это stringb , не могли бы вы показать это, пожалуйста!
@P0W P0W Предоставленная декларация.
@g-makulik g-makulik Предоставлена декларация.
Только одна вещь, чтобы выбросить их, что происходит, когда вы вызываете strlen для указателя, который указывает на ноль? У вас есть подрядчик по умолчанию, который по умолчанию указывает на NULL. Однако, если вызывается конструктор копирования, он вызывает strlen для указателя, который указывает на ноль. Это также приведет к сбою вашего кода. Тем не менее, правильный ответ g-makulik ниже.

πάντα ῥεῖ · Answer 1 · 2013-11-23T13-15-00.000Z

Ваша проблема здесь:

~stringb()
{
    delete p; // <<<<<
}

Это будет выполнено, если temp выходит за пределы области определения вашего оператора.

Чтобы заставить ваш код работать, вам нужна правильная реализация правила 3 в вашем классе stringb. Вы также можете посмотреть этот хороший IDE один образец, P0W имеет настройку из вашего кода.

я должен удалить это? удаление его приводит к сбою кода.
Нет! Вам нужно будет реализовать правильное правило 3 в stringb
Вот хорошее объяснение и пример: Википедия «Правило трех»
Вклад SO в стек stackoverflow.com/questions/4172722/what-is-the-rule-of-three
@john THX, быстро не нашел, использовал неверные параметры поиска ...
@Anurupa_Dey Посмотрите на вашу реализацию и Правило трех
@ g-makulik Вы должны добавить утверждение «правило трех» в свой ответ
@ g-makulik Не думаю, что ОП понимает ваш ответ. Может быть, проведя его через что может привести к сбою его кода и почему? Я бы ответил, но вы ответите правильно. Я просто не думаю, что он достаточно знаком с правилом трех. Просто мысль, отличный ответ, кстати.

Eutherpy · Answer 2 · 2013-11-23T13-47-00.000Z

0

Попробуйте вернуть ссылку:

stringb& operator+(const stringb &a, const stringb &b);
{
   stringb *temp = new stringb();
   temp->len = a.len + b.len;
   [...]
   return *(temp);
}

Eutherpy 23 нояб. 2013, в 13:47

0

Сделал это: stringb *temp = new stringb; temp->len = a.len - b.len; temp->p=new char[temp->len+1]; strcpy(temp->p,ap); return *(temp); Все еще падает.
Anurupa_Dey 23 нояб. 2013, в 16:27
0

Кажется, ошибка в реализации -operator. + работает просто отлично.
Anurupa_Dey 23 нояб. 2013, в 16:39
0

Ну, а если b длиннее? Как насчет абс (a.len-b.len)?
Eutherpy 23 нояб. 2013, в 16:42
0

все еще вылетает даже после использования abs, в то время как + one работает превосходно!
Anurupa_Dey 23 нояб. 2013, в 16:48
0

Не возвращайте ссылку на память, выделенную в свободном хранилище (куче). После возврата этой функции нет указателей на ресурс, который она выделяет.
Yakk - Adam Nevraumont 23 нояб. 2013, в 23:31

Показать ещё 3 комментария