feat: Add centralized ErrorService for consistent error handling

Author: TenFinges

lib/controllers/authentication_controller.dart

@@ -3,6 +3,7 @@ import 'dart:developer';
 import 'package:appwrite/appwrite.dart';
 import 'package:flutter/rendering.dart';
 import 'package:resonate/services/appwrite_service.dart';
+import 'package:resonate/services/error_service.dart';
 import 'package:flutter/material.dart';
 import 'package:get/get.dart';
 import 'package:resonate/controllers/auth_state_controller.dart';
@@ -36,7 +37,7 @@ class AuthenticationController extends GetxController {
       emailController.clear();
       passwordController.clear();
     } on AppwriteException catch (e) {
-      log(e.toString());
+      // Handle specific Appwrite auth errors with custom messages
       if (e.type == userInvalidCredentials) {
         customSnackbar(
           AppLocalizations.of(context)!.tryAgain,
@@ -53,14 +54,16 @@ class AuthenticationController extends GetxController {
           AppLocalizations.of(context)!.passwordShort,
           LogType.error,
         );
-
         SemanticsService.announce(
           AppLocalizations.of(context)!.passwordShort,
           TextDirection.ltr,
         );
+      } else {
+        // Use ErrorService for other Appwrite errors
+        ErrorService.handle(e, context: 'login');
       }
     } catch (e) {
-      log(e.toString());
+      ErrorService.handle(e, context: 'login');
     } finally {
       isLoading.value = false;
     }
@@ -75,14 +78,7 @@ class AuthenticationController extends GetxController {
       );
       return true;
     } catch (e) {
-      log(e.toString());
-      customSnackbar(
-        AppLocalizations.of(context)!.oops,
-        e.toString(),
-        LogType.error,
-      );
-      SemanticsService.announce(e.toString(), TextDirection.ltr);
-
+      ErrorService.handle(e, context: 'signup');
       return false;
     } finally {
       isLoading.value = false;
@@ -93,15 +89,15 @@ class AuthenticationController extends GetxController {
     try {
       await authStateController.loginWithGoogle();
     } catch (error) {
-      log(error.toString());
+      ErrorService.handle(error, context: 'Google login');
     }
   }
 
   Future<void> loginWithGithub() async {
     try {
       await authStateController.loginWithGithub();
     } catch (error) {
-      log(error.toString());
+      ErrorService.handle(error, context: 'GitHub login');
     }
   }
 

lib/controllers/create_room_controller.dart

@@ -1,8 +1,10 @@
 import 'dart:developer';
+
 import 'package:flutter/material.dart';
 import 'package:get/get.dart';
 import 'package:resonate/controllers/auth_state_controller.dart';
 import 'package:resonate/controllers/tabview_controller.dart';
+import 'package:resonate/services/error_service.dart';
 import 'package:resonate/themes/theme_controller.dart';
 import 'package:resonate/utils/enums/room_state.dart';
 import 'package:textfield_tags/textfield_tags.dart';
@@ -88,10 +90,9 @@ class CreateRoomController extends GetxController {
       tagsController.clearTags();
       descriptionController.clear();
     } catch (e) {
-      log(e.toString());
-
       // Close the loading dialog
       Get.back();
+      ErrorService.handle(e, context: 'creating room');
     } finally {
       isLoading.value = false;
     }

lib/controllers/rooms_controller.dart

@@ -9,11 +9,10 @@ import 'package:loading_animation_widget/loading_animation_widget.dart';
 import 'package:resonate/controllers/tabview_controller.dart';
 import 'package:resonate/models/appwrite_room.dart';
 import 'package:resonate/services/appwrite_service.dart';
+import 'package:resonate/services/error_service.dart';
 import 'package:resonate/services/room_service.dart';
 import 'package:resonate/themes/theme_controller.dart';
-import 'package:resonate/utils/enums/log_type.dart';
 import 'package:resonate/utils/enums/room_state.dart';
-import 'package:resonate/views/widgets/snackbar.dart';
 
 import '../utils/constants.dart';
 import 'auth_state_controller.dart';
@@ -88,7 +87,7 @@ class RoomsController extends GetxController {
       }
       update();
     } catch (e) {
-      log(e.toString());
+      ErrorService.handleSilently(e, context: 'fetching rooms');
     } finally {
       isLoading.value = false;
     }
