我在代码的特定部分遇到异常

Question

我在编译重载运算符时遇到指针异常。异常显示“抛出异常：写访问冲突。 TempString.Pointer 是 0x1110112。”我希望操作员将两个自定义字符串合二为一。

重载运算符：

MyString MyString::operator+(MyString &String) {
    MyString TempString;
    TempString.StringLength = this->StringLength + String.StringLength;
for (int i = 0; i < this->StringLength + String.StringLength - 1; i++) {
    if (i < this->StringLength) {
        TempString.Pointer[i] = this->Pointer[i]; // Exception pops up here
    }
    else {
        TempString.Pointer[i] = String.Pointer[i];
    }
}
return TempString;
};

构造函数：

MyString::MyString() {
    StringLength = 0;
    Pointer = nullptr;
}

MyString::MyString(char* String) {
    Pointer = String;
    for (int i = 0; Pointer[i]; i++) {
        StringLength++;
    }
}

Main.cpp：

#include "MyString.h"
#include <iostream>

int main() {

    char String1[] = "abcdef";
    MyString NewString(String1);

    char String2[] = "cdefa";
    MyString SubString(String2);

    MyString BrandNewString(NewString + SubString);
    cout << BrandNewString.GetStringLength() << endl;
    system("pause");
    return 0;
}

Answer 1

行：

TempString.StringLength = this->StringLength + String.StringLength;

不为您的字符串分配内存。 您需要在写入之前分配内存（在TempString.Pointer[i] = this->Pointer[i] 行）。

你能展示你的构造函数内容吗？

你写的是一个只读的字符串类。它附加到外部字符数组。

这很不幸，但你不能改变它们。请注意，您可以修改外部 char 数组的内容，但不能修改大小。 要进行连接（如在abc+def 中），您需要创建一个与两个初始数组的长度之和一样大的数组，但您的代码中没有这样的数组。

所以你需要先创建它们（它被称为分配，通常使用operator new，也可以使用malloc），但是你的类将无法知道它指向的数组是否由本身或来自其他任何地方。因此，您的班级将无法释放数组。

因此，您应该选择您希望班级执行的场景：

1. 保持只读类（您将无法执行串联或任何改变数组大小的操作）。
2. 成为一个真正的字符串类（您必须在构造函数中分配一个字符数组并在析构函数中释放它），然后将接收到的字符串复制到您自己的托管数组中。虽然您将编写这样一个类，但您应该看看 std::string 类，它现在正在这样做。

你应该有什么，逻辑上在一个连接运算符中：

MyString MyString::operator+(const MyString & o) const {
   MyString ret(length + o.length);
   for(int i = 0; i < length; i++)
       ret.pointer[i] = pointer[i];
   for(int i = 0; i < o.length; i++)
       ret.pointer[i+length] = o.pointer[i];
   return ret;
}

MyString::MyString(int length) 
: length(length), pointer(new char[length+1]) 
{
  pointer[length] = 0;
}

MyString::~MyString() { delete[] pointer; pointer = nullptr; length = 0; }

MyString::MyString(const char * str)
: length(strlen(str)), pointer(new char[length+1])
{
    memcpy(pointer, str, length+1);
}