ベクターにpush_back構造体を挿入

Question

//prototype  
void Split(char c, vector <MyString> &outputVector) const   

//partial code inside split function 
// create new MyString object to push into output vector 
MyString substr; 
substr.mString = newString; 
substr.mLength = size; 

// push new item 
outputVector.push_back(substr); 

outputVector.push_back()行をステップ実行すると、mStringデータメンバーは保持されません。ベクターへ

//I have two constructors 
MyString() 
{ 
    mString = NULL; 
    mLength = 0; 
} 

/************************************************* 
* MyList copy constructor 
* creates a deep copy of a MyString item 
************************************************/ 
MyString(const MyString &copy) 
{ 
    mString = new char[copy.mLength]; 
    int i; 

    for(; i < copy.mLength; i++) 
    { mString[i] = copy.mString[i]; } 

    mString[i] = '\0'; 
    mLength = copy.mLength; 

} 

Answer 1

にあなたは私たちは何が起こっていることができますが、最も可能性の高いiれるものi見当がつかないここundefined behavior

int i; 

for(; i < copy.mLength; i++) 

で初期化されていない変数を使用していますcopy.mLengthより大きいので、forループを入力することはありません。正しい動作を得るためにあなたは私たちがそのラインi == copy.mLengthに達するが、配列が唯一の私たちはあるのでcopy.mLengthの大きさを持っている時間によって

mString[i] = '\0'; 

と別の問題が発生している

int i = 0; 

のように0にiを設定配列は0のインデックスに基づいています。ほとんどの場合、割り当てをヌルターミネータのスペースに変更する必要があります。

mString = new char[copy.mLength + 1]; 

Answer 2

http://www.cplusplus.com/reference/vector/vector/push_back/

一backコピー値。 MyStringクラスはmStringメンバをコピーするコピーコンストラクタを正しく定義していますか？私はこれがあなたの問題かもしれないと思います。

Answer 3

コピーコンストラクタの正しいバージョン

MyString(const MyString &copy) 
{ 
    mString = new char[copy.mLength + 1]; 
    int i = 0; 

    for(; i < copy.mLength; i++) 
    { mString[i] = copy.mString[i]; } 

    mString[i] = '\0'; 
    mLength = copy.mLength; 

} 

Answer 4

私は、あなたが 1.for行っている2つのミスがあると思います（; I < copy.mLength;私は++）{ mString [I] = copy.mString [i]は、 } ループがどこで始まるか言及する必要があります。 2.mString =新しいchar [copy.mLength + 1]; mString [i] = '\ 0'; 私はあなたが答えを持っていると思う：