Bug fix

Maticzpl · Jun 30, 2021 · be36300 · be36300
1 parent ef1ad6e
commit be36300
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 4 deletions.
diff --git a/MoreHazards/MoreHazards/Doors.cs b/MoreHazards/MoreHazards/Doors.cs
@@ -85,10 +85,18 @@ public IEnumerator<float> RandomDoorMalfunction()
                         continue;
 
                     //random door of the room the player is inside
-                    var door = CollectionUtils<DoorVariant>.GetRandomElement((Map.FindParentRoom(player.GameObject).Doors));
-
-                    door.NetworkTargetState = false;
-                    Log.Debug("Door closed on player:"+player.Nickname);
+                    try
+                    {
+                        var door = CollectionUtils<DoorVariant>.GetRandomElement((Map.FindParentRoom(player.GameObject).Doors));
+
+                        door.NetworkTargetState = false;
+                        Log.Debug("Door closed on player:" + player.Nickname);
+                    }
+                    catch (Exception e)
+                    {
+                        Log.Debug(e + " No doors found", MoreHazards.Instance.Config.Debug);
+                        throw;
+                    }
                 }
             }
         }

diff --git a/MoreHazards/MoreHazards/utils.cs b/MoreHazards/MoreHazards/utils.cs
@@ -111,6 +111,9 @@ public static TValue GetRandomElement(List<TValue> collection)
         public static TValue GetRandomElement(IEnumerable<TValue> enumerable)
         {
             var collection = (enumerable as IReadOnlyList<TValue>) ?? enumerable.ToArray();
+            if (collection.Count <= 0)
+                throw new Exception("Collection is empty");
+
             int index = UnityEngine.Random.Range(0, collection.Count - 1);
             return collection[index];
         }