refactor: core functions

Author: ram-amoncar

src/case.ts

@@ -157,7 +157,7 @@ function snakeCase(str: string, inWords = false, locales?: Intl.LocalesArgument)
     const numbersInWords = ['zero', 'one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine']
     if (inWords) {
         for (let i = 0; i < numbersInWords.length; i++) {
-            str = str.replace(RegExp(i.toString(), 'g'), (_char: string) => ' ' + numbersInWords[i] + ' ')
+            str = str.replace(RegExp(i.toString(), 'g'), () => ' ' + numbersInWords[i] + ' ')
         }
     }
 
@@ -181,7 +181,7 @@ function kebabCase(str: string, inWords = false, locales?: Intl.LocalesArgument)
     const numbersInWords = ['zero', 'one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine']
     if (inWords) {
         for (let i = 0; i < numbersInWords.length; i++) {
-            str = str.replace(RegExp(i.toString(), 'g'), (_char: string) => ' ' + numbersInWords[i] + ' ')
+            str = str.replace(RegExp(i.toString(), 'g'), () => ' ' + numbersInWords[i] + ' ')
         }
     }
 

src/merge.ts

@@ -22,12 +22,12 @@
  */
 function merge(sep: string | boolean, ...strings: string[] | string[][]): string {
     strings = strings.flat()
-    for (let i = 0, val = ''; i < strings.length; i++) {
-        if (strings.length - 1 === i) return val + strings[i]
-        else if (typeof sep === 'boolean') val += sep ? strings[i] + ' ' : strings[i]
-        else val += strings[i] + sep
+    if (typeof sep === 'boolean') {
+        const separator = sep ? ' ' : ''
+        return strings.length === 0 ? separator : strings.join(separator)
     }
-    return typeof sep === 'boolean' ? (sep ? ' ' : '') : sep
+    if (strings.length === 0) return sep
+    return strings.join(sep)
 }
 
 export { merge }

src/padding.ts

@@ -24,14 +24,12 @@ interface PaddingOptions {
  * padStart('hello', 'abc', 3, 8) // abchello
  */
 function padStart(val: string, fillString = '\u0020', repeatCount = 1, maxLen?: number): string {
-    if (typeof maxLen !== 'undefined' && maxLen <= val.length) return val
-    else if (typeof maxLen === 'undefined') {
-        for (let i = 0; i < repeatCount; i++) val = fillString + val
-        return val
-    }
-    for (let i = 0; i < repeatCount && val.length <= maxLen; i++) val = fillString + val
-    const finalVal = val.length > maxLen ? val.substring(Math.abs(maxLen - val.length)) : val
-    return finalVal
+    const padding = fillString.repeat(repeatCount)
+    if (typeof maxLen === 'undefined') return padding + val
+    if (maxLen <= val.length) return val
+
+    const remSpace = maxLen - val.length
+    return padding.slice(-remSpace) + val
 }
 
 /**
@@ -49,13 +47,12 @@ function padStart(val: string, fillString = '\u0020', repeatCount = 1, maxLen?:
  * padEnd('hello', 'abc', 3, 8); // helloabc
  */
 function padEnd(val: string, fillString = '\u0020', repeatCount = 1, maxLen?: number): string {
-    if (typeof maxLen !== 'undefined' && maxLen <= val.length) return val
-    else if (typeof maxLen === 'undefined') {
-        for (let i = 0; i < repeatCount; i++) val += fillString
-        return val
-    }
-    for (let i = 0; i < repeatCount && val.length <= maxLen; i++) val += fillString
-    return val.substring(0, maxLen)
+    const padding = fillString.repeat(repeatCount)
+    if (typeof maxLen === 'undefined') return val + padding
+    if (maxLen <= val.length) return val
+
+    const remSpace = maxLen - val.length
+    return val + padding.substring(0, remSpace)
 }
 
 /**

src/regionMatchers.ts

@@ -83,12 +83,17 @@ function looseRegionMatch(
     end?: number,
     locales?: Intl.LocalesArgument
 ): boolean {
-    if (typeof str1 === 'string') str1 = str1.toLocaleLowerCase(locales)
-    else str1.str = str1.str.toLocaleLowerCase(locales)
-    if (typeof str2 === 'string') str2 = str2.toLocaleLowerCase(locales)
-    else str2.str = str2.str.toLocaleLowerCase(locales)
+    if (typeof str1 === 'string') {
+        const e = end ?? str1.length
+        str1 = { str: str1.toLocaleLowerCase(locales), start, end: e }
+    } else str1.str = str1.str.toLocaleLowerCase(locales)
+
+    if (typeof str2 === 'string') {
+        const e = end ?? str2.length
+        str2 = { str: str2.toLocaleLowerCase(locales), start, end: e }
+    } else str2.str = str2.str.toLocaleLowerCase(locales)
 
-    return regionMatch(str1, str2, start, end)
+    return regionMatch(str1, str2)
 }
 
 export { regionMatch, looseRegionMatch, type StringRegion }