Flutter: Is there a better way to handle calling setState() for each condition in an if/else statement?

Question

I have code that I think is not optimized. Basically, I want to change the value of errorString if a FirebaseAuthException occurs. Do I use a ChangeNotifier on this instead?

Future<void> _signInWithEmailAndPassword(BuildContext context,AsyncSnapshot<String> userSnapshot,AsyncSnapshot<String> passwordSnapshot) async {
    try {
      final auth = Provider.of<AuthService>(context, listen: false);
      await auth.signInWithEmailAndPassword(
          userSnapshot.data.trim(), passwordSnapshot.data.trim());
          setState(() => errorString = 'Success');
    } on FirebaseAuthException catch (e) {
      // I don't want to call setState() in each condition
      if (e.code == 'invalid-credential') {
        setState(() {
          errorString = "Email address appears to be malformed/expired";
        });
      } else if (e.code == 'wrong-password') {
        setState(() {
          errorString = "Password associated with this email is wrong";
        });
      } else if (e.code == 'user-not-found') {
        setState(() {
          errorString = "Email has not been registered, please sign up :)";
        });
      } else if (e.code == 'user-disabled') {
        setState(() {
          errorString = "User with this email has been disabled :(";
        });
      } else if (e.code == 'too-many-requests') {
        setState(() {
          errorString = "Too many requests, please try again later.";
        });
      } else if (e.code == 'operation-not-allowed') {
        setState(() {
          errorString = "Signing in with email and password is not enabled";
        });
      } else if (e.code == 'account-exists-with-different-credential') {
        setState(() {
          errorString =
              "Email has already been registered. Reset your password.";
        });
      }
    } 
  }

score 3 · Accepted Answer · answered Sep 05 '20 at 07:08

I would just do something like this:

const errMap = {
    'invalid-credential': "Email address appears to be malformed/expired",
    'wrong-password': "Password associated with this email is wrong",
    'user-not-found': "Email has not been registered, please sign up :)",
    'user-disabled': "User with this email has been disabled :(",
    'too-many-requests': "Too many requests, please try again later.",
    'operation-not-allowed': "Signing in with email and password is not enabled",
    'account-exists-with-different-credential': "Email has already been registered. Reset your password."
};

setState(() {
    errorString = errMap[e.code];
});

score 2 · Answer 2 · answered Sep 05 '20 at 07:06

Just put setState at end of all Like

 Future<void> _signInWithEmailAndPassword(BuildContext context,AsyncSnapshot<String> userSnapshot,AsyncSnapshot<String> passwordSnapshot) async {
    try {
      final auth = Provider.of<AuthService>(context, listen: false);
      await auth.signInWithEmailAndPassword(
          userSnapshot.data.trim(), passwordSnapshot.data.trim());
      setState(() => errorString = 'Success');
    } on FirebaseAuthException catch (e) {
      // I don't want to call setState() in each condition
      if (e.code == 'invalid-credential') {
       
          errorString = "Email address appears to be malformed/expired";
       
      } else if (e.code == 'wrong-password') {
      
          errorString = "Password associated with this email is wrong";
        
      } else if (e.code == 'user-not-found') {
        
          errorString = "Email has not been registered, please sign up :)";
       
      } else if (e.code == 'user-disabled') {
        
          errorString = "User with this email has been disabled :(";
        
      } else if (e.code == 'too-many-requests') {
         
          errorString = "Too many requests, please try again later.";
       
      } else if (e.code == 'operation-not-allowed') {
        
          errorString = "Signing in with email and password is not enabled";
        
      } else if (e.code == 'account-exists-with-different-credential') {
     
          errorString =
          "Email has already been registered. Reset your password.";
        
      }
      setState(() {})
    }
  }

Flutter: Is there a better way to handle calling setState() for each condition in an if/else statement?

2 Answers2