Question

我在C ++中遇到了新手问题我有一个结构Edge定义如下：

struct Edge {
    int position[4];
    int average;
};

现在我需要创建很多这些结构，然后我为它创建了一个辅助方法，它根据一些参数创建了这些结构的数组：

Edge* createEdges(int some_parameters){
    Edge *edges = new Edge[8];

    for(int i = 0 ; i < 8; i++){
        Edge edge;
        edge.position[0] = 1; //fill the rest of the edge array in the same manner
        edge.average = 10;

        edges[i] = edge;
    }

    return edges;
}

但是，当我现在调用：Edge *edges = createEdges(int some_parameters)时，Edge数组中没有合理的数据（超出范围？）。

我想我在这里混淆了一些东西，但我更愿意，如果我能够在不诉诸vector数据结构的情况下完成这项工作。这是处理这个的常规方法还是应该自己声明边数组并将其传递给辅助方法来填充它？

修改：

首先，我要感谢大家的意见/提示/提示/建议/ ......他们帮助我找到了我很容易忽视的问题。
在我看到代码应该有效的回复之后，我测试了简化代码（我应该在第一位做的事情），令人惊讶的是，它有效！那么我检查了为什么我的真实代码不起作用，简化版本没有我的真实代码看起来像这样：

Edge* createEdges(int some_parameters){
     Edge* edges = new Edge[8];
     if(some_parameter != 0){
          //create the edges as in my 1st snippet
          return NULL; //doh, should return edges here !
     } else { 
          return NULL;
     }
}

我没有看到我只是返回错误的值（NULL）的原因是因为调试器向我显示了一些0xf6f6f6 addressess以及edge.position的一些负值（我不太了解明白，它应该只是告诉我0x000000，也许我只是想象事情。

总而言之，这是一个重要的教训，为什么在凌晨3点之后永远不会编码，没有什么好处的呢！

Answer 1

这是C ++使用std::vector：

std::vector<Edge> createEdges(int some_parameters){
    std::vector<Edge> edges;

    for(int i = 0 ; i < 8; i++){
        Edge edge;
        edge.position[0] = 1;
        edge.average = 10;
        edges.push_back(edge);
    }

    return edges;
}

Answer 2

你在这里工作得很好。我觉得你忘记的是，当你打电话时

Edge *edges = createEdges(int some_parameters)

edge变量是指向列表中第一个元素的指针。如果你那么做：

edges[0]

或：

edges[1]

您会看到这些是不同的值，您现在需要循环访问该数组。请记住，在循环期间不要通过数组的末尾。这就是大多数人说使用向量的原因，因为你可以使用迭代器来使用更多的安全功能。

Answer 3

欢迎使用C ++！用于创建类实例的辅助方法（C ++中的结构是一个所有字段都是公共的类）称为构造函数。这是我如何使用构造函数创建边类的实例。

class Edge {
public:
    int position[4];
    int average;
    Edge();
};

Edge::Edge() //constructor
{
    position[0] = 1;//fill the rest of the edge array in the same manner
    average = 10;
}

int main()
{
    Edge* myEdge = new Edge(); //constructor called
    Edge* myEdges[8];
    for (int i = 0; i < 8; i++)
    {
        myEdges[i] = new Edge();
    }
    return 0;
}

Answer 4

这是一个修改后的版本，它在函数中创建边，并返回包含所有创建边的矢量。

#include <iostream>
#include <vector>

struct Edge {
    int position[4];
    int average;
};

std::vector< Edge > createEdges(int some_parameters){
    std::vector< Edge > edges( 8 );

    for(int i = 0 ; i < 8; i++){
        const Edge edge{ { 1, 0, 0, 0 }, 10 };

        edges[i] = edge;
    }

    return edges;
}


int main()
{
    auto e( createEdges(5));

    std::cout<<e[0].average<<std::endl;
}

几乎没有修改：

我没有分配边数组，而是返回指针，而是返回向量
创建边缘对象的方式，并进行初始化（我认为这就是问题）

用C ++创建结构数组

4 个答案: