473,382 Members | 1,355 Online
Bytes | Software Development & Data Engineering Community
Post Job

Home Posts Topics Members FAQ

Join Bytes to post your question to a community of 473,382 software developers and data experts.

Yet Another Question About Smart Pointers (YAQAS)

Hello,

I've been trying to implement smart pointers in C++ (combined with a
reference counter) because I want to do some memory management. My code
is based on the gamedev enginuity articles, various books and ...
whatever I could find on the subject :-)

I'll leave out the reference counter part because its pretty basic, but
my reference counter class is called xObject. My smart pointer class is
called xPointer.

My problem is that I can't figure out how to make my smart pointer
object a pointer. Look at the following code:

--------------- CUT
void test()
{
//xShaderResource is derived from xObject
class xPointer<xShaderResource> myShader2 = new xShaderResource();
myShader2->loadResource(NULL); //testing
}
--------------- CUT

This works like a charm. The xShaderResource object will be destroyed in
my garbage collector because myShader2 is beeing destructed after it
goes out of scope when test() returns.

So far so good, but what if let's say I have a vector/list with 1000+
xPointer objects (imagine a resource manager)? They will never go out of
scope, right?

What I need is to do something like this:

--------------- CUT
class xPointer<xShaderResource> *myShader2 = new
xPointer<xShaderResource>(new xShaderResource());
myShader2->loadResource(NULL);
--------------- CUT

This should enable me to run through the list and do something like this:

delete (*iterator);

on relevant objects thus forcing them to go away and get collcected. Can
this be done? I kinda figure I need to change/add my operators
somewhere, but I can't figure out how :-(

Can anyone help?

-------------------------------- Relevant code
template<class X>
class xPointer
{
protected:
X* obj;
public:

xPointer()
{
obj = 0;
}

xPointer(X *oP)
{
obj = 0;
*this = oP;
}

~xPointer()
{
if( obj )
obj->Release();
}

X& operator*(void) { return *obj; }

void operator =(X *oP)
{
if( obj )
obj->Release();
obj = oP;
if( obj )
obj->Alloc();
}

void operator =(const xPointer<X> &oP)
{
if( obj )
obj->Release();
obj = oP.obj;
if( obj )
obj->Alloc();
}

X* operator ->(void)
{
return obj;
}
};
Jul 22 '05 #1
6 1787
"Johnny Hansen" <we*******@suxx.dk> wrote in message
news:cj**********@news.cybercity.dk...
Hello,

I've been trying to implement smart pointers in C++ (combined with a
reference counter) because I want to do some memory management. My code
is based on the gamedev enginuity articles, various books and ...
whatever I could find on the subject :-)

I'll leave out the reference counter part because its pretty basic, but
my reference counter class is called xObject. My smart pointer class is
called xPointer.

My problem is that I can't figure out how to make my smart pointer
object a pointer. Look at the following code:

--------------- CUT
void test()
{
//xShaderResource is derived from xObject
class xPointer<xShaderResource> myShader2 = new xShaderResource();
myShader2->loadResource(NULL); //testing
}
--------------- CUT

This works like a charm. The xShaderResource object will be destroyed in
my garbage collector because myShader2 is beeing destructed after it
goes out of scope when test() returns.

So far so good, but what if let's say I have a vector/list with 1000+
xPointer objects (imagine a resource manager)? They will never go out of
scope, right?

What I need is to do something like this:

--------------- CUT
class xPointer<xShaderResource> *myShader2 = new
xPointer<xShaderResource>(new xShaderResource());
myShader2->loadResource(NULL);
--------------- CUT

This should enable me to run through the list and do something like this:

delete (*iterator);

on relevant objects thus forcing them to go away and get collcected. Can
this be done? I kinda figure I need to change/add my operators
somewhere, but I can't figure out how :-(

Can anyone help?
[...]


