diff --git a/cmd/client/advanced.go b/cmd/client/advanced.go index b22f63b..e212d5d 100644 --- a/cmd/client/advanced.go +++ b/cmd/client/advanced.go @@ -38,7 +38,8 @@ func advancedMenu(app *application) { fmt.Printf("\nChanged units from %s to %s...\n\n", current, app.Config.Units) } else if option == "2" { getPreciseLocation(app) + getCurrent(app) + printWeather(app) } } - app.WaitGroup.Done() } diff --git a/cmd/client/location.go b/cmd/client/location.go index b0c38f3..6c0b5a1 100644 --- a/cmd/client/location.go +++ b/cmd/client/location.go @@ -33,8 +33,6 @@ func getLocation(app *application) { loc := strings.Split(app.Config.Location, ",") app.Config.Latitude, app.Config.Longitude = loc[0], loc[1] - - app.WaitGroup.Done() } func getPreciseLocation(app *application) { @@ -55,5 +53,4 @@ func getPreciseLocation(app *application) { log.Println(err) } app.Config.Latitude = strings.TrimSuffix(input, "\n") - //app.WaitGroup.Done() } diff --git a/cmd/client/main.go b/cmd/client/main.go index 38212e8..c3e1d75 100644 --- a/cmd/client/main.go +++ b/cmd/client/main.go @@ -6,7 +6,6 @@ import ( "log" "os" "strings" - "sync" "github.com/joho/godotenv" ) @@ -42,22 +41,21 @@ type config struct { } type application struct { - Forecast forecast - Config config - WaitGroup sync.WaitGroup - Version string + Forecast forecast + Config config + Version string } func printWeather(app *application) { - var unitString string + var unitStr string if app.Config.Units == "imperial" { - unitString = "Farhenheit/mph" + unitStr = "Farhenheit/mph" } else { - unitString = "Celsius/kph" + unitStr = "Celsius/kph" } - fmt.Printf("\nUnits: %s (%s)\n\n", app.Config.Units, unitString) + fmt.Printf("\nUnits: %s (%s)\n\n", app.Config.Units, unitStr) fmt.Printf("Current temperature: %.2f\n", app.Forecast.Main.Temp) fmt.Printf("Feels like: %.2f\n", app.Forecast.Main.FeelsLike) fmt.Printf("High: %.2f\n", app.Forecast.Main.HighTemp) @@ -82,13 +80,10 @@ func main() { fcst := forecast{} - var wg sync.WaitGroup - app := application{ - Config: cfg, - Forecast: fcst, - WaitGroup: wg, - Version: version, + Config: cfg, + Forecast: fcst, + Version: version, } var option string @@ -114,17 +109,11 @@ func main() { option = strings.TrimSuffix(input, "\n") if option == "1" || option == "" { - app.WaitGroup.Add(1) - go getLocation(&app) - app.WaitGroup.Wait() - app.WaitGroup.Add(1) - go getCurrent(&app) - app.WaitGroup.Wait() + getLocation(&app) + getCurrent(&app) printWeather(&app) } else if option == "2" { - app.WaitGroup.Add(1) - go advancedMenu(&app) - app.WaitGroup.Wait() + advancedMenu(&app) } else if option == "0" { return } else { diff --git a/cmd/client/weather.go b/cmd/client/weather.go index b33e766..fdba3a1 100644 --- a/cmd/client/weather.go +++ b/cmd/client/weather.go @@ -30,5 +30,4 @@ func getCurrent(app *application) { if err != nil { log.Println(err) } - app.WaitGroup.Done() }