Refactored out the static method on RandomGridSeeder and created an interface.

Author: keithwillcode

ConsoleApplication/Program.cs

@@ -14,7 +14,8 @@ class Program
         static void Main(string[] args)
         {
             var grid = new LifeGrid(NumberOfRows, NumberOfColumns);
-            RandomGridSeeder.Seed(grid);
+            var randomGridSeeder = new RandomGridSeeder();
+            randomGridSeeder.Seed(grid);
 
             while (true)
             {

Core/GameOfLife.Core.csproj

@@ -43,6 +43,7 @@
   </ItemGroup>
   <ItemGroup>
     <Compile Include="Coordinate.cs" />
+    <Compile Include="IGridSeeder.cs" />
     <Compile Include="LifeGrid.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
     <Compile Include="RandomGridSeeder.cs" />

Core/IGridSeeder.cs

@@ -0,0 +1,7 @@
+namespace GameOfLife.Core
+{
+    public interface IGridSeeder
+    {
+        void Seed(LifeGrid grid);
+    }
+}

Core/RandomGridSeeder.cs

@@ -2,11 +2,10 @@
 
 namespace GameOfLife.Core
 {
-    public class RandomGridSeeder
+    public class RandomGridSeeder : IGridSeeder
     {
-        public static void Seed(LifeGrid grid)
+        public void Seed(LifeGrid grid)
         {
-            var numberOfCells = grid.NumberOfRows * grid.NumberOfColumns;
             var random = new Random();
 
             for (var x = 0; x < grid.NumberOfColumns; x++)

SharpGlWpfApplication/MainWindow.xaml.cs

@@ -38,7 +38,8 @@ public MainWindow()
         private void StartSimulation()
         {
             lifeGrid = new LifeGrid(NumberOfRows, NumberOfColumns);
-            RandomGridSeeder.Seed(lifeGrid);
+            var randomGridSeeder = new RandomGridSeeder();
+            randomGridSeeder.Seed(lifeGrid);
             timer.Start();
         }
 

SignalRApplication/GameOfLifeExecutor.cs

@@ -10,19 +10,16 @@ namespace SignalRApplication
     public class GameOfLifeExecutor
     {
         private readonly static Lazy<GameOfLifeExecutor> instance = new Lazy<GameOfLifeExecutor>(() => new GameOfLifeExecutor(GlobalHost.ConnectionManager.GetHubContext<GameOfLifeHub>().Clients));
-        private readonly static LifeGrid lifeGrid = new LifeGrid(50, 50);
+        private readonly static Lazy<LifeGrid> lifeGrid = new Lazy<LifeGrid>(() => new LifeGrid(150, 150));
+        private readonly static Lazy<IGridSeeder> gridSeeder = new Lazy<IGridSeeder>(() => new RandomGridSeeder());
 
         private readonly TimeSpan updateInterval = TimeSpan.FromMilliseconds(250);
-        private readonly Object updateLivingCellsLock = new Object();
         private readonly Timer timer;
-        private volatile Boolean updatingLivingCells = false;
 
         private GameOfLifeExecutor(IHubConnectionContext clients)
         {
             Clients = clients;
-
-            RandomGridSeeder.Seed(lifeGrid);
-
+            gridSeeder.Value.Seed(lifeGrid.Value);
             timer = new Timer(UpdateLivingCells, null, updateInterval, updateInterval);
         }
 
@@ -39,34 +36,18 @@ private IHubConnectionContext Clients
 
         public IEnumerable<Coordinate> GetLivingCells()
         {
-            return lifeGrid.GetLivingCells();
+            return lifeGrid.Value.GetLivingCells();
         }
 
         private void UpdateLivingCells(Object state)
         {
-            lock (updateLivingCellsLock)
-            {
-                if (!updatingLivingCells)
-                {
-                    updatingLivingCells = true;
-                    lifeGrid.Tick();
-                    Clients.All.updateLivingCells(lifeGrid.GetLivingCells());
-                    updatingLivingCells = false;
-                }
-            }
+            lifeGrid.Value.Tick();
+            Clients.All.updateLivingCells(lifeGrid.Value.GetLivingCells());
         }
 
         public void Restart()
         {
-            lock (updateLivingCellsLock)
-            {
-                if (!updatingLivingCells)
-                {
-                    updatingLivingCells = true;
-                    RandomGridSeeder.Seed(lifeGrid);
-                    updatingLivingCells = false;
-                }
-            }
+            gridSeeder.Value.Seed(lifeGrid.Value);
         }
     }
 }

SignalRApplication/Scripts/game-of-life.js

@@ -11,7 +11,7 @@
             context.fillStyle = 'black';
             context.fill();
             context.lineWidth = 1;
-            context.strokeStyle = 'black';
+            context.strokeStyle = 'white';
             context.stroke();
         });
     };

UnitTests/LifeGridSpeedTests.cs

@@ -14,7 +14,8 @@ public class LifeGridSpeedTests
         public void TestInitialize()
         {
             grid = new LifeGrid(1000, 1000);
-            RandomGridSeeder.Seed(grid);
+            var randomGridSeeder = new RandomGridSeeder();
+            randomGridSeeder.Seed(grid);
         }
 
         [TestMethod]

WpfApplication/MainWindow.xaml.cs

@@ -87,7 +87,8 @@ protected void backgroundWorker_ProgressChanged(Object sender, ProgressChangedEv
         protected void backgroundWorker_DoWork(Object sender, DoWorkEventArgs e)
         {
             var grid = new LifeGrid(NumberOfRows, NumberOfColumns);
-            RandomGridSeeder.Seed(grid);
+            var randomGridSeeder = new RandomGridSeeder();
+            randomGridSeeder.Seed(grid);
 
             while (true)
             {