@@ -106,7 +105,8 @@ class RoomsController extends GetxController {
       AppwriteRoom appwriteRoom = await createRoomObject(room, userUid);
       return appwriteRoom;
     } catch (e) {
-      log(e.toString());
+      ErrorService.handleSilently(e, context: 'fetching room by ID');
+      return null;
     }
   }
 
@@ -140,11 +140,11 @@ class RoomsController extends GetxController {
       // Open the Room Bottom Sheet to interact in the room
       Get.find<TabViewController>().openRoomSheet(room);
     } catch (e) {
-      log(e.toString());
-      getRooms();
-      update();
       // Close the loading dialog
       Get.back();
+      ErrorService.handle(e, context: 'joining room');
+      getRooms();
+      update();
     }
   }
 
@@ -163,12 +163,11 @@ class RoomsController extends GetxController {
             room.description.toLowerCase().contains(lowerQuery);
       }).toList();
     } catch (e) {
-      log('Error searching rooms: $e');
       filteredRooms.value = rooms;
-      customSnackbar(
-        AppLocalizations.of(Get.context!)!.error,
-        AppLocalizations.of(Get.context!)!.searchFailed,
-        LogType.error,
+      ErrorService.handle(
+        e,
+        context: 'searching rooms',
+        userMessage: AppLocalizations.of(Get.context!)?.searchFailed,
       );
     } finally {
       isSearching.value = false;

lib/l10n/app_en.arb

@@ -1808,7 +1808,66 @@
 "usernameAlreadyTaken": "This username is already taken. Try a different one.",
 "@usernameAlreadyTaken": {
   "description": "Error shown when the chosen username is unavailable because another user has already registered it."
-}
+},
+
+"networkError": "Please check your internet connection and try again.",
+"@networkError": {
+  "description": "Error message shown when a network-related error occurs."
+},
 
+"authenticationError": "Authentication failed. Please try again.",
+"@authenticationError": {
+  "description": "Generic error message for authentication failures."
+},
+
+"storageError": "Failed to save or load data. Please try again.",
+"@storageError": {
+  "description": "Error message shown when file storage operations fail."
+},
+
+"databaseError": "Failed to access data. Please try again.",
+"@databaseError": {
+  "description": "Error message shown when database operations fail."
+},
+
+"validationError": "Please check your input and try again.",
+"@validationError": {
+  "description": "Error message shown when input validation fails."
+},
+
+"generalError": "Something went wrong. Please try again.",
+"@generalError": {
+  "description": "Generic fallback error message for unexpected errors."
+},
+
+"connectionError": "Connection Error",
+"@connectionError": {
+  "description": "Title for network connection error messages."
+},
+
+"authError": "Authentication Error",
+"@authError": {
+  "description": "Title for authentication error messages."
+},
+
+"invalidInput": "Invalid Input",
+"@invalidInput": {
+  "description": "Title for validation error messages."
+},
+
+"userAlreadyExists": "An account with this email already exists.",
+"@userAlreadyExists": {
+  "description": "Error message when trying to create an account with an existing email."
+},
+
+"userNotFound": "User not found.",
+"@userNotFound": {
+  "description": "Error message when a user account cannot be found."
+},
+
+"dataNotFound": "The requested data was not found.",
+"@dataNotFound": {
+  "description": "Error message when requested data does not exist."
+}
 
 }

lib/l10n/app_localizations.dart

