Вектор с указателями на объекты

@exkilla · Регистрация: 28.12.2009

Author24 — интернет-сервис помощи студентам

Всем привет! Решил разобраться с векторами. Написал программку. На выводе ожидал увидеть 10 случайных числ. Но получаю краш приложения. Подскажите пожалуйста, что я делаю не так.

Код

#include <iostream>
#include <vector>
using namespace std;

class Xray
{
	public:
		int id;
};

int main()
{
	Xray * oxi;
	vector<Xray*> v(10);

	for(int i = 0; i < 10; i++)
	{
		oxi = new Xray;
		oxi->id = rand();
		v.insert(v.end(), oxi);
	}

	for(vector<Xray*>::iterator it = v.begin(); it!=v.end(); it++)
		cout << (*it)->id << endl;
}

@ValeryLaptev · 03.05.2011, 20:02

вот:
v.insert(v.end(), oxi);
Поставь просто v[i] = oxi;

@neske · 03.05.2011, 20:09

Или измените 14 строку на:

C++

1	vector<Xray*> v;

@ValeryLaptev · 03.05.2011, 20:11

Сообщение от neske

Или измените 14 строку на:

C++

1	vector<Xray*> v;

И так можно.
Тогда вместо v.insert() гораздо лучше ставить v.push_back().

@neske · 03.05.2011, 20:11

Да, вы правы, push_back в таком случае будет лучше смотреться.

@rangerx · 03.05.2011, 20:19

Не по теме:

Сообщение от exkilla

for(vector<Xray*>::iterator it = v.begin(); it!=v.end(); it++)

for(vector<Xray*>::iterator it = v.begin(); it!=v.end(); ++it)
для НЕ встроенных типов это принципиально

@exkilla · 03.05.2011, 20:28 **[ТС]**

Спасибо. Работает.

После того, как мне уже не будут нужны объекты, каким образом мне освободить занимаемую память? Просто вызвать метод clear?

Сообщение от rangerx

Не по теме:

for(vector<Xray*>::iterator it = v.begin(); it!=v.end(); ++it)
для НЕ встроенных типов это принципиально

Объясните пожалуйста почему.

Сообщение от neske

Или измените 14 строку на:

C++

1	vector<Xray*> v;

На сколько элементов резервируется место в данном случае?

@ValeryLaptev · 03.05.2011, 20:30

Сообщение от exkilla

На сколько элементов резервируется место в данном случае?

Там внутри - зависит от реализации. Но для того, кто использует - элементов нет. Нужно добавлять каждый элемент операцией v.push_back(элемент);

@rangerx · 03.05.2011, 20:55

Сообщение от exkilla

После того, как мне уже не будут нужны объекты, каким образом мне освободить занимаемую память? Просто вызвать метод clear?

Нет, вызвать delete для каждого элемента.

Сообщение от exkilla

Объясните пожалуйста почему.

Потому что постфиксная запись предполагает создание временного объекта.

Сообщение от exkilla

На сколько элементов резервируется место в данном случае?

Для тебя, как для пользователя контейнером, в нём ровно v.size() элементов )

@exkilla · 03.05.2011, 21:12 **[ТС]**

Сообщение от rangerx

Нет, вызвать delete для каждого элемента.

Верно?

Код

	for(vector<Xray*>::iterator it = v.begin(); it!=v.end(); ++it)
		delete (*it);

Сообщение от rangerx

Потому что постфиксная запись предполагает создание временного объекта.

Спасибо.

Сообщение от rangerx

Для тебя, как для пользователя контейнером, в нём ровно v.size() элементов )

Верно, но я имел в виду v.capacity() =) Проверил. После создания вектора равно 0.

@alex_x_x · 03.05.2011, 21:17

Сообщение от exkilla

Верно, но я имел в виду v.capacity() =) Проверил. После создания вектора равно 0.

даже это не означает что там не выделена память

@exkilla 3 / 3 / 1 Регистрация: 28.12.2009 Сообщений: 73
		1
	Вектор с указателями на объекты 03.05.2011, 19:57. Показов 23122. Ответов 10 Метки нет (Все метки) Всем привет! Решил разобраться с векторами. Написал программку. На выводе ожидал увидеть 10 случайных числ. Но получаю краш приложения. Подскажите пожалуйста, что я делаю не так. Код #include <iostream> #include <vector> using namespace std; class Xray { public: int id; }; int main() { Xray * oxi; vector<Xray> v(10); for(int i = 0; i < 10; i++) { oxi = new Xray; oxi->id = rand(); v.insert(v.end(), oxi); } for(vector<Xray>::iterator it = v.begin(); it!=v.end(); it++) cout << (*it)->id << endl; } 0

@ValeryLaptev 1069 / 848 / 60 Регистрация: 30.04.2011 Сообщений: 1,659
	03.05.2011, 20:02	2
	вот: v.insert(v.end(), oxi); Поставь просто v[i] = oxi; 1

@neske 1552 / 918 / 193 Регистрация: 26.03.2010 Сообщений: 3,105
	03.05.2011, 20:11	5
	Да, вы правы, push_back в таком случае будет лучше смотреться. 1

@rangerx
	03.05.2011, 20:19 #6
	Не по теме: Сообщение от exkilla for(vector<Xray>::iterator it = v.begin(); it!=v.end(); it++) for(vector<Xray>::iterator it = v.begin(); it!=v.end(); ++it) для НЕ встроенных типов это принципиально 1

@ValeryLaptev 1069 / 848 / 60 Регистрация: 30.04.2011 Сообщений: 1,659
	03.05.2011, 20:30	8
	Сообщение от exkilla На сколько элементов резервируется место в данном случае? Там внутри - зависит от реализации. Но для того, кто использует - элементов нет. Нужно добавлять каждый элемент операцией v.push_back(элемент); 0

@rangerx 2022 / 1621 / 489 Регистрация: 31.05.2009 Сообщений: 3,005
	03.05.2011, 20:55	9
	Сообщение от exkilla После того, как мне уже не будут нужны объекты, каким образом мне освободить занимаемую память? Просто вызвать метод clear? Нет, вызвать delete для каждого элемента. Сообщение от exkilla Объясните пожалуйста почему. Потому что постфиксная запись предполагает создание временного объекта. Сообщение от exkilla На сколько элементов резервируется место в данном случае? Для тебя, как для пользователя контейнером, в нём ровно v.size() элементов ) 1

@exkilla 3 / 3 / 1 Регистрация: 28.12.2009 Сообщений: 73
	03.05.2011, 21:12 [ТС]	10
	Сообщение от rangerx Нет, вызвать delete для каждого элемента. Верно? Код for(vector<Xray>::iterator it = v.begin(); it!=v.end(); ++it) delete (it); Сообщение от rangerx Потому что постфиксная запись предполагает создание временного объекта. Спасибо. Сообщение от rangerx Для тебя, как для пользователя контейнером, в нём ровно v.size() элементов ) Верно, но я имел в виду v.capacity() =) Проверил. После создания вектора равно 0. 0

@alex_x_x бжни 2473 / 1684 / 135 Регистрация: 14.05.2009 Сообщений: 7,162
	03.05.2011, 21:17	11
	Сообщение от exkilla Верно, но я имел в виду v.capacity() =) Проверил. После создания вектора равно 0. даже это не означает что там не выделена память 0