Собственный класс String в C++

[Bitter_Schokolade]

Задача такая:
Составить описание класса для работы со вектором строк (строки
произвольной длины), с операциями включения в вектор, удаления из
вектора элемента с заданным значением, удаления всего вектора или
конца вектора, начиная с заданного элемента.

Класс Vector я описала, но до него мне необходимо описать класс String, так как нельзя пользоваться соответствующими библиотеками. Помогите, пожалуйста.

Вот описание класса Vector,

class Vector
{
	int size;
	Stroka* v;
	void add (Stroka s) {Stroka* new_v = new Stroka [size+1];
	for (int i=0; i<size; ++i)
	{
		new_v[i] = v[i];
	}
	new_v[size] = s;
		delete[]v;
	v = new_v;
	++size;
	}
};

Это добавление записи в вектор. Остальные методы дописать смогу на основе этого примера (лектор объяснял)

[rrrFer]

чего вы хотите от String?

Кстати, пример ужасен, как вы собираетесь его использовать?

[Bitter_Schokolade]

Говорят, что в идеале используются библиотеки string и vector, но нам запретили этим пользоваться. Вот посмотрите, я набросала кое-что, можете ответить, что должно быть на месте "//?" ? Я перегрузила операции в Stroka, там нужно еще что-нибудь?

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

class Stroka
{
private:
	char* s;
public: Stroka()
		{
			s = new char [50];
			*s = '\0';
		}
		bool operator==(Stroka n)
        {
			return (!strcmp(n.s,s)) ? true : false;
		}
		bool operator!=(Stroka n)
        {
			return (strcmp(n.s,s)) ? true : false;
		}
		~ Stroka ()
		{
			delete [] s;
		}
};

class Vector: Stroka
{
	int size;
	Stroka* v;
	char* s;
public: Vector ()
		{
			s = new char [50];
			*s = '\0';
		}
	void add (Stroka s) {Stroka* new_v = new Stroka [size+1];
	for (int i=0; i<size; ++i)
	{
		new_v[i] = v[i];
	}
	new_v[size] = s;
		delete[]v;
	v = new_v;
	++size;
	}
	void sub (Stroka s) {Stroka* new_v = new Stroka [size-1];
	for (int i = 0; i<size; --i)
	{
		//? дальше верно?
		if (s == v[i])
		{
			new_v [size-1] = s;
		}
		delete[]v;
		v = new_v;
		--size;
		}
	}
	void delfrom (Stroka s) {Stroka* new_v = new Stroka [size-1];
	for (int i = 0; i<size; --i)
	{
		//?
	}    
	}
	void delall (Stroka s) {Stroka* new_v = new Stroka [size-1];
	//?
	}
};

int _tmain(int argc, _TCHAR* argv[])
{
	system ("pause");
	return 0;
}

[rrrFer]

оператор присваивания, наверное.
вот тут вызываеца: new_v [size-1] = s;
вам же компилятор поди сообщает чего не хватает?

[Bitter_Schokolade]

Компилятор при таком коде не ругается:

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

class Stroka
{
private:
	char* s;
public: Stroka()
		{
			s = new char [50];
			*s = '\0';
		}
		bool operator==(Stroka n)
        {
			return (!strcmp(n.s,s)) ? true : false;
		}
		bool operator!=(Stroka n)
        {
			return (strcmp(n.s,s)) ? true : false;
		}
		~ Stroka ()
		{
			delete [] s;
		}
};

class Vector
{
	int size;
	Stroka* v;
public: Vector ()
		{
			//?
		}
	void add (Stroka s) {Stroka* new_v = new Stroka [size+1];
	for (int i=0; i<size; ++i)
	{
		new_v[i] = v[i];
	}
	new_v[size] = s;
		delete[]v;
	v = new_v;
	++size;
	}
	void del (Stroka s) {Stroka* new_v = new Stroka [size-1];
	for (int i = 0; i<size; --i)
	{
		//? дальше верно?
		if (s == v[i])
		{
			new_v [size-1] = s;
		}
		delete[]v;
		v = new_v;
		--size;
		}
	}
	void delfrom (Stroka s) {Stroka* new_v = new Stroka [size-1];
	for (int i = 0; i<size; --i)
	{
		//?
	}    
	}
	void delall (Stroka s) {Stroka* new_v = new Stroka [size-1];
	//?
	}
};

