如何访问 std::vector 类型的成员

Question

我的程序有一个函数来计算一些总和，为此我需要访问向量中对象的属性：

向量声明：

class trilateration {
public:
    ...
    std::vector<tip> *potential;
    ...
};

然后在构造函数中对其进行初始化：

trilateration::trilateration()
{
...
potential = new std::vector<tip>();
...
}

类提示如下所示：

class tip {
public:

double sum;
Point2d *pt;
tip();
tip(double x, double y);
virtual ~tip();
};

提示构造函数：

tip::tip(double x, double y)
{
pt = new Point2d(x,y);
sum=0;
}

在一些函数中将对象添加到向量中，如下所示：

potential->push_back(tip1);

然后我想像这样访问向量中的一些对象：

void trilateration::get3points()
{

for(int i=0; i<potential->size(); ++i)
{
    for(int j=0; j<potential->size(); ++j)
    {
        potential[i].sum=potential[i].sum+normalize(potential[i].pt,potential[j].pt);
    }

}
}

编译时出现以下错误：

error: ‘class std::vector<tip>’ has no member named ‘sum’
error: ‘class std::vector<tip>’ has no member named ‘pt’

如何从向量中访问这些属性？

编辑：

在将电位更改为三边测量的成员并将pt更改为提示的成员后，程序编译但遇到

potential.push_back(tip1);

抛出：

*** glibc detected *** ./loktest: malloc(): memory corruption: 0x00792f10 ***

Answer 1

除非有强烈理由拥有指针数据成员并使用new 在堆上分配对象，否则不要这样做。

您的代码似乎使用了一种 Java 风格，但这是 C++，而不是 Java。享受 C++ 的自动资源管理，以及 C++ 析构函数的强大功能。只需定义数据成员，无需使用指针和动态分配。

在课堂trilateration，从vector<tip>*更改为vector<tip>：

class trilateration {
public:
    ...
    // WAS std::vector<tip> *potential;
    std::vector<tip> potential;
    ...
};

在trilateration的构造函数中，不需要动态创建向量；只需删除使用new 分配向量的行：

trilateration::trilateration()
{
    ...
    // REMOVED:
    // potential = new std::vector<tip>();
    ...
}

请注意，在您之前的代码中，当您使用 new 分配向量时，您必须在析构函数中正确地 delete 它，并提供正确的复制操作（复制构造函数和复制赋值），或者禁止复制标记上述操作=delete.

相反，如果您有一个普通的简单非指针数据成员，则不需要所有这些复杂的东西。

tip 课程也是如此。
除非有充分的理由拥有 Point2d* 数据成员，否则只需使用 Point2d（非指针）数据成员：

class tip {
public:

  double sum;

  // WAS: Point2d *pt;
  Point2d pt; // <-- note: no pointers here

  tip();
  tip(double x, double y);

  // NOTE: Do you really need a virtual destructor here?? 
  virtual ~tip();
};

也改变构造函数：

tip::tip(double x, double y)
    : pt(x, y), sum(0)
{
  // REMOVE:
  // pt = new Point2d(x,y);
  //sum=0;
}

您的代码得到了简化，您将避免一些错误和令人头疼的问题。

Answer 2

由于trilateration::potential 是std::vector<tip>*，potential[i] 是位于trilateration::potential + i 的std::vector<tip>。这不是你想要的。

for(int j=0; j<potential->size(); ++j)
{
    (*potential)[i].sum=potential[i].sum+normalize((*potential)[i].pt,(*potential)[j].pt);
}

是一种可能的黑客攻击。但是您真正想要的是摆脱所有那些无用的指针。将向量设为类的成员变量，并在不需要的地方停止使用 new。