Question

我有一个函数，我想从函数中检索字符串并返回错误代码：

int Connection::RecvMsg(std::string* buffer)
{
  buffer = &RecvBuffer.front();
  RecvBuffer.pop();
  return error_state; 
}

我想从RecvBuffer弹出一个字符串，我想通过参数传递字符串。

我意识到pop之后缓冲区会无效。是在该函数中声明静态字符串的唯一方法吗？我应该采取更好的方法吗？

Answer 1

我想你可能想要更像这样的东西：

*buffer = RecvBuffer.front();

这会将字符串分配给buffer个点。

Answer 2

在C ++中传播错误的推荐方法是使用异常机制。

如果您由于某种原因不想使用例外，则可以使用引用将信息传递回调用者。

int Connection::RecvMsg(std::string &buffer)
{
    buffer = RecvBuffer.front();
    RecvBuffer.pop();
    return error_state; 
}

当然，您也可以使用指针，但在这种情况下，最好确保调用者没有传递nullptr。所以你需要写这样的东西

int Connection::RecvMsg(std::string *buffer)
{
    error_state = false;

    if(buffer != nullptr)
    {
        *buffer = RecvBuffer.front();
        RecvBuffer.pop();
        error_state = true;
    }

    return error_state; 
}

Answer 3

Outparameters不是唯一（和IMO不是最好的）选择来建模几个参数。相反，我建议在pair：

中返回两个值

#include <utility>
#include <string>

std::pair<std::string,int> Connection::RecvMsg() {
  if (recvBuffer.empty()) {
    return std::make_pair<std::string,int>(std::string(),error_code);
  }
  std::string const front = recvBuffer.front();
  recvBuffer.pop();
  return std::make_pair<std::string,int>(front,no_error);
}

从参数和范围返回

3 个答案: