#395 closed ожидается проверка (задача сдана)
WW #5
Reported by: | sukhodolskiy.maksim | Owned by: | Sokolov Viacheslav |
---|---|---|---|
Component: | WW_c_io | Version: | 2.0 |
Keywords: | Cc: |
Description
Без допа
Change History (6)
comment:1 Changed 5 years ago by
comment:2 Changed 5 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
В целом работа выполнена отлично.
11 #define MAX_VALUE (int32_t)pow(BYTE, BYTES_IN_NUMBER)
pow лучше не использовать в таких случаях, потому что он floating-point, это дорогая операция, теряется точность, куча проблем на ровном месте.
Нужно поправить на допбалл:
fprintf / fwrite / fputc может не получиться, если на файловой системе место кончилось.
Нужно как-нибудь специфицировать поведение программы в таком случае - как минимум сообщать об этом с помощью кода возврата. Для этого стоит либо поменять сигнатуру apply (протащить возможность рапортовать ошибку), либо использовать exit. Кроме того, стоит решить, что делать с частично записанными данными - либо оставить, как есть, либо удалить файл целиком, либо оставить только успешно записанные точки, но в любом случае поведение должно быть донесено до конечного пользователя утилиты либо комментарием в main, либо с помощью --help.
comment:3 Changed 4 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
comment:4 Changed 4 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
file system memory
это disk space
В описании стоило бы уточнить, в каком состоянии окажется файл в случае, если кончилось место (не специфицировано, число может оказаться частично записано)
comment:5 Changed 4 years ago by
Я добавил описание программы, специально чтобы добавить строчку про то, что делать с частично записанными данными, но её не написал, вспомнил и добавил её после дедлайна, но до того, как вы проверили. Это считается?
Или ошибка в описании есть в последней версии?
Теперь с допом