Возвращение объекта из функции класса

[atenon]

Здравствуйте.
Возникла такая проблема. При возвращении объекта из метода класса, вместо того что бы его просто вывести на экран, как должно быть по коду(вернее как хотелось бы), начинается какаято фигня с вызовом перегружаемого конструктора.
Пожалуйста подскажите в чем ошибка!

#include "stdafx.h"
#include <iostream>
#include <time.h> 
#include <math.h> 
#include <iomanip> 
#include <ctype.h> 
#include <stdlib.h> 
#include <string.h> 
#include <assert.h> 
using namespace std;	
class String { 
	friend ostream &operator<< (ostream &, const String &); // Перегрузка операции вывода
public: 
	String(const char * = ""); // конструктор преобразования 
	String(const String &); // конструктор копии 
	~String(); // деструктор 
	String operator()(int, int); // возвращение подстроки 
	int getLength() const; // возвращение длины строки 
private: 
	char *sPtr; // указатель на начало строки 
	int length; // длина строки 
};
// Конструктор преобразования: преобразовывает char * в String 
String::String(const char *s) 
{
	cout << "Конструктор преобразования: " << s << endl; 
	length = strlen(s); // вычисление длины 
	sPtr = new char[length + 1]; // выделение памяти 
	strcpy(sPtr, s); // копирование строки аргумента в объект 
}
// Конструктор копии 
String::String(const String &copy) 
{ 
	cout << "Конструктор копии: " << copy.sPtr << endl; 
	length = copy.length; // копирование длины 
	sPtr = new char[length +1]; // выделение памяти 
	assert(sPtr != 0); // завершение, если память не выделена 
	strcpy(sPtr, copy.sPtr); // копирование строки 
}
// Деструктор 
String::~String() 
{ 
	cout << "Деструктор:"<< sPtr << endl; 
 
	delete [ ] sPtr; // освобождение памяти, отведенной строке 
}
	

String String::operator() (int index, int subLength) 
{ 
	cout<<"\n"<<"Вход в функцию"<<"\n";
	// подтверждение того, что индекс в диапазоне и длина подстроки >= 0 
	assert(index >= 0 && index < length && subLength >= 0); 
	 String sub; // пустой String 

	 // Определение длины подстроки 
	if ((subLength ==0) || (index + subLength > length)) // Если subLength== 0 то подстрока заканчивается на томже символе что и строка
		sub.length = length - index + 1; 
	else 
		sub.length = subLength + 1; 
	
	// удаление символа из объекта 
	delete sub.sPtr;
	// выделение памяти для подстроки 
	sub.sPtr = new char[sub.length]; 	

	// копирование подстроки в новый String 
	strncpy(sub.sPtr, &sPtr[index], sub.length); 
	sub.sPtr[sub.length] = '\0'; // завершение нового String 

	cout<<"Попытка вывести суб  "<<sub<<"\n";
return sub; // возвращение копии sub объекта String 
}
// Возвращение длины строки 
int String::getLength() const { return length;} 
	// Перегруженная операция вывода данных 
ostream &operator<< (ostream &output, const String &s)
{
	output << s.sPtr; 
	return output; // возможность сцепления 
}


int _tmain(int argc, _TCHAR* argv[])

{
	setlocale(LC_ALL, "Russian");
	String s1("поздравляем" ), s2( " с днем рождения" ), s3; 

	cout<<"---------------------------"<<"\n";
	// проверка перегруженной функции вызова операции ()// для подстроки 
	cout << "Подстрока si начиная с " << endl; 
	cout<< "О до 27, s1 (0, 27), равна: "; 
	cout<<s1(2, 5); cout << endl << endl; 
		cout<<"---------------------------"<<"\n";
	// проверка опции подстроки "до конца String" 
	cout << "Подстрока si начиная с" << endl 
	<< "28, si (28, 0), равна: " 
	<< s1(28, 0) << endl <<endl; // 0 означает до конца строки 
	
	return 0; 	 
}

[BOBAH13]

Хм... может не в этом проблема, но объявлять сначала это

#include <string.h>

, а потом писать это

using namespace std;	
class String {

Ой как не красиво, и бог его знает что будет. Вообщем измените название вашего класса.

[atenon]

Ой как не красиво, и бог его знает что будет. Вообщем измените название вашего класса.

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

[Carbon]

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

Ой как не красиво, и бог его знает что будет. Вообщем измените название вашего класса.

Да, некрасиво, но работе совершенно не мешает.

начинается какаято фигня с вызовом перегружаемого конструктора

Какая фигня? Можно описание фигни, а то я по понедельникам плохой телепат.

[atenon]

Какая фигня? Можно описание фигни, а то я по понедельникам плохой телепат.

После выполнения строки

cout<<"Попытка вывести суб  "<<sub<<"\n";

Происходит вызов вот этого конструктора

// Конструктор копии 
String::String(const String &copy) 
{ 
	cout << "Конструктор копии: " << copy.sPtr << endl; 
	length = copy.length; // копирование длины 
	sPtr = new char[length +1]; // выделение памяти 
	assert(sPtr != 0); // завершение, если память не выделена 
	strcpy(sPtr, copy.sPtr); // копирование строки 
}

и сразу вызов деструктора.
После этого программа зависает.

[Stilet]

и сразу вызов деструктора.

assert is a macro that expands to an if statement; if test evaluates to zero, the assert macro calls the _assert function

void _RTLENTRY _EXPFUNC _assert(char * __cond, char * __file, int __line);

and aborts the program.

Вот видимо он и срабатывает у тебя

[atenon]

Мой английский хромает на все четыре ноги, поэтому я тупо удалил строку

	assert(index >= 0 && index < length && subLength >= 0);

Результата нет.. Не могу понять почему вызывается конструктор и деструктор???

[netrino]

Если срабатывает ассерт, то его не удалять надо, а смотреть где неправильные параметры переданы )

	// удаление символа из объекта 
	delete sub.sPtr; //  (1)
	// выделение памяти для подстроки 
	sub.sPtr = new char[sub.length]; // (2)

	// копирование подстроки в новый String 
	strncpy(sub.sPtr, &sPtr[index], sub.length); 
	sub.sPtr[sub.length] = '\0'; // (3)

В (1) нужно delete[]
В (2), в следствии (3), наверное нужно length+1.

	cout << "Подстрока si начиная с" << endl 
	<< "28, si (28, 0), равна: " 
	<< s1(28, 0) << endl <<endl; // 0 означает до конца строк

По-видимому это и есть причина срабатывания ассерта. Вы уверены, что в строке будет 28+ символов? )

[atenon]

Работает!! (из-за такой мелочи столько просидел). Всем спасибо за помощь.

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

[netrino]

String String::operator() (int index, int subLength)

Возврат происходит по значению, поэтому перед выходом из ф-ции происходит копирование объекта в "результат ф-ции"(потому вызывается конструктор копии), затем происходит разрушение локального объекта sub(вызывается деструктор)