Fix negative height

Author: Elxas866

lib/features/profile/presentation/widgets/set_height_dialog.dart

@@ -2,9 +2,9 @@ import 'package:flutter/material.dart';
 import 'package:horizontal_picker/horizontal_picker.dart';
 import 'package:opennutritracker/generated/l10n.dart';
 
-class SetHeightDialog extends StatelessWidget {
+class SetHeightDialog extends StatefulWidget {
   static const _heightRangeCM = 100.0;
-  static const _heightRangeFt = 10;
+  static const _heightRangeFt = 10.0;
 
   final double userHeight;
   final bool usesImperialUnits;
@@ -15,9 +15,29 @@ class SetHeightDialog extends StatelessWidget {
     required this.usesImperialUnits,
   });
 
+  @override
+  State<SetHeightDialog> createState() => _SetHeightDialogState();
+}
+
+class _SetHeightDialogState extends State<SetHeightDialog> {
+  late double selectedHeight;
+
+  @override
+  void initState() {
+    super.initState();
+    selectedHeight = widget.userHeight;
+  }
+
   @override
   Widget build(BuildContext context) {
-    double selectedHeight = userHeight;
+    final minValue = widget.usesImperialUnits
+        ? widget.userHeight - SetHeightDialog._heightRangeFt
+        : widget.userHeight - SetHeightDialog._heightRangeCM;
+
+    final maxValue = widget.usesImperialUnits
+        ? widget.userHeight + SetHeightDialog._heightRangeFt
+        : widget.userHeight + SetHeightDialog._heightRangeCM;
+
     return AlertDialog(
       title: Text(S.of(context).selectHeightDialogLabel),
       content: Wrap(
@@ -27,18 +47,18 @@ class SetHeightDialog extends StatelessWidget {
               HorizontalPicker(
                 height: 100,
                 backgroundColor: Colors.transparent,
-                minValue: usesImperialUnits
-                    ? selectedHeight - _heightRangeFt
-                    : selectedHeight - _heightRangeCM,
-                maxValue: usesImperialUnits
-                    ? selectedHeight + _heightRangeFt
-                    : selectedHeight + _heightRangeCM,
+                // Prevent negative minimum height
+                minValue: minValue < 0 ? 1 : minValue, // setting it to 1, because 0 triggers zero-division error
+                maxValue: maxValue,
                 divisions: 400,
-                suffix: usesImperialUnits
+                suffix: widget.usesImperialUnits
                     ? S.of(context).ftLabel
                     : S.of(context).cmLabel,
                 onChanged: (value) {
-                  selectedHeight = value;
+                  setState(() {
+                    // Prevent negative height values
+                    selectedHeight = value < 0 ? 1 : value;
+                  });
                 },
               ),
             ],
@@ -62,4 +82,4 @@ class SetHeightDialog extends StatelessWidget {
       ],
     );
   }
-}
+}