Problem inserting into Vector

[wutongjoe]

Hi,I had a problem while I was doing the assignment .the code is like
below(not exact):
//=============== =============== ========

class Buyer
{
private:
int _product;

public:
//Buyer(void){ _product=-1;);
int get_product(voi d);
Buyer( int n ){ _product = n;};
};

int Buyer::get_prod uct(void){
return _product;
};

std::vector<Buy er *> buyer_queue;

int main( int argc, char *argv[] )
{
for(;;){
int i=int(1+rand()% 10);
Buyer b=Buyer(i);

buyer_queue.pus h_back(&b);
printf("\nB[%d]",(*buyer_q ueue[buyer_queue.siz e()-1]).get_product() );

for (int j=0;j<buyer_que ue.size();j++){
printf("\t\n looped [%d]",(*buyer_q ueue[j]).get_product() );

}
usleep(1000000) ;
}
}

//=============== =============== ======
the problem is in the for loop,why it outputs the same value for
member"product" of each Buyer?

[red floyd]

wu*******@hotma il.com wrote:

Hi,I had a problem while I was doing the assignment .the code is like
below(not exact):
//=============== =============== ========

class Buyer
{
private:
int _product;

public:
//Buyer(void){ _product=-1;);
int get_product(voi d);
Buyer( int n ){ _product = n;};
};

int Buyer::get_prod uct(void){
return _product;
};

std::vector<Buy er *> buyer_queue;

int main( int argc, char *argv[] )
{
for(;;){
int i=int(1+rand()% 10);
Buyer b=Buyer(i);

buyer_queue.pus h_back(&b);
printf("\nB[%d]",(*buyer_q ueue[buyer_queue.siz e()-1]).get_product() );

for (int j=0;j<buyer_que ue.size();j++){
printf("\t\n looped [%d]",(*buyer_q ueue[j]).get_product() );

}
usleep(1000000) ;
}
}

//=============== =============== ======
the problem is in the for loop,why it outputs the same value for
member"product" of each Buyer?

Don't see any of the "Miranda" functions.

Provide a default constructor, copy constructor, and assignment operator.

Also, you want to call srand() before you call rand().

[DaKoadMunky]

> Buyer b=Buyer(i);

buyer_queue.pus h_back(&b);

The address you are storing is the address of a local variable with automatic
storage duration. That variable is being destroyed at the end of the for loop
in which it is declared and thusly your vector contains a pointer to an object
that has been destroyed.

However, in my runtime environment every pass through the loop results in that
variable being reconstructed at the same memory location as the last pass
through the loop. As a result all the pointers stored in the vector point to
the same object and at the time those pointers are accessed they are infact
pointing to a valid object.

Is there some reason you just didn't use std::vector<Buy er>?

[wutongjoe]

DaKoadMunky wrote:

Buyer b=Buyer(i);

buyer_queue.pus h_back(&b);

The address you are storing is the address of a local variable with automatic
storage duration. That variable is being destroyed at the end of the for loop
in which it is declared and thusly your vector contains a pointer to an object
that has been destroyed.

However, in my runtime environment every pass through the loop results in that
variable being reconstructed at the same memory location as the last pass
through the loop. As a result all the pointers stored in the vector point to
the same object and at the time those pointers are accessed they are infact
pointing to a valid object.

Is there some reason you just didn't use std::vector<Buy er>?

declare buyer as a gloabl variable would do ?if I want to insert many
buyers with random product.

[DaKoadMunky]

>declare buyer as a gloabl variable would do ?if I want to insert many

buyers with random product.

The object passed to vector<T>::push _back will be copied using the T copy
constructor. It is the copy that becomes part of the controlled sequence.

vector<Buyer> buyers;

for(int i=0;i<10;++i)
{
Buyer buyer(i);
buyers.push_bac k(buyer);
}

At this point the vector contains 10 elements that are copies of the elements
that were arguments to vector<T>::push _back.

[Richard Herring]

In message <OC************ *@newssvr14.new s.prodigy.com>, red floyd
<no*****@here.d ude> writes

wu*******@hotm ail.com wrote:

Hi,I had a problem while I was doing the assignment .the code is like
below(not exact):
//=============== =============== ========
class Buyer
{
private:
int _product;
public:
//Buyer(void){ _product=-1;);
int get_product(voi d);
Buyer( int n ){ _product = n;};
};
int Buyer::get_prod uct(void){
return _product;
};
std::vector<Buy er *> buyer_queue;
int main( int argc, char *argv[] )
{
for(;;){
int i=int(1+rand()% 10);
Buyer b=Buyer(i);
buyer_queue.pus h_back(&b);
printf("\nB[%d]",(*buyer_q ueue[buyer_queue.siz e()-1]).get_product() );
for (int j=0;j<buyer_que ue.size();j++){
printf("\t\n looped [%d]",(*buyer_q ueue[j]).get_product() );

}
usleep(1000000) ;
}
}
//=============== =============== ======
the problem is in the for loop,why it outputs the same value for
member"produc t" of each Buyer?
Don't see any of the "Miranda" functions.

Provide a default constructor, copy constructor, and assignment operator.

Why? The class doesn't use pointers or manage resources, so the
compiler-generated copy and assignment will have the correct semantics.
Writing your own, unnecessarily, is just another opportunity to add
errors.
Also, you want to call srand() before you call rand().

The real problem is that every entry in the vector is a pointer to
(effectively) the *same* object, 'b'. So no matter which element of the
vector you look at, you see the last value assigned to b.

--
Richard Herring

[Howard]

<wu*******@hotm ail.com> wrote in message
news:10******** *******@clint.i ts.waikato.ac.n z...

DaKoadMunky wrote:

Buyer b=Buyer(i);

buyer_queue.pus h_back(&b);

The address you are storing is the address of a local variable with automatic storage duration. That variable is being destroyed at the end of the for loop in which it is declared and thusly your vector contains a pointer to an object that has been destroyed.

However, in my runtime environment every pass through the loop results in that variable being reconstructed at the same memory location as the last pass through the loop. As a result all the pointers stored in the vector point to the same object and at the time those pointers are accessed they are infact pointing to a valid object.

Is there some reason you just didn't use std::vector<Buy er>?

declare buyer as a gloabl variable would do ?if I want to insert many
buyers with random product.

No. Simply store Buyer objects instead of pointers to them. Then each
object in the vector will be unique, because push_back will make a copy of
the object and store that. Then when you re-use the object, it will be
different.

Also, you should use

Buyer b(i);

instead of

Buyer b=Buyer(i);

There's no need for the assignment. (Perhaps you were thinking of "Buyer* b
= new Buyer(i);"?)

-Howard

[red floyd]

Richard Herring wrote:

[ reply to my incredibly dumb comment redacted]

You're right... I missed the fact that he was storing *pointers*. I've
been bitten by bad or missing Miranda functions with vectors before, so
that's why I suggested that.