Skip to content

interface#13

Open
FedotovFrontalex wants to merge 5 commits into
masterfrom
4-interfaces
Open

interface#13
FedotovFrontalex wants to merge 5 commits into
masterfrom
4-interfaces

Conversation

@FedotovFrontalex
Copy link
Copy Markdown
Owner

No description provided.

Copy link
Copy Markdown
Collaborator

@IgorTirador IgorTirador left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

. main.go
Проблема: Отсутствие обработки ошибок
В текущем коде ошибки обрабатываются не везде, что может привести к неожиданным завершениям программы или трудностям при отладке.

Добавьте обработку ошибок для всех операций, особенно для методов, работающих с файлами и хранилищем

storage. Go
Проблема: Ошибки игнорируются при чтении и записи данных
Если произошла ошибка чтения/записи, возвращается пустой список BinList, что скрывает реальную ошибку.

Решение:
Возвращайте ошибки, чтобы код на уровне main мог их обработать

Bins.go
В функции CreateBin идентификатор задается статическим значением, что может вызвать конфликты идентификаторов в будущем.

Решение:
Используйте UUID для генерации уникальных идентификаторов

file.go
Проблема: Ошибки обработки файлов не всегда корректно обрабатываются
Вы открываете файл и не используете defer для его закрытия сразу после открытия. Также ошибки могут теряться.

Решение:
Используйте defer и добавьте обработку ошибок при записи файла:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants