Opened 4 years ago
Closed 4 years ago
#682 closed ожидается проверка (задача сдана)
WW #11 subbotina.olesya
| Reported by: | subbotina.olesya | Owned by: | Дмитрий Лапшин (lapshin) |
|---|---|---|---|
| Component: | WW cpp_io | Version: | 2.0 |
| Keywords: | Cc: |
Description
Change History (3)
comment:1 Changed 4 years ago by
| Owner: | changed from Дмитрий Лапшин (lapshin) to subbotina.olesya |
|---|---|
| Summary: | WW #11 Subbotina Olesia → WW #11 subbotina.olesya |
| Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 4 years ago by
| Owner: | changed from subbotina.olesya to Дмитрий Лапшин (lapshin) |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
| Version: | 1.0 → 2.0 |
comment:3 Changed 4 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
Читать из файла много рабочих всё ещё не получилось. Ассёрт про зарплату забавен, но не ясно зачем. Что-то не так с чтением бинарных чисел.
У работника как экземпляра класса есть поле тип, но оно лишнее: ведь не может поменяться.
= default лучше писать с пробелами.
Ещё вижу волшебные константы (лучше не полагаться, что 4 и int32_t согласованы, а достать sizeof). Дублируется код создания (фабрики).
7.
Note: See
TracTickets for help on using
tickets.
Валгринд ругается. Бинарные файлы плохо читаешь и пишешь. Ошибки ввода-вывода зачем-то ассертаешь внутри. Для бинарного ввода-вывода надо использовать read/write, что не работает у тебя.
Многозагадочно. Начнём с undefined behaviour. Закончим тем, откуда эта память взялась-то?!
Дублируется код создания работника при чтении.
Чинить.