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: | new → assigned |
Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.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
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 2.0 → 3.0 |
comment:5 Changed 3 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Тесты пройдены
Стиль:
много волшебных констант
9.5/10
Note: See
TracTickets for help on using
tickets.
да, обычный массив реализован без вопросов 7/10