Problem with inserting custom types into STL map

[uartem]

Hello, this is my problem:

I have a map:

typedef map<string, bfdb_recordBFDB ;

where:

typedef struct
{
size_t seq_no;
StdBloomFilter bf;
} bfdb_record;

where:

class StdBloomFilter : public BloomFilter
{
private:
vector<bool__fi lter;
size_t __size;
public:
StdBloomFilter( ):__filter(0),_ _size(0){};
StdBloomFilter( size_t _size): __filter(_size) , __size(_size) {};
StdBloomFilter( const StdBloomFilter& _source);

virtual ~StdBloomFilter (){};

operator CountingBloomFi lter();
StdBloomFilter& operator=(const StdBloomFilter& );

void set(size_t _pos);
bool test(size_t _pos) const;

void clear();
size_t size() const;
void merge(const StdBloomFilter& _source);

string to_string();
};

now I want to insert structure into my map:

__db.insert(mak e_pair(_id,_inf o));
or
__db[_id] = _info;
or
__db.insert(BFD B::value_type(_ id, _info));

I get a runtime error:

8 [main] LE 2560 _cygtls::handle _exceptions: Exception:
STATUS_ACCESS_V IOLATION
985 [main] LE 2560 open_stackdumpf ile: Dumping stack trace to
LE.exe.stackdum p

Is something in StdBloomFilter missing. If I change bf to int type in
the bfdb_record struct then everything works, but that is not what I
need!!!

Please help

[Ian Collins]

ua****@gmail.co m wrote:

Hello, this is my problem:

I have a map:

typedef map<string, bfdb_recordBFDB ;

where:

typedef struct
{
size_t seq_no;
StdBloomFilter bf;
} bfdb_record;

That's C. You want

struct bfdb_record
{
size_t seq_no;
StdBloomFilter bf;
};

Are you sure you want to copy the entire StdBloomFilter object each time
you do an insert?

where:

