perf: [10196] build a dict of the colums metadata before the main parsing loop

Author: arianber7

src/FileUtils/AgGridUtils.js

@@ -90,34 +90,39 @@ const _validateFormat = (rows, hasHeader, cols, options) => {
   const errors = [];
   const knownColsCount = colsData.length;
   const startIndex = hasHeader ? 1 : 0;
+
+  const colMeta = cols.map((col) => ({
+    field: col.field,
+    type: _getColTypeFromTypeArray(col.type),
+    acceptsEmptyFields: col.acceptsEmptyFields ?? col.cellEditorParams?.acceptsEmptyFields ?? false,
+    colOptions: {
+      ...options,
+      enumValues: col.enumValues ?? col.cellEditorParams?.enumValues,
+      minValue: col.minValue,
+      maxValue: col.maxValue,
+    },
+  }));
+
   for (let rowIndex = startIndex; rowIndex < rows.length; rowIndex++) {
     const row = rows[rowIndex];
     while (row[row.length - 1] === undefined && row.length > knownColsCount) row.pop();
-    if (row.length !== knownColsCount || row.includes(undefined))
-      _forgeColumnsCountError(row, rowIndex + 1, colsData, errors);
-    row.forEach((rowCell, colIndex) => {
-      if (colIndex < knownColsCount) {
-        const colType = colsData[colIndex].type;
-        if (colType && rowCell !== undefined) {
-          // use of cellEditorParams is deprecated
-          const colOptions = {
-            ...options,
-            enumValues: colsData[colIndex]?.enumValues ?? colsData[colIndex]?.cellEditorParams?.enumValues,
-            minValue: colsData[colIndex]?.minValue,
-            maxValue: colsData[colIndex]?.maxValue,
-          };
-          const acceptsEmptyFields =
-            // use of cellEditorParams is deprecated
-            colsData[colIndex].acceptsEmptyFields ?? colsData[colIndex].cellEditorParams?.acceptsEmptyFields ?? false;
-          const validationResult = ValidationUtils.isValid(rowCell, colType, colOptions, acceptsEmptyFields);
-          if (validationResult !== true) {
-            const { summary: errorSummary, context: errorContext } = validationResult;
-            const errorLoc = `Line ${rowIndex + 1}, Column ${colIndex + 1} ("${colsData[colIndex].field}")`;
-            errors.push(new PanelError(errorSummary, errorLoc, errorContext));
-          }
-        }
+    if (row.length !== knownColsCount || row.includes(undefined)) {
+      _forgeColumnsCountError(row, rowIndex + 1, cols, errors);
+      continue;
+    }
+
+    for (let colIndex = 0; colIndex < knownColsCount; colIndex++) {
+      const { type, colOptions, acceptsEmptyFields, field } = colMeta[colIndex];
+      const value = row[colIndex];
+      if (value === undefined) continue;
+
+      const validationResult = ValidationUtils.isValid(value, type, colOptions, acceptsEmptyFields);
+      if (validationResult !== true) {
+        const { summary: errorSummary, context: errorContext } = validationResult;
+        const errorLoc = `Line ${rowIndex + 1}, Column ${colIndex + 1} ("${field}")`;
+        errors.push(new PanelError(errorSummary, errorLoc, errorContext));
       }
-    });
+    }
   }
 
   return errors;