Opened 5 years ago
Closed 4 years ago
#407 closed ожидается проверка (задача сдана)
WW #5
| Reported by: | Vladislav Nosivskoy | Owned by: | Sokolov Viacheslav |
|---|---|---|---|
| Component: | WW_c_io | Version: | 2.0 |
| Keywords: | Cc: |
Description
Change History (3)
comment:1 Changed 5 years ago by
| Type: | ожидается проверка → ожидаются исправления |
|---|
comment:2 Changed 4 years ago by
| Type: | ожидаются исправления → ожидается проверка |
|---|---|
| Version: | 1.0 → 2.0 |
comment:3 Changed 4 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
like за --help
строки сообщений лучше вынести в именованные константы, так будет проще их модифицировать; должно же быть
Try ./lab_05 --help?data тоже not NULL
byte_to_int, int_to_byte проще реализовать не побитовыми операциями, а побайтовыми
Кусок
174 if (argc < 5) { 175 remove_all_points(&list); 176 printf("Error: unknown command\nTry ./lab_05 -help"); 177 return 0; 178 }дублируется много раз, лучше вынести в отдельную функцию / макрос / звать один раз на программу; в некоторых блоках при этом неправильное сообщение
Нужно поправить на допбалл:
fprintf / fwrite / fputc может не получиться, если на файловой системе место кончилось.
Нужно как-нибудь специфицировать поведение программы в таком случае - как минимум сообщать об этом с помощью кода возврата. Для этого стоит либо поменять сигнатуру apply (протащить возможность рапортовать ошибку), либо использовать exit. Кроме того, стоит решить, что делать с частично записанными данными - либо оставить, как есть, либо удалить файл целиком, либо оставить только успешно записанные точки, но в любом случае поведение должно быть донесено до конечного пользователя утилиты либо комментарием в main, либо с помощью --help.