zwracanie obiektu w przeciążonych operatorach +

Question

Witam.
Czemu moje operatory + nie chcą zwracać obiektu utworzonego w ich ciele ? Debugger przechodzi przez wszystkie linie tych metod a program sypie się przy próbie zwrócenia obiektu.

Stringw  Stringw::operator+(const Stringw &st)
{
	 Stringw sum;
	sum.str = new char[len + st.len +1];
	strcpy(sum.str, str);
	strcat(sum.str, st.str);
	sum.len = len + st.len;
	return sum;
}

//ta funkcja jest zaprzyjaźniona z klasą
Stringw operator+( const char*s, const Stringw &st)
{
	 Stringw sum;
	sum.str = new char[strlen(s) + st.len + 1];
	strcpy(sum.str, s);
	strcat(sum.str, st.str);
	sum.len = strlen(s) + st.len;
	return sum;
}

Stringw Stringw::operator+(const char* s)
{
	 Stringw sum;
	sum.str = new char[len + strlen(s) + 1];
	sum.len = len + strlen(s);
	strcpy(sum.str, str);
	strcat(sum.str,s);
	return sum;

}

 

Comments

Co masz na myśli sypie się wywala błąd? Jaki komunikat dostajesz?

---

Dokładnie coś takiego

Unhandled exception at 0x000007FED5BDFD0E (ucrtbased.dll) in ConsoleApplication1.exe: 0xC0000005: Access violation reading location 0xFFFFFFFFFFFFFFFF.

---

Pokaż wszystkie metody klasy.

---

Stringw::Stringw()
{
	str = new char[1];
	str[0] = '\0';
	len = 0;
}

Stringw::Stringw(const char *s)
{
	delete[] str;
	len = strlen(s);
	str = new char[len+1];
	strcpy(str, s);
}

Stringw::Stringw(const Stringw &s)
{	
	delete[] str;
	len = s.len;
	str = new char[len + 1];
	strcpy(str, s.str);
}

Stringw::~Stringw()
{
	delete[] str;
}

Stringw & Stringw::operator=(const Stringw &s)
{
	if (this == &s)
		return *this;
	delete[] str;
	len = s.len;
	str = new char[len + 1];
	strcpy(str, s.str);
	return *this;
}

Stringw & Stringw::operator=(const char * s)
{
	if (this->str == s)
		return *this;
	delete[] str;
	len = strlen(s);
	str = new char[len + 1];
	strcpy(str, s);
	return *this;
}
Stringw  Stringw::operator+(const Stringw &st)
{
	 Stringw sum;
	 
	sum.str = new char[len + st.len +1];
	strcpy(sum.str, str);
	strcat(sum.str, st.str);
	sum.len = len + st.len;
	return sum;
}


void Stringw::stringlow()
{
	for (int i = 0;i < len;i++)
	{
		str[i]=tolower(str[i]);
	}
}

void Stringw::stringup()
{
	for (int i = 0;i < len;i++)
	{
		str[i]=toupper(str[i]);
	}
}

int Stringw::count(char s)
{
	int x = 0;
	for (int i = 0;i < len;i++)
	{
		if (str[i] == s)
			x++;
	}
	return x;
}

const char & Stringw::operator[](int i) const
{
	return str[i];
}

char & Stringw::operator[](int i)
{
	return str[i];
}

bool operator<(const Stringw &st, const Stringw &st2)
{
	return(std::strcmp(st.str, st2.str) < 0);
}

bool operator>(const Stringw &st, const Stringw &st2)
{
	return st2 < st;
}

bool operator==(const Stringw &st, const Stringw &st2)
{
	return (std::strcmp(st.str, st2.str) == 0);
}

ostream & operator<<(ostream &os, const Stringw &st)
{
	os << st.str;
	return os;
}

istream & operator >> (istream & is, Stringw & st)
{
	char temp[Stringw::CINLIM];
	is.get(temp,Stringw::CINLIM);

	if(is)
		st = temp;
	while (is && (is.get() != '\n'))
		continue;
	return is;
}

Stringw Stringw::operator+(const char* s)
{
	 Stringw sum;
	 
	sum.str = new char[len + strlen(s) + 1];
	sum.len = len + strlen(s);
	strcpy(sum.str, str);
	strcat(sum.str,s);
	return sum;

}
Stringw operator+( const char*s, const Stringw &st)
{
	 Stringw sum;
	
	sum.str = new char[strlen(s) + st.len + 1];
	strcpy(sum.str, s);
	strcat(sum.str, st.str);
	sum.len = strlen(s) + st.len;
	return sum;
}

 

---

Przede wszystkim to w konstruktorach masz nadmiarowe delete[] na wskaźniku, który nie ma żadnej konkretnej wartości. (debugger powinien na to wskazać :/ )

---

Wcześniej tego nie było, poprostu dużo kombinowałem sam i próbowałem różnych sposobów, bez tych delete jest dokładnie taki sam komunikat :/

---

A nie, przepraszam, po usunięciu delete działa, być może problem był jak nie dodawałem len w operatorach ponieważ przedtem nie dodawałem ich do siebie w przeciązeniu, dziękuje za pomoc :)

Answer 1

Dodaj konstruktor kopiujący i operator przypisania, żeby było do pary...

Comments

Fakt, jeżeli używane są operatory i dynamiczna alokacja, to byłoby dobrze od razu dodać konstruktor kopiujący i operator przypisania, bo inaczej można się dorobić wielu kłopotów.

Answer 2

Na pierwszy rzut oka to mi brakuje delete [] str przed alokacją pamięci dla nowego obiektu.

Answer 3

Wydaje mi się że zapomniałeś o gwiazdzce.