、オーバーロード代入演算子のコピーではないchar型のポインタが正しく

Question

私はこのようになりますクラスのユーザーを持っている：

class User 
{ 
    private: 
     char* p_username; 
     int nProcesses; 
     struct time 
     { 
      int mins; 
      int secs; 
     } totalTime;   
     int longestPID;  
     char* p_longestPath;  
    public: 
     User(); 
     User(const char[],int,int,int,const char[]); 
     ~User(); 
     User operator=(const User&); 
     // Other functions 
}; 

そして、オーバーロード代入演算子の機能は次のとおりです。

User User::operator=(const User &u) 
{ 
    if (this != &u) 
    { 
     delete [] p_username; 
     delete [] p_longestPath; 
     p_username = new char[strlen(u.p_username)+1]; 
     strcpy(p_username,u.p_username); 
     nProcesses = u.nProcesses; 
     totalTime.mins = u.totalTime.mins; 
     totalTime.secs = u.totalTime.secs; 
     longestPID = u.longestPID; 
     p_longestPath = new char[strlen(u.p_longestPath)+1]; 
     strcpy(p_longestPath,u.p_longestPath); 
    } 
    return *this; 
} 

使用したサンプルのメインプログラム代入演算子：

int main() 
{ 
    cout << "\n\nProgram\n\n"; 

    User u("Username",20,30,112233,"Pathname"),u2; 
    u2 = u; 
} 

代入演算子をu2 = uという行は、動的char配列以外のすべてが正しく割り当てられます。

operator = functionの最後のテスト出力は、割り当ての終わりにすべてが完全に機能していることを示しています（ユーザ名とパス名は正しい）が、割り当て直後のmain関数からのテスト出力は、突然char配列が変更されました。突然u2のユーザー名は空で、パス名の最初の半分はガベージです。

代入演算子関数の最後にユーザー名とパス名が完璧である場合、呼び出し元関数でどのように間違って戻ることができますか？

これは本当に私は困惑している

...

編集：ここでは

User::User() 
{ 
    p_username = 0; 
    nProcesses = 0; 
    totalTime.mins = 0; 
    totalTime.secs = 0; 
    longestPID = -1; 
    p_longestPath = 0; 
} 


User::User(const char UID[],int minutes,int seconds,int PID,const char path[]) 
{ 
    p_username = new char[strlen(UID)+1]; 
    strcpy(p_username,UID); 
    nProcesses = 1; 
    totalTime.mins = minutes; 
    totalTime.secs = seconds; 
    longestPID = PID; 
    p_longestPath = new char[strlen(path)+1]; 
    strcpy(p_longestPath,path); 
} 

Answer 1

あなたは割り当て関数から値で戻ってきているコンストラクタです。コピーコンストラクタに欠陥がある可能性があります。あなたがここにこのチュートリアルをチェックアウトする場合があります

Answer 2

： http://courses.cms.caltech.edu/cs11/material/cpp/donnie/cpp-ops.html

ここでは、そこからの例です：

MyClass& MyClass::operator=(const MyClass &rhs) { 

    // Only do assignment if RHS is a different object from this. 
    if (this != &rhs) { 
     ... // Deallocate, allocate new space, copy values... 
    } 

    return *this; 
    }