kaufi/kantinen-wrapper
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #9 from TauNeutrino/performance-optimization-2237739564462318710

Browse Source 
⚡ optimize menu data load to fetch concurrently
This commit is contained in:
Michael
2026-03-10 15:06:18 +01:00
committed by GitHub
parent 1eb2034c61 d365b71ee6
commit 029bcf012c
4 changed files with 128 additions and 60 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
benchmark_loadMenuDataFromAPI.js Normal file
View File

@@ -0,0 +1,22 @@
const fs = require('fs');
async function benchmark() {
// We will simulate the same exact loop in src/actions.js
let availableDates = Array.from({length: 30}).map((_, i) => ({ date: `2024-01-${i+1}`}));
const totalDates = availableDates.length;
let completed = 0;
console.log(`Starting benchmark for ${totalDates} items (Sequential with 100ms artificial delay)`);
const start = Date.now();
for (const dateObj of availableDates) {
// mock fetch
await new Promise(r => setTimeout(r, 50)); // simulate network delay
completed++;
await new Promise(r => setTimeout(r, 100)); // the artificial delay in codebase
}
const end = Date.now();
console.log(`Sequential loading took ${end - start}ms`);
}
benchmark();

26
benchmark_loadMenuDataFromAPI_concurrent.js Normal file
View File

@@ -0,0 +1,26 @@
const fs = require('fs');
async function benchmark() {
let availableDates = Array.from({length: 30}).map((_, i) => ({ date: `2024-01-${i+1}`}));
const totalDates = availableDates.length;
let completed = 0;
console.log(`Starting benchmark for ${totalDates} items (Concurrent batch=5 without 100ms artificial delay)`);
const start = Date.now();
// Simulate Promise.all batching approach
const BATCH_SIZE = 5;
for (let i = 0; i < totalDates; i += BATCH_SIZE) {
const batch = availableDates.slice(i, i + BATCH_SIZE);
await Promise.all(batch.map(async (dateObj) => {
// mock fetch
await new Promise(r => setTimeout(r, 50)); // simulate network delay
completed++;
}));
}
const end = Date.now();
console.log(`Concurrent loading took ${end - start}ms`);
}
benchmark();

32
dist/kantine.bundle.js vendored
View File

@@ -783,13 +783,12 @@ async function loadMenuDataFromAPI() {
const allDays = [];
let completed = 0;
for (const dateObj of availableDates) {
const BATCH_SIZE = 5;
for (let i = 0; i < totalDates; i += BATCH_SIZE) {
const batch = availableDates.slice(i, i + BATCH_SIZE);
const results = await Promise.all(batch.map(async (dateObj) => {
const dateStr = dateObj.date;
const pct = Math.round(((completed + 1) / totalDates) * 100);
progressFill.style.width = `${pct}%`;
progressPercent.textContent = `${pct}%`;
progressMessage.textContent = `Lade Menü für ${dateStr}...`;
let dayData = null;
try {
const detailResp = await fetch(`${_constants_js__WEBPACK_IMPORTED_MODULE_2__/* .API_BASE */ .tE}/venues/${_constants_js__WEBPACK_IMPORTED_MODULE_2__/* .VENUE_ID */ .eW}/menu/${_constants_js__WEBPACK_IMPORTED_MODULE_2__/* .MENU_ID */ .YU}/${dateStr}/`, {
headers: (0,_api_js__WEBPACK_IMPORTED_MODULE_3__/* .apiHeaders */ .H)(token)
@@ -805,19 +804,30 @@ async function loadMenuDataFromAPI() {
}
}
if (dayItems.length > 0) {
allDays.push({
dayData = {
date: dateStr,
menu_items: dayItems,
orders: dateObj.orders || []
});
};
}
}
} catch (err) {
console.error(`Failed to fetch details for ${dateStr}:`, err);
}
} finally {
completed++;
await new Promise(r => setTimeout(r, 100));
const pct = Math.round((completed / totalDates) * 100);
progressFill.style.width = `${pct}%`;
progressPercent.textContent = `${pct}%`;
progressMessage.textContent = `Lade Menü für ${dateStr}...`;
}
return dayData;
}));
for (const result of results) {
if (result) {
allDays.push(result);
}
}
}
const weeksMap = new Map();

32
src/actions.js
View File

@@ -753,13 +753,12 @@ export async function loadMenuDataFromAPI() {
const allDays = [];
let completed = 0;
for (const dateObj of availableDates) {
const BATCH_SIZE = 5;
for (let i = 0; i < totalDates; i += BATCH_SIZE) {
const batch = availableDates.slice(i, i + BATCH_SIZE);
const results = await Promise.all(batch.map(async (dateObj) => {
const dateStr = dateObj.date;
const pct = Math.round(((completed + 1) / totalDates) * 100);
progressFill.style.width = `${pct}%`;
progressPercent.textContent = `${pct}%`;
progressMessage.textContent = `Lade Menü für ${dateStr}...`;
let dayData = null;
try {
const detailResp = await fetch(`${API_BASE}/venues/${VENUE_ID}/menu/${MENU_ID}/${dateStr}/`, {
headers: apiHeaders(token)
@@ -775,19 +774,30 @@ export async function loadMenuDataFromAPI() {
}
}
if (dayItems.length > 0) {
allDays.push({
dayData = {
date: dateStr,
menu_items: dayItems,
orders: dateObj.orders || []
});
};
}
}
} catch (err) {
console.error(`Failed to fetch details for ${dateStr}:`, err);
}
} finally {
completed++;
await new Promise(r => setTimeout(r, 100));
const pct = Math.round((completed / totalDates) * 100);
progressFill.style.width = `${pct}%`;
progressPercent.textContent = `${pct}%`;
progressMessage.textContent = `Lade Menü für ${dateStr}...`;
}
return dayData;
}));
for (const result of results) {
if (result) {
allDays.push(result);
}
}
}
const weeksMap = new Map();