fix: search error and cache videos

Author: kkweon

client/lib/screens/main_screen.dart

@@ -46,19 +46,15 @@ class _MainScreenState extends State<MainScreen> {
               body: IndexedStack(
                 index: _selectedBottomNavIndex,
                 children: [
-                  MainScreenListView(
-                      cleanList: cleanList,
-                      videoSearchDelegate: videoSearchDelegate),
-                  MainScreenFavoriteView(
-                      cleanList: cleanList,
-                      videoSearchDelegate: videoSearchDelegate)
+                  MainScreenListView(cleanList: cleanList),
+                  MainScreenFavoriteView(cleanList: cleanList)
                 ],
               ),
               bottomNavigationBar: CustomBottomNavigationBar(
                   selectedBottomNavIndex: _selectedBottomNavIndex,
                   onTap: (index) => setState(() {
                         _selectedBottomNavIndex = index;
-                        videoSearchDelegate.ignoreBookmarkIcon = index == 1;
+                        videoSearchDelegate.showOnlyBookmarkItems = index == 1;
                       })));
         });
   }

client/lib/service.dart

@@ -11,6 +11,8 @@ class GrpcClient {
   final Pr12erServiceClient _client =
       Pr12erServiceClient(getKkweonOktetoChannel());
 
+  List<Video>? _videoCache;
+
   Future<String> sendMessage(String message) async {
     final request = HelloRequest()..body = message;
     final response = await _client.getHello(request);
@@ -19,10 +21,14 @@ class GrpcClient {
   }
 
   Future<List<Video>> getVideos() async {
+    if (_videoCache != null) return _videoCache!;
+
     final request = GetVideosRequest();
     final response = await _client.getVideos(request);
 
-    return response.videos;
+    _videoCache = response.videos;
+
+    return _videoCache!;
   }
 
   Future<Detail> getDetail(int videoId) async {

client/lib/widgets/main/main_screen_favorite_view.dart

@@ -10,11 +10,9 @@ class MainScreenFavoriteView extends StatelessWidget {
   const MainScreenFavoriteView({
     Key? key,
     required this.cleanList,
-    required this.videoSearchDelegate,
   }) : super(key: key);
 
   final List<Video> cleanList;
-  final VideoSearchDelegate videoSearchDelegate;
 
   @override
   Widget build(BuildContext context) {
@@ -24,13 +22,11 @@ class MainScreenFavoriteView extends StatelessWidget {
         if (!snapshot.hasData) {
           return const Center(child: CircularProgressIndicator());
         }
-
-        final Set<int> favoriteVideoMap = snapshot.data!;
+        final favoriteMap = snapshot.requireData;
         return PRVideos(
           cleanList: cleanList
-              .where((element) => favoriteVideoMap.contains(element.prId))
-              .toList(growable: false),
-          videoSearchDelegate: videoSearchDelegate,
+              .where((element) => favoriteMap.contains(element.prId))
+              .toList(),
           hideBookmarkIcon: true,
         );
       },

client/lib/widgets/main/main_screen_list_view.dart

@@ -7,17 +7,14 @@ class MainScreenListView extends StatelessWidget {
   const MainScreenListView({
     Key? key,
     required this.cleanList,
-    required this.videoSearchDelegate,
   }) : super(key: key);
 
   final List<Video> cleanList;
-  final VideoSearchDelegate videoSearchDelegate;
 
   @override
   Widget build(BuildContext context) {
     return PRVideos(
       cleanList: cleanList,
-      videoSearchDelegate: videoSearchDelegate,
       hideBookmarkIcon: false,
     );
   }

client/lib/widgets/main/pr12video.dart

@@ -3,24 +3,17 @@ import 'package:provider/provider.dart';
 import 'package:pr12er/protos/pkg/pr12er/messages.pb.dart';
 import 'package:pr12er/screens/detail_screen.dart';
 import 'package:pr12er/view_models/view_model_videos.dart';
-import 'package:pr12er/widgets/main/video_search_delegate.dart';
 
 class PRVideos extends StatelessWidget {
-  final VideoSearchDelegate videoSearchDelegate;
   final List<Video> cleanList;
   final bool hideBookmarkIcon;
 
   const PRVideos(
-      {Key? key,
-      required this.cleanList,
-      required this.videoSearchDelegate,
-      required this.hideBookmarkIcon})
+      {Key? key, required this.cleanList, required this.hideBookmarkIcon})
       : super(key: key);
 
   @override
   Widget build(BuildContext context) {
-    videoSearchDelegate.dataRef = cleanList;
-
     return ListView.builder(
         padding: const EdgeInsets.all(8),
         itemCount: cleanList.length,

client/lib/widgets/main/video_search_delegate.dart

@@ -1,10 +1,13 @@
 import 'package:flutter/material.dart';
 import 'package:pr12er/protos/pkg/pr12er/messages.pb.dart';
+import 'package:pr12er/service.dart';
+import 'package:pr12er/view_models/view_model_videos.dart';
 import 'package:pr12er/widgets/main/pr12video.dart';
+import 'package:provider/provider.dart';
 
 class VideoSearchDelegate extends SearchDelegate {
-  late List<Video> dataRef;
-  late bool ignoreBookmarkIcon;
+  List<Video> dataRef = [];
+  bool showOnlyBookmarkItems = false;
 
   @override
   ThemeData appBarTheme(BuildContext context) {
@@ -33,30 +36,71 @@ class VideoSearchDelegate extends SearchDelegate {
     );
   }
 
-  Widget getSearchResult() {
-    final videos = dataRef.where((video) {
+  Widget getSearchResult(BuildContext context) {
+    return FutureBuilder<List<Video>>(
+      future: context.read<GrpcClient>().getVideos(),
+      builder: (context, snapshot) {
+        if (!snapshot.hasData) {
+          return const Center(child: CircularProgressIndicator());
+        }
+
+        dataRef = snapshot.requireData;
+
+        if (showOnlyBookmarkItems) {
+          return FutureBuilder<Set<int>>(
+            future: context.read<FavoriteVideoViewModel>().favoriteVideosMap,
+            builder: (context, snapshot) {
+              if (!snapshot.hasData) {
+                return const Center(child: CircularProgressIndicator());
+              }
+              final map = snapshot.requireData;
+              dataRef = dataRef
+                  .where((element) => map.contains(element.prId))
+                  .toList(growable: false);
+
+              final List<Video> videos = getInterestedVideos();
+
+              return ListView.builder(
+                  padding: const EdgeInsets.all(8),
+                  itemCount: videos.length,
+                  itemBuilder: (BuildContext context, int index) => PR12Video(
+                      prID: index,
+                      video: videos[index],
+                      hideBookmarkIcon: showOnlyBookmarkItems));
+            },
+          );
+        }
+
+        final List<Video> videos = getInterestedVideos();
+
+        return ListView.builder(
+            padding: const EdgeInsets.all(8),
+            itemCount: videos.length,
+            itemBuilder: (BuildContext context, int index) => PR12Video(
+                prID: index,
+                video: videos[index],
+                hideBookmarkIcon: showOnlyBookmarkItems));
+      },
+    );
+  }
+
+  List<Video> getInterestedVideos() {
+    var videos = dataRef.where((video) {
       final searchLower = query.toLowerCase();
       final titleLower = video.title.toLowerCase();
 
       return titleLower.contains(searchLower);
     }).toList();
-
-    return ListView.builder(
-        padding: const EdgeInsets.all(8),
-        itemCount: videos.length,
-        itemBuilder: (BuildContext context, int index) => PR12Video(
-            prID: index,
-            video: videos[index],
-            hideBookmarkIcon: ignoreBookmarkIcon));
+    return videos;
   }
 
   @override
   Widget buildResults(BuildContext context) {
-    return getSearchResult();
+    return getSearchResult(context);
   }
 
   @override
   Widget buildSuggestions(BuildContext context) {
-    return getSearchResult();
+    return getSearchResult(context);
   }
 }