diff --git a/examples/app-architecture/todo_data_service/lib/main_app_viewmodel.dart b/examples/app-architecture/todo_data_service/lib/main_app_viewmodel.dart index 215584c2a6..70a2cce200 100644 --- a/examples/app-architecture/todo_data_service/lib/main_app_viewmodel.dart +++ b/examples/app-architecture/todo_data_service/lib/main_app_viewmodel.dart @@ -24,16 +24,11 @@ class MainAppViewModel extends ChangeNotifier { bool get isDarkMode => _isDarkMode; Future _load() async { - try { - final result = await _themeRepository.isDarkMode(); - if (result is Ok) { - _isDarkMode = result.value; - } - } on Exception catch (_) { - // handle error - } finally { - notifyListeners(); + final result = await _themeRepository.isDarkMode(); + if (result is Ok) { + _isDarkMode = result.value; } + notifyListeners(); } @override diff --git a/examples/app-architecture/todo_data_service/lib/ui/theme_config/viewmodel/theme_switch_viewmodel.dart b/examples/app-architecture/todo_data_service/lib/ui/theme_config/viewmodel/theme_switch_viewmodel.dart index d1c5518d6f..072d08919a 100644 --- a/examples/app-architecture/todo_data_service/lib/ui/theme_config/viewmodel/theme_switch_viewmodel.dart +++ b/examples/app-architecture/todo_data_service/lib/ui/theme_config/viewmodel/theme_switch_viewmodel.dart @@ -26,29 +26,20 @@ class ThemeSwitchViewModel extends ChangeNotifier { /// Load the current theme setting from the repository Future> _load() async { - try { - final result = await _themeRepository.isDarkMode(); - if (result is Ok) { - _isDarkMode = result.value; - } - return result; - } on Exception catch (e) { - return Result.error(e); - } finally { - notifyListeners(); + final result = await _themeRepository.isDarkMode(); + if (result is Ok) { + _isDarkMode = result.value; } + notifyListeners(); + return result; } /// Toggle the theme setting Future> _toggle() async { - try { - _isDarkMode = !_isDarkMode; - return await _themeRepository.setDarkMode(_isDarkMode); - } on Exception catch (e) { - return Result.error(e); - } finally { - notifyListeners(); - } + _isDarkMode = !_isDarkMode; + final result = await _themeRepository.setDarkMode(_isDarkMode); + notifyListeners(); + return result; } } diff --git a/src/content/app-architecture/design-patterns/key-value-data.md b/src/content/app-architecture/design-patterns/key-value-data.md index 525d025463..dd673f9e7f 100644 --- a/src/content/app-architecture/design-patterns/key-value-data.md +++ b/src/content/app-architecture/design-patterns/key-value-data.md @@ -148,29 +148,20 @@ class ThemeSwitchViewModel extends ChangeNotifier { /// Load the current theme setting from the repository Future> _load() async { - try { - final result = await _themeRepository.isDarkMode(); - if (result is Ok) { - _isDarkMode = result.value; - } - return result; - } on Exception catch (e) { - return Result.error(e); - } finally { - notifyListeners(); + final result = await _themeRepository.isDarkMode(); + if (result is Ok) { + _isDarkMode = result.value; } + notifyListeners(); + return result; } /// Toggle the theme setting Future> _toggle() async { - try { - _isDarkMode = !_isDarkMode; - return await _themeRepository.setDarkMode(_isDarkMode); - } on Exception catch (e) { - return Result.error(e); - } finally { - notifyListeners(); - } + _isDarkMode = !_isDarkMode; + final result = await _themeRepository.setDarkMode(_isDarkMode); + notifyListeners(); + return result; } } ``` @@ -317,16 +308,11 @@ class MainAppViewModel extends ChangeNotifier { bool get isDarkMode => _isDarkMode; Future _load() async { - try { - final result = await _themeRepository.isDarkMode(); - if (result is Ok) { - _isDarkMode = result.value; - } - } on Exception catch (_) { - // handle error - } finally { - notifyListeners(); + final result = await _themeRepository.isDarkMode(); + if (result is Ok) { + _isDarkMode = result.value; } + notifyListeners(); } @override