Cant concatenate two strings that are the result of two functions.

[roxorsoxor2345]

I have this custom string class that I have created.

#include <cstdlib>
#include <iostream>

using namespace std;

class Cstring
{
private:
//array that will hold the strings
char array[50];
public:
//default constructor
Cstring();
//constructor that turns a single character into a string
Cstring(char stuff);
//constructor that creates a copy of a string
Cstring(const Cstring& other);

//function that returns the first character of the string
char head();
//function that returns the string with the first character removed
Cstring tail();

//////////////////////////////////
//overloaded operators for strings
//////////////////////////////////

//this operator adds a single character to the end of a string
friend Cstring operator+ (Cstring& left, const char right);
//this operator stores a copy of a string in a result string
Cstring Cstring::operat or= (const Cstring& right);
//these are my input/output operators
friend istream& operator>(istre am& input, Cstring& right);
friend ostream& operator<< (ostream& output, Cstring right);
};

/////////////////////////////////////
//Constructors
////////////////////////////////////

//default constructor
Cstring::Cstrin g()
{
sprintf(array," ");
}

//constructor that makes character a string
Cstring::Cstrin g(char stuff)
{
sprintf(array, "%c", stuff);
}

//constructor that copies string to result string
Cstring::Cstrin g(const Cstring &other)
{
sprintf(array, "%s", other.array);
}

/////////////////////////////////////
//Functions
/////////////////////////////////////

//this is the head function that returns first character of string
char Cstring::head()
{
return array[0];
}

//this is the tail function that returns the string without the first
character
Cstring Cstring::tail()
{
Cstring temp;
for(int i=1; i<50; i++)
{
temp.array[i-1] = array[i];
}
return temp;
}

///////////////////////////////////
//Operator Overloads
///////////////////////////////////

Cstring operator+ (Cstring& left, const char right)
{
Cstring temp;
for(int i=0; i<50; i++)
temp.array[i] = left.array[i];

for(int j=0; j<strlen(temp.a rray)+1; j++)
{
if(temp.array[j]=='\0')
{
temp.array[j] = right;
temp.array[j+1] = '\0';
break;
}
}
cout << temp << endl;
return temp;
}

ostream& operator<<(ostr eam& output, Cstring right)
{
Cstring temp;
int length = strlen(right.ar ray);
for(int i=0; i<length; i++)
{
output << right.array[i];
}
return output;
}

istream& operator>>(istr eam& input, Cstring& right)
{
input.getline(r ight.array,50);
return input;
}
Cstring Cstring::operat or=(const Cstring& right)
{
int i=0;
Cstring temp;
temp.array[i] = right.array[i];
return temp;
}
And I have this Program, in which I am merely trying to output the
results of string.fail() + string.head()

#include <cstdlib>
#include <iostream>
#include "cstringclass.h "

using namespace std;

int main(int argc, char* argv[])
{
Cstring reverse(Cstring & string);
Cstring string1, string2;

cout << "Enter a string" << endl;
cin >string1;

cout << string1 << endl;
cout << "head: " << string1.head() << endl;
cout << "tail: " << string1.tail() << endl;

cout << "tail,head: " << string1.tail() + string1.head() << endl;
system("pause") ;
return 0;

}

when i do this though i get a bunch of compiler errors griping about my
parameter, and that basically I can't add them together like that.

Any help would be appreciated.

[roxorsoxor2345]

Can I somehow cast the functions?
cout << "tail,head: " << Cstring(string1 .tail()) + char(string1.he ad())
<< endl;

but that doesnt work, how would I do this?

[Peter]

roxorsoxor2345 wrote:

I have this custom string class that I have created.

since stl exists there is no need anymore for a custom string class.

[Thomas J. Gritzan]

roxorsoxor2345 schrieb:

I have this custom string class that I have created.

[...]

when i do this though i get a bunch of compiler errors griping about my
parameter, and that basically I can't add them together like that.

Read the FAQ:
http://www.parashift.com/c++-faq-lite/

Especially here:
http://www.parashift.com/c++-faq-lit...rrectness.html
and here:
http://www.parashift.com/c++-faq-lit...t.html#faq-5.8

I also recommend to use std::string instead.

--
Thomas
http://www.netmeister.org/news/learn2quote.html

[Nate Barney]

Peter wrote:

roxorsoxor2345 wrote:

>I have this custom string class that I have created.

since stl exists there is no need anymore for a custom string class.

Not for real programs, but it could be an interesting exercise for
someone who has never done it.

Nate