Vector Initialisation in C++

Question

I am using Vectors in my code.

The line that is causing the error is as follows :

vector alt_seq ;
alt_seq = vector(1000);
for(in         


        

Answer 1

I can see one problem; when you declare 1000 vector objects and then do a push_back you're adding to the end of the vector, i.e. the objects that you're trying to add are at 1001, 1002.... etc. If declaring 1000 vector objects doesn't give you a runtime error, I would first see what's the default definition (since the first 1000 node objects have default values), since that's not error out and compare against the actual data I'm trying to load. Hope this helps.

Answer 2

Your Node class has a pointer member of type CombinedFragment* called cfrag. This creates dangling references if you don't defined your own copy ctor and assignment operators. Further, a proper dtor is required to prevent leaks if Node is responsible for allocating/deallocating cfrag. If you don't want to deep-copy CombinedFragment you can use a shared_ptr.

Also, the default ctor for Node probably doesn't even need to be there (it leaves cfrag to an uninitialized state).