Opened 3 years ago

Closed 3 years ago

#535 closed ожидаются исправления (задача сдана)

WW #10

Reported by: Азат Валеев Owned by: Святослав Власов
Component: WW cpp_io Version: 1.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 3 years ago by Азат Валеев

Помянем вторую проверку :pepeSadge:

comment:2 Changed 3 years ago by Святослав Власов

Type: ожидается проверка → ожидаются исправления
  1. Не нужно открывать файл на добавление (ios::app), если этого явно не указано в задании. В итоге тесты начали дозаписывать в конец файлов с прошлых тестов и дружно попадали
  2. В Employee::set_info делаешь много лишних телодвижений. Всё это покороче и попроще можно было написать. Есть вариант утечки -- если salary прочитается с ошибкой, то память под name не очистится
  3. set_info и read у тебя практически идентичны, там сплошная копи-паста, это плохо
  4. Да и вообще, у тебя есть std::string, который тебя избавит от кучи проблем с буферами и ручным управлением памятью. Разрешили же -- пользуйся!

7/10 пока что.

comment:3 Changed 3 years ago by Святослав Власов

Resolution: → задача сдана
Status: assigned → closed
Note: See TracTickets for help on using tickets.