int _tmain(int argc, _TCHAR* argv[])
{
	
	system ("pause");
	return 0;
}

[rrrFer]

Компилятор при таком коде не ругается:

если при этом все работает как задумано - то проблем нет, верно? )
попробуйте использовать свою строку и вектор - если будет работать - радуйтесь, компилятор сообщит об ошибках - тоже, ведь нашли ответ на "там нужно еще что-нибудь?"

я набросала кое-что, можете ответить, что должно быть на месте "//?"

поясните зачем вам вот эти функции:

void delfrom (Stroka s) {Stroka* new_v = new Stroka [size-1];
	for (int i = 0; i<size; --i)
	{
		//?
	}    
	}
	void delall (Stroka s) {Stroka* new_v = new Stroka [size-1];
	//?
	}

и может быть мы придумаем что должно быть на месте "//?".
Пока что мне не понятно почему в функции delall (которая, судя по названию, должна лишь все удалить выделяется память. И форматирование кода очень не нравится.

вместо вопросительных знаков в конструкторе по умолчанию:

Vector() : size_(0), str_(0) {
}

[Bitter_Schokolade]

Премного благодарна!

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

Я просто не знаю, вдруг еще что-то надо, а я еще main() не заполняла. функции вызывать надо. А у меня в классах ввод-вывод строк не заявлен нигде. Это нормально?

[Abstraction]

Компилятор при таком коде не ругается:

Ну давайте я за него поругаюсь.
1) В классе Stroka не объявлены конструктор копии и оператор присваивания. Не в меру инициативный компилятор, к сожалению, в таком случае создаст их за Вас и это будет простое побитовое копирование. В частности, конструктор копии задействуется при передаче аргумента по значению в operator==. Копия будет создана, а на выходе из функции вызовется деструктор, а в нём delete[]... короче, оригинал будет безнадёжно испорчен. Во-первых, объявите конструктор копии и оператор присваивания; во-вторых, желательно бы передавать в качестве аргумента не объект типа Stroka, а константную ссылку на него, если это возможно (const Stroka&).

2) В методе Vector :: del Вы выделяете массив new_v размера X, а потом обращаетесь к new_v[X]. Поздравляю, Вы только что уронили программу.

3) Там же Вы в цикле вызываете delete[] v. Да и сам цикл получается почти бесконечный...

4)

А у меня в классах ввод-вывод строк не заявлен нигде. Это нормально?

Зависит от определения термина "нормально". Поскольку единственная строка, которую можно создать с использованием Вашего кода - это пустая строка, то вывод, собственно, не очень и нужен...

[Bitter_Schokolade]

А можете объяснить, как это делать?
[QUOTE=Abstraction;1207078]Во-первых, объявите конструктор копии и оператор присваивания; во-вторых, желательно бы передавать в качестве аргумента не объект типа Stroka, а константную ссылку на него, если это возможно (const Stroka&).[/QUOTE=Abstraction;1207078]

И как исправить 2 и 3 ошибки?

[Abstraction]

А можете объяснить, как это делать?

public: 
Stroka() {
	s = new char [50];
	*s = '\0';
}

//Конструктор копии
Stroka(const Stroka& cpy){
  s = new char[50];
  strncpy(cpy.s, s, 49);
}

//Оператор присваивания
Stroka& operator=(const Stroka& src){
  if(this == &src) return *this; //Проверка на присваивание объекта самому себе

  strncpy(s, src.s, 49);
  return *this;
}

И как исправить 2 и 3 ошибки?

А самостоятельно? Логика Vector :: del лично мне недоступна, я даже не вполне понимаю, чего Вы хотели сделать такими действиями. "Если очередной элемент вектора равен удаляемому, скопировать его туда-то" - действие достаточно странное.