U++ forum: Welcome to the forum

Search on this site

Search in forums

Home » Community » Newbie corner » stl question

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

stl question [message #32796]

Fri, 10 June 2011 05:37

GaroRobe
Messages: 30
Registered: May 2011
Location: Khabarovsk, Russia

Member

So.

I've got a method in OpenCV I need to use. It has parameters with type vector<vector<Point3d>>. It is a data about primitive regular mesh (in model coords) for each frame. Filling it manually feels wrong, so I do it like:

vector<Point2f> checkboardMesh;

for ( int i = 0; i < board_h; i++ )
		for ( int j = 0; j < board_w;i++ )
			checkboardMesh.push_back ( * new Point2f ( j, i ) );

And here I get stuck: will I not get memory leak here? AFAIK std::vector copies element on insert. Should I do it like

for ( int i = 0; i < board_h; i++ )
for ( int j = 0; j < board_w; j++ )
{
	tmpP = new Point2f ( j, i );
	checkboardMesh.push_back ( *tmpP );
	delete tmpP;
}

maybe? If I do, then I'd rather look for a better idea.

Or maybe there is some way of using Upp containers here?

P.S.: The method I mentioned is cv::calibrateCamera btw:

double cv::calibrateCamera( const vector<vector<Point3f> >& objectPoints,
                          const vector<vector<Point2f> >& imagePoints,
                          Size imageSize, Mat& cameraMatrix, Mat& distCoeffs,
                          vector<Mat>& rvecs, vector<Mat>& tvecs, int flags )

Report message to a moderator

Re: stl question [message #32797 is a reply to message #32796]

Fri, 10 June 2011 07:55

dolik.rce
Messages: 1789
Registered: August 2008
Location: Czech Republic

Ultimate Contributor

Hi GaroRobe,

You don't have to use new at all Wink

Just creating a temporary object is fine, since std::vector will make a copy:

vector<Point2f> checkboardMesh;

for ( int i = 0; i < board_h; i++ )
		for ( int j = 0; j < board_w; j++ )
			checkboardMesh.push_back ( Point2f ( j, i ) );

Using U++ containers should be possible, but it would be an ugly hack Smile

Something allong the lines of:

	using std::vector;
	using cv::Point2f;

	Vector<Point2f> v; //this requires to do NAMESPACE_UPP; NTL_MOVEABLE(cv::Point2f); END_UPP_NAMESPACE; in global scope

	vector<Point2f> checkboardMesh;

	for ( int i = 0; i < board_h; i++ )
		for ( int j = 0; j < board_w; j++ )
			v.Add( Point2f ( j, i ) );
	
	checkboardMesh.resize(v.GetCount());
	memmove(&checkboardMesh[0],v.Begin(),v.GetCount()*sizeof(Point2f));

In other words: not elegant and definitelly not recomended. Please never do something like that Wink

Honza

Report message to a moderator

Re: stl question [message #32799 is a reply to message #32797]

Fri, 10 June 2011 08:21

GaroRobe
Messages: 30
Registered: May 2011
Location: Khabarovsk, Russia

Member

Wouldn't have guessed. Thanks a lot Smile

Report message to a moderator

Previous Topic:	Restricting GridCtrl Selection to just One row
Next Topic:	Please recommend a scripting language

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Sun Apr 28 04:48:36 CEST 2024

Total time taken to generate the page: 0.02590 seconds