remove switch

Signed-off-by: francesco <[email protected]>
fastify · Apr 11, 2024 · 16c54b9 · 16c54b9
1 parent 49e059c
commit 16c54b9
Showing 1 changed file with 23 additions and 30 deletions.
diff --git a/lib/serializer.js b/lib/serializer.js
@@ -4,7 +4,7 @@
 const STR_ESCAPE = /[\u0000-\u001f\u0022\u005c\ud800-\udfff]/
 
 module.exports = class Serializer {
-  constructor (options) {
+  constructor(options) {
     switch (options && options.rounding) {
       case 'floor':
         this.parseInteger = Math.floor
@@ -23,30 +23,23 @@ module.exports = class Serializer {
     this._options = options
   }
 
-  asInteger (i) {
-    switch (typeof i) {
-      case 'number':
-      case 'boolean':
-      case 'string':
-      case 'object': {
-        if (Number.isInteger(i)) {
-          return '' + i
-        }
-        /* eslint no-undef: "off" */
-        const integer = this.parseInteger(i)
-        // check if number is Infinity or NaN
-        // eslint-disable-next-line no-self-compare
-        if (integer === Infinity || integer === -Infinity || integer !== integer) {
-          throw new Error(`The value "${i}" cannot be converted to an integer.`)
-        }
-        return '' + integer
-      }
-      case 'bigint': return i.toString()
-      default: throw new Error(`The value "${i}" cannot be converted to an integer.`)
+  asInteger(i) {
+    if (Number.isInteger(i)) {
+      return '' + i
+    } else if (typeof i === 'bigint') {
+      return i.toString()
+    }
+    /* eslint no-undef: "off" */
+    const integer = this.parseInteger(i)
+    // check if number is Infinity or NaN
+    // eslint-disable-next-line no-self-compare
+    if (integer === Infinity || integer === -Infinity || integer !== integer) {
+      throw new Error(`The value "${i}" cannot be converted to an integer.`)
     }
+    return '' + integer
   }
 
-  asNumber (i) {
+  asNumber(i) {
     // fast cast to number
     const num = +i
     // check if number is NaN
@@ -60,11 +53,11 @@ module.exports = class Serializer {
     }
   }
 
-  asBoolean (bool) {
+  asBoolean(bool) {
     return bool && 'true' || 'false' // eslint-disable-line
   }
 
-  asDateTime (date) {
+  asDateTime(date) {
     if (date === null) return '""'
     if (date instanceof Date) {
       return '"' + date.toISOString() + '"'
@@ -75,7 +68,7 @@ module.exports = class Serializer {
     throw new Error(`The value "${date}" cannot be converted to a date-time.`)
   }
 
-  asDate (date) {
+  asDate(date) {
     if (date === null) return '""'
     if (date instanceof Date) {
       return '"' + new Date(date.getTime() - (date.getTimezoneOffset() * 60000)).toISOString().slice(0, 10) + '"'
@@ -86,7 +79,7 @@ module.exports = class Serializer {
     throw new Error(`The value "${date}" cannot be converted to a date.`)
   }
 
-  asTime (date) {
+  asTime(date) {
     if (date === null) return '""'
     if (date instanceof Date) {
       return '"' + new Date(date.getTime() - (date.getTimezoneOffset() * 60000)).toISOString().slice(11, 19) + '"'
@@ -97,7 +90,7 @@ module.exports = class Serializer {
     throw new Error(`The value "${date}" cannot be converted to a time.`)
   }
 
-  asString (str) {
+  asString(str) {
     const len = str.length
     if (len < 42) {
       // magically escape strings for json
@@ -133,15 +126,15 @@ module.exports = class Serializer {
     }
   }
 
-  asUnsafeString (str) {
+  asUnsafeString(str) {
     return '"' + str + '"'
   }
 
-  getState () {
+  getState() {
     return this._options
   }
 
-  static restoreFromState (state) {
+  static restoreFromState(state) {
     return new Serializer(state)
   }
 }