use getIt for the api connection and add changing pages with the drawer

Author: Narzgul

lib/api_connection.dart

@@ -0,0 +1,31 @@
+import 'dart:convert';
+
+import 'package:ka_client/product.dart';
+import 'package:http/http.dart' as http;
+
+class APIConnection {
+  String url = 'http://127.0.0.1:8000/products';
+  late Future<List<Product>> products;
+
+  APIConnection() {
+    products = fetchProducts();
+  }
+
+  Future<List<Product>> fetchProducts() async {
+    final response = await http.get(Uri.parse(url));
+
+    if (response.statusCode == 200) {
+      List<Product> products = []; // TODO: Write sleak oneliner
+      for (Map<String, dynamic> productString
+      in jsonDecode(utf8.decode(response.body.codeUnits))) {
+        // Convert to codeUnits to fix Umlauts
+        products.add(Product.fromJson(productString));
+      }
+      return products;
+    } else {
+      throw Exception('Failed to load product');
+    }
+  }
+
+
+}

lib/main.dart

@@ -1,12 +1,18 @@
 import 'package:flutter/material.dart';
+import 'package:get_it/get_it.dart';
+import 'package:ka_client/api_connection.dart';
 import 'package:ka_client/style_components/app_scaffold.dart';
 
 void main() {
-  runApp(const MyApp());
+  runApp(MyApp());
 }
 
 class MyApp extends StatelessWidget {
-  const MyApp({super.key});
+  MyApp({super.key}) {
+    _initAPI();
+  }
+
+  void _initAPI() => GetIt.I.registerSingleton<APIConnection>(APIConnection());
 
   @override
   Widget build(BuildContext context) {
@@ -19,7 +25,7 @@ class MyApp extends StatelessWidget {
         ),
         useMaterial3: true,
       ),
-      home: const AppScaffold(title: "KA Scraper"),
+      home: const AppScaffold(),
     );
   }
 }

lib/overview.dart

@@ -1,25 +1,8 @@
-import 'dart:convert';
-
 import 'package:flutter/material.dart';
-import 'package:http/http.dart' as http;
-import 'package:ka_client/price_chart.dart';
+import 'package:get_it/get_it.dart';
 import 'package:ka_client/product.dart';
 
