vector.resize assignment operator problem

[Bram Kuijper]

Hi all,

I am trying to resize a vector of objects (MyObj below), which contain
references to other objects (OtherObj, see below). However, apparently
somewhere in the resize operation an assignment is done of the
referenced OtherObj object (according to the compiler messages). This is
strange, since the referenced OtherObj is initialized using member
initialization lists. Anyone a clue?

thanks,

Bram Kuijper

this is the source. Compiler messages (g++ 4.1.3) are listed below.
#include<vector >

using namespace std;
class OtherObj
{
public:
OtherObj()
{}
};

class MyObj
{
private:
OtherObj const &objref;

public:
MyObj(OtherObj const &obj)
:
objref(obj)
{}

MyObj(MyObj const &other)
:
objref(other.ob jref)
{}
};

int main()
{
OtherObj obj = OtherObj(); //fine

vector<MyObjfir st(3, MyObj(obj)); //okay

first.resize(10 , MyObj(obj)); //ERROR

return 0;
}

test.cpp: In member function ‘MyObj& MyObj::operator =(const MyObj&)’:
test.cpp:14: instantiated from ‘static void std::__fill<<an onymous>

>::fill(_Forwar dIterator, _ForwardIterato r, const _Tp&) [with

_ForwardIterato r = __gnu_cxx::__no rmal_iterator<M yObj*,
std::vector<MyO bj, std::allocator< MyObj >, _Tp = MyObj, bool
<anonymous= false]’
/usr/include/c++/4.1.3/bits/stl_algobase.h: 568: instantiated from
‘void std::fill(_Forw ardIterator, _ForwardIterato r, const _Tp&) [with
_ForwardIterato r = __gnu_cxx::__no rmal_iterator<M yObj*,
std::vector<MyO bj, std::allocator< MyObj >, _Tp = MyObj]’
/usr/include/c++/4.1.3/bits/vector.tcc:330: instantiated from ‘void
std::vector<_Tp ,
_Alloc>::_M_fil l_insert(__gnu_ cxx::__normal_i terator<typenam e
std::_Vector_ba se<_Tp, _Alloc>::_Tp_al loc_type::point er,
std::vector<_Tp , _Alloc, size_t, const _Tp&) [with _Tp = MyObj,
_Alloc = std::allocator< MyObj>]’
/usr/include/c++/4.1.3/bits/stl_vector.h:65 8: instantiated from ‘void
std::vector<_Tp , _Alloc>::insert (__gnu_cxx::__n ormal_iterator< typename
std::_Vector_ba se<_Tp, _Alloc>::_Tp_al loc_type::point er,
std::vector<_Tp , _Alloc, size_t, const _Tp&) [with _Tp = MyObj,
_Alloc = std::allocator< MyObj>]’
/usr/include/c++/4.1.3/bits/stl_vector.h:42 6: instantiated from ‘void
std::vector<_Tp , _Alloc>::resize (size_t, _Tp) [with _Tp = MyObj, _Alloc
= std::allocator< MyObj>]’
test.cpp:36: instantiated from here
test.cpp:14: error: non-static reference member ‘const OtherObj&
MyObj::objref’, can't use default assignment operator

*** Especially this error message above is puzzling me ***

/usr/include/c++/4.1.3/bits/stl_algobase.h: In static member function
‘static void std::__fill<<an onymous::fill(_ ForwardIterator ,
_ForwardIterato r, const _Tp&) [with _ForwardIterato r =
__gnu_cxx::__no rmal_iterator<M yObj*, std::vector<MyO bj,
std::allocator< MyObj >, _Tp = MyObj, bool <anonymous= false]’:
/usr/include/c++/4.1.3/bits/stl_algobase.h: 529: note: synthesized method
‘MyObj& MyObj::operator =(const MyObj&)’ first required here

[Victor Bazarov]

Bram Kuijper wrote:

Hi all,

I am trying to resize a vector of objects (MyObj below), which contain
references to other objects (OtherObj, see below). However, apparently
somewhere in the resize operation an assignment is done of the
referenced OtherObj object (according to the compiler messages). This
is strange, since the referenced OtherObj is initialized using member
initialization lists. Anyone a clue?

One of the requirements for the contained type ('MyObj' in your case) is
that it is *Assignable*. You need to provide your user-defined operator
if the compiler is unable to generate one for you.

It is up to you how you do it. You will not be able to re-seat the
reference member to refer to the other object's referee, so if you need
the assignment to change that, you should consider storing a pointer
to 'OtherObj' in 'MyObj' instead.

[..]

V
--
Please remove capital 'A's when replying by e-mail
I do not respond to top-posted replies, please don't ask

[Markus Moll]

Hi

Bram Kuijper schreef:

Hi all,

I am trying to resize a vector of objects (MyObj below), which contain
references to other objects (OtherObj, see below). However, apparently
somewhere in the resize operation an assignment is done of the
referenced OtherObj object (according to the compiler messages). This is
strange, since the referenced OtherObj is initialized using member
initialization lists. Anyone a clue?

thanks,

You are violating the container requirements:

"The type of objects stored in these components must meet the
requirements of CopyConstructib le types (20.1.3), and the additional
requirements of Assignable types."

Markus

[Jim Langston]

"Bram Kuijper" <a.***********@ rug.nlwrote in message
news:f4******** **@info.service .rug.nl...

Hi all,

I am trying to resize a vector of objects (MyObj below), which contain
references to other objects (OtherObj, see below). However, apparently
somewhere in the resize operation an assignment is done of the referenced
OtherObj object (according to the compiler messages). This is strange,
since the referenced OtherObj is initialized using member initialization
lists. Anyone a clue?

thanks,

Bram Kuijper

this is the source. Compiler messages (g++ 4.1.3) are listed below.
#include<vector >

using namespace std;
class OtherObj
{
public:
OtherObj()
{}
};

class MyObj
{
private:
OtherObj const &objref;

public:
MyObj(OtherObj const &obj)
:
objref(obj)
{}

MyObj(MyObj const &other)
:
objref(other.ob jref)
{}
};

int main()
{
OtherObj obj = OtherObj(); //fine

vector<MyObjfir st(3, MyObj(obj)); //okay

first.resize(10 , MyObj(obj)); //ERROR

return 0;
}

[snip bunch of error deetail]

test.cpp:14: error: non-static reference member ‘const OtherObj&
MyObj::objref’, can't use default assignment operator

OtherObj& is a reference. A reference in your class can't be assigned by
the default assignment operator. References in a class need to be
initialized, not assigned, that's where the problem is coming in.

*** Especially this error message above is puzzling me ***

/usr/include/c++/4.1.3/bits/stl_algobase.h: In static member function
‘static void std::__fill<<an onymous::fill(_ ForwardIterator ,
_ForwardIterato r, const _Tp&) [with _ForwardIterato r =
__gnu_cxx::__no rmal_iterator<M yObj*, std::vector<MyO bj,
std::allocator< MyObj >, _Tp = MyObj, bool <anonymous= false]’:
/usr/include/c++/4.1.3/bits/stl_algobase.h: 529: note: synthesized method
‘MyObj& MyObj::operator =(const MyObj&)’ first required here