Question

我试图在不使用字符串库的情况下创建自己的字符串类。 #include <string>

我被允许使用c风格的字符串！

我必须制作iString ctor，复制ctor，dtor并重载运算符＆gt;＆gt;，＆lt;＆lt;，+，*

到目前为止，我的问题是，当我尝试编译我的作品时，我的每个operator+和operator*都会收到此错误...

warning: reference to local variable s3 returned [enabled by default]

s3只是iString对象名称。

这是我到目前为止所做的：

.h文件

#ifndef __ISTRING_H__
#define __ISTRING_H__

struct iString{
  char * chars;
  unsigned int length;
  unsigned int capacity;
  iString();  // ctor
  iString(const char *); // copy ctor
  iString(const iString&); // copy ctor
  ~iString(); // dtor
};

std::istream &operator>>(std::istream &in, const iString &s);
std::ostream &operator<<(std::ostream &out, const iString &s);
iString &operator+(const iString &s1, const iString &s2);
iString &operator+(const iString &s1, const char *c_str);
iString &operator*(const iString &s, const int k);
iString &operator*(const int k, const iString &s);


#endif

.cc文件

#include <iostream>
#include "istring.h"
#include <cstring>

using namespace std;

iString::iString(){  // this is my ctor
  chars = new char[1];
  length = 0;
  capacity = length + 1;
  chars[0] = '\0';
}

iString::iString(const char *word){ // this is my copy ctor from a c-string
  length = strlen(word);
  capacity = length + 1;
  chars = new char[capacity];
  strcpy(chars, word);
}

iString::iString(const iString &s){ // this is my copy ctor from an iString
  length = strlen(s.length);
  capacity = length + 1;
  chars = new char[capacity];
  strcpy(chars, s.chars);
}


iString::~iString() { //dtor
  delete [] this->chars;
}

istream &operator>>(istream &in, const iString &s){  // not done, don't worry

}

ostream &operator<<(ostream &out, const iString &s){
  //out<<s.chars;
  return out<<s.chars;
}

iString &operator+(const iString &s1, const iString &s2){ 
  iString s3;

  char *new_str;// = new char[size];
  new_str = strcat(s1.chars, s2.chars);

  s3 = iString(new_str);
  return s3;
}

// tried a different approach 
iString &operator+(const iString &s1, const char *c_str){ 

  int size = s1.length + strlen(c_str) + 1;
  char *new_str = new char[size];

  strcpy(new_str, s1.chars);
  strcat(new_str, c_str); 

  iString s3(new_str);
  delete [] new_str;
  return s3;
}

iString &operator*(const iString &s, const int k){

  int size = (s.length * k) + 1;
  char *c_str = new char[size];
  for(int i = 0; i < k; i++){
    strcat(c_str, s.chars);
  }

  iString t(c_str);
  delete[] c_str;
  return t;
}

iString &operator*(const int k, const iString &s){
  return s * k;
}

Answer 1

您不应该从operator+( , )

返回引用

iString operator+(const iString &s1, const iString &s2){
 ...
}

Answer 2

问题是s3在返回后会被破坏，所以当调用者最终得到字符串时，它将是对已被破坏的字符串的引用，因此无效。

I.E：您正在返回对函数本地字符串的引用 - 当您到达调用者时，它将不再存在。

您需要的是按价值返回。这样，字符串在返回时仍然有效。使用C ++ 11时，由于rvalue引用很长，因此您可以提供移动构造和赋值。此外，在C ++ 98中，您可以指望许多编译器在许多情况下应用返回值优化来提高效率。

Answer 3

警告表示您正在返回对作为局部变量的对象的引用。这是一个问题，因为当函数返回时会破坏局部变量。

std::string的{{1}}返回一个新的字符串对象，而不是对字符串对象的引用;你可能也应该这样做。替代方法是创建一个新对象并返回一个指针，或者修改传递给操作符的一个字符串，这两个字符看起来都不是很好。

创建没有字符串库的字符串类

3 个答案: