--update: another way to check anagrams

Author: ashokdey

src/anagrams/anagrams.test.js

@@ -1,39 +1,79 @@
-const { checkAnagrams } = require('.');
+const { checkAnagrams, checkAnagramUsingHelpers } = require('.');
 
 describe('Anagrams', () => {
-  it('Should return TRUE for `rail safety` & `fairy tales`', () => {
-    expect(
-      checkAnagrams({
-        firstString: 'rail safety',
-        secondString: 'fairy tales',
-      }),
-    ).toBe(true);
-  });
+  describe('Using cutom methods and character map', () => {
+    it('Should return TRUE for `rail safety` & `fairy tales`', () => {
+      expect(
+        checkAnagrams({
+          firstString: 'rail safety',
+          secondString: 'fairy tales',
+        }),
+      ).toBe(true);
+    });
 
-  it('Should return TRUE for `FAIRY tales` & `rail SAFETY`', () => {
-    expect(
-      checkAnagrams({
-        firstString: 'FAIRY tales',
-        secondString: 'rail SAFETY',
-      }),
-    ).toBe(true);
-  });
+    it('Should return TRUE for `FAIRY tales` & `rail SAFETY`', () => {
+      expect(
+        checkAnagrams({
+          firstString: 'FAIRY tales',
+          secondString: 'rail SAFETY',
+        }),
+      ).toBe(true);
+    });
+
+    it('Should return FALSE for `Hello World` & `Bye`', () => {
+      expect(
+        checkAnagrams({
+          firstString: 'Hello World',
+          secondString: 'Bye',
+        }),
+      ).toBe(false);
+    });
 
-  it('Should return FALSE for `Hello World` & `Bye`', () => {
-    expect(
-      checkAnagrams({
-        firstString: 'Hello World',
-        secondString: 'Bye',
-      }),
-    ).toBe(false);
+    it('Should ignore special characters', () => {
+      expect(
+        checkAnagrams({
+          firstString: 'hello world!!',
+          secondString: 'hello  -  world',
+        }),
+      ).toBe(true);
+    });
   });
 
-  it('Should ignore special characters', () => {
-    expect(
-      checkAnagrams({
-        firstString: 'hello world!!',
-        secondString: 'hello  -  world',
-      }),
-    ).toBe(true);
+  describe('Using in built methods and sorting', () => {
+    it('Should return TRUE for `rail safety` & `fairy tales`', () => {
+      expect(
+        checkAnagramUsingHelpers({
+          firstString: 'rail safety',
+          secondString: 'fairy tales',
+        }),
+      ).toBe(true);
+    });
+
+    it('Should return TRUE for `FAIRY tales` & `rail SAFETY`', () => {
+      expect(
+        checkAnagramUsingHelpers({
+          firstString: 'FAIRY tales',
+          secondString: 'rail SAFETY',
+        }),
+      ).toBe(true);
+    });
+
+    it('Should return FALSE for `Hello World` & `Bye`', () => {
+      expect(
+        checkAnagramUsingHelpers({
+          firstString: 'Hello World',
+          secondString: 'Bye',
+        }),
+      ).toBe(false);
+    });
+
+    it('Should ignore special characters', () => {
+      expect(
+        checkAnagramUsingHelpers({
+          firstString: 'hello world!!',
+          secondString: 'hello  -  world',
+        }),
+      ).toBe(true);
+    });
   });
 });

src/anagrams/index.js

@@ -1,6 +1,8 @@
+const pattern = /[^\w]/g;
+
 function createCharMap(str) {
   const charMap = {};
-  const sanitizedString = str.replace(/[^\w]/g, '').toLowerCase();
+  const sanitizedString = str.replace(pattern, '').toLowerCase();
   sanitizedString.split('').forEach((char) => {
     if (!charMap[char]) {
       charMap[char] = 1;
@@ -12,6 +14,15 @@ function createCharMap(str) {
   return charMap;
 }
 
+function sanitizeAndSortString(str) {
+  return str
+    .replace(pattern, '')
+    .toLowerCase()
+    .split('')
+    .sort()
+    .join('');
+}
+
 function checkAnagrams({ firstString, secondString }) {
   const charMapFirst = createCharMap(firstString);
   const charMapSecond = createCharMap(secondString);
@@ -30,7 +41,11 @@ function checkAnagrams({ firstString, secondString }) {
   return true;
 }
 
+function checkAnagramUsingHelpers({ firstString, secondString }) {
+  return sanitizeAndSortString(firstString) === sanitizeAndSortString(secondString);
+}
+
 module.exports = {
   checkAnagrams,
-  createCharMap,
+  checkAnagramUsingHelpers,
 };