chore: minor cleanups to Dimensions

bradenmacdonald · Mar 12, 2024 · 651ff6b · 651ff6b
1 parent 400aeab
commit 651ff6b
Showing 1 changed file with 7 additions and 18 deletions.
diff --git a/dimensions.ts b/dimensions.ts
@@ -52,9 +52,7 @@ export class Dimensions {
         ] = [],
     ) {
         if (dimensions.length < numBasicDimensions) {
-            throw new QuantityError(
-                "not enough dimensions specified for Quantity.",
-            );
+            throw new QuantityError("not enough dimensions specified for Quantity.");
         }
 
         const numCustomDimensions = customDimensionNames.length;
@@ -64,18 +62,12 @@ export class Dimensions {
             );
         }
 
-        if (customDimensionNames.length) {
+        if (numCustomDimensions) {
             // Make sure customDimensionNames is sorted in alphabetical order, for consistency.
             // This also validated that there are no duplicate custom dimensions (["floop", "floop"])
-            const isSorted = customDimensionNames.every((
-                v,
-                i,
-                a,
-            ) => (i === 0 || v! > a[i - 1]!));
+            const isSorted = customDimensionNames.every((v, i, a) => (i === 0 || v! > a[i - 1]!));
             if (!isSorted) {
-                throw new QuantityError(
-                    "customDimensionNames is not sorted into the correct alphabetical order.",
-                );
+                throw new QuantityError("customDimensionNames is not sorted into the correct alphabetical order.");
             }
         }
     }
@@ -180,18 +172,15 @@ export class Dimensions {
 
     /** Raise these dimensions to a power */
     public pow(n: number): Dimensions {
-        if (typeof n !== "number" || isNaN(n) || !Number.isInteger(n)) {
+        if (!Number.isInteger(n)) {
             throw new QuantityError(`Dimensions.pow(n): n must be an integer`);
         }
         if (n === 0) {
             return Dimensionless;
         }
         const newDimArray = this.dimensions.map((d) => d! * n);
-        return new Dimensions(
-            // deno-lint-ignore no-explicit-any
-            newDimArray as any,
-            this.customDimensionNames,
-        );
+        // deno-lint-ignore no-explicit-any
+        return new Dimensions(newDimArray as any, this.customDimensionNames);
     }
 
     /** Use a nice string when logging this with Deno's console.log() etc. */