refactor: streamline CNPJ validation logic and update error messages

Author: gabriel-logan

src/cnpjValidator/cnpjValidator1.ts

@@ -51,22 +51,6 @@ function cnpjIsValid(
   cnpj: string,
   errorMsg: (string | null)[] | null = defaultErrorMsg,
 ): ValidateFunctions {
-  if (typeof cnpj !== "string") {
-    throw new TypeError("The input should be a string.");
-  }
-  // Check para saber se as mensagens que sao passadas sao validas
-  // caso contrario retorna um ERRO
-  if (errorMsg) {
-    if (!Array.isArray(errorMsg)) throw new Error("Must be an Array");
-    for (const element of errorMsg) {
-      if (element != null && typeof element !== "string") {
-        throw new TypeError(
-          "All values within the array must be strings or null/undefined.",
-        );
-      }
-    }
-  }
-
   // Função interna para obter a mensagem de erro
   function getErrorMessage(index: number): string {
     const errorMessage: string | null = errorMsg ? errorMsg[index] : null;
@@ -108,4 +92,5 @@ function cnpjIsValid(
     errorMsg: getErrorMessage(2), // 'CNPJ is not valid'
   };
 }
+
 export default cnpjIsValid;

src/cnpjValidator/cnpjValidator2.ts

@@ -2,7 +2,7 @@ import type { ValidateFunctions } from "../types";
 
 const defaultErrorMsg: string[] = [
   "CNPJ invalid",
-  "CNPJ must have 14 numerical digits",
+  "CNPJ must have 14 alphanumerical digits",
   "CNPJ is not valid",
 ];
 
@@ -73,22 +73,6 @@ function cnpjIsValid(
   cnpj: string,
   errorMsg: (string | null)[] | null = defaultErrorMsg,
 ): ValidateFunctions {
-  if (typeof cnpj !== "string") {
-    throw new TypeError("The input should be a string.");
-  }
-  // Check para saber se as mensagens que sao passadas sao validas
-  // caso contrario retorna um ERRO
-  if (errorMsg) {
-    if (!Array.isArray(errorMsg)) throw new Error("Must be an Array");
-    for (const element of errorMsg) {
-      if (element != null && typeof element !== "string") {
-        throw new TypeError(
-          "All values within the array must be strings or null/undefined.",
-        );
-      }
-    }
-  }
-
   // Função interna para obter a mensagem de erro
   function getErrorMessage(index: number): string {
     const errorMessage: string | null = errorMsg ? errorMsg[index] : null;

src/cnpjValidator/index.ts

@@ -2,22 +2,44 @@ import type { ValidateFunctions } from "../types";
 import cnpjIsValid1 from "./cnpjValidator1";
 import cnpjIsValid2 from "./cnpjValidator2";
 
-const defaultErrorMsg: string[] = [
-  "CNPJ invalid",
-  "CNPJ must have 14 numerical digits",
-  "CNPJ is not valid",
-];
-
 function cnpjIsValid(
   cnpj: string,
-  errorMsg: (string | null)[] | null = defaultErrorMsg,
-  cnpjVersion: "v1" | "v2" = "v1",
+  errorMsg: (string | null)[] | null = null,
+  cnpjVersion: "v1" | "v2" | null = null,
 ): ValidateFunctions {
+  if (typeof cnpj !== "string") {
+    throw new TypeError("The input should be a string.");
+  }
+
+  // Check para saber se as mensagens que sao passadas sao validas
+  // caso contrario retorna um ERRO
+  if (errorMsg) {
+    if (!Array.isArray(errorMsg)) throw new Error("Must be an Array");
+
+    for (const element of errorMsg) {
+      if (element != null && typeof element !== "string") {
+        throw new TypeError(
+          "All values within the array must be strings or null/undefined.",
+        );
+      }
+    }
+  }
+
+  if (cnpjVersion === "v1") {
+    return cnpjIsValid1(cnpj, errorMsg);
+  }
+
   if (cnpjVersion === "v2") {
     return cnpjIsValid2(cnpj, errorMsg);
   }
 
-  return cnpjIsValid1(cnpj, errorMsg);
+  const isOnlyNumbers: boolean = /^\d+$/.test(cnpj.replace(/\D/g, ""));
+
+  if (isOnlyNumbers) {
+    return cnpjIsValid1(cnpj, errorMsg);
+  }
+
+  return cnpjIsValid2(cnpj, errorMsg);
 }
 
 export default cnpjIsValid;