Push_back正在替换当前点,而不是向向量添加新的点

时间:2016-10-02 15:21:53

标签: c++ opencv vector point push-back

我无法将新点添加到当前Vector点数的末尾。它目前正在做的是用我想要添加的新点覆盖现有的向量,使向量仅包含1个元素。

这是类定义(.h文件):

class person
{
public:
    person();
    ~person();

    int ID;
    std::vector<cv::Point> history;

    void addposition(cv::Point);
    void person::drawhistory(cv::Mat*,std::vector<cv::Point>);
};

这就是出现在类的.cpp文件中的函数声明:

person::person()
{
}

person::~person()
{
}

void person::addposition(cv::Point inpt) {
    std::cout << "Adding a position ----" << std::endl;
    std::cout << "Pushing inpt.x: " << inpt.x << std::endl;
    std::cout << "Pushing inpt.y: " << inpt.y << std::endl;
    history.push_back(inpt);
    std::cout << "Current Size: " << history.size() << std::endl;
    if (history.size()>15)
    {
        history.erase(history.begin());
    }
}

void person::drawhistory( cv::Mat* image, std::vector<cv::Point> hist) {
    cv::Point pt;
    for (cv::Point const& pt : hist)
    {
        std::cout << "Printing the History" << std::endl;
        std::cout << "Current Pt.x: " << pt.x << std::endl;
        std::cout << "Current Pt.y: " << pt.y << std::endl;
        cv::circle(*image, pt, 5, cv::Scalar(0, 0, 0), -1);
    }
}

这就是在main函数中调用这两个函数的方法。请注意,detectBox声明为:

vector<RECT> detectBox

它正确地在框架中存储必要的点,所以我很确定它不是问题的原因:

for (RECT const& rect : *detectBox)
{
        std::cout << "Inside the LOOOOOP" << std::endl;
        //This is simply finding the middle point of the rectangle currently stored in detectBox        
        pt.y = (rect.bottom + rect.top) / 2; 
        pt.x = (rect.left + rect.right) / 2;

    person personn;
    personn.addposition(pt);
    personn.drawhistory(&img_8bit, personn.history);

    cv::circle(img_8bit, pt, 3, cv::Scalar(255, 255, 0), -1);
}
cv::imshow("8Bit", img_8bit);

我认为将点推入向量会很简单,但不知何故它不会在向量的底部添加新点。另请注意,我添加了一个擦除步骤,可将存储的点数保持为15。

我的类定义中的函数存在问题(我是类的新手),还是我从主循环调用函数的问题?

1 个答案:

答案 0 :(得分:2)

这很可能不是你想要做的:

person personn;
personn.addposition(pt);
personn.drawhistory(&img_8bit, personn.history);

你的presonn是循环体的局部,因此在每次迭代中你创建一个新的person,添加一个位置并打印出来。只需在循环之外声明personn

PS :问题的一部分可能如下所示:

#include <vector>
#include <iostream>
int main() {
    for (int i=0; i<5; i++) {
        std::vector<int> vect;
        vect.push_back(i);
        std::cout << vect.size() << std::endl;
    }
}

它会重现您所遇到的问题,它会编译并且只有最少的代码才能执行此操作。如果你自己创造了这个,你可能自己找到了错误。当然,找到错误的来源并不总是那么容易。调试器可能有助于找到出错的位置。