Re: vector<> pointer

From:

red floyd <redfloyd@gmail.com>

Newsgroups:

comp.lang.c++

Date:

Fri, 24 Sep 2010 17:44:10 -0700 (PDT)

Message-ID:

<d2fb1715-ab24-41f1-9bbe-7e5f009ddd71@l32g2000prn.googlegroups.com>

On Sep 24, 5:05 pm, Bob <roberto.pagli...@gmail.com> wrote:

Hi All,
I defined the following class:

class Graph {
public:
    Graph() { }
    int genRandGeomGraph(int i_nodes, double i_radius);
    void print();
    ~Graph() {
        if( NodeList ) delete [] NodeList;
    }
private:
    class Node {
    public:
        Node() {}
        Node(int inID) { this->ID = inID; }
        inline int getID() {return this->ID;}
        inline void setID(int newID) { this->ID = newID; }
        inline void setx(double inx) { this->x = inx; }
        inline double getx() { return this->x; }
        inline void sety(double iny) { this->y = iny; }
        inline double gety() { return this->y; }
        ~Node() { if ( neighbors ) delete [] neighbors; }
    private:
        double x, y;
        int ID;
        std::vector<int>* neighbors;

Java-ism. Should be std::vector<int> neighbors;

};
std::vector<Node>* NodeList;

Java-ism. Should be std::vector<Node> NodeList;

};

whenever I try to access NodeList from genRandGeomGraph(...)
int Graph::genRandGeomGraph((int i_nodes, double i_radius) {
NodeList = new vector< Node >(nnodes);

No such variable as nnodes.
Should be:
NodeList.resize(nnodes);

std::cout << NodeList->size();
return 1;

}

I get segmentation fault. Shouldn't I be able to access it?

Also, as a nitpick, the hungarian notation should be completely
dropped.