Thread: Dynamic Memory Allocation

operator+() is clearly the problem since you are incurring in a buffer overflow both on strcpy() and strcat() calls. You should read more about these two functions. It's important you use them correctly since they are some of the biggest contenders for bugs both in C and C++. strncpy and strncat are somewhat safer, but not too much really...

Anyways,

My first advise to you is to immediately define a length() member function that returns the size of the str data member array. I believe it should return the size of the array minus the null element. In order to do this, a quick way is to simply iteratate through str until you find the null terminator.

This member function will be your life savior. You will be using it everywhere. It tells you the size of String without the null character. Defining operator+() with strncpy() and strncat() will then become much, much easier.

The second thing you must make sure is that your parameterized constructor should check for the existence of a null terminator and add one to the end of str in case it doesn't exist. As is right now, your String(char s[]) constructor can construct the str data member without a null terminator. That would foil your length() function member... and the whole of your class.

On a second note, I'm very limited on the type of help I can provide here. I believe the above are right choices that will help you tremendously to simplify the rest of your class construction. But wrapping a c style string is no trivial task... Your teacher was not kind at all. To that I add the fact, I'm not really that comfortable with c strings anyways.

One of the problems you will face is exactly with my second advise. How do you really check for the presence of a null terminator? The problem here is that the char array is passed without a size. You simply don't know the size of the array, so you could end up looking outside its bounds. I don't think making it String(const char* s) will be enough. Since it is still possible to pass a non null terminated char array to it.

Someone else may provide an answer to this.

EDIT: One solution is to go ahead and place '\0' at the end of the string anyways. That's kind of hackish but quiet safe. No matter what your str will always hold a null terminated string. Of course, if the string passed to the constructor already contains one, you will be wasting 8 bits of memory. A small price to pay, I guess.

Also, I was looking at your class again... you cannot expect to do this with fixed size arrays. You have to use dynamic ones. Come to think of it, that was probably the original intent of your thread. go here for an introduction http://www.cprogramming.com/tutorial/lesson9.html and check google for "c++ c-style strings" or something similar.

Since you intend to use dynamic memory allocation, I suggest that you have a char* member variable to store the string content, and two unsigned int (or std::size_t) member variables for size and capacity.

Make operator+ a free function, and implement it in terms of a member function operator+=, e.g.

Code:

String operator+(const String& a, const String& b) {
    return String(a) += b;
}