Merge pull request '🐛 Fix call to Random.Next()' (#82) from bug-fix-abstract-die into main

Reviewed-on: #82
2 years ago · c63729b0b2
parent b4bcd40190 7f10eaed24
commit c63729b0b2
1 changed files with 1 additions and 1 deletions
--- a/Sources/Model/Dice/AbstractDie.cs
+++ b/Sources/Model/Dice/AbstractDie.cs
@ -18,7 +18,7 @@ namespace Model.Dice

        public T GetRandomFace()
        {
-            int faceIndex = rnd.Next(1, ListFaces.Count() + 1);
+            int faceIndex = rnd.Next(0, ListFaces.Count());
            return ListFaces.ElementAt(faceIndex);
        }
    }