【问题标题】:Error while passing a private data member as default argument to a public method of that class将私有数据成员作为默认参数传递给该类的公共方法时出错
【发布时间】:2020-02-22 03:54:34
【问题描述】:

我认为标题说明了一切。 MSVS 显示的错误是

非静态成员引用必须相对于特定对象

我的代码:

struct Node
{
    Node(size_t id, int length, int size);
    size_t id_;
    int length_;
    std::vector<Node*> children_;
};

class SuffixTree
{
public:
    SuffixTree();
    void printSuffixes();
    void printSuffixes(Node* current = root_);    // ERROR
    ~SuffixTree();
private:
    Node *root_;
};

还有一些类似的方法,我希望用户从 main 调用这些方法,但由于 root_ 是私有的,我必须重载所有这些方法,而用户现在调用重载的方法。这些方法的定义很简单:

void SuffixTree::printSuffixes()
{
    printSuffixes(root_);
}

有什么办法吗?

编辑:

void SuffixTree::printSuffixes(Node* current)
{
    if (current == nullptr)
        return;
    if (current->length_ == -1)
        std::cout << "leaf" << std::endl;
    for (size_t i = 0; i < current->children_.size(); ++i)
        printSuffixes(current->children_[i]);
}

【问题讨论】:

  • 您不能使用类成员作为默认参数。摆脱它。
  • @NathanOliver 摆脱了什么?
  • = root_ 部分。
  • @NathanOliver 我知道我已经让代码工作了。我只是问是否有任何方法可以使其仅使用一个名为 printSuffixes(Node*) 的函数工作
  • 没有。您需要使用成员变量作为默认参数,但您不能这样做。你必须像你一样使用重载。

标签: c++


【解决方案1】:

Default arguments 有很多限制。

请考虑使用 nullptr 作为默认值:

void SuffixTree::printSuffixes(Node* current = nullptr)
{
    if (current == nullptr)
        current = root_;
    // ...
}

【讨论】:

  • 为什么要增加运行时间成本?
  • printSuffixes(Node*) 函数是一个递归函数,当 current nullptr 时,它已经有一个案例要处理。
  • @NathanOliver 这个答案只是为由于语言限制而无法使用默认参数的情况提供一个简单的替代方案。 that 是否是最佳解决方案,这取决于(有时甚至无法使用)。无论如何,如果内联,则没有运行时成本。
  • @lucieon 然后你应该在问题中说明这一点(也许举一个调用者和成员函数主体的例子)。听起来您没有正确设计界面。
  • @Acorn 它使用了更多与问题无关的变量,因此我决定不包含它。如果您愿意,我可以,但是您能解释一下我的界面不正确吗?编辑:我添加了第二个重载方法。
猜你喜欢
  • 2021-05-19
  • 2015-06-29
  • 1970-01-01
  • 2013-11-21
  • 2011-12-29
  • 2013-11-06
  • 2019-12-28
相关资源
最近更新 更多