verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 86 Releases Wiki Activity

#8113 create arrayDataStore map #979

Closed
jorgep wants to merge 17 commits from 8113-preventDuplicateRecords into dev
pull from: 8113-preventDuplicateRecords
merge into: verdnatura:dev
Conversation 1 Commits 17 Files Changed 5 +3 -3
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 9b86f2ddd7 - Show all commits

6
src/composables/useArrayData.js
View File

@ -289,10 +289,10 @@ export function useArrayData(key = useRoute().meta.moduleName, userOptions) {
store.map = new Map();
}
if (!map && !append) store.data = data;
if (!Array.isArray(data) || (!map && !append)) store.data = data;
else if (!map && append) for (const row of data) store.data.push(row);
jorgep commented 2024-11-25 16:29:17 +00:00
Review
Copy Link

Se podría usar ... pero he leído que en términos de rendimiento a la hora de manejar arrays muy grande es más rápido un for.

Se podría usar **...** pero he leído que en términos de rendimiento a la hora de manejar arrays muy grande es más rápido un for.
else
data.forEach((row) => {
for (const row of data) {
const key = row[store.mapKey];
const val = { ...row, key };
if (store.map.has(key)) {
@ -305,7 +305,7 @@ export function useArrayData(key = useRoute().meta.moduleName, userOptions) {
store.map.set(key, val);
store.data.push(val);
}
});
}
}
const totalRows = computed(() => (store.data && store.data.length) || 0);