IvanCesnokov / Labs_Cesnokov

0 stars 0 forks source link

Lab 5 Notes #3

Open AntonC9018 opened 9 months ago

AntonC9018 commented 9 months ago

https://github.com/IvanCesnokov/Labs_Cesnokov/blob/e58edec998ce69f34327c2ad402cd5b666030253/Cesnokov_lab5/Cesnokov_lab5/Cesnokov_lab5.cpp#L25 Это неправильно, ты должен вписывать до нулевого символа. Чекни документацию -- https://cplusplus.com/reference/ostream/ostream/write/

AntonC9018 commented 9 months ago

https://github.com/IvanCesnokov/Labs_Cesnokov/blob/e58edec998ce69f34327c2ad402cd5b666030253/Cesnokov_lab5/Cesnokov_lab5/Cesnokov_lab5.cpp#L64 Та же фигня.

AntonC9018 commented 9 months ago

https://github.com/IvanCesnokov/Labs_Cesnokov/blob/e58edec998ce69f34327c2ad402cd5b666030253/Cesnokov_lab5/Cesnokov_lab5/Cesnokov_lab5.cpp#L79-L80 Это не тут должно происходить, ошибка должна каким-то образом подняться выше, чтобы функция, которая вызвала эту, могла прерваться. Почитай про валидацию, у меня есть в теме про сериализацию.

AntonC9018 commented 9 months ago

https://github.com/IvanCesnokov/Labs_Cesnokov/blob/e58edec998ce69f34327c2ad402cd5b666030253/Cesnokov_lab5/Cesnokov_lab5/Cesnokov_lab5.cpp#L94-L95 Добавь, бросать эксепшены только не надо. Прочитай опять же приложение про валидацию.

AntonC9018 commented 9 months ago

https://github.com/IvanCesnokov/Labs_Cesnokov/blob/e58edec998ce69f34327c2ad402cd5b666030253/Cesnokov_lab5/Cesnokov_lab5/Cesnokov_lab5.cpp#L122 Технически, можно из без этого буфера обойтись, но сойдет.

AntonC9018 commented 9 months ago

https://github.com/IvanCesnokov/Labs_Cesnokov/blob/e58edec998ce69f34327c2ad402cd5b666030253/Cesnokov_lab5/Cesnokov_lab5/Cesnokov_lab5.cpp#L102C40-L104 Тут проверку ошибок после считывания каждого поля.

AntonC9018 commented 9 months ago

https://github.com/IvanCesnokov/Labs_Cesnokov/blob/e58edec998ce69f34327c2ad402cd5b666030253/Cesnokov_lab5/Cesnokov_lab5/Cesnokov_lab5.cpp#L126 И тут тоже

AntonC9018 commented 9 months ago

https://github.com/IvanCesnokov/Labs_Cesnokov/blob/e58edec998ce69f34327c2ad402cd5b666030253/Cesnokov_lab5/Cesnokov_lab5/Cesnokov_lab5.cpp#L133-L137 Ты эти переменные не ипользуешь дальше, всунь в скоуп (блок).

{
    этот код.
}

И ниже тоже.

close вызывать тогда не надо. Файл закроется через RAII.