Never using named result parameters
Mistake
Name result parameters should be used when would provide better context.
type locator interface {
getCoordinates(address string) (float32, float32, error)
}
Name result parameters should not be used when they provide no value
func ReadFull(r io.Reader, buf []byte) (n int, err error) {
// ...
}
Fix
type locator interface {
getCoordinates(address string) (lng, lat float32, error)
}
func ReadFull(r io.Reader, buf []byte) (int, error) {
// ...
}