class StdBloomFilter : public BloomFilter
{
private:
vector<bool__fi lter;
size_t __size;

Those are illegal names, anything starting with double underscore is
reserved.

>
now I want to insert structure into my map:

__db.insert(mak e_pair(_id,_inf o));
or
__db[_id] = _info;
or
__db.insert(BFD B::value_type(_ id, _info));

I get a runtime error:

What does your StdBloomFilter copy constructor do?

--
Ian Collins.

[uartem]

StdBloomFilter& StdBloomFilter: :operator=(cons t StdBloomFilter&
_source)
{
this->__size=_source .__size;
this->__filter =_source.__filt er;
}

And what should it do? These are no pointers. These should be copies
by value, right? It was not working, so I added this dummy copy
constructor, but it still does not work.

[Ian Collins]

ua****@gmail.co m wrote:

Please quote the appropriate context, this post makes no sense on its own.

StdBloomFilter& StdBloomFilter: :operator=(cons t StdBloomFilter&
_source)
{
this->__size=_source .__size;
this->__filter =_source.__filt er;

You realy should get rid of these illegal names.

}

And what should it do? These are no pointers. These should be copies
by value, right? It was not working, so I added this dummy copy
constructor, but it still does not work.

Run you code under a debugger and see what it is doing. You don't show
the base class, so the problem may well be in there.

--
Ian Collins.

[uartem]

On Apr 2, 4:29 pm, Ian Collins <[e-mail] mwrote:

[e-mail] m wrote:

Please quote the appropriate context, this post makes no sense on its own.

StdBloomFilter& StdBloomFilter: :operator=(cons t StdBloomFilter&
_source)
{
this->__size=_source .__size;
this->__filter =_source.__filt er;

You realy should get rid of these illegal names.

}

And what should it do? These are no pointers. These should be copies
by value, right? It was not working, so I added this dummy copy
constructor, but it still does not work.

Run you code under a debugger and see what it is doing. You don't show
the base class, so the problem may well be in there.

--
Ian Collins.

I can put StdBloomFilter object into a vector<StdBloom Filterand I
tried to get rid of any base class - does not help. Base class is an
abstract interface.

[Ian Collins]

ua****@gmail.co m wrote:

On Apr 2, 4:29 pm, Ian Collins <[e-mail] mwrote:

>>uar...@gmail. com wrote:

Please quote the appropriate context, this post makes no sense on its own.

>>>StdBloomFilt er& StdBloomFilter: :operator=(cons t StdBloomFilter&
_source)
{
this->__size=_source .__size;
this->__filter =_source.__filt er;

You realy should get rid of these illegal names.

>>>}

>>>And what should it do? These are no pointers. These should be copies
by value, right? It was not working, so I added this dummy copy
constructo r, but it still does not work.

Run you code under a debugger and see what it is doing. You don't show
the base class, so the problem may well be in there.

*Please* don't quote signatures - the bit after the "-- ".

>
I can put StdBloomFilter object into a vector<StdBloom Filterand I
tried to get rid of any base class - does not help. Base class is an
abstract interface.

What did your debugger show?

--
Ian Collins.

[uartem]

On Apr 2, 4:40 pm, Ian Collins <[e-mail] mwrote:

[e-mail] m wrote:

On Apr 2, 4:29 pm, Ian Collins <[e-mail] mwrote:

>uar...@gmail.c om wrote:

>Please quote the appropriate context, this post makes no sense on its own.

>>StdBloomFilte r& StdBloomFilter: :operator=(cons t StdBloomFilter&
_source)
{
this->__size=_source .__size;
this->__filter =_source.__filt er;

>You realy should get rid of these illegal names.

>>}

>>And what should it do? These are no pointers. These should be copies
by value, right? It was not working, so I added this dummy copy
constructor , but it still does not work.

>Run you code under a debugger and see what it is doing. You don't show
the base class, so the problem may well be in there.

*Please* don't quote signatures - the bit after the "-- ".

I can put StdBloomFilter object into a vector<StdBloom Filterand I
tried to get rid of any base class - does not help. Base class is an
abstract interface.

What did your debugger show?

--
Ian Collins.

This is what I can catch:

mi_cmd_var_crea te: unable to create variable object
Single stepping until exit from function ntdll!
LdrDisableThrea dCalloutsForDll ,
which has no line number information.
Single stepping until exit from function ntdll!
LdrFindCreatePr ocessManifest,
which has no line number information.
Single stepping until exit from function ntdll!RtlCheckR egistryKey,
which has no line number information.

I compile it in gcc. I installed cygwin on a Windows system and I
debug it in Eclipse. So, this is all I managed to get out of it.

[Ian Collins]

ua****@gmail.co m wrote:

On Apr 2, 4:40 pm, Ian Collins <[e-mail] mwrote:

>>uar...@gmail. com wrote:

>>>On Apr 2, 4:29 pm, Ian Collins <[e-mail] mwrote:

>>>>uar...@gmai l.com wrote:

>>>>Please quote the appropriate context, this post makes no sense on its own.

>>>>>StdBloomFi lter& StdBloomFilter: :operator=(cons t StdBloomFilter&
>_source)
>{
this->__size=_source .__size;
this->__filter =_source.__filt er;

>>>>You realy should get rid of these illegal names.

>>>>>}

>>>>>And what should it do? These are no pointers. These should be copies
>by value, right? It was not working, so I added this dummy copy
>constructo r, but it still does not work.

>>>>Run you code under a debugger and see what it is doing. You don't show
the base class, so the problem may well be in there.

*Please* don't quote signatures - the bit after the "-- ".

>>>I can put StdBloomFilter object into a vector<StdBloom Filterand I
tried to get rid of any base class - does not help. Base class is an
abstract interface.

What did your debugger show?

*Please* *Please* don't quote signatures - the bit after the "-- ".

>
This is what I can catch:

I compile it in gcc. I installed cygwin on a Windows system and I
debug it in Eclipse. So, this is all I managed to get out of it.

You should be able to see the last line in your code.

I assume you have implemented the assignment operator as well?

--
Ian Collins.

[uartem]

On Apr 2, 4:52 pm, Ian Collins <[e-mail] mwrote:

[e-mail] m wrote:

On Apr 2, 4:40 pm, Ian Collins <[e-mail] mwrote:

>uar...@gmail.c om wrote:

>>On Apr 2, 4:29 pm, Ian Collins <[e-mail] mwrote:

>>>uar...@gmail .com wrote:

>>>Please quote the appropriate context, this post makes no sense on its own.

>>>>StdBloomFil ter& StdBloomFilter: :operator=(cons t StdBloomFilter&
_source)
{
this->__size=_source .__size;
this->__filter =_source.__filt er;

>>>You realy should get rid of these illegal names.

>>>>}

>>>>And what should it do? These are no pointers. These should be copies
by value, right? It was not working, so I added this dummy copy
constructor , but it still does not work.

>>>Run you code under a debugger and see what it is doing. You don't show
the base class, so the problem may well be in there.

>*Please* don't quote signatures - the bit after the "-- ".

>>I can put StdBloomFilter object into a vector<StdBloom Filterand I
tried to get rid of any base class - does not help. Base class is an
abstract interface.

>What did your debugger show?

*Please* *Please* don't quote signatures - the bit after the "-- ".

This is what I can catch:

I compile it in gcc. I installed cygwin on a Windows system and I
debug it in Eclipse. So, this is all I managed to get out of it.

You should be able to see the last line in your code.

I assume you have implemented the assignment operator as well?

--
Ian Collins.

Well the assignment is smth like:
bool BFDatabase::upd ate(const string& _id, const bfdb_record& _info)
{
BFDB::iterator iter = __db.find(_id);

if (iter==__db.end ())
{
//__db.insert(mak e_pair(_id,_inf o));
__db[_id] = _info;
//__db.insert(BFD B::value_type(_ id, _info));
return true;
}
return false;
}

But the problem is the following:

This works:
StdBloomFilter bf(100);
vector<StdBloom Filtervec;
vec.push_back(b f);

And this does not:
map<int,StdBloo mFilterm;
m.insert(make_p air(5,bf));

[Ralph D. Ungermann]

ua****@gmail.co m wrote:

StdBloomFilter& StdBloomFilter: :operator=(cons t StdBloomFilter&
_source)

[...]

It was not working, so I added this dummy copy
constructor,

This is not the copy ctor. Show us your

StdBloomFilter: :StdBloomFilter ( const StdBloomFilter& );