mempersimple codingan

lib/ui/settings/general.dart

@@ -12,66 +12,60 @@ class GeneralSetting extends StatefulWidget {
 class _GeneralSettingState extends State<GeneralSetting> {
   @override
   Widget build(BuildContext context) {
-    final bool? isOrderActive = prefs.getBool('isOrderActive');
-    final bool? isAdActive = prefs.getBool('isAdActive');
+    final bool isOrderActive = prefs.getBool('isOrderActive') ?? true;
+    final bool isAdActive = prefs.getBool('isAdActive') ?? true;
     return Scaffold(
       appBar: AppBar(
         title: const Text("General Setting"),
         backgroundColor: backgroundAppBar,
       ),
-      body: Column(
+      body: SizedBox(
+        height: MediaQuery.of(context).size.height,
+        child: ListView(
           children: [
-          Container(
-            color: Colors.black.withOpacity(0.5),
-            height: 50,
-            width: MediaQuery.of(context).size.width * 1,
-            child: Container(
-              margin: EdgeInsets.only(left: 10),
-              child: const Align(
-                  alignment: Alignment.centerLeft, child: Text("General")),
-            ),
-          ),
-          Container(
-            margin: const EdgeInsets.only(left: 10, right: 10),
-            child: Row(
-              mainAxisAlignment: MainAxisAlignment.spaceBetween,
-              children: [
-                Text(
-                  "Tampilkan Order",
-                  style: TextStyle(
-                      fontSize: MediaQuery.of(context).size.height * 0.025,
-                      fontWeight: FontWeight.w500),
-                ),
-                Switch(
-                    value: (isOrderActive == null) ? true : isOrderActive,
-                    onChanged: (bool newValue) async {
-                      await prefs.setBool('isOrderActive', newValue);
-                      setState(() {});
-                    })
+            titleHeader(context, "General"),
+            listWithSwitch(isOrderActive, "Tampilkan Order", "isOrderActive"),
+            listWithSwitch(isAdActive, "Tampilkan Iklan", "isAdActive"),
           ],
         ),
       ),
-          Container(
-            margin: const EdgeInsets.only(left: 10, right: 10),
+    );
+  }
+
+  Container listWithSwitch(bool isOrderActive, String title, String prefName) {
+    return Container(
+      margin: const EdgeInsets.only(left: 10, right: 10, top: 15),
       child: Row(
         mainAxisAlignment: MainAxisAlignment.spaceBetween,
         children: [
           Text(
-                  "Tampilkan Iklan",
-                  style: TextStyle(
-                      fontSize: MediaQuery.of(context).size.height * 0.025,
-                      fontWeight: FontWeight.w500),
+            title,
+            style: const TextStyle(fontSize: 20, fontWeight: FontWeight.w500),
           ),
           Switch(
-                    value: (isAdActive == null) ? true : isAdActive,
+              value: isOrderActive,
               onChanged: (bool newValue) async {
-                      await prefs.setBool('isAdActive', newValue);
+                await prefs.setBool(prefName, newValue);
                 setState(() {});
               })
         ],
       ),
-          )
-        ],
+    );
+  }
+
+  Container titleHeader(BuildContext context, String title) {
+    return Container(
+      color: Colors.black.withOpacity(0.5),
+      height: 50,
+      width: MediaQuery.of(context).size.width * 1,
+      child: Container(
+        margin: const EdgeInsets.only(left: 10),
+        child: Align(
+            alignment: Alignment.centerLeft,
+            child: Text(
+              title,
+              style: const TextStyle(fontWeight: FontWeight.w600, fontSize: 20),
+            )),
       ),
     );
   }