1

我正在尝试构造一个对象，然后将其中一个变量设置为一个值。

类头：

class Book {
public:
Book();
Book(string newSelection);
string getSelection();
string setSelection(string newSelection);

private:
string selection;
}

类 cpp

Book::Book() {}

Book::Book(string newSelection) {
selection = newSelection;
}


string Book::getSelection(){
return selection;
}

string Book::setSelection(string newSelection){
selection = newSelection;
}

司机

Book* book1 = new Book();
book1->setSelection("The Book Title");
cout << "Book selected:    " << book1->getSelection() << endl;

我遇到了分段错误，不知道为什么。有人可以指出我的问题可能出在哪里吗？

4

2 回答 2

2

您的setSelection()函数被声明为返回一个字符串，但实际上没有返回任何字符串。你至少应该得到一个编译警告。

通过在调试器中运行它，我相信正在发生的事情是在调用setSelection()析构函数之后在返回的字符串上调用。由于这个字符串并不真正存在，这会导致abort()运行时出现。

作为一般规则，'setters' 倾向于不返回任何东西，所以会写成如下：

void setSelection(const string& newSelection);

...

void Book::setSelection(const string& newSelection)
{
    selection = newSelection;
}

另请注意，字符串是通过 const 引用而不是更有效的值传递的。不过，这不会是您的问题的原因。

另一个建议是确保将您的“getter”声明为 const，因为它不会更改对象中的任何内容：

string getSelection() const;

...

string Book::getSelection() const
{
    return selection;
}
于 2012-08-08T16:34:39.970 回答
1

该setSelection方法被定义为返回一个字符串，但它没有返回任何内容。也许添加一个 return 语句（或将其更改为 void）将修复它：

string Book::setSelection(string newSelection){
  selection = newSelection;
  return selection;
}
于 2012-08-08T16:34:07.513 回答