verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 71 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 +2 -2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 4c2a576915 - Show all commits

4
src/composables/useArrayData.js
View File

@ -284,12 +284,12 @@ export function useArrayData(key = useRoute().meta.moduleName, userOptions) {
} }
function processData(data, { map = true, append = true }) { function processData(data, { map = true, append = true }) {
jorgep commented 2024-11-25 15:51:50 +00:00
Review
Copy Link

Se puede meter directamente el snippet en la fn fetch, pero , por legibilidad creo que es mejor dejarlo en esta fn.

Se puede meter directamente el snippet en la fn fetch, pero , por **legibilidad** creo que es mejor dejarlo en esta fn.
if (!store.data) { if (!append) {
store.data = []; store.data = [];
store.map = new Map(); store.map = new Map();
} }
if (!Array.isArray(data) || (!map && !append)) store.data = data; if (!Array.isArray(data)) store.data = data;
else if (!map && append) for (const row of data) store.data.push(row); 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 else
for (const row of data) { for (const row of data) {