Merge pull request #25 from kagikn/CoroutineEnhancement_

IsCoroutineActiveの追加とtypo修正
TORISOUP · Oct 27, 2015 · d96f7cd · d96f7cd
2 parents 91bf63a + f7c8bdf
commit d96f7cd
Show file tree

Hide file tree

Showing 4 changed files with 58 additions and 9 deletions.
diff --git a/Inferno/InfernoScripts/InfernoCore/CoroutineSystem.cs b/Inferno/InfernoScripts/InfernoCore/CoroutineSystem.cs
@@ -33,7 +33,7 @@ public CoroutineSystem(DebugLogger logger = null)
         /// </summary>
         /// <param name="coroutine">登録するコルーチン</param>
         /// <returns></returns>
-        public uint AddCrotoutine(IEnumerable<Object> coroutine)
+        public uint AddCoroutine(IEnumerable<Object> coroutine)
         {
             lock (_lockObject)
             {
@@ -60,6 +60,15 @@ public void RemoveCoroutine(uint id)
             }
         }
 
+        /// <summary>
+        /// コルーチンが存在するかどうかチェックする
+        /// </summary>
+        /// <param name="id">存在するかどうか確認したいコルーチンID</param>
+        public bool ContainsCoroutine(uint id)
+        {
+            return _coroutines.ContainsKey(id);
+        }
+
 
         /// <summary>
         /// コルーチンの処理を行う

diff --git a/Inferno/InfernoScripts/InfernoCore/InfernoScript.cs b/Inferno/InfernoScripts/InfernoCore/InfernoScript.cs
@@ -116,7 +116,7 @@ protected IObservable<Unit> CreateTickAsObservable(int millsecond)
 
         protected uint StartCoroutine(IEnumerable<Object> coroutine)
         {
-            return coroutineSystem.AddCrotoutine(coroutine);
+            return coroutineSystem.AddCoroutine(coroutine);
         }
 
         protected void StopCoroutine(uint id)
@@ -127,6 +127,18 @@ protected void StopCoroutine(uint id)
             }
         }
 
+        protected bool IsCoroutineActive(uint id)
+        {
+            if (coroutineSystem != null)
+            {
+                return coroutineSystem.ContainsCoroutine(id);
+            }
+            else
+            {
+                return false;
+            }
+        }
+
         /// <summary>
         /// 指定秒数待機するIEnumerable
         /// </summary>

diff --git a/InfernoTest/ChaosModeSettingLoader/ChaosModeSettingLoaderTest.cs b/InfernoTest/ChaosModeSettingLoader/ChaosModeSettingLoaderTest.cs
@@ -16,7 +16,7 @@ namespace InfernoTest
     public class ChaosModeSettingLoaderTest
     {
         [TestMethod]
-        public void 全て正常値が設定されたJosonからChaosSettingが生成できる()
+        public void 全て正常値が設定されたJsonからChaosSettingが生成できる()
         {
             var testLoader =
                 new TestChaosModeSettingLoader(

diff --git a/InfernoTest/CoroutineSystemTest.cs b/InfernoTest/CoroutineSystemTest.cs
@@ -18,11 +18,11 @@ public void Initialize()
         }
 
         [TestMethod]
-        public void AddCroutine時にそれぞれ別のidが連番で割り振られる()
+        public void AddCoroutine時にそれぞれ別のidが連番で割り振られる()
         {
             for (uint expected = 0; expected < 10; expected++)
             {
-                var resultId = testCoroutineSystem.AddCrotoutine(testEnumerable(1));
+                var resultId = testCoroutineSystem.AddCoroutine(testEnumerable(1));
                 Assert.AreEqual(expected, resultId);
             }
 
@@ -42,10 +42,10 @@ IEnumerable<Object> ExecuteEnumerator()
 
 
         [TestMethod]
-        public void AddCroutine時に最初のコルーチンが実行される()
+        public void AddCoroutine時に最初のコルーチンが実行される()
         {
             actionCount = 0;
-            testCoroutineSystem.AddCrotoutine(ExecuteEnumerator());
+            testCoroutineSystem.AddCoroutine(ExecuteEnumerator());
 
             //登録直後に実行されているはず
             Assert.AreEqual(1, actionCount);
@@ -63,7 +63,7 @@ public void RemoveCoroutineで該当のCoroutineが削除される()
             //１０個登録
             for (uint expected = 0; expected < 10; expected++)
             {
-                testCoroutineSystem.AddCrotoutine(testEnumerable(5));
+                testCoroutineSystem.AddCoroutine(testEnumerable(5));
             }
 
             //IDは１０個全て存在する
@@ -110,11 +110,39 @@ public void RemoveCoroutineで存在しないキーを削除しても例外に
             testCoroutineSystem.RemoveCoroutine(0);
         }
 
+        [TestMethod]
+        public void ContainsCoroutineで該当のCoroutineが存在するかどうかチェックされる()
+        {
+            //１０個登録
+            for (uint expected = 0; expected < 10; expected++)
+            {
+                testCoroutineSystem.AddCoroutine(testEnumerable(5));
+            }
+
+            //IDは１０個全て存在するか確認し、すぐに除去
+            Assert.AreEqual(10, testCoroutineSystem.RegisteredCoroutineCount);
+            for (uint id = 0; id < 10; id++)
+            {
+                Assert.IsTrue(testCoroutineSystem.ContainsCoroutine(id));
+                testCoroutineSystem.RemoveCoroutine(id);
+            }
+
+            //一度コルーチンを回す
+            testCoroutineSystem.CoroutineLoop();
+
+            //IDは１０個全て存在していない
+            Assert.AreEqual(0, testCoroutineSystem.RegisteredCoroutineCount);
+            for (uint id = 0; id < 10; id++)
+            {
+                Assert.IsFalse(testCoroutineSystem.ContainsCoroutine(id));
+            }
+        }
+
 
         [TestMethod]
         public void NestしたIEnumerableを展開して実行できる()
         {
-            testCoroutineSystem.AddCrotoutine(nestEnumerable());
+            testCoroutineSystem.AddCoroutine(nestEnumerable());
             var i = 0;
 
             //コルーチンが終了するまでの実行回数を数える