refactor(#674): add allGuidelineGenes as CachedDrugs extension

hpi-dhc · Dec 5, 2023 · 90588f4 · 90588f4
1 parent c783702
commit 90588f4
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 8 deletions.
diff --git a/app/lib/common/models/drug/cached_drugs.dart b/app/lib/common/models/drug/cached_drugs.dart
@@ -56,3 +56,17 @@ Future<void> initCachedDrugs() async {
   final cachedDrugs = Hive.box<CachedDrugs>(_boxName);
   CachedDrugs._instance = cachedDrugs.get('data') ?? CachedDrugs();
 }
+
+extension CachedDrugsMethods on CachedDrugs {
+  Set<String> get allGuidelineGenes {
+    final guidelineGenes = <String>{};
+    if (CachedDrugs.instance.drugs != null) {
+      for (final drug in CachedDrugs.instance.drugs!) {
+        for (final guideline in drug.guidelines) {
+          guideline.lookupkey.keys.forEach(guidelineGenes.add);
+        }
+      }
+    }
+    return guidelineGenes;
+  }
+}
diff --git a/app/lib/report/pages/report.dart b/app/lib/report/pages/report.dart
@@ -15,15 +15,8 @@ class ReportPage extends StatelessWidget {
 
   Widget _buildReportPage(BuildContext context, ActiveDrugs activeDrugs) {
     final hasActiveInhibitors = activeDrugs.names.any(isInhibitor);
-    final genes = <String>{};
-    for (final drug in CachedDrugs.instance.drugs!) {
-      for (final guideline in drug.guidelines) {
-        guideline.lookupkey.keys.forEach(genes.add);
-      }
-    }
-
     final notTestedString = context.l10n.general_not_tested;
-    final userPhenotypes = genes.map(
+    final userPhenotypes = CachedDrugs.instance.allGuidelineGenes.map(
       (geneSymbol) => UserData.instance.lookups![geneSymbol] ??
       CpicPhenotype(
         geneSymbol: geneSymbol,