multimap

[Гость]

Здравствуйте.
Начал изучать С++ - подскажите может можно как то более изящно все это сделать?

Код: Выделить всё

get(std::multimap<std::string, msg>* msg_pool, std::string id_thread)
{
	typedef std::multimap<std::string, msg>::iterator iterator;
	std::pair< iterator, iterator > pos;
	std::pair< iterator, iterator > pos_old;
	

	pos = _pool.equal_range( id_thread );
	
	pos_old = pos;

	for (; pos.first != pos.second; pos.first++ )
	{
		msg_pool->insert(std::make_pair(pos.first->first, pos.first->second));
	}

	_pool.erase(pos_old.first, pos_old.second);

....................................................................

[Хостинг HostFood.ru]

Тарифы на хостинг в России, от 12 рублей: https://www.host-food.ru/tariffs/hosting/
Тарифы на виртуальные сервера (VPS/VDS/KVM) в РФ, от 189 руб.: https://www.host-food.ru/tariffs/virtualny-server-vps/
Выделенные сервера, Россия, Москва, от 2000 рублей (HP Proliant G5, Intel Xeon E5430 (2.66GHz, Quad-Core, 12Mb), 8Gb RAM, 2x300Gb SAS HDD, P400i, 512Mb, BBU):
https://www.host-food.ru/tariffs/vydelennyi-server-ds/
Недорогие домены в популярных зонах: https://www.host-food.ru/domains/

[Fastman]

>>Начал изучать С++
и
>>typedef std::multimap<std::string, msg>::iterator iterator;

Плохо начал. Не усложняй себе жизнь.
И напиши для начала чего ты хочешь получить.

[Гость]

Хочу из одной map перенести в другую, удовлетворяющие определенному условию

[Dambo]

Нда прочитайте для начало описание того что вы используете... А именно по multimap и qual_range.
Вообще multimap хорош в кучах сортированных, в вашем коде вы перетаскиваете почему то 1 элемент и убиваете весь диапазон(думаю ваша ошибка)... если хотите так делать почему не vector?
Код который перетаскивает кучу удовлетворяющую ключу и грохает её:

Код: Выделить всё

get(std::multimap<std::string, msg>* msg_pool, std::string id_thread)
{
   typedef std::multimap<std::string, msg>::iterator iterator;
   std::pair< iterator, iterator > pos;   

   pos = _pool.equal_range( id_thread );

   msg_pool->insert(pos.first, pos.second);

   _pool.erase(pos.first, pos.second);

....................................................................

После erase, pos указывает на мусор перед употреблением его, надо присвоить другое ему, если он дальше по коду используется....