1

我目前正在写一个日志类(只是为了练习)并遇到了一个问题。我有两个类: Buffer 类充当临时缓冲区,并在其析构函数中刷新自身。还有返回 Buffer 实例的类 Proxy,这样我就不用一直写 Buffer() 了。

无论如何,这里是代码:

#include <iomanip>
#include <iostream>
#include <sstream>
#include <string>

class Buffer
{
private:
  std::stringstream buf;
public:
  Buffer(){};
  template <typename T>
  Buffer(const T& v)
  {
    buf << v;
    std::cout << "Constructor called\n";
  };
  ~Buffer()
  {
    std::cout << "HEADER: " << buf.str() << "\n";
  }
  Buffer(const Buffer& b)
  {
    std::cout << "Copy-constructor called\n";
    // How to get rid of this?
  };
  Buffer(Buffer&&) = default;
  Buffer& operator=(const Buffer&) & = delete;
  Buffer& operator=(Buffer&&) & = delete;
  template <typename T>
  Buffer& operator<<(const T& v)
  {
    buf << v;
    return *this;
  }
};

class Proxy
{
public:
  Proxy(){};
  ~Proxy(){};
  Proxy(const Proxy&) = delete;
  Proxy(Proxy&&) = delete;
  Proxy& operator=(const Proxy&) & = delete;
  Proxy& operator=(Proxy&&) & = delete;
  template <typename T>
  Buffer operator<<(const T& v) const
  {
    if(v < 0)
      return Buffer();
    else
      return Buffer(v);
  }
};

int main () {  
  Buffer(Buffer() << "Test") << "what";
  Buffer() << "This " << "works " << "fine";
  const Proxy pr;
  pr << "This " << "doesn't " << "use the copy-constructor";
  pr << "This is a " << std::setw(10) << " test";
  return 0;
}

这是输出:

Copy-constructor called
HEADER: what
HEADER: Test
HEADER: This works fine
Constructor called
HEADER: This doesn't use the copy-constructor
Constructor called
HEADER: This is a       test

该代码完全符合我的要求,但它取决于 RVO。我多次读到你不应该依赖 RVO,所以我想问我怎么能:

  1. 完全避免 RVO,以便每次都调用复制构造函数
  2. 避免复制构造函数

我已经尝试通过返回引用或移动来避免复制构造函数,但是会出现段错误。我猜那是因为 Proxy::operator<< 中的临时变量在返回期间被删除。

我也对大致相同的完全不同的方法感兴趣。

4

1 回答 1

1

这似乎是一个人为的问题:首先,无论 RVO 是启用还是禁用,代码都可以工作(您可以使用带有flagno-elide-constructorsG++ 来测试它)。其次,您设计Buffer用于<<运算符的对象返回的方式只能通过复制来完成†</sup>:该Proxy::operator<<(const T& v)函数在堆栈上创建一个新Buffer实例,然后在您离开函数调用时将其删除(即 ) 中的每个串联之间pr << "This " << "doesn't " << "use the copy-constructor";;这就是尝试从函数外部引用此对象时出现分段错误的原因。

或者,您可以定义一个<<运算符来使用动态内存,例如返回 a unique_ptr<Buffer>

#include <memory>

...

std::unique_ptr<Buffer> operator<<(const T& v) const
{
    if(v < 0)
        return std::unique_ptr<Buffer>(new Buffer());
    else
        return std::unique_ptr<Buffer>(new Buffer(v));
}

但是,您的原始连接语句将无法编译,因为Proxy::operator<<(const T& v)现在返回类型std::unique_ptr<Buffer>而不是的对象Buffer,这意味着此返回的对象没有Proxy::operator<<(const T& v)定义自己的函数,因此如果没有首先明确地 de- 多个连接将不起作用引用返回的指针:

const Proxy pr;
std::unique_ptr<Buffer> pb = pr << "This ";
//  pb << "doesn't " << "use the copy-constructor"; // This line doesn't work
*pb << "doesn't " << "use the copy-constructor";

换句话说,你的类本质上依赖于复制,因此,如果你真的想避免复制,你应该把它们扔掉并完全重新设计你的日志功能。


†</sup> 我敢肯定有一些黑魔法巫术可以被调用来使这成为可能——尽管以一个人的理智为代价。

于 2016-04-06T11:32:06.497 回答