[MINOR] getMeasurementsSince is ongebruikte export #11

New issue

Open

opened 2026-08-05 21:11:35 +02:00 by eddy · 0 comments

eddy commented

2026-08-05 21:11:35 +02:00

Owner

Probleem

src/database/db.ts:78-84: getMeasurementsSince() is gedefinieerd en geëxporteerd maar wordt nergens in de codebase geïmporteerd. Het is dode code.

export async function getMeasurementsSince(date: string): Promise<Measurement[]> {
  const db = await initDatabase();
  return db.getAllAsync<Measurement>(
    'SELECT * FROM measurements WHERE createdAt > ? ORDER BY createdAt ASC', [date]
  );
}

Oplossing

Twee opties:

Verwijderen — als de functie achterhaald is
Gebruiken — in DashboardScreen.tsx voor charts in plaats van getAllMeasurements(). Bijv. alleen metingen van de afgelopen 365 dagen ophalen voor de WeightChart in plaats van alles

Optie 2 is zinvoller: het vermindert de data die geladen wordt voor charts op het dashboard.

## Probleem `src/database/db.ts:78-84`: `getMeasurementsSince()` is gedefinieerd en geëxporteerd maar wordt nergens in de codebase geïmporteerd. Het is dode code. ```ts export async function getMeasurementsSince(date: string): Promise<Measurement[]> { const db = await initDatabase(); return db.getAllAsync<Measurement>( 'SELECT * FROM measurements WHERE createdAt > ? ORDER BY createdAt ASC', [date] ); } ``` ## Oplossing Twee opties: 1. **Verwijderen** — als de functie achterhaald is 2. **Gebruiken** — in `DashboardScreen.tsx` voor charts in plaats van `getAllMeasurements()`. Bijv. alleen metingen van de afgelopen 365 dagen ophalen voor de WeightChart in plaats van alles Optie 2 is zinvoller: het vermindert de data die geladen wordt voor charts op het dashboard.

eddy added this to the Codereview augustus 2025 milestone

2026-08-05 21:14:31 +02:00