Opened 4 years ago

Closed 4 years ago

#992 closed ожидается проверка (задача сдана)

WW #15

Reported by: gabitov.daniil Owned by: Vasily Alferov
Component: WW_linq Version: 2.0
Keywords: Cc:

Description


Change History (4)

comment:1 Changed 4 years ago by Vasily Alferov

Type: ожидается проверка → ожидаются исправления

Неплохо, но пока половина баллов.

По корректности:

  • valgrind на примере ругается. Там небольшая бага в where_enumerator.
  • Логика у until_enumerator не та, что в задании нужна. В задании "выдавай x, ПОКА НЕ f(x)". У тебя "ПОКА f(x)".
  • В нём же, проверь случай, когда он должен выдать пустой вектор. Там будет найдена ошибка.
  • Лишние копирования функторов. Функторы у нас перемещаемые, но не копируемые.

По стилю:

  • Читать parent.operator bool() не очень приятно. Поправь на (bool)parent, раз они у тебя даже не explicit. То же про operator*. Там, где они вызываются у this, можно оставить, там так и так будет нечитаемо.

comment:2 Changed 4 years ago by gabitov.daniil

Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

comment:3 Changed 4 years ago by gabitov.daniil

Да, не знаю что на меня нашло: parent.operator bool() - это действительно ужасно

comment:4 Changed 4 years ago by Vasily Alferov

Resolution: → задача сдана
Status: assigned → closed

Топ, теперь зачтено, 10/10.

Note: See TracTickets for help on using tickets.