✏️ ♻️ Rename Faces and override GetRandomFace()

"ListFaces" was misleading because it is not a list
GetRandomFace should return a Face<T> when called from a HomogeneousDie

Sources/Model/Dice/Die.cs

@@ -7,21 +7,21 @@ namespace Model.Dice
 {
     public abstract class Die
     {
-        public IEnumerable<Face> ListFaces => listFaces;
+        public IEnumerable<Face> Faces => faces;
 
         protected static readonly Random rnd = new();
 
-        private readonly List<Face> listFaces = new();
+        private readonly List<Face> faces = new();
 
         protected Die(params Face[] faces)
         {
-            listFaces.AddRange(faces);
+            this.faces.AddRange(faces);
         }
 
-        public Face GetRandomFace()
+        public virtual Face GetRandomFace()
         {
-            int faceIndex = rnd.Next(0, ListFaces.Count());
+            int faceIndex = rnd.Next(0, Faces.Count());
-            return ListFaces.ElementAt(faceIndex);
+            return Faces.ElementAt(faceIndex);
         }
     }
 }

Sources/Model/Dice/HomogeneousDie.cs

@@ -7,5 +7,10 @@ namespace Model.Dice
         protected HomogeneousDie(params Face<T>[] faces) : base(faces)
         {
         }
+
+        public override Face<T> GetRandomFace()
+        {
+            return (Face<T>)base.GetRandomFace();
+        }
     }
 }