добавлен функционал#5
Conversation
just-musha
left a comment
just-musha
left a comment
There was a problem hiding this comment.
Привет! Спасибо за работу. Оставила пару комментов как можно ещё чуть улучшить код. Для задания не критично. Принято.
| package encoding | ||
|
|
||
| import ( | ||
| "encoding/json" |
There was a problem hiding this comment.
Кстати, можешь в IDE-шке подключить форматирование через goimports, он красиво отформатирует порядок импортов
| var dockerCompose models.DockerCompose | ||
|
|
||
| yamlFile, err := os.Open(y.FileInput) | ||
|
|
There was a problem hiding this comment.
Здесь и дальше по коду - можно не оставлять пустые строки перед проверкой ошибки - код будет чуть более компактным
|
|
||
| yamlData, err := io.ReadAll(yamlFile) | ||
|
|
||
| yamlFile.Close() |
There was a problem hiding this comment.
В Go принято сразу после проверки ошибки добавлять закрытие файла через defer, тогда метод Close вызовется автоматически при выходе из функции. Вот тут пример https://otus.ru/nest/post/2550/
|
Ой, не заметила. На будущее пул реквест нужно создавать в свой репозиторий, а не в Яндекс Практикум. Обрати на это внимание, пожалуйста. |
2 participants
No description provided.