C++. Тема "Классы"

[S1av0k]

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

#include <iostream>
using namespace std;
#include "string.h"

class String
{
	private:
		char *mass;
	public:
		String();
		String(char[]);
		String(const String&);
		~String(){delete []mass;}								//деструктор
		friend ostream& operator << (ostream&,String&);
		friend istream& operator >> (istream&,const String&);
		String& operator += (const char a);	
		String& operator = (const String&);
		int operator () (const char[]);
};

String::String()
	{	
		mass = NULL;
	}

String::String(char p[])
	{
		int length = strlen(p);
		int i = 0;
		mass = new char [length];
		while(p[i])
			{
				mass[i] = p[i];	
				++i;
			}
			
		mass[length] = '\0';
	}	
	
String::String(const String& qw)									//копирующий конструктор
	{
		if (this != &qw)
			{
				int length = strlen(qw.mass);
				mass = new char[length];
				strcpy(qw.mass,this->mass);
			}	
	}	
	
ostream& operator << (ostream &s,String &qw)
	{
		return s << qw.mass;
		cout << "qwer" << endl;
	}	
	
istream& operator >> (istream& s,const String& qw)	
	{
		return s >> qw.mass;	
	}
	
String& String::operator += (const char a)
	{
		int length = strlen(mass);
		mass = new char [length+2];	
		strcpy(this->mass,mass);
		mass[length+2] = '\0';
		return *this;
	}	
	
String& String::operator = (const String& qw)	
	{
		int length = strlen(qw.mass);
		mass = new char [length + 1];	
		strcpy(qw.mass,mass);
		return *this;
	}
	
int String::operator () (const char p[])
	{
		int length = strlen(mass);	
		int k, i = 0;
		const char* pptr = p;
		if (mass[i] && *pptr)
			{
				if (*pptr == mass[i])
					return i;
				else		
					{
						++i;
						++pptr;
					}	
			}
	}	
		
int main (int argc, char *argv[])
{
	String str1;
	String str2("abc");
	
	cout << str1 << endl;
	cout << str2 << endl;
	
	str1 = str2;
	cout << str1 << endl;
	
	str1 += 'D';
	cout << str1 << endl;
	
	cout << "Hello world!" << endl;
	
	return 0;
}

[Obey-Kun]

В глаза бросилась дурацкая опечатка в ostream& operator << (ostream &s,String &qw). Сначала ретарн, потом дебаговое действие. В потоках не разбираюсь, так что по теме не помогу.

[pacniwassano]

friend ostream& operator << (ostream &stream, MyClass &obj)
{
        stream << obj.value ...;
        ...
        return stream;
}

[S1av0k]

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

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

ostream& operator << (ostream &s,String &qw)
	{
		if (qw.mass)
			s << qw.mass;
		else
			s << "Cтрока пустая.";	
		return s;
	}

Допустимо ли это?

И ещё остается вопрос по поводу оператора ввода...так и не придумал что сделать, чтобы он заработал...прошу помощи!

Исправил копирующий конструктор:

String::String(const String& qw)									//копирующий конструктор
	{
		if (this != &qw)
			{
				int length = strlen(qw.mass);
				mass = new char[length];
				strcpy(mass,qw.mass);
			}	
	}

[S1av0k]

Всё исправил, всё работает
Прошу помощи только с оператором ввода!

[Obey-Kun]

Твой код чреват утечками памяти, не забывай про delete.

[S1av0k]

Сообщение от Obey-Kun

Твой код чреват утечками памяти, не забывай про delete.

Например, где?...

В конечном варианте программы всё учтено, память не теряется
Помогите только с перегрузкой оператора ввода в поток!

[Obey-Kun]

Да хоть в вышеприведённом тобой

String::String(const String& qw)									//копирующий конструктор
	{
		if (this != &qw)
			{
				int length = strlen(qw.mass);
				mass = new char[length];
				strcpy(mass,qw.mass);
			}	
	}

Перед заменой значения mass надо сделать delete mass. Иначе это утечка памяти.

[S1av0k]

Уже учел
Дело только за перегрузкой оператора ввода...
Кстати, Obey-Kun, а как Вы пишете собственные классы, если не умеете перегружать операторы ввода/вывода?

[An1ka]

И если

переменная = new тип[число];

То освобождать надо с помощью delete[]

delete[] переменная;

Если delete без скобочек, то освобождается только первый элемент массива... А остальные остаются ! Такая память называется мусором)))