Opened 4 years ago
Closed 4 years ago
#1068 closed ожидается проверка (задача сдана)
WW #18
| Reported by: | vorobeva.aleksandra | Owned by: | Vasily Alferov |
|---|---|---|---|
| Component: | WW_format | Version: | 1.0 |
| Keywords: | Cc: |
Description
Коммит задержался, потому что я долго разбиралась, что надо сюда отправлять:с Все тесты проходятся, новых я не писала. Valgrind орет надеюсь исправить до завтра)
Change History (2)
comment:1 Changed 4 years ago by
comment:2 Changed 4 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Очень круто.
Тут 8/10.
Во-первых, компилятор в меня начал кидаться ворнингами, а тесты при первом запуске попадали.
Потому что в
FormatHelper<bool>ты хранишьconst bool&.Это довольно смешной объект. Как минимум, бул занимает один байт, а константная ссылка на бул занимает восемь.
Вообще со ссылками на внутренности вектора надо быть предельно осторожным, потому что векторы могут перевыделиться в другом месте, а ссылки об этом в курсе не будут. Хотя тут не этот случай, но надо иметь ввиду.
Тут скорее проблема с вектором булов, который, как известно, объект очень капризный.
Короче, надо
const bool&заменить наbool, и проблема исчезнет.По оставшейся части кода замечаний нет.