实现 C++字符串运算符

Question

我有一个关于 char*、字符串相加的问题 像这样：

enter code here
s2 = s3 + "," + s1; 

我下面有三个运算符

friend Mystring operator+( const Mystring &lhs, const Mystring &rhs);  -- 1
friend Mystring operator+( const Mystring &mystr, const char *ch ); -- 2
friend Mystring operator+( const char *ch, const Mystring &mystr ); -- 3

，但我使用 1 和 3 它会崩溃，但我使用 1 和 3 可以做得很好。

我的问题是顺序不是 s3 + "," 首先，所以首先使用运算符 w 结果使用运算符3，但事实并非如我所想。

谁能解释为什么会发生这种情况？

Mystring operator+( const Mystring &mystr,const char *ch )
{
  Mystring tmp;
  tmp.str_ = new char[ strlen(mystr.str_)+2 ];
  strcpy( tmp.str_, mystr.str_ );
  strcat( tmp.str_, ch );
  return tmp;
}


Mystring operator+( const char *ch, const Mystring &mystr )
{
  Mystring tmp;
  tmp.str_ = new char[ strlen(mystr.str_)+strlen(mystr.str_)+1 ];
  strcpy( tmp.str_, mystr.str_ );
  strcat( tmp.str_, mystr.str_ );
  return tmp;
}

Mystring operator+( const Mystring &lhs, const Mystring &rhs )
{
  Mystring tmp;
  tmp.str_ = new char[ strlen(lhs.str_)+strlen(rhs.str_)+1 ];
  strcpy( tmp.str_, lhs.str_ );
  strcat( tmp.str_, rhs.str_ );
  return tmp;
}

Answer 1

首先尝试测试更简单的事情：

s2 = s3 + ",";
s2 = "," + s3;
s3 = s1 + s2;

然后再进行链接：

s2 = s3 + ","  + s1;

这样您就可以更清楚地判断问题所在。

Answer 2

在

Mystring operator+( const Mystring &mystr,const char *ch )
{
  Mystring tmp;
  tmp.str_ = new char[ strlen(mystr.str_)+2 ];

你应该写：

  tmp.str_ = new char[ strlen(mystr.str_) + strlen(ch) + 1 ];

在这里：

Mystring operator+( const char *ch, const Mystring &mystr )
{
  Mystring tmp;
  tmp.str_ = new char[ strlen(mystr.str_)+strlen(mystr.str_)+1 ];

你应该写：

  tmp.str_ = new char [ strlen(ch) + strlen(mystr.str_) + 1 ];

Answer 3

你读过这篇文章吗？ 什么是三法则？

基于您手动管理内存的方式这些功能，我认为可以肯定地假设您没有。不管怎样，一旦你正确地实施了三巨头，这就是你应该做的。首先你应该有两个完整的成员。一种用于存储字符串的大小，另一种用于存储动态数组的容量。然后，您需要一个私有函数，我们将其称为 increase_capacity，用于在必要时增加容量。所有的内存分配都将在一个地方进行。它将大大简化事情。它可能看起来像这样：

void increase_capacity(int cap)
{
    if (cap <= capacity_) return;
    char * temp = new char[cap];
    capacity_ = cap;
    memcpy(temp, str_, size_); // Or size_ + 1 if your string is null terminated.
                               // It doesn't have to be.
    delete [] str_;
    str_ = temp;
}

现在，您还应该有一个调整大小功能，可以在必要时调整大小。

void resize(int size)
{
    if (size > capacity_)
    {
        int cap = capacity_ * 2;
        cap = cap > size ? cap : size;
        increase_capacity(cap);
    }

    size_ = size;
    // Fill in new elements with some default value, or don't.
}

比上面所有的更好，您应该只使用 vector，但也许您正在尝试了解手动内存管理，这很好。

现在，您应该实现operator+=，作为成员：

Mystring & operator+=(const Mystring & rhs)
{
    int old_size = size_;
    resize(size_ + rhs.size_);
    memcpy(str_ + old_size, rhs.str_, rhs.size_);
    return *this;
}

如果您愿意，您还可以实现一个采用const char * 的运算符，这将节省依赖隐式转换时可能发生的分配。

最后，您可以实现您的 operator+：

Mystring operator+(Mystring lhs, const Mystring & rhs)
{
    return lhs += rhs;
}

如果您有一个来自 const char * 的隐式转换，那么它应该涵盖所有这些。但是，如果您编写了采用 const char * 的额外 operator+= 来减少分配次数，那么您可能还应该编写采用 const char * 的操作符+= 在它的右侧。看起来和上面的一样，只是第二个参数的类型改变了。您不需要为反向操作编写一个，因为无论如何 lhs 都需要分配为 Mystring。