✨ Enhance LoadSwitchController with disposal handling to prevent state changes after disposal, and update SpinnerWidget to require animation duration while simplifying widget construction.

Author: esentis

lib/src/load_switch_controller.dart

@@ -1,4 +1,5 @@
 import 'package:flutter/foundation.dart';
+import 'package:load_switch/src/load_switch_types.dart';
 
 /// Controller for managing the state of a [LoadSwitch] widget.
 ///
@@ -17,6 +18,7 @@ class LoadSwitchController extends ChangeNotifier {
   bool _value;
   bool _isLoading;
   bool _isActive;
+  bool _isDisposed = false;
 
   /// The current value of the switch (on/off).
   bool get value => _value;
@@ -29,6 +31,9 @@ class LoadSwitchController extends ChangeNotifier {
 
   /// Sets the value of the switch and notifies listeners.
   set value(bool newValue) {
+    if (_isDisposed) {
+      return;
+    }
     if (_value != newValue) {
       _value = newValue;
       notifyListeners();
@@ -37,6 +42,9 @@ class LoadSwitchController extends ChangeNotifier {
 
   /// Sets the loading state of the switch and notifies listeners.
   set isLoading(bool loading) {
+    if (_isDisposed) {
+      return;
+    }
     if (_isLoading != loading) {
       _isLoading = loading;
       notifyListeners();
@@ -45,6 +53,9 @@ class LoadSwitchController extends ChangeNotifier {
 
   /// Sets the active state of the switch and notifies listeners.
   set isActive(bool active) {
+    if (_isDisposed) {
+      return;
+    }
     if (_isActive != active) {
       _isActive = active;
       notifyListeners();
@@ -53,6 +64,9 @@ class LoadSwitchController extends ChangeNotifier {
 
   /// Toggles the value of the switch if it's not loading and is active.
   void toggle() {
+    if (_isDisposed) {
+      return;
+    }
     if (!_isLoading && _isActive) {
       value = !value;
     }
@@ -64,24 +78,43 @@ class LoadSwitchController extends ChangeNotifier {
   /// the future completes or throws an error. Updates the value of the switch
   /// based on the result of the future.
   ///
-  /// If [onChange] is provided, it will be called with the new value.
+  /// If [onChanged] is provided, it will be called with the new value.
   /// If [onError] is provided, it will be called with any error that occurs.
   Future<void> executeWithLoading(
-    Future<bool> Function() future, {
-    Function(bool)? onChange,
-    Function(Object)? onError,
+    LoadSwitchToggleCallback onToggle, {
+    ValueChanged<bool>? onChanged,
+    LoadSwitchErrorCallback? onError,
   }) async {
-    if (_isLoading || !_isActive) return;
+    if (_isDisposed || _isLoading || !_isActive) {
+      return;
+    }
 
     isLoading = true;
 
     try {
-      value = await future.call();
-      onChange?.call(value);
-    } catch (error) {
-      onError?.call(error);
+      value = await onToggle();
+      if (_isDisposed) {
+        return;
+      }
+      onChanged?.call(value);
+    } catch (error, stackTrace) {
+      if (_isDisposed) {
+        return;
+      }
+      onError?.call(error, stackTrace);
     } finally {
-      isLoading = false;
+      if (!_isDisposed) {
+        isLoading = false;
+      }
+    }
+  }
+
+  @override
+  void dispose() {
+    if (_isDisposed) {
+      return;
     }
+    _isDisposed = true;
+    super.dispose();
   }
 }