Homemade String Class

[csnook]

I am attempting to create a driver main for a homemade string class. My program appears to run correctly up to the point it calls the overload= member function for char* and again with the friend function that should append char* to String, but then it breaks. I know each function is getting the appropriate data(I tested cout << for each member item in the function to be sure) but I am unsure how to assign/append a char* to a string based upon the definitions I wrote throughout the class. Any help/pointers for either of those two functions specifically would be greatly appreciate.

The main calls should:
S="Text";
and
"Text " + S;

#include <iostream>
#include <string>
#include <cstring>

using namespace std;

class String1 {
public:
	// Constructors
	String1();
	String1(char *);

	// Fill a character buffer argument
	void GetString(char *);

	// Concatenation Operators
	String1 operator+(String1 &);
	String1 operator+(char *);
	friend String1 operator+(char *, String1 &);

	// Assignment Operators
	// would work without this cuz we do not have
	// any pointer members but do anyway for practice
	String1 operator=(String1 &);
	String1 operator=(char *);

	// Console Stream Output
	friend ostream &operator<<(ostream &, const String1 &);
private:
	char str[80];
};

String1::String1()
{
	str[0] = '\0';
}

String1::String1(char * inputString)
{
	strcpy(str, inputString);
}

void String1::GetString(char * bufferString)
{
	strcpy(bufferString, str);
}

String1 String1::operator+(String1 & stringIn)
{
	String1 temp(this->str);
	strcat(temp.str, stringIn.str);
	return temp;
}

String1 String1::operator+(char *bufferString)//this is were it breaks
{
	String1 temp(this->str);
	//cout << bufferString; //test to see whether function was taking "Test"...
	//cout << temp; //test to see if the function gathered the pointer to the string value
	strcat(temp.str, bufferString);
	return temp;
}

String1 String1::operator=(String1 & stringIn)
{
	String1 temp(this->str);
	strcpy(stringIn.str, temp.str);
	return temp;
}

String1 String1::operator=(char *bufferString)
{
	String1 temp(this->str);
	//strcpy(temp.str, bufferString);
	return *this;
}

String1 operator+(char *bufferString, String1 &stringIn)
{
	String1 temp;
	cout << stringIn;
	cout << bufferString;
	//temp = bufferString + stringIn;
	return temp;
}

ostream & operator<<(ostream &stream, const String1 & stringIn)
{
	stream << stringIn.str;
	return stream;
}

int main()
{
	String1 S1("First ");
	String1 S2("Last ");
	String1 S3;
	String1 S4;
	String1 S5;
	String1 S6;

	cout << "\t\tHOMEMADE STRING CLASS" << endl;
	cout << "\nLets use a friend function to see whats stored in the string array(s): " << endl;
	cout << "String 1: " << S1 << endl;
	cout << "String 2: " << S2 << endl;
	
	cout << "\nTesting the function which concatenates two strings" << endl;
	cout << "String 1 ( " << S1 << ") + String 2 ( " << S2 << ") appended = ";
	cout << S1 + S2 << endl;
	
	cout << "\nTesting the function which overloads the string assignment operator=" << endl;
	cout << "For example, copy string 1 into string 3" << endl;
	S1 = S3;
	cout << "String 3 now carries the string " << S3 << endl;

	cout << "\nTesting the string plus char* operator+ overload function ( S + 'Test')" << endl;
	cout << "String 1 ( " << S1 << ") + 'Test' appended = ";
	cout << S1 + "Test" << endl;
	cout << "String 2 ( " << S2 << ") + 'Test' appended = ";
	cout << S2 + "Test" << endl;

	cout << "\nTesting the function which overloads the char* assignment operator=" << endl;
	cout << "For example, copy char 'Test' into string 4" << endl;
	S6 = "Test";
	cout << "String 4 now carries the string " << S4 << endl;

	cout << "\nTesting the friend function which again overloads the operator+ ('Test2' + S)" << endl;
	cout << "Test2 " + S1 << endl;
	cout << "Test2 " + S2 << endl;	

	system("pause");
}

[GRex2595]

*** WARNING ***
strcpy can and will cause buffer overflows if you do not have enough space where you're sending it.
http://www.cplusplus.com/reference/cstring/strcpy/

You can use strncpy to prevent buffer overflows.
http://www.cplusplus.com/reference/cstring/strncpy/

You also seem to be using the overloads incorrectly. Somebody correct me if I'm wrong, but

1. If you have two arguments, the first is on the left-hand side of the operator and the second is on the right-hand side of the operator. If you only have one argument, then the object itself is on the left-hand side, and the argument is on the right-hand side

2. There was going to be a 2, but I misread the code, sorry.

Also, you should use const type &argument for the arguments to make sure they aren't changed and to decrease function call overhead (if it's a variable, but you seem to have that covered).

Aside from that, I can't really tell just by looking. Hope this helps.