hw07 complete #7
Conversation
| @@ -9,12 +9,6 @@ type ( | |||
| type Stage func(in In) (out Out) | |||
There was a problem hiding this comment.
Это я влил правки по 6ой домашке, чтобы отдельно не отправлять ПР на неё. Правки очень незначительные и не хочу отвлекать преподавателя ещё и на это, поэтому сюда добавил
There was a problem hiding this comment.
Желательно, чтобы PR содержал код только одного задания.
Правки по предыдущему заданию можно добавить к прежнему PR (отдельный создавать не нужно). Если в нём уже есть аппрув, то можно сразу мержить
There was a problem hiding this comment.
У меня ситуация такая что я смержил и только потом увидел комментарии.
| @@ -9,12 +9,6 @@ type ( | |||
| type Stage func(in In) (out Out) | |||
There was a problem hiding this comment.
Желательно, чтобы PR содержал код только одного задания.
Правки по предыдущему заданию можно добавить к прежнему PR (отдельный создавать не нужно). Если в нём уже есть аппрув, то можно сразу мержить
hw07_file_copying/copy.go
Outdated
| return ErrLimitCannotBeNegative | ||
| } | ||
|
|
||
| fileTo, _ := os.Create(toPath) |
There was a problem hiding this comment.
Очень важно обработывать ошибки.
Файл toPath лучше открыть после всех проверок, т.к. есть вероятность, что он не пригодится (в случае ошибок ErrUnsupportedFile, ErrOffsetExceedsFileSize и т. д.)
| fromPath := "testdata/input.txt" | ||
| toPath := "out.txt" |
There was a problem hiding this comment.
В тестах при работе с файлами, особенно при удалении, рекомендую использовать https://pkg.go.dev/os#CreateTemp
Домашнее задание №7 «Утилита для копирования файлов»
Чек-лист студента (Что это?)
go mod tidy..syncфайл. Зачем его удалять?test.sh).Критерии оценки
Зачёт от 7 баллов