WW #15

[Egor Suvorov]

Корректность 6/7: не работает until на пустом входе.

Стиль 1.25/3:

• Не хватает слов final, explicit (конструктор), noexcept, rvalue-ref-qualifier у методов.
• Не хватает виртуального деструктора в полиморфном классе enumerator. Конкретно в этой задаче это ни на что не влияет, но привычка полезная.
• У enumerator стоит запретить копирование и перемещение — это точно ошибки. Перемещение можно запрещать, начиная с C++17 из-за copy elision.
• static_cast<bool>(parent_) не нужен — explicit operator bool в таких контекстах как раз прекрасно работает. explicit запрещает parent_ + 1.
• Не очень консистентно используются () и {} в member initialization list: выберите что-нибудь одно.
• В where_enumerator есть некоторое дублирование кода.

[Egor Suvorov]

Можно переоткрыть и дорешать.

[sukhodolskiy.maksim]

В тг договорились на дорешку до 3 июня 22:59 на баллы.

[sukhodolskiy.maksim]

Оказывается, нужно было ещё тикет открыть.

[Egor Suvorov]

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

Стиль 2.5/3:

• drop может быть noexcept
• Не надо писать operator++(), вызывайте операторы как обычно. Например: ++*this.
• Итераторы хорошо бы мувать, а не копировать в range_enumerator. Они, конечно, маленькие, но код тут обобщённый.
• Вместо cached_+current_ — std::optional<T>
• Перед : в member initializer list стоит сделать перевод строки и отступ, чтобы не было отступа по 100500 символов перед очередным членом.
• findFirstSuitable — не совсем first. Скорее next. Ну и Suitable я бы убрал.