尝试制作我自己的字符串类

Question

这些是我的程序遇到的错误。

myString1.cpp: In constructor ‘MyString1::MyString1(char*, int)’:
myString1.cpp:6: error: expected primary-expression before ‘]’ token
myString1.cpp:6: error: expected primary-expression before ‘]’ token
myString1.cpp: In member function ‘MyString1 MyString1::append(MyString1)’: 
myString1.cpp:11: error: invalid use of member (did you forget the ‘&’ ?) 
myString1.cpp: In member function ‘void MyString1::clear()’:
myString1.cpp:25: error: expected primary-expression before ‘]’ token 
myString1.cpp:25: error: expected primary-expression before ‘{’ token
myString1.cpp:25: error: expected `;' before ‘{’ token 
myString1.cpp: In member function ‘bool MyString1::empty()’:
myString1.cpp:29: error: expected primary-expression before ‘]’ token
myString1.cpp:31: error: expected primary-expression before ‘else’ 
myString1.cpp:31: error: expected `;'  before ‘else’

这是我的三个不同部分的程序。

myString1.h

#ifndef MYSTRING1_H
#define MYSTRING1_H

class MyString1
{
  private:
   char chars[];
   int size;
  public:
   MyString1();
   MyString1(char chars[], int size);
   MyString1 append(MyString1 s);
   char at(int index);
   int length();
   void clear();
   bool empty();
   int find(char ch);
};
#endif

myString1.cpp

#include "myString1.h"
using namespace std;

MyString1::MyString1(char chars[], int size)
{
  this->chars[] = chars[];
  this->size = size;
}
MyString1 MyString1::append(MyString1 s)
{
  for(int i = size; i > size - s.length; i++)
    chars[i] = s.at(i);
}
char MyString1::at(int index)
{
  return chars[index];
}
int MyString1::length()
{
  return size;
}
void MyString1::clear()
{
  size = 0;
  chars[] = {};
}
bool MyString1::empty()
{
  if(chars[]){
    return true;
    else
      return false;
  }
}
int MyString1::find(char ch)
{
  for(int i = 0; i < size; i++)
    if(chars[i] = ch)
      return i;
}

testMyString1.cpp

#include <iostream>
#include "myString1.h"
using namespace std;

int main()
{
  MyString1 first("cat", 4);
  MyString1 second("dog", 4);
  cout << first.at(1) << " and " << second.at(1) << endl;
  first.append(second);
  cout << first.at(6) << endl;

  return 0;
}

我是一个新手，只是想学习如何使用 g++ 编译器，所以只是寻求一些帮助来阅读错误消息并调试我的代码。另外我确信有一些非常糟糕的代码，因此我们将不胜感激。

Answer 1

该代码有很多错误，所以我不知道从哪里开始，但我想通常给你一些指导来帮助你理解你的代码就可以了。

在我看来，您不需要在 String 类中拥有大小索引，因为有 strlen() 函数很乐意为您计算它。
现在，对于您的类声明，请检查如何声明将为您保存字符串的指针。您需要像下面这样做：

class MyString1
{
  private:
   char* chars;//this declares a pointer to a char that will hold the string for you
  public:
  ...

而且您永远不会分配保存字符串的 char* 。你的构造函数应该是：

MyString1::MyString1(const char* chars)
{
  this->chars = (char*) malloc(strlen(chars)+1); //this will allocate an array of strlen() chars +1
  strcpy(this->chars,chars);
}

正如你所看到的，我没有使用大小索引，因为 strlen 可以非常有效地为你找到它。 +1 代表“\0”，表示字符串结束。

现在要向字符串添加一些内容，这会很棘手。

void MyString1::append(const MyString1& s) //it's good to give a constant reference to the string here
{
  //first of all we gotta reallocate the pointer,since we don't have enough memory for the string
  int newsize = strlen(this->chars) + strlen(s)+1;
  this->chars = (char*) realloc(this->chars,newSize); \\ no check for realloc failure, I know but this is just an example

  strcat(this->chars,s.chars);

}

追加时不需要返回任何内容。你正在对这个字符串做一些事情。

你的 ::at() 函数几乎没问题。想象一下，如果字符串的大小为 10 并且您请求 MyString1::at(12) 会发生什么。这可能会导致分段错误（这不好）。

因此，您应该更改代码以进行如下所示的边界检查：

char MyString1::at(int index)
{
  //if it's out of bounds let's return -1 which will signify that we got an out of bounds value  (could also throw an exception here but that's a different subject altogether)
  if(index > strlen(this->chars) || index <0)
      return -1;

  return chars[index];
}

同样在 C/C++ 中，您必须释放分配的内存。因此，为了做到这一点，您应该声明一个称为析构函数的东西。

MyString1::~MyString1()
{
  free(this->chars);
}

最后，isempty 函数可以像这样：

bool MyString1::empty()
{
  return (this->chars[0] == '\0';
}