Printable View
In my class, a data member is a pointer to a character array......which is assigned to memory from the heap when the constructor is called.
One of the arguments to the constructor is a string, and I want to copy that string to the memory in the heap. Using the copy() member function of the string class does the job, but my compiler (Visual Studio 2005) complains that it is deprecated. Is there a better way to do this?
Bling bling, it's a microsoft thing:
http://msdn2.microsoft.com/fr-fr/lib...69(VS.80).aspx
http://msdn.microsoft.com/msdnmag/is...C/default.aspx
The problem with switching to checked_copy is that it is not portable. If you are correctly setting the size of your dynamic array, it should be fine. Of course the truly safe solution is to use a string or vector<char> inside the class itself instead of the character array. Is there a reason you cannot do that?
You could easily write your own copy function. Is there any reason not to write it?
I would do something like this:
Code:#include <string>
#include <iostream>
#include <cstring>
using namespace std;
class MyClass
{
char* str;
void mycopy (string&);
public:
MyClass(string s)
{
str = new char[strlen(s.c_str()) + 1];
mycopy(s);
}
void show()
{
cout << str;
}
//take care of rest (deallocation etc...)
};
void MyClass::mycopy(string& s)
{
size_t i;
for(i = 0; i < strlen(s.c_str()); i++)
{
str[i] = s[i];
}
str[i] = '\0';
}
int main()
{
string q = "Test";
MyClass t(q);
t.show();
}
Why would you write your own when good ones already exist? Your own would have to be debugged and tested, the existing ones have gone through that already.