Clean up @turf/simplify

Rework the simplify-js library to actually just take number[] directly
Style cleanup around the type inference and passthrough functions

packages/turf-simplify/index.ts

@@ -54,9 +54,9 @@ function simplify<T extends AllGeoJSON>(
   } = {}
 ): T {
   // Optional parameters
-  options = options || {};
+  options = options ?? {};
   if (!isObject(options)) throw new Error("options is invalid");
-  const tolerance = options.tolerance !== undefined ? options.tolerance : 1;
+  const tolerance = options.tolerance ?? 1;
   const highQuality = options.highQuality ?? false;
   const mutate = options.mutate ?? false;
 
@@ -96,66 +96,36 @@ function simplifyGeom(
 
   if (type !== "GeometryCollection") {
     // TODO should this cater for GeometryCollections too?
-    const coordinates = geometry.coordinates;
     switch (type) {
       case "LineString":
-        geometry.coordinates = simplifyLine(
-          coordinates as Position[],
+        geometry.coordinates = simplifyJS(
+          geometry.coordinates,
           tolerance,
           highQuality
         );
         break;
       case "MultiLineString":
-        geometry.coordinates = (coordinates as Position[][]).map(
-          function (lines) {
-            return simplifyLine(lines, tolerance, highQuality);
-          }
+        geometry.coordinates = geometry.coordinates.map((lines) =>
+          simplifyJS(lines, tolerance, highQuality)
         );
         break;
       case "Polygon":
         geometry.coordinates = simplifyPolygon(
-          coordinates as Position[][],
+          geometry.coordinates,
           tolerance,
           highQuality
         );
         break;
       case "MultiPolygon":
-        geometry.coordinates = (coordinates as Position[][][]).map(
-          function (rings) {
-            return simplifyPolygon(rings, tolerance, highQuality);
-          }
+        geometry.coordinates = geometry.coordinates.map((rings) =>
+          simplifyPolygon(rings, tolerance, highQuality)
         );
     }
   }
 
   return geometry;
 }
 
-/**
- * Simplifies the coordinates of a LineString with simplify-js
- *
- * @private
- * @param {Array<number>} coordinates to be processed
- * @param {number} tolerance simplification tolerance
- * @param {boolean} highQuality whether or not to spend more time to create a higher-quality
- * @returns {Array<Array<number>>} simplified coords
- */
-function simplifyLine(
-  coordinates: Position[],
-  tolerance: number,
-  highQuality: boolean
-) {
-  return simplifyJS(
-    coordinates.map(function (coord) {
-      return { x: coord[0], y: coord[1], z: coord[2] };
-    }),
-    tolerance,
-    highQuality
-  ).map(function (coords) {
-    return coords.z ? [coords.x, coords.y, coords.z] : [coords.x, coords.y];
-  });
-}
-
 /**
  * Simplifies the coordinates of a Polygon with simplify-js
  *
@@ -171,25 +141,15 @@ function simplifyPolygon(
   highQuality: boolean
 ) {
   return coordinates.map(function (ring) {
-    const pts = ring.map(function (coord) {
-      return { x: coord[0], y: coord[1] };
-    });
-    if (pts.length < 4) {
+    if (ring.length < 4) {
       throw new Error("invalid polygon");
     }
-    let simpleRing = simplifyJS(pts, tolerance, highQuality).map(
-      function (coords) {
-        return [coords.x, coords.y];
-      }
-    );
-    //remove 1 percent of tolerance until enough points to make a triangle
+    let ringTolerance = tolerance;
+    let simpleRing = simplifyJS(ring, ringTolerance, highQuality);
+    // remove 1 percent of tolerance until enough points to make a triangle
     while (!checkValidity(simpleRing)) {
-      tolerance -= tolerance * 0.01;
-      simpleRing = simplifyJS(pts, tolerance, highQuality).map(
-        function (coords) {
-          return [coords.x, coords.y];
-        }
-      );
+      ringTolerance -= ringTolerance * 0.01;
+      simpleRing = simplifyJS(ring, ringTolerance, highQuality);
     }
     if (
       simpleRing[simpleRing.length - 1][0] !== simpleRing[0][0] ||

packages/turf-simplify/lib/simplify.d.ts

@@ -23,10 +23,7 @@ HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABI
 TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
 SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 */
-interface Point {
-  x: number;
-  y: number;
-}
+type Point = number[]; // [number, number] | [number, number, number];
 
 declare function simplify<T extends Point>(
   points: T[],

packages/turf-simplify/lib/simplify.js

@@ -9,33 +9,33 @@
 
 // square distance between 2 points
 function getSqDist(p1, p2) {
-  var dx = p1.x - p2.x,
-    dy = p1.y - p2.y;
+  var dx = p1[0] - p2[0],
+    dy = p1[1] - p2[1];
 
   return dx * dx + dy * dy;
 }
 
 // square distance from a point to a segment
 function getSqSegDist(p, p1, p2) {
-  var x = p1.x,
-    y = p1.y,
-    dx = p2.x - x,
-    dy = p2.y - y;
+  var x = p1[0],
+    y = p1[1],
+    dx = p2[0] - x,
+    dy = p2[1] - y;
 
   if (dx !== 0 || dy !== 0) {
-    var t = ((p.x - x) * dx + (p.y - y) * dy) / (dx * dx + dy * dy);
+    var t = ((p[0] - x) * dx + (p[1] - y) * dy) / (dx * dx + dy * dy);
 
     if (t > 1) {
-      x = p2.x;
-      y = p2.y;
+      x = p2[0];
+      y = p2[1];
     } else if (t > 0) {
       x += dx * t;
       y += dy * t;
     }
   }
 
-  dx = p.x - x;
-  dy = p.y - y;
+  dx = p[0] - x;
+  dy = p[1] - y;
 
   return dx * dx + dy * dy;
 }