重新检查时指针属性会更改

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::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);

导致未定义的行为。您的程序可能会崩溃，一切看起来都正确，或者您的计算机可能会订购披萨。

很难说只用一些代码片段来解决这个问题的最佳方法是什么。一种方法是将std::vector<order *> orders;更改为std::vector<order> orders;。另一种方法是使用智能指针。

Answer 2

State::order* o;
o->id = (obj->ClOrdID);

指针具有不确定的值。通过此未初始化指针进行定向的行为是未定义的。

您可能应该改用它：

std::vector<order> orders;

以便向量包含订单实例。