From aea3ccd07b5043e4d189f792d5ee9be76848393a Mon Sep 17 00:00:00 2001
From: LimaniBhavik <6619586+LimaniBhavik@users.noreply.github.com>
Date: Wed, 27 May 2026 06:37:24 +0000
Subject: [PATCH] Performance: Optimize map conversion loop in DatabaseHelper

* Use `List.generate(..., growable: false)` instead of `map(...).toList()`
* Avoids closure overhead and prevents reallocation for large result sets
---
 lib/Utils/DatabaseHelper.dart | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/lib/Utils/DatabaseHelper.dart b/lib/Utils/DatabaseHelper.dart
index 10289f9..e0765bf 100644
--- a/lib/Utils/DatabaseHelper.dart
+++ b/lib/Utils/DatabaseHelper.dart
@@ -23,8 +23,8 @@ class DatabaseHelper {
 
   factory DatabaseHelper() {
     if (_databaseHelper == null) {
-      _databaseHelper =
-          DatabaseHelper._createInstance(); // This is executed only once, singleton object
+      _databaseHelper = DatabaseHelper
+          ._createInstance(); // This is executed only once, singleton object
     }
     return _databaseHelper!;
   }
@@ -121,9 +121,11 @@ class DatabaseHelper {
   Future<List<VideoModel>> getVidList() async {
     var vidMapList = await getVidMapList(); // Get 'Map List' from database
 
-    List<VideoModel> vidList = vidMapList
-        .map((map) => VideoModel.fromMapObject(map))
-        .toList();
+    List<VideoModel> vidList = List.generate(
+      vidMapList.length,
+      (i) => VideoModel.fromMapObject(vidMapList[i]),
+      growable: false,
+    );
 
     return vidList;
   }