From 92eb59d2efd363a2bf4eebe88136902487589ced Mon Sep 17 00:00:00 2001 From: Julian Appel Date: Wed, 19 Aug 2026 20:39:12 +0200 Subject: [PATCH] Teamdatenbank ohne lowdb, Pfad unabhaengig vom Startverzeichnis lowdb v7 ist ESM-only, das Projekt ist CommonJS. Der bisherige Umweg ueber eine async IIFE mit await import() hatte zur Folge, dass db bei einem sehr fruehen Request noch undefined sein konnte. Fuer eine Liste von Teamnamen genuegt fs: read/write lesen und schreiben db.json direkt und synchron, die Race-Condition entfaellt und lowdb kann als Abhaengigkeit raus. Die exportierte API bleibt unveraendert, score.js und routes/admin.js brauchen keine Anpassung. Der Pfad zu db.json wird jetzt ueber __dirname aufgeloest statt relativ zum Arbeitsverzeichnis, damit die Datei unabhaengig vom Startverzeichnis immer an derselben Stelle liegt. Fehlt oder bricht die Datei, wird auf eine leere Teamliste zurueckgefallen. Co-Authored-By: Claude Opus 5 --- scoreboard/controllers/db.js | 61 ++++++++++++++++++++++-------------- scoreboard/package-lock.json | 26 --------------- scoreboard/package.json | 1 - 3 files changed, 38 insertions(+), 50 deletions(-) diff --git a/scoreboard/controllers/db.js b/scoreboard/controllers/db.js index a135404..73e29d3 100644 --- a/scoreboard/controllers/db.js +++ b/scoreboard/controllers/db.js @@ -1,46 +1,61 @@ "use strict"; -let db; +// Datenbank-Controller +// Haelt die Liste der bekannten Teamnamen persistent in db.json vor, +// damit sie im Adminpanel als Auswahl-Dropdown angeboten werden koennen. -// Import ESM into CJS file using immediately invoked function expression because of missing top-level await in CJS -(async () => { - const { LowSync } = await import('lowdb'); - const { JSONFileSync } = await import('lowdb/node'); - db = new LowSync(new JSONFileSync('db.json'), { teams: []}) -})(); +const fs = require('node:fs'); +const path = require('node:path'); +// Pfad relativ zum Modul, nicht zum Arbeitsverzeichnis, damit die Datei +// unabhaengig vom Startverzeichnis immer an derselben Stelle liegt +const dbFile = path.join(__dirname, '..', 'db.json'); + +// Liest db.json ein, faellt bei fehlender oder defekter Datei auf eine leere Liste zurueck +function read() { + try { + const data = JSON.parse(fs.readFileSync(dbFile, 'utf-8')); + if(!Array.isArray(data.teams)) data.teams = []; // Fehlender oder falscher teams-Key + return data; + } catch (err) { + console.error("db.json konnte nicht gelesen werden, verwende leere Teamliste: ", err.message); + return { teams: [] }; + } +} + +// Schreibt den uebergebenen Datensatz zurueck nach db.json +function write(data) { + fs.writeFileSync(dbFile, JSON.stringify(data, null, 2)); +} + +// Returns all teams from db function getTeams() { - db.read(); - return db.data.teams + return read().teams; } // Adds passed teamName to db if not already existing function addTeam(teamName) { - console.log(teamName) - db.read(); - if(!db.data.teams.includes(teamName) && teamName != "") { // Teamname not in db - db.data.teams.push(teamName); - db.write(); - } else { // Teamname already in db + const data = read(); + if(!data.teams.includes(teamName) && teamName != "") { // Teamname not in db + data.teams.push(teamName); + write(data); } } // Deletes passed teamName from db if existing function deleteTeam(teamName) { - db.read(); - if(db.data.teams.includes(teamName) && teamName != "") { // Teamname in db - db.data.teams.splice(db.data.teams.indexOf(teamName), 1) // Delete corresponding array entry - db.write(); + const data = read(); + if(data.teams.includes(teamName) && teamName != "") { // Teamname in db + data.teams.splice(data.teams.indexOf(teamName), 1); // Delete corresponding array entry + write(data); } } // Returns important values from db function getValues() { - db.read(); - console.log(db.data) - return db.data + return read(); } module.exports = { getTeams, addTeam, deleteTeam, getValues -} \ No newline at end of file +} diff --git a/scoreboard/package-lock.json b/scoreboard/package-lock.json index 4eea9ad..5e9c263 100644 --- a/scoreboard/package-lock.json +++ b/scoreboard/package-lock.json @@ -14,7 +14,6 @@ "express-ws": "^5.0.2", "hbs": "~4.0.4", "http-errors": "~1.6.3", - "lowdb": "^7.0.1", "moment": "^2.30.1", "morgan": "~1.9.1", "socket.io": "^4.7.5", @@ -470,20 +469,6 @@ "node": ">= 0.10" } }, - "node_modules/lowdb": { - "version": "7.0.1", - "resolved": "https://registry.npmjs.org/lowdb/-/lowdb-7.0.1.tgz", - "integrity": "sha512-neJAj8GwF0e8EpycYIDFqEPcx9Qz4GUho20jWFR7YiFeXzF1YMLdxB36PypcTSPMA+4+LvgyMacYhlr18Zlymw==", - "dependencies": { - "steno": "^4.0.2" - }, - "engines": { - "node": ">=18" - }, - "funding": { - "url": "https://github.com/sponsors/typicode" - } - }, "node_modules/media-typer": { "version": "0.3.0", "resolved": "https://registry.npmjs.org/media-typer/-/media-typer-0.3.0.tgz", @@ -858,17 +843,6 @@ "node": ">= 0.6" } }, - "node_modules/steno": { - "version": "4.0.2", - "resolved": "https://registry.npmjs.org/steno/-/steno-4.0.2.tgz", - "integrity": "sha512-yhPIQXjrlt1xv7dyPQg2P17URmXbuM5pdGkpiMB3RenprfiBlvK415Lctfe0eshk90oA7/tNq7WEiMK8RSP39A==", - "engines": { - "node": ">=18" - }, - "funding": { - "url": "https://github.com/sponsors/typicode" - } - }, "node_modules/type-is": { "version": "1.6.18", "resolved": "https://registry.npmjs.org/type-is/-/type-is-1.6.18.tgz", diff --git a/scoreboard/package.json b/scoreboard/package.json index 63f7a92..3fad970 100644 --- a/scoreboard/package.json +++ b/scoreboard/package.json @@ -12,7 +12,6 @@ "express-ws": "^5.0.2", "hbs": "~4.0.4", "http-errors": "~1.6.3", - "lowdb": "^7.0.1", "moment": "^2.30.1", "morgan": "~1.9.1", "socket.io": "^4.7.5",