That function shows the state of error handling in Go.
Don't get me wrong, error handling is verbose and annoying, but not even displaying actual error is just horrible coding practice.
should be at the very least
err = json.Unmarshal(weatherBody, &WeatherRes)
if err != nil {
fmt.Printf("error decoding weather json: %s\n",err)
return
}
or just be function that returns that error or the weather err = json.Unmarshal(weatherBody, &WeatherRes)
if err != nil {
return fmt.Errorf("error decoding weather json: %s\n",err)
} return err if err != nil
everything better than three fucking lines every time something returns err.Especially that some clown decided that apparently
if err != nil { return err }
is too terse and go fmt just goes back to expanding it to 3 lines...