Conversation
sofiiakulish
force-pushed
the
hw08_envdir_tool
branch
from
6cd8ea3 to
2c3143b
Compare
agneum
left a comment
agneum
left a comment
There was a problem hiding this comment.
Спасибо за выполненное задание. Хорошая работа
Итого: 10 баллов из 10. Принято
|
|
||
| result := make(Environment) | ||
|
|
||
| files, err := ioutil.ReadDir(dir) |
There was a problem hiding this comment.
С Go 1.16 функции пакета ioutil помечены как deprecated, рекомендуется использовать функции пакета io и os: https://go.dev/doc/go1.16#ioutil
| return "", true, nil | ||
| } | ||
|
|
||
| file, err := os.Open(dir + "/" + fileInfo.Name()) |
There was a problem hiding this comment.
Для получения пути к файлу рекомендую использовать https://pkg.go.dev/path/filepath#Join
| func createTestDir(dir string) { | ||
| err := os.MkdirAll(dir, os.ModePerm) | ||
| if err != nil { | ||
| panic(fmt.Errorf("Error creating test directory: %w", err)) |
There was a problem hiding this comment.
Желательно не паниковать, даже в тестах. Предложил бы использовать функции пакета testify/require
|
|
||
| err := command.Start() | ||
| if err != nil { | ||
| panic(err) |
There was a problem hiding this comment.
В программе особенно не рекомендуется паниковать. Обработанная ошибка для пользователя будет выглядеть гораздо понятнее
| @@ -1,3 +1,5 @@ | |||
| module github.com/fixme_my_friend/hw08_envdir_tool | |||
| module github.com/sofiiakulish/hw08_envdir_tool | |||
There was a problem hiding this comment.
| module github.com/sofiiakulish/hw08_envdir_tool | |
| module github.com/sofiiakulish/hw/hw08_envdir_tool |
2 participants
Домашнее задание №8 «Утилита envdir»
Чек-лист студента (Что это?)
go mod tidy..syncфайл. Зачем его удалять?test.sh).Критерии оценки
Зачёт от 7 баллов