Opened 4 years ago

Closed 4 years ago

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

WW #15

Reported by: onofriychuk.ilya Owned by: Egor Suvorov
Component: WW_linq Version: 2.0
Keywords: Cc:

Description


Change History (2)

comment:1 Changed 4 years ago by onofriychuk.ilya

Version: 1.02.0

comment:2 Changed 4 years ago by Egor Suvorov

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

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

  • Вы зря копируете элементы внутри where_neq/until_eq внутрь функторов. [x] — это всегда захват по значению с копированием.

Стиль 1.9/3:

  • Не хватает слов final, explicit (конструктор и оператор), noexcept, const qualification, rvalue-ref-qualifier у методов.
  • Виртуальный деструктор лучше объявить наверху, рядом с остальными "техническими" методами.
  • У enumerator стоит запретить копирование и перемещение — это точно ошибки. Перемещение можно запрещать, начиная с C++17 из-за copy elision.
  • static_cast<bool> не нужен в assert.
  • В конструкторе select_enumerator дублирование кода с куском operator++. Аналогично в until-enumerator.
Note: See TracTickets for help on using tickets.