Opened 4 years ago

Closed 4 years ago

#1012 closed ожидаются исправления (задача сдана)

WW #15

Reported by: Bagryanova Ekaterina Owned by: Egor Suvorov
Component: WW_linq Version: 2.0
Keywords: Cc:

Description


Change History (2)

comment:1 Changed 4 years ago by Bagryanova Ekaterina

Version: 1.02.0

comment:2 Changed 4 years ago by Egor Suvorov

Resolution: задача сдана
Status: assignedclosed
Type: ожидается проверкаожидаются исправления

Проверялась версия 4358. Если хотите — можно договориться и дорешать на баллы. Очень хороший код по модулю стандартных подвохов и пары мелочей.

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

Стиль 1.75/3:

  • Не хватает слов final, explicit (конструктор и оператор), noexcept, rvalue-ref-qualifier у методов.
  • Не хватает виртуального деструктора в полиморфном классе enumerator. Конкретно в этой задаче это ни на что не влияет, но привычка полезная.
  • У enumerator стоит запретить копирование и перемещение — это точно ошибки. Перемещение можно запрещать, начиная с C++17 из-за copy elision.
  • static_cast<bool> не нужен даже для explicit operator bool внутри while/if/assert/... В общем, везде, где он очевидно используется как логическое условие.
  • to_vector лучше через copy_to + back_inserter.
  • Скобочки в return не нужны: return A && B;
Note: See TracTickets for help on using tickets.