feat(api): Improve performance

dataesr · Dec 6, 2023 · 5b887c5 · 5b887c5
1 parent 2de356b
commit 5b887c5
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 8 deletions.
diff --git a/server/src/routes/works.routes.js b/server/src/routes/works.routes.js
@@ -36,8 +36,9 @@ router.route('/works')
         console.time(`4. Sort works ${options.affiliations}`);
         const publications = [];
         const datasets = [];
-        // TODO IMPROVE PERF by replacing the forEach by a for loop
-        deduplicatedWorks.forEach((deduplicatedWork) => {
+        const deduplicatedWorksLength = deduplicatedWorks.length;
+        for (let i = 0; i < deduplicatedWorksLength; i += 1) {
+          const deduplicatedWork = deduplicatedWorks[i];
           if (
             (deduplicatedWork.datasource.includes('fosm') && deduplicatedWork.type !== 'dataset')
             || (deduplicatedWork.datasource.includes('openalex') && deduplicatedWork.type !== 'dataset')
@@ -51,7 +52,7 @@ router.route('/works')
           } else {
             console.error(`Work not sorted : ${JSON.stringify(deduplicatedWork)}`);
           }
-        });
+        }
         console.timeEnd(`4. Sort works ${options.affiliations}`);
         // Compute distinct types & years for facet
         console.time(`5. Facet ${options.affiliations}`);

diff --git a/server/src/utils/utils.js b/server/src/utils/utils.js
@@ -113,10 +113,12 @@ const defaultDiacriticsRemovalMap = [
 ];
 
 const diacriticsMap = {};
-for (let i = 0; i < defaultDiacriticsRemovalMap.length; i += 1) {
-  const { letters } = defaultDiacriticsRemovalMap[i];
-  for (let j = 0; j < letters.length; j += 1) {
-    diacriticsMap[letters[j]] = defaultDiacriticsRemovalMap[i].base;
+const defaultDiacriticsRemovalMapLength = defaultDiacriticsRemovalMap.length;
+for (let i = 0; i < defaultDiacriticsRemovalMapLength; i += 1) {
+  const { base, letters } = defaultDiacriticsRemovalMap[i];
+  const lettersLength = letters.length;
+  for (let j = 0; j < lettersLength; j += 1) {
+    diacriticsMap[letters[j]] = base;
   }
 }
 

diff --git a/server/src/utils/works.js b/server/src/utils/works.js
@@ -216,7 +216,8 @@ const groupByAffiliations = ({ options, works }) => {
   // Compute distinct affiliations of works
   const allAffiliationsTmp = works.reduce((deduplicatedAffiliations, work) => {
     const { affiliations = [] } = work;
-    for (let i = 0; i < affiliations.length; i += 1) {
+    const affiliationsLength = affiliations.length;
+    for (let i = 0; i < affiliationsLength; i += 1) {
       const affiliation = affiliations[i];
       const normalizedAffiliation = removeDiacritics(affiliation);
       if (normalizedAffiliations.some((aff) => normalizedAffiliation.includes(aff))) {