Question

Let’s assume I have a singleton class:

class Singleton {
public:
    static Singleton* getInstance();
    void doit();
    std::vector<Object*>& getVector();

private:
    std::vector<Object*> _vector;
    static Singleton *instance;
    Singleton();
    ~Singleton();
    Singleton(const Singleton&);
};

class Delegator {
public:
        void main();
}

• In the doit method I populate the _vector with pointers to objects.
• In the main() from the Delegator class, I call getVector() and display the results.

Given this abstraction, I have the following questions:

1. Can I delete all the pointers to instances of Object in the main() from Delegator (after displaying the results). If yes, is this recommended?
2. Does the singleton ever get destroyed? In other words, will the reference returned in getVector() always be valid?
3. I return a reference to a vector instead of a copy of the vector in getVector(). Given that the vector only contains pointers to objects and will not modify the vector content outside the Singleton class, is there any efficiency gained from returning a reference?

Thank you in advance

Answer 1

My first answer to the question you didn’t ask is this:

Do not use Singleton. It’s an anti-pattern. It makes the code it touches worse. It kills testability and makes it harder to modify your program in the future.

This really excellent article talks in great detail about why Singleton is a bad idea.

To answer the questions you did ask…

1. Yes, you can. Of course, those pointers will still be in the vector so the vector will now contain a ton of dangling pointers. So you ought to clear out the vector too.
2. With the code you have it will never be destroyed. By some definitions that makes your Singleton a memory leak. So yes, the reference will always be valid.
3. Yes, there is a huge performance improvement to returning a reference. Copying a vector means making a copy of all the elements in the vector. Note that this means all the pointers will be copied, not the Objects they point to. Additionally, if you return a copy and have the main function delete all the pointers you are absolutely guaranteed to have a vector lying around with a whole ton of dangling pointers in it.