Re: Me again with same problem.

From:

"Jim Langston" <tazmaster@rocketmail.com>

Newsgroups:

comp.lang.c++

Date:

Sun, 28 May 2006 17:47:42 -0700

Message-ID:

<Swreg.333$0l.208@fe04.lga>

"JoeC" <enki034@yahoo.com> wrote in message
news:1148862143.613998.32370@y43g2000cwc.googlegroups.com...

Sorry, I got fustrated and careless in my attempts to make this work.

I made some changes that should work but my program crashes.

class player{
string name;
vector<BYTE>gData;
void create();

public:
   player();
   BYTE data(const int);
   void dummy(){MessageBox(NULL, "Dummy" , "Notice", MB_OK); }
};

BYTE player::data(const int n){return gData[n];}

class space{

player * play;
....

graphic& space::graphicOut(){

    if(play){
     vector<BYTE>t;
     for(int lp = 0; lp != 32; lp++){
       t.push_back(play->data(lp));
       }

what is play->data(lp) supposed to be giving you? It would be nice if you
actually showed us what BYTE data( const int); was doing, as that's where
your problem might be.
Also, what is BYTE defined as? I could presume unsigned char, but I might
be presuming wrong.

How do you know how much data the vector? Are there actually 32 values?
Are you sure? How do you know? How do we know?

Why do you need to copy it anyway? Why don't you just return a pointer or a
reference to it?

    cgr = new graphic(t);
    return *cgr;
    }
    //if(play){return *cgr;}
    if(seen){return *gr;}
       else {return *grDefault;}
    }

What I am trying to do is read the vector from my player into a new
vector in my graphicOut function. My problem is my program keeps
crashing when I try to get data from my player object.