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

Ситуация такова: есть строковый класс rstring (распространенный учебный пример для начинающих), содержащий два приватных элемента: char* (без нуль-терминатора) и unsigned int, описывающий его длину. При нем определены операторы сравнения, присвоения и конкатенации. В последних и есть проблема. Есть простая тестовая программка, где объявляются три объекта этого класса: два содержат разные строки, третий служит для получения результата конкатенации. Вроде бы все работает, но не работает: после распечатки сообщения, при завершении выдает Segmentation fault. Вроде понятно, что должна быть ошибка с выделением/освобождением памяти, но вот где точно - не знаю. Удалось выяснить, что проблема в операторе "+" (скорее всего), поскольку без него все работает нормально. Вот исходный код оператора:

Код

rstring operator+ (rstring a, rstring b) {    char *ts, *p, *ad, *bd;    int i, al, bl;    al = a.getlength ();    bl = b.getlength ();    ad = a.getptr ();    bd = b.getptr ();    ts = new char [al + bl + 1];    p = ts;    for (i = 0; i < al; i++)        *p++ = ad[i];    for (i = 0; i < bl; i++)        *p++ = bd[i];    *p = 0;    rstring str = ts;    delete [] ts;    return str; }

Кстати, аналогичный opertator+ (rstring, char*) вообще возвращает пустую строку... Не подскажете, в чем может быть проблема?

В глаза сразу это бросилось, в остальной код я не вглядывался...

Код

rstring str = ts;   delete [] ts;   return str;

Тут к str присваивается указатель ts, а потом мы удаляем все, на что этот указатель указывал.
И в результате возвращаем нечто...

Цитата(Baa @ 20.12.2002, 11:02)

Тут к str присваивается указатель ts, а потом мы удаляем все, на что этот указатель указывал. И в результате возвращаем нечто...

  Никогда не мог понять зачем нужны функции работы со строками которые возвращают указатели, память по которому рано или позно будет задействовано другими функциями.

На самом деле, возврат указателя - штука очень удобная... но вот надо быть полностью увереным в добросовестности других функций, дабы они за собой почистили память.

Цитата(Baa @ 20.12.2002, 15:02)

На самом деле, возврат указателя - штука очень удобная... но вот надо быть полностью увереным в добросовестности других функций, дабы они за собой почистили память.

Это конецно интересно, только вот возникает проблема когда два раза подряд эту функцию вызываеш.

Цитата(Baa @ 20.12.2002, 11:02)

Код rstring str = ts;   delete [] ts;   return str; Тут к str присваивается указатель ts, а потом мы удаляем все, на что этот указатель указывал. И в результате возвращаем нечто...

Смотри внимательнее. Переменныя str имеет тип rstring, а значит тут будет вызван конструктор rstring([const] char*), который скорее всего произведет копирование.

Вообще, на первый взгляд ошибок не видно. Походи деббагером - все проявиться.

Цитата(NightGoblin @ 20.12.2002, 09:31)

...

Вообще-то по-моему, лучше реализовать это примерно так:

Код

void shstrcpy(char* source,const char*& dest,int count); {  for (int i=0; i<count; ++i)     *source++=*dest++; } rstring operator+ (rstring a, rstring b) {    rstring new_str;    new_str.SetLenght(a.getlength ()+b.getlength ());        char* p=new_str.getbuffer();    shstrcpy(a.getbuffer(),p,a.getlength());    shstrcpy(b.getbuffer(),p,b.getlength());    return new_str; }

Понятнее и эффективнее. Можно, конечно, еще лучше, но это уже надо класс редактировать. getbuffer() должна возвращать внутренний указатель на char* и возможно следует сделать ее приватной. Функции getbuffer() и getlenght() (а можно и SetLenght()) нужно сделать inline.

Да! Совсем забыл. Убедись, что у тебя правильно определен конструктор копий.

Еще вылетать может быть если либо не инициализируется член содержащий длину или же он не меняется при изменении члена содержащего собсвенно строку, например в том же констукторе копирования. Вообщем все места в классе с ним связанные лучше проверить.

1. rstring str = ts;//Посмотри внимательно!!!!
2.   delete [] ts;
3. return str;

В строке 1, ты присваиваешь переменной "занчение" которое содержится в ts!
таким образом, если ты вызываешь после строки 2, что ты и сделал в строке 3, то ты обращаешся туда. чего уже нет!

Цитата(DrMasik @ 20.12.2002, 23:49)

1. rstring str = ts;//Посмотри внимательно!!!! 2.   delete [] ts; 3. return str; В строке 1, ты присваиваешь переменной "занчение" которое содержится в ts! таким образом, если ты вызываешь после строки 2, что ты и сделал в строке 3, то ты обращаешся туда. чего уже нет!

(а также 2 Baa)

Да, прошу прощения, забыл сказать, что конструктор rstring (const char*) для rstring действительно определен так, что char* копируется в rstring, а не просто указатель на него присваивается массиву char в rstring. Иначе бы вообще не работало...

Цитата(Fantasist @ 20.12.2002, 19:05)

Вообще-то по-моему, лучше реализовать это примерно так: ...

Хм, спасибо, надо попробовать

2Fantasist:

Короче, говоря, спасибо за улучшенную реализацию. Однако, при переделке обнаружилась некая странная вещь: проблема вроде как даже и не в "+": если это выражение поставить в конструктор (rstring c = a + b;), то все работает без всяких Segmentation fault'ов, даже со старой функцией оператора. С другой стороны, если написать просто c = a, все выполнится опять же гладко. И только когда объекту присваивается результат конкатенации, начинаются чудеса... В общем, будем копать

Прошу прощения за флуд. ))

Проблема выяснена: стоило в операторе присваивания (rstring::operator=) поставить аргументом rstring вместо rstring*, все заработало как надо Всем  огромное спасибо за помощь

ну и программер из меня однако

Цитата(NightGoblin @ 21.12.2002, 10:40)

Однако, при переделке обнаружилась некая странная вещь: проблема вроде как даже и не в "+":

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

Кстати, при передаче аргументов rstring& rstring::operator= (rstring); в виде rstring, а не rstring* использует указатели или так и пихает всю структуру в стек?...