Iter25 #26
Iter25 #26
Conversation
internal/server/server.go
Outdated
| return nil, fmt.Errorf("failed to create database storage: %w", err) | ||
| } | ||
| return s, nil | ||
| } else { |
There was a problem hiding this comment.
Использование else после return в блоке if избыточно. Потому что если условие в if истинно, функция завершится и вернёт значение на этом этапе, и выполнение кода дальше не пойдет. Это значит, что все, что находится в else, будет выполнено только если условие if не истинно, и нет нужды явно оборачивать этот код в else. Таким образом, код становится проще и уменьшается его вложенность, что облегчает его чтение и понимание. Поэтому вместо твоего варианта типа:
if условие {
return что-то
} else {
// другой код
}Можно написать кратко:
if условие {
return что-то
}
// другой кодДоп. ссылки:
internal/agent/agent.go
Outdated
| logger.Log.Fatal("failed to get encryption key", zap.Error(err)) | ||
| var s Sender | ||
| if !cfg.UseRPC { | ||
| var key *rsa.PublicKey = nil |
There was a problem hiding this comment.
сылочный тип инициализируется как nil
присваивание лишнее.
internal/agent/senderrpc/sender.go
Outdated
| delay := retryDelays[n] | ||
| return delay | ||
| }), | ||
| retry.OnRetry(func(n uint, err error) { |
There was a problem hiding this comment.
Параметр функции, названный err, не используется в теле функции. В Go, если определили параметр в функции, но не использовали его внутри, это считается неоптимальным стилем, поскольку параметр занимает место в сигнатуре функции и может сбивать с толку при чтении кода.
В данном случае, есть два возможных решения:
- Удаление параметра: Если параметр
errдействительно не нужен для работы функции, его можно просто удалить из сигнатуры функции. Это делает код чище и проще для понимания. - Переименование в
_: Если параметр не используется, но его наличие обязательно по какой-то причине (например, для соответствия интерфейсу или для использования в будущем), его можно переименовать в_. В Go, символ_используется для игнорирования переменной. Переменные, именованные как_, не обязаны быть использованы, т.е. не будет копирования неиспользуемого значения внутрь функции. Это позволяет сохранить сигнатуру функции неизменной, но явно указывает на то, что данный параметр в настоящее время не используется и избежать лишнего использования памяти.
No description provided.