C++指针问题:如何修复这些代码中的错误?

您只分配 1 个node对象，然后将 10 个node*指针存储到您的向量中，这些指针都指向该 1 个node对象。 因此，在您的第一个循环中，所有x作业都在 1node上。 当第一个循环完成时，无论最后一个值分配给 1node的x，这就是你的第二个循环打印出 10 次的内容。

对于您正在尝试的内容，您需要分配并释放 10 个单独的node对象，例如：

#include <vector>
struct node{
int x;
};
node* temp;
std::vector<node*> ptrs;
for (int i = 0; i < 10; ++i){
temp = new node; // <-- MOVED HERE!!!
temp->x = i;
ptrs.push_back(temp);
}
std::vector<node*>::iterator j;
for (j = ptrs.begin(); j != ptrs.end(); ++j){
std::cout << (*j)->x;
}
for (j = ptrs.begin(); j != ptrs.end(); ++j){ // <-- ADD THIS LOOP!!!
temp = *j;
delete temp;
}

但是，你为什么要使用指针？ 这根本不是一个值得使用指针的好例子。 您应该创建node对象的vector，而不是指向node对象的指针vector，例如：

#include <vector>
struct node{
int x;
};
std::vector<node> nodes;
for (int i = 0; i < 10; ++i){
node temp;
temp.x = i;
nodes.push_back(temp);
}
std::vector<node>::iterator j;
for (j = nodes.begin(); j != nodes.end(); ++j){
std::cout << j->x; // or (*j).x
}

或者：

#include <vector>
struct node{
int x;
};
std::vector<node> nodes(10);
std::vector<node>::iterator j;
for (j = nodes.begin(); j != nodes.end(); ++j){
j->x = i; // or (*j).x
}
for (j = nodes.begin(); j != nodes.end(); ++j){
std::cout << j->x; // or (*j).x
}

你忘了声明函数 main。在任何函数之外，您只能使用声明。所以至少这部分代码

for (int i = 0; i < 10; i++){
temp->x = i;
ptrs.push_back(temp);
}
std::vector<node*>:: iterator j;
for (j = ptrs.begin(); j != ptrs.end(); j++){
std::cout << (*j)->x;
}
delete temp;

必须放置在函数中。

此循环

for (int i = 0; i < 10; i++){
temp->x = i;
ptrs.push_back(temp);
}

在矢量中放置相同的指针。所以这个说法

temp->x = i;

更改由同一指针指向的同一对象的数据成员xtemp其副本在矢量中推送。也就是说，在此循环之后，向量的所有元素都将指向同一个动态分配的对象。

您需要为每个值创建节点类型的新对象i。

程序可以通过以下方式查找示例

#include <iostream>
#include <memory>
#include <vector>
struct node{
int x;
};
int main() 
{
std::vector<std::unique_ptr<node>> ptrs;
for ( int i = 0; i < 10; i++ )
{
ptrs.emplace_back( new node { i } );
}
for ( const auto &p : ptrs )
{
std::cout << p->x << ' ';
}
std::cout << 'n';
return 0;
}

它的输出是

0 1 2 3 4 5 6 7 8 9

使用智能指针unique_ptr可以让您不必为删除分配的内存而烦恼。