C ++访问类对象向量中的元素

Question

我正在尝试访问类对象向量中的元素，但我没有把它弄好。 我想构造函数/解构函数和引用有问题，但即使是其他问题，如C ++析构函数问题，类对象的std :: vector，类对象指针的c ++向量或对象的 C ++向量与指向对象的指针向量 。 希望有人可以帮我修改我的代码片段。

node.h

class Node {

public:
    Node();
    Node(int id, const std::string& name);
    Node(const Node& orig);
    virtual ~Node();

    void cout(void);

    int get_id(void);

private:
    int _id;
    std::string _name;

};

node.cpp

#include "node.h"

Node::Node() {
}

Node::Node(int id, const std::string& name) : _id(id), _name(name) {
    this->cout();
}

Node::Node(const Node& orig) {
}

Node::~Node() {
}

void Node::cout(void) {
    std::cout << "NODE " << _id << " \"" << _name << "\"" std::endl;
}

int Node::get_id(void) {
    return _id;
}

communication.h

#include "node.h"

class Com {

public:
    std::vector<Node> nodes;

    Com();
    com(const Com& orig);
    virtual ~Com();

    void cout_nodes(void);

private:

};

communication.cpp

#include "communication.h"

Com::Com() {
    nodes.push_back(Node(169, "Node #1"));
    nodes.push_back(Node(170, "Node #2"));
}

Com::Com(const Com& orig) {
}

Com::~Com() {
}

void Com::cout_nodes(void) {
    for (uint8_t i = 0; i < nodes.size(); i++) {
        nodes[i].cout();
    }
}

如果我运行Com com; 我得到了预期的输出：

[I 171218 13:10:10 Cpp:22] < NODE 169 "Node #1"
[I 171218 13:10:10 Cpp:22] < NODE 170 "Node #2"

但是运行com.cout_nodes(); 结果是：

[I 171218 13:10:14 Cpp:22] < NODE 0 ""
[I 171218 13:10:14 Cpp:22] < NODE 0 ""

就像C ++向量对象与指向对象指针的向量一样，当我使用引用时，一切正常，但是我无法使std::iterator和find_if工作。

更新：工作find_if语句和索引计算

auto iterator = std::find_if(nodes.begin(), nodes.end(), [&](Node node) {
    return node.get_id() == 169;
});

if (iterator != nodes.end()) {
    size_t index = std::distance(nodes.begin(), iterator );
    std::cout << "Index of ID #169: " << index << std::endl;
}

Answer 1

您定义了此复制构造函数：

Node::Node(const Node& orig) {
}

它没有做任何复制。 它默认初始化正在构造的Node所有成员。 由于std::vector::push_back会复制其参数，因此您的get伪造副本。

而不是强制定义一个编译器可以很容易地自己合成的操作（你只有一个int和一个std::string作为成员），而不是声明它。

或者，如果你想要显式（或者需要，例如使用默认的c'tor），只需显式默认它：

class Node {

    Node()                 = default;
    Node(const Node& orig) = default;

};