ColourMeOKLABGame/Assets/Scripts/Backend.cs

@@ -11,48 +11,10 @@ using UnityEngine;
 /// </summary>
 public class Backend : MonoBehaviour
 {
-    /// <summary>
-    ///     enum for the result of the authentication process
-    /// </summary>
-    // ReSharper disable once MemberCanBePrivate.Global
-    public enum AuthenticationResult
-    {
-        Ok,
-        AlreadyAuthenticated,
-        NonExistentUser,
-        AlreadyExistingUser,
-        InvalidCredentials,
-        GenericError
-    }
-
-    /// <summary>
-    ///     generic enum for the result of a database transaction
-    /// </summary>
-    // ReSharper disable once MemberCanBePrivate.Global
-    public enum DatabaseTransactionResult
-    {
-        Ok,
-        Unauthenticated,
-        Error
-    }
-
-    /// <summary>
-    ///     enum for the connection status of the firebase backend
-    /// </summary>
-    public enum FirebaseConnectionStatus
-    {
-        NotConnected,
-        Connected,
-        UpdateRequired, // "a required system component is out of date"
-        Updating, // "a required system component is updating, retrying in a bit..."
-        ExternalError, // "a system component is disabled, invalid, missing, or permissions are insufficient"
-        InternalError // "an unknown error occurred"
-    }
-
     /// <summary>
     ///     whether the backend is connected to the firebase backend
     /// </summary>
-    public FirebaseConnectionStatus status = FirebaseConnectionStatus.NotConnected;
+    public bool connected;
 
     /// <summary>
     ///     the firebase authentication object
@@ -94,8 +56,6 @@ public class Backend : MonoBehaviour
     private void OnDestroy()
     {
         SignOutUser();
-        _auth.StateChanged -= AuthStateChanged;
-        _auth = null;
     }
 
     /// <summary>
@@ -105,7 +65,6 @@ public class Backend : MonoBehaviour
     {
         FirebaseApp.CheckAndFixDependenciesAsync().ContinueWith(task =>
         {
-            // cher is this robust enough
             switch (task.Result)
             {
                 case DependencyStatus.Available:
@@ -113,33 +72,28 @@ public class Backend : MonoBehaviour
                     _auth = FirebaseAuth.GetAuth(app);
                     _auth.StateChanged += AuthStateChanged;
                     _db = FirebaseDatabase.DefaultInstance.RootReference;
-                    status = FirebaseConnectionStatus.Connected;
+                    connected = true;
+                    Debug.Log("firebase initialised");
                     break;
 
                 case DependencyStatus.UnavailableDisabled:
                 case DependencyStatus.UnavailableInvalid:
                 case DependencyStatus.UnavilableMissing:
                 case DependencyStatus.UnavailablePermission:
-                    status = FirebaseConnectionStatus.ExternalError;
-                    break;
-
-                case DependencyStatus.UnavailableUpdating:
-                    status = FirebaseConnectionStatus.Updating;
-                    RetryInitialiseAfterDelay();
+                    Debug.LogError("firebase error outside of our control");
                     break;
 
                 case DependencyStatus.UnavailableUpdaterequired:
-                    status = FirebaseConnectionStatus.UpdateRequired;
+                case DependencyStatus.UnavailableUpdating:
+                    Debug.LogError("firebase is updating");
+                    RetryInitialiseAfterDelay();
                     break;
 
                 case DependencyStatus.UnavailableOther:
                 default:
-                    status = FirebaseConnectionStatus.InternalError;
                     Debug.LogError("firebase ??? blew up or something," + task.Result);
                     break;
             }
-
-            Debug.Log("firebase status is" + status);
         });
     }
 
@@ -150,18 +104,17 @@ public class Backend : MonoBehaviour
     /// <param name="eventArgs">the event arguments</param>
     private void AuthStateChanged(object sender, EventArgs eventArgs)
     {
-        // if the user hasn't changed, do nothing
         if (_auth.CurrentUser == _user) return;
-
-        // if the user has changed, check if they've signed in or out
         var signedIn = _user != _auth.CurrentUser && _auth.CurrentUser != null;
-
-        // if we're not signed in, but we still hold _user locally, we've signed out
-        if (!signedIn && _user != null) _onSignOutCallback(_user);
-
-        // they have signed in, update _user
-        _user = _auth.CurrentUser;
-        if (signedIn) _onSignInCallback(_user);
+        switch (signedIn)
+        {
+            case false when _user != null:
+                _onSignOutCallback(_user);
+                break;
+            case true:
+                _onSignInCallback(_auth.CurrentUser);
+                break;
+        }
     }
 
     /// <summary>
@@ -215,7 +168,7 @@ public class Backend : MonoBehaviour
     {
         return _user;
     }
-
+    
     /// <summary>
     ///     abstraction function to sign out the user
     /// </summary>
@@ -227,10 +180,10 @@ public class Backend : MonoBehaviour
     /// <summary>
     ///     abstraction function to submit a play to the database
     /// </summary>
-    /// <param name="playData">play data</param>
+    /// <param name="averageMatchAccuracy">the float percentage (0-100) of how accurate the user was when colour matching</param>
     /// <param name="callback">callback function that takes in one DatabaseTransactionResult argument</param>
     private void SubmitPlay(
-        PlayData playData,
+        float averageMatchAccuracy,
         Action<DatabaseTransactionResult> callback)
     {
         throw new NotImplementedException();
@@ -282,15 +235,27 @@ public class Backend : MonoBehaviour
     }
 
     /// <summary>
-    ///     struct for play data
+    ///     enum for the result of the authentication process
     /// </summary>
     // ReSharper disable once MemberCanBePrivate.Global
-    public struct PlayData
+    public enum AuthenticationResult
     {
-        public int RoundsPlayed;
-        public float AverageOverallAccuracy;
-        public float AverageLuminanceAccuracy;
-        public float AverageRedGreenAccuracy;
-        public float AverageBlueYellowAccuracy;
+        Ok,
+        AlreadyAuthenticated,
+        NonExistentUser,
+        AlreadyExistingUser,
+        InvalidCredentials,
+        GenericError
+    }
+
+    /// <summary>
+    ///     generic enum for the result of a database transaction
+    /// </summary>
+    // ReSharper disable once MemberCanBePrivate.Global
+    public enum DatabaseTransactionResult
+    {
+        Ok,
+        Unauthenticated,
+        Error
     }
 }