Выделение памяти вектора, тип которого является собственным классом C++

У меня пока мало сообщений о выделении памяти для векторов

( Выделение памяти для класса в C++ Выделение памяти для класса в C++, например), но мне не удалось найти решение проблемы, которую я м столкнулся прямо сейчас.

Поехали.. По словам С.Прата,

tab.push_back(); 

выделяет память для нового объекта, который мы помещаем в вектор с именем «tab». Если это так, нам не нужно указывать длину вектора в объявлении, верно?

vecotr<Type> tab;

Итак, зная эти факты, я хочу поделиться с вами.

инструменты.ч

#pragma once
#include "portfel.h"
#include <string>
#include <memory>

class Spolka;

class Akcja
{
    std::shared_ptr<Spolka> firma;
    double cena_zakupu;
    double cena_aktualna;
public:
    Akcja();
    ~Akcja();
};

class Obligacja
{
    int a;
public:
    Obligacja();
    ~Obligacja();
};

class Kontrakt
{
    int a;
public:
    Kontrakt();
    ~Kontrakt();
};

Теперь идет "portfel.h"

#pragma once
#include <fstream>
#include <vector>
#include <memory>
#include "instrumenty.h"

class Akcja;
class Obligacja;
class Kontrakt;

class Portfel
{
    friend class Inwestor;
    int a;
    std::vector< std::unique_ptr<Akcja> > akcje;
    std::vector< std::unique_ptr<Obligacja> > obligacje;
    std::vector< std::unique_ptr<Kontrakt> > kontrakty;
public:
    Portfel();
    ~Portfel();
    friend std::ofstream& operator<<(std::ofstream& zapisz, Portfel& p1);
};

portfel.cpp (причина проблемы)

Portfel::Portfel()
{
    a=0;
    akcje.push_back(NULL);     //   <------- THIS THING
    //akcje.reserve(class Akcja);  //  <-- How to properly define this according to
                                       portfel.h?
}

На выходе какой-то куст

введите здесь описание изображения

Как правильно выделить память для таких типов векторов, которые состоят из интеллектуальных указателей на собственный определенный класс как тип?

==3238== Invalid read of size 2
==3238==    at 0x409336: operator<<(std::basic_ofstream<char, std::char_traits<char> >&, Inwestor*) (zapis.cpp:13)
==3238==    by 0x4066B5: Inwestor::nowy_profil() (inwestor.cpp:116)
==3238==    by 0x404B00: menu1() (funkcje.cpp:97)
==3238==    by 0x40208E: main (main.cpp:27)
==3238==  Address 0x5a07140 is 8 bytes after a block of size 8 alloc'd
==3238==    at 0x4C2B1C7: operator new(unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==3238==    by 0x408B13: __gnu_cxx::new_allocator<std::unique_ptr<Akcja, std::default_delete<Akcja> > >::allocate(unsigned long, void const*) (new_allocator.h:94)
==3238==    by 0x40892A: std::_Vector_base<std::unique_ptr<Akcja, std::default_delete<Akcja> >, std::allocator<std::unique_ptr<Akcja, std::default_delete<Akcja> > > >::_M_allocate(unsigned long) (in /home/rideofyourlife/Pulpit/DM/all)
==3238==    by 0x4084C3: void std::vector<std::unique_ptr<Akcja, std::default_delete<Akcja> >, std::allocator<std::unique_ptr<Akcja, std::default_delete<Akcja> > > >::_M_emplace_back_aux<std::unique_ptr<Akcja, std::default_delete<Akcja> > >(std::unique_ptr<Akcja, std::default_delete<Akcja> >&&) (vector.tcc:405)
==3238==    by 0x408234: void std::vector<std::unique_ptr<Akcja, std::default_delete<Akcja> >, std::allocator<std::unique_ptr<Akcja, std::default_delete<Akcja> > > >::emplace_back<std::unique_ptr<Akcja, std::default_delete<Akcja> > >(std::unique_ptr<Akcja, std::default_delete<Akcja> >&&) (vector.tcc:102)
==3238==    by 0x407E5D: std::vector<std::unique_ptr<Akcja, std::default_delete<Akcja> >, std::allocator<std::unique_ptr<Akcja, std::default_delete<Akcja> > > >::push_back(std::unique_ptr<Akcja, std::default_delete<Akcja> >&&) (stl_vector.h:900)
==3238==    by 0x407AE2: Portfel::Portfel() (portfel.cpp:6)
==3238==    by 0x405D07: Inwestor::Inwestor() (inwestor.cpp:13)
==3238==    by 0x402126: __static_initialization_and_destruction_0(int, int) (main.cpp:14)
==3238==    by 0x402161: _GLOBAL__sub_I_q (main.cpp:31)
==3238==    by 0x40985C: __libc_csu_init (in /home/rideofyourlife/Pulpit/DM/all)
==3238==    by 0x536D6FF: (below main) (libc-start.c:185)

person user1960582    schedule 07.05.2013    source источник
comment
Как вы объявляете свой объект Portfel? Что вы делаете в конструкторе Inwestor?   -  person Some programmer dude    schedule 07.05.2013
comment
В Inwestor::Inwestor делаю Portfel p1;   -  person user1960582    schedule 07.05.2013
comment
Вставленная вами ошибка возникает, когда Inwestor::nowy_profil() вызывает operator<<. Пожалуйста, покажите этот метод.   -  person Useless    schedule 07.05.2013
comment
Где вектор, тип которого является его собственным классом?   -  person M.M    schedule 10.06.2014


Ответы (1)


Чтобы добавить указатель на новый Akcja в вектор, вы должны сказать

akcje.push_back(new Akcja);

Обычно вам вообще не нужно управлять vector памятью — для этого и предназначен класс.

В частности, reserve используется, чтобы убедиться, что есть место для всех элементов, когда вы заранее знаете, сколько вам понадобится, чтобы вы могли сэкономить время, необходимое для перераспределения и копирования вектора в новое место по мере его роста или для избежать фрагментации памяти (о которой нужно беспокоиться очень редко).
Она не используется для добавления элементов.

person molbdnilo    schedule 07.05.2013