Refactoring and input validation of nearest_* fns

Author: sveinbjornt

src/iceaddr/addresses.py

@@ -199,16 +199,7 @@ def nearest_addr(
 ) -> list[dict[str, Any]]:
     """Find the address closest to the given coordinates."""
 
-    results_with_dist = find_nearest(
-        lat=lat,
-        lon=lon,
-        rtree_table="stadfong_rtree",
-        main_table="stadfong",
-        id_column="hnitnum",
-        limit=limit,
-        max_dist=max_dist,
-        post_process=_postprocess_addr,
-    )
+    results_with_dist = nearest_addr_with_dist(lat=lat, lon=lon, limit=limit, max_dist=max_dist)
 
     # Strip out distances for backward compatibility
     return [addr for addr, _dist in results_with_dist]
@@ -224,6 +215,12 @@ def nearest_addr_with_dist(
     - float: Distance from the search point in kilometers
     """
 
+    if lat > 90.0 or lat < -90.0 or lon > 180.0 or lon < -180.0:
+        raise ValueError("Invalid latitude or longitude value: {}, {}".format(lat, lon))
+
+    if limit < 0 or max_dist < 0.0:
+        raise ValueError("limit and max_dist must be non-negative")
+
     return find_nearest(
         lat=lat,
         lon=lon,

src/iceaddr/placenames.py

@@ -93,15 +93,9 @@ def nearest_placenames(
     lat: float, lon: float, limit: int = 1, max_dist: float = 0.0
 ) -> list[dict[str, Any]]:
     """Find the placename closest to the given coordinates."""
-    results_with_dist = find_nearest(
-        lat=lat,
-        lon=lon,
-        rtree_table="ornefni_rtree",
-        main_table="ornefni",
-        id_column="id",
-        limit=limit,
-        max_dist=max_dist,
-        post_process=None,  # No extra processing needed for placenames
+
+    results_with_dist = nearest_placenames_with_dist(
+        lat=lat, lon=lon, limit=limit, max_dist=max_dist
     )
 
     # Strip out distances for backward compatibility
@@ -117,6 +111,13 @@ def nearest_placenames_with_dist(
     - dict: Placename information
     - float: Distance from the search point in kilometers
     """
+
+    if lat > 90.0 or lat < -90.0 or lon > 180.0 or lon < -180.0:
+        raise ValueError("Invalid latitude or longitude value: {}, {}".format(lat, lon))
+
+    if limit < 0 or max_dist < 0.0:
+        raise ValueError("limit and max_dist must be non-negative")
+
     return find_nearest(
         lat=lat,
         lon=lon,