-Future<List<Product>> fetchProducts() async {
-  final response = await http.get(Uri.parse('http://127.0.0.1:8000/products'));
-
-  if (response.statusCode == 200) {
-    List<Product> products = []; // TODO: Write sleak oneliner
-    for (Map<String, dynamic> productString
-        in jsonDecode(utf8.decode(response.body.codeUnits))) {
-      // Convert to codeUnits to fix Umlauts
-      products.add(Product.fromJson(productString));
-    }
-    return products;
-  } else {
-    throw Exception('Failed to load product');
-  }
-}
+import 'api_connection.dart';
 
 class Overview extends StatefulWidget {
   const Overview({super.key});
@@ -29,35 +12,32 @@ class Overview extends StatefulWidget {
 }
 
 class _OverviewState extends State<Overview> {
-  late Future<List<Product>> products;
-
-  @override
-  void initState() {
-    super.initState();
-    products = fetchProducts();
-  }
+  Future<List<Product>> products = GetIt.I<APIConnection>().products;
 
   @override
   Widget build(BuildContext context) {
-    return Scaffold(
-      appBar: AppBar(
-        title: const Text('Overview'),
-        backgroundColor: Theme.of(context).colorScheme.primary,
-      ),
-      body: FutureBuilder(
-        builder: (context, snapshot) {
-          if (snapshot.hasData && snapshot.data != null) {
-            List<Product> products = snapshot.data!; // Can't be null
-            return PriceChart(prices: products.map((e) => e.price).toList());
-          } else if (snapshot.hasData && snapshot.data == null) {
-            return const Center(child: Text('No data'));
-          } else if (snapshot.hasError) {
-            return Center(child: Text('${snapshot.error}'));
-          }
-          return const Center(child: CircularProgressIndicator());
-        },
-        future: products,
-      ),
+    return FutureBuilder(
+      builder: (context, snapshot) {
+        if (snapshot.hasData && snapshot.data != null) {
+          List<Product> products = snapshot.data!; // Can't be null
+          return ListView.builder(
+            itemCount: products.length,
+            itemBuilder: (context, index) {
+              return ListTile(
+                title: Text(products[index].title),
+                subtitle: Text(products[index].location),
+                trailing: Text(products[index].price.toString()),
+              );
+            },
+          );
+        } else if (snapshot.hasData && snapshot.data == null) {
+          return const Center(child: Text('No data'));
+        } else if (snapshot.hasError) {
+          return Center(child: Text('${snapshot.error}'));
+        }
+        return const Center(child: CircularProgressIndicator());
+      },
+      future: products,
     );
   }
 }

lib/price_chart_page.dart

@@ -0,0 +1,30 @@
+import 'package:flutter/material.dart';
+import 'package:get_it/get_it.dart';
+import 'package:ka_client/product.dart';
+import 'package:ka_client/style_components/price_chart.dart';
+
+import 'api_connection.dart';
+
+class PriceChartPage extends StatelessWidget {
+  final Future<List<Product>> products = GetIt.I<APIConnection>().products;
+
+  PriceChartPage({super.key});
+
+  @override
+  Widget build(BuildContext context) {
+    return FutureBuilder(
+      builder: (context, snapshot) {
+        if (snapshot.hasData && snapshot.data != null) {
+          List<Product> products = snapshot.data!; // Can't be null
+          return PriceChart(prices: products.map((e) => e.price).toList());
+        } else if (snapshot.hasData && snapshot.data == null) {
+          return const Center(child: Text('No data'));
+        } else if (snapshot.hasError) {
+          return Center(child: Text('${snapshot.error}'));
+        }
+        return const Center(child: CircularProgressIndicator());
+      },
+      future: products,
+    );
+  }
+}

lib/style_components/app_scaffold.dart

@@ -1,13 +1,12 @@
 import 'package:flutter/material.dart';
+import 'package:ka_client/price_chart_page.dart';
 import 'package:ka_client/style_components/nav_drawer.dart';
 
 import '../overview.dart';
-import '../price_chart.dart';
 
 class AppScaffold extends StatefulWidget {
-  final String title;
 
-  const AppScaffold({super.key, required this.title});
+  const AppScaffold({super.key});
 
   @override
   State<AppScaffold> createState() => _AppScaffoldState();
@@ -16,9 +15,7 @@ class AppScaffold extends StatefulWidget {
 class _AppScaffoldState extends State<AppScaffold> {
   final _pages = {
     "Overview": const Overview(),
-    "Price Chart": const PriceChart(
-      prices: [1, 2, 3, 4, 5, 6],
-    ),
+    "Price Chart": PriceChartPage(),
   };
 
   String _currentPage = "";
@@ -46,7 +43,7 @@ class _AppScaffoldState extends State<AppScaffold> {
           },
         ),
         appBar: AppBar(
-          title: Text(widget.title),
+          title: Text(_currentPage),
         ),
         body: _pages[_currentPage],
       ),

lib/style_components/nav_drawer.dart

@@ -29,9 +29,13 @@ class NavDrawer extends StatelessWidget {
               Navigator.pop(context);
             },
           ),
-          const ListTile(
-            title: Text('Price Graph'),
-            leading: Icon(Icons.show_chart),
+          ListTile(
+            title: const Text('Price Chart'),
+            leading: const Icon(Icons.show_chart),
+            onTap: () {
+              updateCurrentPage("Price Chart");
+              Navigator.pop(context);
+            },
           ),
         ],
       ),

lib/price_chart.dart renamed to lib/style_components/price_chart.dart

@@ -3,7 +3,6 @@ import 'package:flutter/material.dart';
 
 class PriceChart extends StatelessWidget {
   final List<int> prices;
-  // add prices to constructor
   const PriceChart({super.key, required this.prices});
 
   @override

pubspec.lock

@@ -91,6 +91,14 @@ packages:
     description: flutter
     source: sdk
     version: "0.0.0"
+  get_it:
+    dependency: "direct main"
+    description:
+      name: get_it
+      sha256: e6017ce7fdeaf218dc51a100344d8cb70134b80e28b760f8bb23c242437bafd7
+      url: "https://pub.dev"
+    source: hosted
+    version: "7.6.7"
   http:
     dependency: "direct main"
     description:

pubspec.yaml

@@ -37,6 +37,7 @@ dependencies:
   cupertino_icons: ^1.0.2
   http:
   fl_chart:
+  get_it: ^7.6.7
 
 dev_dependencies:
   flutter_test:

test/widget_test.dart

@@ -13,7 +13,7 @@ import 'package:ka_client/main.dart';
 void main() {
   testWidgets('Counter increments smoke test', (WidgetTester tester) async {
     // Build our app and trigger a frame.
-    await tester.pumpWidget(const MyApp());
+    await tester.pumpWidget(MyApp());
 
     // Verify that our counter starts at 0.
     expect(find.text('0'), findsOneWidget);