Question

我正在使用yaml-cpp作为项目。我想为某些类重载<<和>>运算符，但我遇到了一个问题，即如何“正确地”执行此操作。以Note类为例。这很无聊：

class Note {
  public:
    // constructors
    Note( void );
    ~Note( void );

    // public accessor methods
    void            number( const unsigned long& number ) { _number = number; }
    unsigned long   number( void ) const                  { return _number; }
    void            author( const unsigned long& author ) { _author = author; }
    unsigned long   author( void ) const                  { return _author; }
    void            subject( const std::string& subject ) { _subject = subject; }
    std::string     subject( void ) const                 { return _subject; }
    void            body( const std::string& body )       { _body = body; }
    std::string     body( void ) const                    { return _body; }

  private:
    unsigned long   _number;
    unsigned long   _author;
    std::string     _subject;
    std::string     _body;
};

<<运营商很容易。在.h：

YAML::Emitter& operator << ( YAML::Emitter& out, const Note& v );

在.cpp：

YAML::Emitter& operator << ( YAML::Emitter& out, const Note& v ) {
  out << v.number() << v.author() << v.subject() << v.body();
  return out;
}

没有汗水。然后我去宣布>>运营商。在.h：

void operator >> ( const YAML::Node& node, Note& note );

但在.cpp我得到了：

void operator >> ( const YAML::Node& node, Note& note ) {
  node[0] >> ?
  node[1] >> ?
  node[2] >> ?
  node[3] >> ?
  return;
}

如果我写node[0] >> v._number;之类的东西，那么我需要更改CV限定符来制作所有Note字段public（这会使我所教的一切都失败（教授，关于数据隐藏的书籍和经验）））。

我觉得在整个地方做node[0] >> temp0; v.number( temp0 );不仅乏味，容易出错，而且很丑陋，而是浪费（带有额外的副本）。

然后我明智了：我试图将这两个运算符移动到Note类本身，并将它们声明为friend s，但编译器（GCC 4.4）不喜欢这样：< / p>

src / note.h：44：错误：'YAML :: Emitter＆amp;注意:: operator＆lt;＆lt;（YAML :: Emitter＆amp;，const Note＆amp;）'必须只有一个参数
    src / note.h：45：错误：'void Note :: operator＆gt;＆gt;（const YAML :: Node＆amp;，Note＆amp;）'必须只取一个参数

问题：如何“正确”重载某个类的>>运算符

不违反信息隐藏原则？
没有过多的复制？

Answer 1

在不违反封装的情况下执行此操作的典型方法是使运算符＆gt;＆gt;朋友的功能。您的朋友操作员声明一定存在语法问题（不清楚错误消息的确切内容）。我不使用YAML，但是根据你的问题，以下是它的主旨：

class Note{
    ...
    friend void operator >> ( const YAML::Node& node, Note& note );
    ....
 };
 void operator >> ( const YAML::Node& node, Note& note ){
    node[0] >> note._number;
    node[1] >> note._author;
    node[2] >> note._subject;
    node[3] >> note._body;
 }

友元函数与私有成员具有相同的成员函数访问权限。

或者，您可以为所有成员数据声明setter，但是friend函数方法更清晰。

Answer 2

我喜欢使用辅助方法。由于该方法是该类的一部分，因此它将具有对所有私有字段的完全访问权限：

class Note {
public:
    void read(const YAML::Node& node)
    {
        node >> ...;
    }
};

然后让operator>>转发电话：

const YAML::Node &operator >> ( const YAML::Node& node, Note& note ) {
    note.read(node);
    return node;
}

Answer 3

您可以在Note中定义更多的setter方法，例如

void number(YAML::Immitter& e) { e>>_number; }

等，然后你将syntax-sugar >>定义为

void operator >> ( YAML::Immitter& e, Note& note ) {
  note.number(e);
  note.author(e);
  note.subject(e);
  note.body(e);
}

我不熟悉您正在使用的YAML命名空间（我知道yaml但我从未在C ++中处理过它），但这大致与普通流有关（除了void返回类型;-)，我确信它可以很容易地适应您的确切需求。

Answer 4

你的班级已经有了setter方法。只需使用临时值来读取值并使用setter方法配置对象：

void operator >> ( const YAML::Emitter& node, Note& note ) {
  unsigned long number;
  unsigned long author;
  // ...
  node[0] >> number;
  node[1] >> author;
  // ... everything properly read, edit the node:
  node.number(number);
  node.author(author);
  // ...
  return;

}

其他一些评论：几乎没有封装所有属性的setter / getter的类。您为用户提供了相同的访问级别，就像您的字段实际上是公共的一样（唯一的好处是您可以在以后添加检查，但仍然是封装很弱）。

在建议添加采用YAML节点的成员方法的解决方案中，这将为您的类的所有用户添加额外的依赖关系。虽然您可以使用前向声明来避免强制它们包含YAML标头，但您将无法使用Note拉出库以在不使用YAML的其他项目中使用。

潜在的浪费资源的使用可能会非常有限。然后，一如既往，首先测量，然后尝试解决问题，如果你有它们。

Answer 5

嗯，这是您可能会考虑的想法。你说你与非朋友非成员＆lt;＆lt;＆lt;＆lt;＆lt;＆lt;＆lt;函数是它涉及很多tmp声明。您是否考虑过封装概念并围绕它构建可重用的组件？使用可能看起来像这样：


inputter& operator >> (inputter& in, my_type & obj)
{
  input_helper<my_type> helper(obj);

  in >> helper.setter(&my_type::number);
  in >> helper.setter(&my_type::subject);
  // etc
}

input_helper的责任只是提供模板函数setter()，它返回一个简单读取值并用它调用setter的对象，创建必要的临时变量。像这样的代码需要对模板有一些熟悉的熟悉，但不会特别困难。现在不能完全思考 - 可能会感冒 - 或者我可能只能输出它。也许有点像这样：


template < typename T >
struct input_helper
{
  input_helper(T & t) : obj(t) {}

  template < typename V >
  struct streamer
  {
    streamer(T & t, void (T::*f)(V const&)) : obj(t), fun(f) {}

    template < typename Stream >
    Stream& read_from(Stream & str) const // yeah, that's right...const; you'll be using a temporary.
    {
      V v;
      str >> v;
      obj.(*fun)(v);
      return str;
    }

  private: // you know the drill...
  }

  template < typename V >
  streamer setter(void (T::*fun)(V const&))
  {
    return streamer(obj, fun);
  }
private:
  T & obj;
};
// etc...  operator >> (blah blah) { return setter.read_from(stream); }

肯定有各种各样的错误，但它应该给你这个想法。还需要更多的工作来概括。

重载流插入而不违反信息隐藏？

5 个答案: