1

Проблема при копировании с помощью copy() Объявление:

copy(it, it, back_inserter(sell_medic));

Где it:

list<Medicine>::iterator it = medic.begin();
advance(it, id - 1);

Метод вообще не копирует в нужный list <Medicine> sell_medic;

3
  • Что именно вы хотели скопировать? Что значит it, it? 17 янв 2019 в 21:26
  • sell_medic.push_back(*it) и все дела 17 янв 2019 в 21:35
  • Почему вы откатили правку, уточняющую заголовок вопроса? Сейчас ваш заголовок состоит из 3 используемых меток и предложения "проблема копирования", которое носит крайне общий характер. Человеку, который столкнулся с подобной проблемой, по текущему заголовку будет сложно найти ваш вопрос 19 янв 2019 в 13:30

2 ответа 2

4

В алгоритмах стандартной библиотеки пара итераторов представляет диапазон - [begin, end[ (end указывает на элемент за последним), поэтому диапазон it, it будет всегда представлять пустое множество. Если вы хотели скопировать с начала до it (но не включая элемент, на который it указывает) то код должен быть:

copy(medic.begin(), it, back_inserter(sell_medic));

А если мне надо скопировать конкретный элемент, на который указывает этот итератор?

Тогда вам вообще не нужен std::copy:

sell_medic.push_back( *it );

но если сильно хочется:

std::copy(it, std::next(it), std::back_inserter(sell_medic));

ЗЫ этот код излишне многословен:

list<Medicine>::iterator it = medic.begin();
advance(it, id - 1);

проще написать так:

auto it = std::next( medic.begin(), id - 1 );

так и проще читать и короче std::next()

4
  • А если мне надо скопировать конкретный элемент, на который указывает этот итератор?
    – Norfo4ik
    17 янв 2019 в 21:27
  • std::copy(it, it + 1, back_inserter(sell_medic)); но если вам всегда нужно копировать только один элемент в контейнер - не совсем понятно, зачем вам std::copy.
    – user302909
    17 янв 2019 в 21:29
  • @ARHovsepyan, там как минимум есть advance(it, id - 1);.
    – user302909
    17 янв 2019 в 21:33
  • @RiotBr3aker но такая арифметика с итераторами list не работает, я понял свою ошибку, я просто создал другой итератор и сделал диапазон в один элемент. А copy я использую, потому, что list из объектов класса Medicine и содержит несколько полей, поэтому мне так удобней копировать
    – Norfo4ik
    17 янв 2019 в 21:38
1

Если вам так уж хочется использовать стандартный алгоритм, то можно так:

copy_n(it, 1, back_inserter(sell_medic));

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.