이 + = 연산자를 어떻게 작동시킬 수 있습니까?

Question

난 내 자신의 문자열 클래스를 만들고있어 모든 것이 하나이를 제외하고 잘 작동 : 나는이 할 수 있도록 operator += 연산자를 오버로드하기 위해 노력하고있어 :

string s1 = "Hello", s2 = " World"; 

s1 += s2; 

것은 그래서 내가 뭘하려 :

// In the string class, 

string& operator +=(string const& other) 
{ 
    using std::strcpy; 

    unsigned length = size() + other.size() + 1; // this->size() member 
    char *temp = new char[length];    // function and a "+ 1" for the 
               // null byte '\0' 
    strcpy(temp, buffer); 
    strcpy(temp, other.data()); 

    delete[] buffer; 

    buffer = new char[length]; 
    strcpy(buffer, temp);  // copy temp into buffer 

    return *this; 
} 

그러나 my program 위의 주 코드를 사용할 때 출력 후에 출력이 없습니다. 또한 오류가 발생하지 않습니다 (런타임 오류조차도 없음). 왜 그런가요? 어떻게 구현할 수 있습니까?

참고 : std::string을 사용할 수 있지만이 방법을 직접 배우고 싶습니다.

strcpy(temp, other.data()); 

당신은 이미 (이전 행에서) 버퍼에 첫 번째 문자열을 복사 한, 그러나 이것은 다음 은 다른 문자열의 데이터로을 덮어 :

Answer 1

문제는 여기에있다. 대신 가 strcat를 사용하여 버퍼에 다른 문자열의 데이터를 추가 할 : 제리가 지적 하듯이

이

strcat(temp, other.data()); 

, 다른 문제가 올바르게 처음에 문자열을 초기화하는하지 않을 것입니다. 당신이 등을 strcpy, strcat을 사용하려고하는 경우 여담으로

는, 당신은 정말 잠재적 인 버퍼 오버런 문제를 방지하기 위해 길이 제한 버전 (strncpy, strncat)를 사용합니다.

Answer 2

demo you posted의 주소는 the comment이며 Mac's answer입니다. 이 문제가 발생합니다 :

string(char const *str) : buffer(new char[strlen(str)]), len(strlen(str)) 
{} 

// ... 

string s1 = "Hello"; 

버퍼를 생성자에 할당하지만 데이터를 복사하지는 않습니다. std::cout << s1;을 어떻게하면됩니까?

---

편집 : 당신은 operator +=

귀하의 복사 생성자에 len를 업데이트하지 않는 두 string의 지점을하고있다

• : 그건 그렇고, 나는 적어도 두 개의 다른 문제를 발견 같은 버퍼 이것은 나쁘다, 다른 사람이 그것을 사용하려고 할 때 다른 것 delete[] 그것을 날려 버릴 것이다.