Well, first consider using a reference-counted pointer library. If you want
to write your own, then take a look at the FAQ's answers about reference
counting (http://www.parashift.com/c++-faq-lite/ Section 16 items 21 "How do
I do simple reference counting" on), because they contain useful snippets of
code for implementing reference counting.

Now, once you have a working reference-counted smart pointer stored in a
std::list or std::vector, removing elements so that the managed
reference-counted object may be deallocated is about as simple as removing
any other element from the container. A simple example (using std::list)

std::list<smartpointertype>::iterator it = /* something to remove */;
myList.erase( it );

will remove the smart pointer from the std::list, causing the smart
pointer's destructor to be invoked, which will cause the managed object's
reference count to be decreased by one (and deallocated, etc if
appropriate).

--
David Hilsee
Jul 22 '05 #2

"Johnny Hansen" <we*******@suxx.dk> wrote in message
news:cj**********@news.cybercity.dk...
Hello,

I've been trying to implement smart pointers in C++ (combined with a
reference counter) because I want to do some memory management. My code is
based on the gamedev enginuity articles, various books and ... whatever I
could find on the subject :-)

I'll leave out the reference counter part because its pretty basic, but my
reference counter class is called xObject. My smart pointer class is
called xPointer.

My problem is that I can't figure out how to make my smart pointer object
a pointer. Look at the following code:

--------------- CUT
void test()
{
//xShaderResource is derived from xObject
class xPointer<xShaderResource> myShader2 = new xShaderResource();
myShader2->loadResource(NULL); //testing
}
--------------- CUT

This works like a charm. The xShaderResource object will be destroyed in
my garbage collector because myShader2 is beeing destructed after it goes
out of scope when test() returns.

So far so good, but what if let's say I have a vector/list with 1000+
xPointer objects (imagine a resource manager)? They will never go out of
scope, right?
Wrong. They will go out of scope when they are removed from the vector or
list, or when the vector or list itself is destroyed.

What I need is to do something like this:

--------------- CUT
class xPointer<xShaderResource> *myShader2 = new
xPointer<xShaderResource>(new xShaderResource());
myShader2->loadResource(NULL);
--------------- CUT

This should enable me to run through the list and do something like this:

delete (*iterator);


You are undoing the good work that smart pointers do. You are making life
difficult for yourself by going back to raw pointers. Smart pointers are
more powerful than you realise. You don't need to do any more work. You're
done.

john
Jul 22 '05 #3
John Harrison wrote:
You are undoing the good work that smart pointers do. You are making life
difficult for yourself by going back to raw pointers. Smart pointers are
more powerful than you realise. You don't need to do any more work. You're
done.


I must be missing something, because I can't get it to work. Look at this:

std::vector<xPointer<xShaderResource>*> myList;
//defined in global scope for testing purposes
//should be stored in my resource-manager object

------------------ CUT
void test()
{
class xPointer<xShaderResource> myResource = new xShaderResource();

myList.push_back(&myResource);
(*myList[0])->loadResource(NULL);
}
------------------ CUT

When test() returns my "myResource" still gets destructed. I assume its
because the myList only contains a pointer to the local global scope
object, thus will contain a invalid pointer when test() returns. Not good.

However trying to do:
std::vector<xPointer<xShaderResource>> myList;

fails miserably. How can I store the actual object in the list, not only
the pointer? This is what I want (i think) :-)


Jul 22 '05 #4

"Johnny Hansen" <we*******@suxx.dk> wrote in message
news:ck***********@news.cybercity.dk...
John Harrison wrote:
You are undoing the good work that smart pointers do. You are making life difficult for yourself by going back to raw pointers. Smart pointers are
more powerful than you realise. You don't need to do any more work. You're done.
I must be missing something, because I can't get it to work. Look at this:

std::vector<xPointer<xShaderResource>*> myList;


Why use smart pointers if you then just go and use ordinary pointers on top?

std::vector<xPointer<xShaderResource> > myList;

Now all your problems go away.
//defined in global scope for testing purposes
//should be stored in my resource-manager object

------------------ CUT
void test()
{
class xPointer<xShaderResource> myResource = new xShaderResource();

myList.push_back(&myResource);
myList.push_back(myResource);
(*myList[0])->loadResource(NULL);
myList[0]->loadResource(NULL);
}
------------------ CUT

When test() returns my "myResource" still gets destructed. I assume its
because the myList only contains a pointer to the local global scope
object, thus will contain a invalid pointer when test() returns. Not good.

However trying to do:
std::vector<xPointer<xShaderResource>> myList;

fails miserably. How can I store the actual object in the list, not only
the pointer? This is what I want (i think) :-)


If this fails miserably, it must be because your smart pointer is not coded
correctly. How does it fail? Could you post the code.

john
Jul 22 '05 #5
>
If this fails miserably, it must be because your smart pointer is not coded correctly. How does it fail? Could you post the code.


