CODEWITHSUNDEEP
c++opencv
Dr. Mallikarjun Anandhalli
Dr. Mallikarjun Anandhalli

Reputation: 394

Assign a point to the class member

i trying to implement the following link http://in.mathworks.com/help/vision/examples/motion-based-multiple-object-tracking.html in opencv and c++. I have created a class say ex:

class assign
{
    vector <int> id;
    vector <int> area;
    vector<Point> centroid;
};

After this i have created an object

assign id;

Now i want to assign the centroid value and other values too. what i tried is

id.centroid (p);

where p is a "point" But i'm getting error for this. I don't know where i'm going wrong.

Upvotes: 1

Views: 115

Answers (3)

Francis Cugler
Francis Cugler

Reputation: 7915

To add to the previous answers; if you want to expand on your class this can be done for you during construction of your object.

class Assign {
private:
    std::vector<int> m_vIds;
    std::vector<int> m_vAreas;
    std::vector<Vec2> m_vCentroids;

public:
    Assign();  // Default Constructor Same As What You Have But Not Declared.
    Assign( int* pIds, int* pAreas, int* pCentroids ); // Create By Using Pointers
    // Create By Passing In Either Pre Filled Vectors Or Even An Empty 
    // Vectors To Be Filled Out Later. Passes By Reference. This Will
    // Also Set The Variables That Are Passed In From The Caller.
    Assign( std::vector<int>& vIds, std::vector<int>& vAreas, std::vector<Vec2>& vCentroids );

    // Since You Are Using Vectors Within This Class It Is Also Good To
    // Have A Destructor To Clear These Out Once The Object Is Done And
    // Ready To Be Destroyed Or Removed From Memory
    ~Assign();
};

// The Destructor Would Look Like This
Assign::~Asign() {
    if ( !m_vIds.empty() ) {
        m_vIds.clear();
    }
    if ( !m_vAreas.empty() ) {
        m_vAreas.clear();
    }
    if ( !m_vCentroids.empty() ) {
        m_vCentroids.empty();
    }
} // ~Assign

// NOTE: I used Vec2 instead of point due to my use of programming
// 2D & 3D Graphics Rendering Engines; Most Graphics APIs and Libraries
// along with Most Math Libraries Will Not Have A Point Class; Most Will
// Use Vec2 or Vec3 - Vector2  or Vector3 & Vector4 Since in terms of
// memory they are exactly the same thing. It is up to you to know which
// objects are points or locations, and which are vectors as in forces,
// velocities, accelerations, directions, normals etc. The only major
// difference between a discrete Point Class or Structure versus a Vector
// Class is that the Vector Class usually has operations defined with it
// to do vector mathematics such as addition, subtraction, multiplication by
// value, multiplication by vector, division by value, division by vector,
// cross & dot product, comparisons, testing if vector is 0, setting it to
// be a normal vector, returning the magnitude or length and a few others.
// The general point class or object is usually just data values or
// simply coordinates without operations.

Upvotes: 1

exilit
exilit

Reputation: 1176

The main answer is already given by songyuanyao. What I want to add is a possible solution which allows you to use the member variables like you already tried it.

If you want to get and set the member centroid with id.centroid(p) you could go with the following class declaration:

class Assign
{
public:
    vector<Point> centroid();
    void          centroid(vector<Point> c);
private:
    vector<Point> m_centroid;
};

The definition might then look like this:

// getter
vector<Point> Assign::centroid() {
    return m_centroid;
}

// setter
void Assign::centroid(vector<Point> c) {
    m_centroid = c;
}

Now if you use id.centroid(p) to set the member the overloaded setter will be called and will set the variable. If you call p = id.centroid() (empty parameter list) the overloaded getter will be called and will return the current m_centroid.

Upvotes: 1

songyuanyao
songyuanyao

Reputation: 172964

centroid is a private member of class assign. If you want to access it directly, you should make it public

class assign
{
public:
    vector<Point> centroid;
    //...
};

And if you want to add a Point into centroid, you should

id.centroid.push_back(p);

Upvotes: 3

Related Questions