重新检查时指针属性发生变化

Question

我是 C++ 新手，目前正在从事一个网络项目，遇到对象指针向量的异常错误。

状态类：

struct State
  {
    public:
      int reject_percent_;
      int fill_percent_;
      int partial_fill_;
      bool is_logged_in_;
      struct order
      {
        long id;
        long price;
        int quantity;
        bool is_filled = false;
        bool is_partially_filled = false;
      };
      std::vector<order *> orders;
  };

推入向量：（这里的 state 是 State struct 的一个对象）

State::order* o;

o->id = (obj->ClOrdID);   // obj->ClOrdID = 1
o->price = (obj->Price);  // obj->Price = 1
o->quantity = (obj->OrderQty); // obj->OrderQty = 1
std::cout<<o->id<<"\n";           //outputs 1
state->orders.push_back(o);

在其他功能中：

State::order* ord = NULL;
    for (int i = 0; i < state->orders.size(); ++i) 
    {
      std::cout<<((state->orders).at(i)->id)<<"\n";  //outputs :: 93893845689152
      std::cout<<((state->orders).at(i)->price)<<"\n"; //outputs :: 93893845689184
      std::cout<<((state->orders).at(i)->quantity)<<"\n"; //outputs :: 869246848
      if(obj->ClOrdID==(state->orders).at(i)->id)
      {
        ord=(state->orders).at(i);
        break;
      }
    }

我知道这不是一个最小的可重现示例，但我认为这可能是一个我看不到的微不足道的错误，代码很大并且需要很长时间才能缩短，所以请请耐心等待，您能否指出可能导致此问题的原因，因为这些值似乎是数据类型的垃圾值。

Answer 1

你没有为订单分配内存，也没有初始化指针

State::order* o;

取消引用此指针以写入其中

o->id = (obj->ClOrdID);   // obj->ClOrdID = 1
o->price = (obj->Price);  // obj->Price = 1
o->quantity = (obj->OrderQty); // obj->OrderQty = 1

从中读取

std::cout<<o->id<<"\n";           //outputs 1

或复制它

state->orders.push_back(o);

导致未定义的行为。您的程序可能会崩溃，一切似乎都正确，或者您的计算机可能会点披萨。

仅使用一些代码 sn-ps 很难说解决此问题的最佳方法是什么。一种方法是将std::vector<order *> orders; 更改为std::vector<order> orders;。另一种方法是使用智能指针。

Answer 2

State::order* o;

o->id = (obj->ClOrdID);

指针有一个不确定的值。通过这个未初始化的指针间接的行为是未定义的。

你应该改用这个：

std::vector<order> orders;

使向量包含订单实例。