refactor: code cleanups
authorJérôme Benoit <jerome.benoit@piment-noir.org>
Tue, 21 May 2024 17:57:26 +0000 (19:57 +0200)
committerJérôme Benoit <jerome.benoit@piment-noir.org>
Tue, 21 May 2024 17:57:26 +0000 (19:57 +0200)
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
benchmark-utils.mjs
max.mjs
min.mjs
quick-select.mjs

index d0d5a62291527bc12387e18188991e83e307670a..8e32f90a5eca3bd6923a2fc2b67ee9d346776d3a 100644 (file)
@@ -26,7 +26,7 @@ export function generateRandomFloat (max = Number.MAX_VALUE, min = 0) {
   if (max < min) {
     throw new RangeError('Invalid interval')
   }
-  if (max - min === Infinity) {
+  if (max - min === Number.POSITIVE_INFINITY) {
     throw new RangeError('Invalid interval')
   }
   return (randomBytes(4).readUInt32LE() / 0xffffffff) * (max - min) + min
diff --git a/max.mjs b/max.mjs
index 23fcfdfb8762f65f91ef23d2a6cf3a9af21693fa..8675f8292dfe81ed07f11ce5f84e5bcd8c7a2280 100644 (file)
--- a/max.mjs
+++ b/max.mjs
@@ -11,7 +11,7 @@ const testArray = generateRandomNumberArray(size)
  * @returns
  */
 function loopMax (values) {
-  let maximum = -Infinity
+  let maximum = Number.NEGATIVE_INFINITY
   for (const value of values) {
     if (value > maximum) maximum = value
   }
@@ -26,7 +26,7 @@ function loopMax (values) {
 function reduceTernaryMax (values) {
   return values.reduce(
     (maximum, num) => (maximum > num ? maximum : num),
-    -Infinity
+    Number.NEGATIVE_INFINITY
   )
 }
 
@@ -36,7 +36,10 @@ function reduceTernaryMax (values) {
  * @returns
  */
 function reduceMathMax (values) {
-  return values.reduce((maximum, num) => Math.max(maximum, num), -Infinity)
+  return values.reduce(
+    (maximum, num) => Math.max(maximum, num),
+    Number.NEGATIVE_INFINITY
+  )
 }
 
 /**
diff --git a/min.mjs b/min.mjs
index 858b1eb3fc9c7e5d4b4cc456f1925b9f9205ce6b..51ce4ce060d93d7c7b96245d763d8fee522d4616 100644 (file)
--- a/min.mjs
+++ b/min.mjs
@@ -11,7 +11,7 @@ const testArray = generateRandomNumberArray(size)
  * @returns
  */
 function loopMin (values) {
-  let minimum = Infinity
+  let minimum = Number.POSITIVE_INFINITY
   for (const value of values) {
     if (value < minimum) minimum = value
   }
@@ -26,7 +26,7 @@ function loopMin (values) {
 function reduceTernaryMin (values) {
   return values.reduce(
     (minimum, num) => (minimum < num ? minimum : num),
-    Infinity
+    Number.POSITIVE_INFINITY
   )
 }
 
@@ -36,7 +36,10 @@ function reduceTernaryMin (values) {
  * @returns
  */
 function reduceMathMin (values) {
-  return values.reduce((minimum, num) => Math.min(minimum, num), Infinity)
+  return values.reduce(
+    (minimum, num) => Math.min(minimum, num),
+    Number.POSITIVE_INFINITY
+  )
 }
 
 /**
index 49dc7057b5c70d30ad5894169cabfd02df409516..8f588f2d5dce372f4283d36b4947eac0512911bf 100644 (file)
@@ -27,7 +27,7 @@ const tasksMap = generateRandomTasksMap(60, 20)
  */
 function loopSelect (tasksMap) {
   let minKey
-  let minValue = Infinity
+  let minValue = Number.POSITIVE_INFINITY
   for (const [key, value] of tasksMap) {
     if (value === 0) {
       return key