refactor: perf getAllSessions, zéro warning analyze, docs à jour
- getAllSessions : élimine le N+1 (une requête par session + par analyse) au profit de 3 requêtes groupées (sessions, analyses IN, shots IN), découpées en paquets de 500 pour rester sous la limite SQLite. Assemblage en mémoire via maps — accueil/historique/stats ne ralentiront plus au fil des mois - flutter analyze : 28 -> 0 problème. APIs dépréciées migrées (withOpacity -> withValues, scale -> scaleByDouble, DropdownButtonFormField.value -> initialValue, dialogBackgroundColor -> dialogTheme, activeColor -> activeThumbColor), use_build_context_synchronously corrigés dans l'armurerie (repository capturé avant await + gardes mounted), print -> debugPrint, identifiants TopLeft/... -> lowerCamelCase, blocs if, champ final - _showShotDetails dédupliqué : bottom sheet extraite dans le widget partagé shot_details_sheet.dart, utilisée par l'écran d'analyse et l'éditeur d'impacts - CLAUDE.md : sections Features réécrites (retrait détection auto/références, ajout capture/plotting manuel) ; description pubspec renseignée Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
@@ -380,17 +380,71 @@ class DatabaseHelper {
|
||||
limit: limit,
|
||||
offset: offset,
|
||||
);
|
||||
if (sessionMaps.isEmpty) return [];
|
||||
|
||||
final sessions = <Session>[];
|
||||
for (final sessionMap in sessionMaps) {
|
||||
final sessionId = sessionMap['id'] as String;
|
||||
final session = await getSession(sessionId);
|
||||
if (session != null) {
|
||||
sessions.add(session);
|
||||
}
|
||||
// 3 requêtes groupées au lieu d'une cascade par session puis par analyse
|
||||
// (N+1) : l'ancien code refaisait un getSession complet pour chaque ligne,
|
||||
// ce qui ralentissait l'accueil/historique/stats au fil des mois.
|
||||
final sessionIds = sessionMaps.map((m) => m['id'] as String).toList();
|
||||
final analysisMaps = await _queryIn(
|
||||
db,
|
||||
AppConstants.targetAnalysesTable,
|
||||
'session_id',
|
||||
sessionIds,
|
||||
);
|
||||
final analysisIds = analysisMaps.map((m) => m['id'] as String).toList();
|
||||
final shotMaps = await _queryIn(
|
||||
db,
|
||||
AppConstants.shotsTable,
|
||||
'analysis_id',
|
||||
analysisIds,
|
||||
);
|
||||
|
||||
final shotsByAnalysis = <String, List<Shot>>{};
|
||||
for (final map in shotMaps) {
|
||||
(shotsByAnalysis[map['analysis_id'] as String] ??= [])
|
||||
.add(Shot.fromMap(map));
|
||||
}
|
||||
|
||||
return sessions;
|
||||
final analysesBySession = <String, List<TargetAnalysis>>{};
|
||||
for (final map in analysisMaps) {
|
||||
final analysis = TargetAnalysis.fromMap(
|
||||
map,
|
||||
shotsByAnalysis[map['id'] as String] ?? [],
|
||||
);
|
||||
(analysesBySession[map['session_id'] as String] ??= []).add(analysis);
|
||||
}
|
||||
|
||||
return sessionMaps
|
||||
.map((m) =>
|
||||
Session.fromMap(m, analysesBySession[m['id'] as String] ?? []))
|
||||
.toList();
|
||||
}
|
||||
|
||||
/// SELECT * FROM [table] WHERE [column] IN (values), découpé par paquets
|
||||
/// de 500 pour rester sous la limite de variables d'une requête SQLite (999).
|
||||
Future<List<Map<String, Object?>>> _queryIn(
|
||||
Database db,
|
||||
String table,
|
||||
String column,
|
||||
List<String> values,
|
||||
) async {
|
||||
if (values.isEmpty) return [];
|
||||
const chunkSize = 500;
|
||||
final results = <Map<String, Object?>>[];
|
||||
for (var i = 0; i < values.length; i += chunkSize) {
|
||||
final chunk = values.sublist(
|
||||
i,
|
||||
i + chunkSize > values.length ? values.length : i + chunkSize,
|
||||
);
|
||||
final placeholders = List.filled(chunk.length, '?').join(',');
|
||||
results.addAll(await db.query(
|
||||
table,
|
||||
where: '$column IN ($placeholders)',
|
||||
whereArgs: chunk,
|
||||
));
|
||||
}
|
||||
return results;
|
||||
}
|
||||
|
||||
Future<int> deleteSession(String id) async {
|
||||
|
||||
Reference in New Issue
Block a user