From c8e4b382b754ffac991b70fe613b1511bbc9ccb1 Mon Sep 17 00:00:00 2001 From: williamnoonan Date: Thu, 17 Feb 2022 13:16:14 -0600 Subject: [PATCH] Use of 'locals' func unnecessary. --- defAirport.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/defAirport.py b/defAirport.py index 344a70d..073ad46 100644 --- a/defAirport.py +++ b/defAirport.py @@ -15,16 +15,14 @@ def add_airport_region(airport_dict): def getClosestAirport(latitude, longitude, allowed_types): from geopy.distance import geodesic plane = (latitude, longitude) + closest_airport_dict = dict() with open('./dependencies/airports.csv', 'r', encoding='utf-8') as airport_csv: airport_csv_reader = csv.DictReader(filter(lambda row: row[0] != '#', airport_csv)) for airport in airport_csv_reader: if airport['type'] in allowed_types: airport_coord = float(airport['latitude_deg']), float(airport['longitude_deg']) airport_dist = float((geodesic(plane, airport_coord).mi)) - if "closest_airport_dict" not in locals(): - closest_airport_dict = airport - closest_airport_dist = airport_dist - elif airport_dist < closest_airport_dist: + if not closest_airport_dict or airport_dist < closest_airport_dist: closest_airport_dict = airport closest_airport_dist = airport_dist closest_airport_dict['distance_mi'] = closest_airport_dist