refactor(geom-resample): update Sampler.closestPoint()

- re-use closestPointPolyline()
thi-ng · Sep 18, 2020 · 62472f5 · 62472f5
1 parent 3167a3d
commit 62472f5
Showing 1 changed file with 6 additions and 15 deletions.
diff --git a/packages/geom-resample/src/sampler.ts b/packages/geom-resample/src/sampler.ts
@@ -1,4 +1,8 @@
-import { closestPointSegment, closestT } from "@thi.ng/geom-closest-point";
+import {
+    closestPointPolyline,
+    closestPointSegment,
+    closestT,
+} from "@thi.ng/geom-closest-point";
 import { fit01 } from "@thi.ng/math";
 import {
     dist,
@@ -59,20 +63,7 @@ export class Sampler {
     }
 
     closestPoint(p: ReadonlyVec) {
-        const pts = this.points;
-        const tmp: Vec = [];
-        const closest: Vec = [];
-        let minD = Infinity;
-        for (let i = 0, n = pts.length - 1; i < n; i++) {
-            if (closestPointSegment(p, pts[i], pts[i + 1], tmp)) {
-                const d = distSq(p, tmp);
-                if (d < minD) {
-                    minD = d;
-                    set(closest, tmp);
-                }
-            }
-        }
-        return minD < Infinity ? closest : undefined;
+        return closestPointPolyline(p, this.points);
     }
 
     closestT(p: ReadonlyVec) {