@@ -1410,10 +1410,10 @@ abstract class AppLocalizations {
   /// **'Please check your mail for a new OTP.'**
   String get otpResentMessage;
 
-  /// Generic error message for network connection issues.
+  /// Title for network connection error messages.
   ///
   /// In en, this message translates to:
-  /// **'There is a connection error. Please check your internet and try again.'**
+  /// **'Connection Error'**
   String get connectionError;
 
   /// The word 'seconds', often used after a countdown number.
@@ -1467,7 +1467,7 @@ abstract class AppLocalizations {
   /// Error message when a chosen username is too short.
   ///
   /// In en, this message translates to:
-  /// **'Username should contain more than 5 characters.'**
+  /// **'Username should contain more than 7 characters.'**
   String get usernameCharacterLimit;
 
   /// Generic button text for submitting a form.
@@ -2518,17 +2518,95 @@ abstract class AppLocalizations {
   /// **'Are you sure you want to delete this message?'**
   String get deleteMessageContent;
 
-  /// No description provided for @thisMessageWasDeleted.
+  /// Status text shown when a previously sent message has been deleted.
   ///
   /// In en, this message translates to:
   /// **'This message was deleted'**
   String get thisMessageWasDeleted;
 
-  /// No description provided for @failedToDeleteMessage.
+  /// Error message shown when the system is unable to delete a message.
   ///
   /// In en, this message translates to:
   /// **'Failed to delete message'**
   String get failedToDeleteMessage;
+
+  /// Validation error displayed when the user enters a username with unsupported characters.
+  ///
+  /// In en, this message translates to:
+  /// **'Please enter a valid username. Only letters, numbers, dots, underscores, and hyphens are allowed.'**
+  String get usernameInvalidFormat;
+
+  /// Error shown when the chosen username is unavailable because another user has already registered it.
+  ///
+  /// In en, this message translates to:
+  /// **'This username is already taken. Try a different one.'**
+  String get usernameAlreadyTaken;
+
+  /// Error message shown when a network-related error occurs.
+  ///
+  /// In en, this message translates to:
+  /// **'Please check your internet connection and try again.'**
+  String get networkError;
+
+  /// Generic error message for authentication failures.
+  ///
+  /// In en, this message translates to:
+  /// **'Authentication failed. Please try again.'**
+  String get authenticationError;
+
+  /// Error message shown when file storage operations fail.
+  ///
+  /// In en, this message translates to:
+  /// **'Failed to save or load data. Please try again.'**
+  String get storageError;
+
+  /// Error message shown when database operations fail.
+  ///
+  /// In en, this message translates to:
+  /// **'Failed to access data. Please try again.'**
+  String get databaseError;
+
+  /// Error message shown when input validation fails.
+  ///
+  /// In en, this message translates to:
+  /// **'Please check your input and try again.'**
+  String get validationError;
+
+  /// Generic fallback error message for unexpected errors.
+  ///
+  /// In en, this message translates to:
+  /// **'Something went wrong. Please try again.'**
+  String get generalError;
+
+  /// Title for authentication error messages.
+  ///
+  /// In en, this message translates to:
+  /// **'Authentication Error'**
+  String get authError;
+
+  /// Title for validation error messages.
+  ///
+  /// In en, this message translates to:
+  /// **'Invalid Input'**
+  String get invalidInput;
+
+  /// Error message when trying to create an account with an existing email.
+  ///
+  /// In en, this message translates to:
+  /// **'An account with this email already exists.'**
+  String get userAlreadyExists;
+
+  /// Error message when a user account cannot be found.
+  ///
+  /// In en, this message translates to:
+  /// **'User not found.'**
+  String get userNotFound;
+
+  /// Error message when requested data does not exist.
+  ///
+  /// In en, this message translates to:
+  /// **'The requested data was not found.'**
+  String get dataNotFound;
 }
 
 class _AppLocalizationsDelegate

lib/l10n/app_localizations_bn.dart

@@ -1384,4 +1384,46 @@ class AppLocalizationsBn extends AppLocalizations {
 
   @override
   String get failedToDeleteMessage => 'বার্তা মুছে ফেলা ব্যর্থ হয়েছে';
+
+  @override
+  String get usernameInvalidFormat =>
+      'Please enter a valid username. Only letters, numbers, dots, underscores, and hyphens are allowed.';
+
+  @override
+  String get usernameAlreadyTaken =>
+      'This username is already taken. Try a different one.';
+
+  @override
+  String get networkError =>
+      'Please check your internet connection and try again.';
+
+  @override
+  String get authenticationError => 'Authentication failed. Please try again.';
+
+  @override
+  String get storageError => 'Failed to save or load data. Please try again.';
+
+  @override
+  String get databaseError => 'Failed to access data. Please try again.';
+
+  @override
+  String get validationError => 'Please check your input and try again.';
+
+  @override
+  String get generalError => 'Something went wrong. Please try again.';
+
+  @override
+  String get authError => 'Authentication Error';
+
+  @override
+  String get invalidInput => 'Invalid Input';
+
+  @override
+  String get userAlreadyExists => 'An account with this email already exists.';
+
+  @override
+  String get userNotFound => 'User not found.';
+
+  @override
+  String get dataNotFound => 'The requested data was not found.';
 }