ЛЛМ написала функцию, которая из строки делает int64:
func lockIDFromString(value string) int64 {
hasher := fnv.New64a()
_, _ := hasher.Write([]byte(value))
lockID := int64(hasher.Sum64())
return lockID
}Мне тут не нравится, что игнорируется ошибка И нет комментария, который объясняет, почему так. То есть, чтобы исправить нужно:
1. Или добавить комментарий
2. Не игнорировать ошибку и внутри if err != nil уже не возвращать ошику (но тоже с комментарием, почему так)
Выбираю второй вариант. Делаю промпт:
_, _ = hasher.Write([]byte(value)) Тут игнорируются все возвращаемые значения. Err надо все-таки вернуть, но внутри проверки на err != nil написать, что мы ее игнорируем явно.
ЛЛМ отдает такое:
func lockIDFromString(value string) (int64, error) {
hasher := fnv.New64a()
_, err := hasher.Write([]byte(value))
if err != nil {
return 0, err
}
lockID := int64(hasher.Sum64())
return lockID, nil
}Проблема в том, что error из функции возвращать не надо т.к. он всегда nil. Он всегда nil т.к. метод Write у fnv.New64a() всегда возвращает nil.
Почему так? Потому что в моем промпте есть слово "вернуть" и ллм поняла это как "вернуть из функции", а не вернуть из метода Write? Или она это поняла верно, но не учла, что Write всегда возвращает nil т.к. не проверила реализацию?
Вот такие моменты конечно не могут нравится т.к. всегда есть шанс, что или кривая формулировка промпта, или еще что-то приведут не к лучшей реализации.
А я хотел видеть это:
_, err := hasher.Write([]byte(value))
if err != nil {
// sum64a.Write always returns len(value), nil.
}
или это:
_, _ = hasher.Write([]byte(value)) // err always nil
