0

我有一段用于将数据发送到远程端的代码，它的行为有点像picojson，例如：

server::value::object obj;
obj["cmd"] = server::value("test");
obj["url"] = server::value(url);
...
obj["code"] = server::value(std::to_string(code));

server::value v(obj);
client.send_to_server(v.process());

只要有东西要发送，就会有这样的块。

这些行在我的项目中的很多地方都使用了，我正在考虑使用函数或 << 运算符来替换它，它应该能够处理可变参数。

<< 运算符版本将类似于：

Data d << "cmd" << "test"
       << "url" << url
       << ... << ...
       << "code" << code;
client.send_to_server(d);

这样做是个好主意吗？如何实施？

谢谢。

4

2 回答 2

1

设计这样的 API 不是一个好主意，因为它非常容易出错。

一个看似相似的想法适用于输出流，原因很简单：您放入流中用于输出的数据被统一处理。您作为参数输入的任何内容都将<<成为输出的一部分，除了流操纵器，它控制输出的呈现方式。

您的 API 不同：奇数项与偶数项的处理方式不同。此外，发送奇数个操作数也是错误的。如果由于某种原因您忘记将字符串代码放在其中一行上，所有值都会默默地变成后续行上的代码。这样的 API 非常脆弱，所以我强烈推荐它。

我认为让您的用户成对添加项目的 API 会更好。如果你的编译器是 C++11 兼容的，你也可以使用统一的初始化语法：

Data d = {
    {"cmd", server::value("test")}
,   {"url", server::value(url)}
,   {"code", server::value(std::to_string(code))}
};
于 2013-08-03T11:16:28.517 回答
1

我会用不同的方式写它：

d << add_value("cmd", "test")
  << add_value("url", url)
  << add_value(..., ...)
  << add_value("code", code)
  ;

为什么？它更清晰并允许更好的类型控制。

所以你创建一个类，说它__add_value_temp包含名称和值，一个add_value创建这个类的函数，并编写一个 << 运算符

Data &operator<<(Data &d, const __add_value_temp &val){
  d.add(val.name,val.val);
  return d;
}

更好的是 -__add_value_temp你可以使用std::pair, 而std::make_pair不是add_value!

Data &operator<<(Data &d, const std::pair<std::string,std::string> &val){
  d.add(val.first,val.second);
  return d;
}

...
Data d;
d << std::make_pair("cmd", "test")
  << std::make_pair("url", url)
  << std::make_pair(..., ...)
  << std::make_pair("code", code)
  ;

operator<<（最后一点：在 的类型上编写模板可能很聪明pair，这样你就可以通过引用传递东西，通常可以避免不必要的复制）

于 2013-08-03T11:17:44.270 回答