这个代码有什么问题？

Question

#include <iostream>
#include<string>
using namespace std;

class String
{
public:
    String(const char *str=NULL);
    String(const String &r);
    ~String()
    {
        cout<<"xigou"<<mydata<<endl;
    }
    int getSize();
private:
    char *mydata;
    int size;
};
String::String(const char *str)
{
    cout<<"gouzao "<<str<<endl;
    size=sizeof(str);
}
String::String(const String &r)
{
    size=r.size;
    mydata=new char[size];
    for(int i=0;i<size;i++)
    {
        mydata[i]=r.mydata[i];
    }
    cout<<"kaobei gouzao"<<mydata<<endl;
}
int main()
{
    String s1,s2("hello");
    String s3(s2);
    return 0;
}

Answer 1

你没有搞清楚sizeof（）的用法；第22行，--------------size=sizeof(str);----这里的str是指针，并不是数组，size会等于4，导致你第二个构造函数new的空间跟预想产生出入，建议用string的str.size()或者strlen（）都可以，还有定义构造函数时私有成员没有全部初始化，很危险；再有就是new了之后一定要delete，在析构函数里；暂时就这些了，按照这个思路改应该差不多