WW #15

[abramov.nikita]

Решение обновлено

[Egor Suvorov]

Корректность 7/7.

Стиль 1/3:

• Не хватает слов final, explicit (конструктор и оператор), noexcept, rvalue-ref-qualifier у методов.
• Не хватает виртуального деструктора в полиморфном классе enumerator. Конкретно в этой задаче это ни на что не влияет, но привычка полезная.
• У enumerator стоит запретить копирование и перемещение — это точно ошибки. Перемещение можно запрещать, начиная с C++17 из-за copy elision.
• Во всех методах drop/take и остальных не нужно явно писать возвращаемый тип, он прекрасно выведется сам.
• *it++ = (**this) — скобочки не нужны. Аналогично в ++*this. Там однозначно парсится.
• Нехорошо, что select_enumerator несколько раз вызывает функтор.
• find_current_/update() — неконсистентное наименование, а ещё оба явно должны быть приватными функциями. Плюс, кажется, можно переписать это дело через do-while или как-то ещё, чтобы не требовалась отдельная функция update.