Fix Steamworks null ref bug - Check if client has been initialized/va…

…lidated prior to running Steamworks logic
adrian-miasik · Jan 5, 2023 · 0b7a2b3 · 0b7a2b3
1 parent 11d469c
commit 0b7a2b3
Showing 1 changed file with 26 additions and 22 deletions.
diff --git a/UnityPomodoro/Assets/AdrianMiasik/Scripts/Components/Core/Containers/TomatoCounter.cs b/UnityPomodoro/Assets/AdrianMiasik/Scripts/Components/Core/Containers/TomatoCounter.cs
@@ -155,30 +155,34 @@ public void SetHorizontalScale(Vector3 newScale)
         public void ConsumeTomatoes()
         {
 #if !UNITY_ANDROID
-            // CANNED ACHIEVEMENT
-            // Add discarded pomodoro/tomato to User Stats
-            SteamUserStats.AddStat("pomodoros_disposed", completedTomatoes.Count);
-            SteamUserStats.StoreStats();
-
-            // Fetch canned achievement
-            Achievement ach = new("ACH_CANNED");
-
-            // If achievement is not unlocked...
-            if (!ach.State)
+            // Check if steam client is found...
+            if (SteamClient.IsValid)
             {
-                // Fetch progression
-                int pomodorosDiscarded = SteamUserStats.GetStatInt("pomodoros_disposed");
-
-                if (pomodorosDiscarded >= 8)
-                {
-                    ach.Trigger();
-                    Debug.Log("Steam Achievement Unlocked! 'Canned for L8R: Dispose of 8" +
-                              " pomodoros/tomatoes..'");
-                }
-                else if (pomodorosDiscarded != 0)
+                // CANNED ACHIEVEMENT
+                // Add discarded pomodoro/tomato to User Stats
+                SteamUserStats.AddStat("pomodoros_disposed", completedTomatoes.Count);
+                SteamUserStats.StoreStats();
+                    
+                // Fetch canned achievement
+                Achievement ach = new("ACH_CANNED");
+
+                // If achievement is not unlocked...
+                if (!ach.State)
                 {
-                    // Display progress every time we discard
-                    SteamUserStats.IndicateAchievementProgress(ach.Identifier, pomodorosDiscarded, 8);
+                    // Fetch progression
+                    int pomodorosDiscarded = SteamUserStats.GetStatInt("pomodoros_disposed");
+
+                    if (pomodorosDiscarded >= 8)
+                    {
+                        ach.Trigger();
+                        Debug.Log("Steam Achievement Unlocked! 'Canned for L8R: Dispose of 8" +
+                                  " pomodoros/tomatoes..'");
+                    }
+                    else if (pomodorosDiscarded != 0)
+                    {
+                        // Display progress every time we discard
+                        SteamUserStats.IndicateAchievementProgress(ach.Identifier, pomodorosDiscarded, 8);
+                    }
                 }
             }
 #endif