C++方法参数应选指针还是引用？push_back元素未存入vector问题求助

你的代码没有实现预期写入效果的根本原因是：OrdersList类的getOrdersList()方法返回的是vector<Orders*>的值副本，而非内部成员的引用。你调用push_back方法时，实际是在临时生成的副本上修改，修改完成后临时对象直接销毁，类内部的私有ordersList成员完全没有被改动，因此看不到新增元素。

同时你的类定义存在两处语法错误需要先修正：

1. 私有成员vector<Orders*> ordersList末尾缺少分号
2. 类定义结束的大括号}末尾缺少分号

方案1：修改get方法返回引用（快速修复） #

将getOrdersList()的返回值修改为左值引用，即可直接返回类内部vector的引用，所有操作都会直接作用到原成员变量上：
修正后的类定义：

class OrdersList{
private : 
  vector<Orders*> ordersList; // 补充分号
public:
  // 返回引用而非值
  vector<Orders*>& getOrdersList();
  // 可选：增加const版本用于只读场景
  const vector<Orders*>& getOrdersList() const;
};

方法实现需要同步修改返回类型：

vector<Orders*>& OrdersList::getOrdersList() {
  return ordersList;
}

const vector<Orders*>& OrdersList::getOrdersList() const {
  return ordersList;
}

修改完成后你原有issueOrder方法不需要改动，即可正常将指针写入vector。

方案2：封装新增方法（更符合封装原则） #

如果不需要对外暴露整个vector的操作权限，更推荐在OrdersList类内部封装新增元素的方法，避免内部成员被意外修改：

class OrdersList{
private : 
  vector<Orders*> ordersList;
public:
  // 封装新增订单的方法，不对外暴露整个vector
  void addOrder(Orders* order) {
    ordersList.push_back(order);
  }
  // 只读获取列表时返回const引用即可
  const vector<Orders*>& getOrdersList() const;
};

对应的issueOrder方法调整为调用封装好的新增接口：

void Player::issueOrder(Orders* order)
{
    ordersList->addOrder(order);
}

• 你当前场景vector存储的是Orders*类型，参数传入Orders*是匹配的，逻辑没有问题。需要额外注意保证传入的order指向的对象生命周期大于存储它的vector，避免出现悬空指针、野指针访问问题。
• 如果需要降低内存管理风险，可以将vector的存储类型改为std::shared_ptr<Orders>，通过智能指针自动管理对象生命周期，避免内存泄漏。

内容的提问来源于stack exchange，提问作者Anonymous_programmer2458