OK, looking at the code you posted earlier, the problem is that you have not
defined a copy constructor for your smart pointer.

Something like this is needed

xPointer(const xPointer<X> &oP)
{
obj = oP.obj;
if( obj )
obj->Alloc();
}

john
Jul 22 '05 #6
John Harrison wrote:
If this fails miserably, it must be because your smart pointer is not

coded
correctly. How does it fail? Could you post the code.

OK, looking at the code you posted earlier, the problem is that you have not
defined a copy constructor for your smart pointer.
Something like this is needed
xPointer(const xPointer<X> &oP)
{
obj = oP.obj;
if( obj )
obj->Alloc();
}


Ahh, I've only done the =copy constructor. It works now! I owe you one,
thanks alot :-)
Jul 22 '05 #7

This thread has been closed and replies have been disabled. Please start a new discussion.

Similar topics

16
by: cppaddict | last post by:
Hi, I am deleting some objects created by new in my class destructor, and it is causing my application to error at runtime. The code below compiles ok, and also runs fine if I remove the body...
5
by: Gonçalo Rodrigues | last post by:
Hi all, (note: newbie alert) Suppose you have a hierarchy of objects that you deal with via smart pointers, e.g. something like: template<typename T> class Ref { private:
27
by: Susan Baker | last post by:
Hi, I'm just reading about smart pointers.. I have some existing C code that I would like to provide wrapper classes for. Specifically, I would like to provide wrappers for two stucts defined...
3
by: Vijai Kalyan | last post by:
I have been thinking about this and it may have already been thrashed out and hung out to dry as a topic of no more interest but here goes. I found when implementing a smart pointer that the...
8
by: Axter | last post by:
I normally use a program call Doxygen to document my source code.(http://www.stack.nl/~dimitri/doxygen) This method works great for small and medium size projects, and you can get good...
22
by: craig | last post by:
I've declared the following as a "vector" of "lists" using STL vector< list<Edge*> > I've tried many ways to add objects to this but can't figure out how to do it. Does anyone have any tips,...
16
by: Yong Hu | last post by:
Does the smart pointer never lead to memory leak? Thanks,
92
by: Jim Langston | last post by:
Someone made the statement in a newsgroup that most C++ programmers use smart pointers. His actual phrase was "most of us" but I really don't think that most C++ programmers use smart pointers,...
54
by: Boris | last post by:
I had a 3 hours meeting today with some fellow programmers that are partly not convinced about using smart pointers in C++. Their main concern is a possible performance impact. I've been explaining...
1
by: CloudSolutions | last post by:
Introduction: For many beginners and individual users, requiring a credit card and email registration may pose a barrier when starting to use cloud servers. However, some cloud server providers now...
0
by: Faith0G | last post by:
I am starting a new it consulting business and it's been a while since I setup a new website. Is wordpress still the best web based software for hosting a 5 page website? The webpages will be...
0
isladogs
by: isladogs | last post by:
The next Access Europe User Group meeting will be on Wednesday 3 Apr 2024 starting at 18:00 UK time (6PM UTC+1) and finishing by 19:30 (7.30PM). In this session, we are pleased to welcome former...
0
by: taylorcarr | last post by:
A Canon printer is a smart device known for being advanced, efficient, and reliable. It is designed for home, office, and hybrid workspace use and can also be used for a variety of purposes. However,...
0
by: Charles Arthur | last post by:
How do i turn on java script on a villaon, callus and itel keypad mobile phone
0
by: ryjfgjl | last post by:
If we have dozens or hundreds of excel to import into the database, if we use the excel import function provided by database editors such as navicat, it will be extremely tedious and time-consuming...
0
by: ryjfgjl | last post by:
In our work, we often receive Excel tables with data in the same format. If we want to analyze these data, it can be difficult to analyze them because the data is spread across multiple Excel files...
0
BarryA
by: BarryA | last post by:
What are the essential steps and strategies outlined in the Data Structures and Algorithms (DSA) roadmap for aspiring data scientists? How can individuals effectively utilize this roadmap to progress...
0
by: Hystou | last post by:
There are some requirements for setting up RAID: 1. The motherboard and BIOS support RAID configuration. 2. The motherboard has 2 or more available SATA protocol SSD/HDD slots (including MSATA, M.2...

By using Bytes.com and it's services, you agree to our Privacy Policy and Terms of Use.

To disable or enable advertisements and analytics tracking please visit the manage ads & tracking page.