Opened 3 years ago

Closed 3 years ago

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

WW_array Kiselev Igor lab_13

Reported by: kiselev.igor Owned by: Антон Филатов
Component: WW_array Version: 3.0
Keywords: Cc:

Description

bool пока нет(

Change History (5)

comment:1 Changed 3 years ago by Антон Филатов

Owner: changed from Evgeny Linsky to Антон Филатов
Status: newassigned
Type: ожидается проверкаожидаются исправления

да, обычный массив реализован без вопросов 7/10

comment:2 Changed 3 years ago by kiselev.igor

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

comment:3 Changed 3 years ago by Антон Филатов

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

Упал тест на sizeof(my_array<bool, 1>) == 1

И упал такой тест:

my_array<bool, 50> a;
const my_array<bool, 50> &ca = a;
for (size_t i = 0; i < N / 2; i++) {
    a[i] = какое-то значение
}
for (size_t i = N / 2; i < N; i++) {
    a.at(i) = какое-то значение
}

for (size_t i = 0; i < N; i++) {
    assert(ca[i] == то же самое значение));
}

Стиль:

for (int i = 128, j = 7; i > 0; i /= 2, j--)
  if (tmp >= i) {
    tmp -= i;
    degs[j] = 1;
  }

это у вас такой перевод в двоичную систему? неужели нет способа сделать это попроще? Странно, что вы хотите пользоваться degs[i], но не хотите пользоваться маской data & msk - вроде как одно и то же.

  • много волшебных констант
  • ваше собственное покрытие тестами маловато :)

8/10

Можете попробовать исправить до вечера понедельника

comment:4 Changed 3 years ago by kiselev.igor

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

comment:5 Changed 3 years ago by Антон Филатов

Resolution: задача сдана
Status: assignedclosed

Тесты пройдены
Стиль:

много волшебных констант

9.5/10

Note: See TracTickets for help on using tickets.