Merge pull request #9 from katsumasa/develop

katsumasa · web-flow · commit 5c3119fde4ed · 2022-09-14T12:50:46.000+09:00
Develop
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,8 +1,11 @@
 # Changelog
 
+## [1.1.3] - 2022-09-14
 
-## [0.1.0] - 2021-10-15
+- [Fixed] イベントスクリプトエラーが発生した場合、Playボタンが押せなくなっていた不具合を修正
+- [Fixed] Autoプレイ時に選択したイベントスクリプトではなく、先頭のスクリプトが実行されていた不具合を修正
+- [Changed] イベントスクリプトエラーが発生した場合、Debug.LogではなくDebug.LogErrorでコンソール出力を行うように変更。また、可能な場合、イベントスクリプト名と行数を表示するようにした。
 
-### Changes
+## [0.1.0] - 2021-10-15
 
-- 1st Release
+- [New] 1st Release
diff --git a/README.md b/README.md
@@ -66,16 +66,16 @@ ifls i 100 goto LOOP1
 ## 使用方法
 
 - UnityEditorとUnityPlayer(実機)の両方で実行出来ます。
-- 既存のEventSystemオブジェクトとEventSystemBotオブジェクトを差し替える必要があります。
+- 既存の`Event System`オブジェクトと`Event System Bot`オブジェクトを差し替える必要があります。
 - 具体的な手順は下記の通りです。
 
 ### インストール方法
 
 1. 本リポジトリを使用したいUnityProjectのAsset以下に配置します。</br></br>
-2. Scene上に、`EventSystemBot` を配置します。元々Scene上にある`EventSystem`は無効化して下さい。</br></br>
+2. Scene上に、`Event System Bot` を配置します。元々Scene上にある`EventSystem`は無効化して下さい。</br></br>
 ![7bb999acffa06c965befe08d2e0dfb32](https://user-images.githubusercontent.com/29646672/114997568-f414e000-9eda-11eb-9019-e399679cc537.gif)</br></br>
 3. [イベントスクリプト]を作成します。イベントスクリプトの記述情報に関しては、[こちら](https://github.com/katsumasa/UnityBotKun/wiki/EventScript)をご確認下さい。また、プレイ中の入力を記録し、イベントスクリプトに書き出すことも可能です。</br></br>
-4. イベントスクリプトをEventSystemBot->EventScriptSystem->Scriptsに登録します。</br></br>
+4. イベントスクリプトを`Event System Bot`->`Event Script System`->`Scripts`に登録します。</br></br>
 ![4cc62410ddd69f7453220c85b54bae02](https://user-images.githubusercontent.com/29646672/115168940-9f9a7c00-a0f7-11eb-9f37-8630c06d885c.gif)</br></br>
 5. プログラム中に[Input](https://docs.unity3d.com/ja/2018.4/ScriptReference/Input.html)を使用している箇所は`Input2`に置き換えて下さい。</br> example</br>
 
@@ -91,7 +91,7 @@ var horizontal = Input2.GetAxsisRow("Horizontal");
 
 ### UnityEditorでの実行方法
 
-Unity EditorをPlay Modeで実行し、任意のタイミングでEventScriptSystemのPlayボタンを押します。</br></br>
+Unity EditorをPlay Modeで実行し、任意のタイミングで`Event Script System`のPlayボタンを押します。</br></br>
 ![223d79121d8f60d04063952a468103fb](https://user-images.githubusercontent.com/29646672/115173162-9f9f7980-a101-11eb-9bc1-88bb9615ca79.gif)</br></br>
 
 ### UnityPlayer(実機)での実行方法
@@ -105,13 +105,13 @@ UnityBotKun Remote Clientを使ってApplicationをUnityEditorからリモート
 
 ## Component
 
-ここでは、`EventSystemBot`　にAddされているコンポーネントを説明します。
+ここでは、`Event System Bot`　にAddされているコンポーネントを説明します。
 
 ### Event System Bot
 
-EventSystemBotはEventSystem,StandaloneInputModuleOverrider,ScriptBot,InputBot,InputRecorder,DontDestory等の複数のコンポーネントで構成されています。
+`Event System Bot`はEventSystem,StandaloneInputModuleOverrider,ScriptBot,InputBot,InputRecorder,DontDestory等の複数のコンポーネントで構成されています。
 
-### EventSystem
+### Event System
 
 ![img](https://user-images.githubusercontent.com/29646672/115169576-65ca7500-a0f9-11eb-95cf-c1f649bcf857.png)
 
@@ -125,7 +125,7 @@ EventSystemBotはEventSystem,StandaloneInputModuleOverrider,ScriptBot,InputBot,I
 Axisやボタンの名称を変更する場合はこちらで設定を行います。
 詳細に関しては、[スクリプトリファレンス](https://docs.unity3d.com/ja/2018.4/ScriptReference/EventSystems.StandaloneInputModule.html)をご確認下さい。</br></br>
 
-### EventScriptSystem
+### Event Script System
 
 ![img](https://user-images.githubusercontent.com/29646672/115514564-31081a80-a2bf-11eb-9ca6-991f5ed9b4e2.png)
 
@@ -175,12 +175,12 @@ InputをHackするコンポーネントです。
 
 ### Dont Destory
 
-Sceneを跨いでEventSystemBotを使用する為のコンポーネントです。
+Sceneを跨いで`Event System Bot`を使用する為のコンポーネントです。
 
 ![img](https://user-images.githubusercontent.com/29646672/115174476-44bb5180-a104-11eb-9dc0-43120e0f571a.png)
 
 - Is Dont Destroy On Load
-  Scene切り替え時のEventSystemBotを破棄したくない場合は有効にする必要があります。
+  Scene切り替え時の`Event System Bot`を破棄したくない場合は有効にする必要があります。
 
 ### Remote Player
 
@@ -190,7 +190,7 @@ Sceneを跨いでEventSystemBotを使用する為のコンポーネントです
 
 <img width="426" alt="RemoteClient" src="https://user-images.githubusercontent.com/29646672/116061453-d48d6c80-a6bd-11eb-93f5-2dcfc7384654.png">
 
-UnityPlayer(実機)上のEventSystemBotをUnityEditorから制御する為のWindowです。
+UnityPlayer(実機)上の`Event System Bot`をUnityEditorから制御する為のWindowです。
 
 ① Refleshボタン。Player上のUnityBotKunの情報を取得します。</br>
 ② イベントスクリプト実行ボタン。④で指定されたイベントスクリプトを実行/停止を行います。
@@ -217,10 +217,10 @@ UnityPlayer(実機)上のEventSystemBotをUnityEditorから制御する為のWin
 - A</br>`InpurRecorder`はInputのみを記録しています。処理落ちや乱数など、結果に対して様々な要員がある為、同じ結果にならない場合があります。
 
 - Q</br>イベントスクリプトを実行しても、Inputが反映されません。
-- A</br>他の`EventSystem`が有効になっている可能性があります。実行時に`EventSystemBot`以外の`EventSystem`が`Scene`上に存在しないか確認してみて下さい。また、`Standalone Iput Module Override`の`Force Module Active`を有効にすることで改善する可能性があります。
+- A</br>他の`Event System`が有効になっている可能性があります。実行時に`Event System Bot`以外の`Event System`が`Scene`上に存在しないか確認してみて下さい。また、`Standalone Iput Module Override`の`Force Module Active`を有効にすることで改善する可能性があります。
 
 - Q</br>`uGUI`にはタッチやマウスのクリックが反応しますが、3D等他のオブジェクトに反応しません。
-- A</br>`EventSystem`を利用している為、`MonoBehaviour.OnMouseXXX`系のイベントは発生しません。[IPointerEnterHandler](https://docs.unity3d.com/ja/2018.4/ScriptReference/EventSystems.IPointerEnterHandler.html)を継承する等してイベントをキャッチして下さい。また、Cameraオブジェクトに[PhysicsRaycaster](https://docs.unity3d.com/ja/2018.4/ScriptReference/EventSystems.PhysicsRaycaster.html)や[Physics2DRaycaster](https://docs.unity3d.com/ja/2018.4/ScriptReference/EventSystems.Physics2DRaycaster.html)をAddすることもお忘れなく。
+- A</br>`Event System`を利用している為、`MonoBehaviour.OnMouseXXX`系のイベントは発生しません。[IPointerEnterHandler](https://docs.unity3d.com/ja/2018.4/ScriptReference/EventSystems.IPointerEnterHandler.html)を継承する等してイベントをキャッチして下さい。また、Cameraオブジェクトに[PhysicsRaycaster](https://docs.unity3d.com/ja/2018.4/ScriptReference/EventSystems.PhysicsRaycaster.html)や[Physics2DRaycaster](https://docs.unity3d.com/ja/2018.4/ScriptReference/EventSystems.Physics2DRaycaster.html)をAddすることもお忘れなく。
 
 [^1]:リリース済みのアプリケーションを制御出来る訳ではありません。）
 [^2]:Input単体での再現の為、再現性の精度は低いです。
diff --git a/Runtime/Scripts/EventScriptSystem.cs b/Runtime/Scripts/EventScriptSystem.cs
@@ -393,7 +393,7 @@ IEnumerator Start()
                 {
                     if (BaseInputOverride.instance != null)
                     {                        
-                        Play();
+                        Play(m_currentScriptIndex);
                         break;
                     }
                     else
@@ -601,7 +601,7 @@ bool CommandWait(string[] args)
                     m_waitSceneChanged = GetInt(args[2]);
                 }
             }            
-            return false;
+            return true;
         }
 
 
@@ -619,7 +619,7 @@ bool CommandGoto(string[] args)
             }
             else
             {
-                Debug.Log("Invalid Label :" + args);
+                Debug.LogError("Invalid Label :" + args[1]);
                 isError = true;
                 return false;
             }
@@ -667,7 +667,7 @@ bool CommandSet(string[] args)
             }
             else
             {
-                Debug.Log("Invalid variable name : " + args[1]);
+                Debug.LogError("Invalid variable name : " + args[1]);
                 isError = true;
                 return false;
             }
@@ -701,7 +701,8 @@ bool CommandAdd(string[] args)
             else
             {
                 isError = true;
-                Debug.Log("Invalid args in add "+ args);
+                Debug.LogError("Invalid args in add "+ args[1]);
+                return false;
             }
             return true;
         }
@@ -722,6 +723,12 @@ bool CommandSub(string[] args)
             {
                 variableFloats[args[1]] -= GetFloat(args[2]) - GetFloat(args[3]);
             }
+            else
+            {
+                isError = true;
+                Debug.LogError("Invalid args in Sub " + args[1]);
+                return false;
+            }
             return true;
         }
 
@@ -744,7 +751,7 @@ bool CommandMul(string[] args)
             else
             {
                 isError = true;
-                Debug.Log("Invalid args in Mul " + args);
+                Debug.LogError("Invalid args in Mul " + args[1]);
                 return false;
             }
             return true;
@@ -769,7 +776,7 @@ bool CommandDiv(string[] args)
             else
             {
                 isError = true;
-                Debug.Log("Invalid args in div " + args);
+                Debug.LogError("Invalid args in div " + args[1]);                
                 return false;
             }
             return true;
@@ -1182,7 +1189,7 @@ bool SyntaxAnalysis(string[] args)
             }
             else
             {
-                Debug.Log("Invalid Command Name "+ args[0]);
+                Debug.LogError("Invalid Command Name "+ args[0]);
                 isError = true;
                 return false;
             }            
@@ -1246,6 +1253,11 @@ void Interpreter(float deltaTime)
 
                 // 構文解析
                 if (SyntaxAnalysis(args) == false) {
+                    isPlay = false;
+                    break;
+                }
+                if (m_waitFrame > 0 || (m_waitTime > 0f) || m_waitSceneChanged > 0)
+                {
                     break;
                 }
             }
@@ -1386,10 +1398,11 @@ void PreProcessor()
 
 
 
-
+                var lineNo = 0;
                 while (!m_textAssetReader.EndOfStream)
                 {
                     var line = m_textAssetReader.ReadLine();
+                    lineNo++;
                     if (line == null)
                     {
                         break;
@@ -1398,7 +1411,14 @@ void PreProcessor()
                     {
                         //var args = line.Split(new char[] { '#' }, StringSplitOptions.RemoveEmptyEntries);
                         var args = line.Split(new char[] { ' ' }, StringSplitOptions.RemoveEmptyEntries);
-                        labelOfsts.Add(args[1], m_textAssetReader.Position);
+                        if (args.Length == 2)
+                        {
+                            labelOfsts.Add(args[1], m_textAssetReader.Position);
+                        }
+                        else
+                        {                                                        
+                            Debug.LogError($@"<a href=""{m_textAssetReader.Name}"" line=""{lineNo}"">{m_textAssetReader.Name}:{lineNo}</a> Invalid Label format {line}");
+                        }
                     }
                     else if (line.StartsWith("int"))
                     {
diff --git a/Runtime/Scripts/TextAssetReader.cs b/Runtime/Scripts/TextAssetReader.cs
@@ -2,7 +2,9 @@
 using System.Collections.Generic;
 using System.IO;
 using UnityEngine;
-
+#if UNITY_EDITOR
+using UnityEditor;
+#endif
 
 namespace Utj.UnityBotKun
 {
@@ -23,6 +25,12 @@ public long Position
             set;            
         }
 
+        public string Name
+        {
+            get;
+            private set;
+        }
+
 
         public bool EndOfStream
         {
@@ -52,7 +60,11 @@ public TextAssetReader(TextAsset textAsset)
         {
             Position = 0;
             lines = new List<string>();
-
+#if UNITY_EDITOR
+            Name = AssetDatabase.GetAssetPath(textAsset);
+#else
+            Name = textAsset.name + ".txt";            
+#endif
             using (var ms = new MemoryStream(textAsset.bytes))
             {
                 using (var sr = new StreamReader(ms))
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "com.utj.unitybotkun",
   "displayName": "UnityBotKun",
-  "version": "1.1.2",
+  "version": "1.1.3",
   "unity": "2019.3",
   "keywords": [
     "unity",

Original file line number	Diff line number	Diff line change
`@@ -393,7 +393,7 @@ IEnumerator Start()`
`393`	`393`	`{`
`394`	`394`	`if (BaseInputOverride.instance != null)`
`395`	`395`	`{`
`396`		`- Play();`
	`396`	`+ Play(m_currentScriptIndex);`
`397`	`397`	`break;`
`398`	`398`	`}`
`399`	`399`	`else`
`@@ -601,7 +601,7 @@ bool CommandWait(string[] args)`
`601`	`601`	`m_waitSceneChanged = GetInt(args[2]);`
`602`	`602`	`}`
`603`	`603`	`}`
`604`		`- return false;`
	`604`	`+ return true;`
`605`	`605`	`}`
`606`	`606`
`607`	`607`
`@@ -619,7 +619,7 @@ bool CommandGoto(string[] args)`
`619`	`619`	`}`
`620`	`620`	`else`
`621`	`621`	`{`
`622`		`- Debug.Log("Invalid Label :" + args);`
	`622`	`+ Debug.LogError("Invalid Label :" + args[1]);`
`623`	`623`	`isError = true;`
`624`	`624`	`return false;`
`625`	`625`	`}`
`@@ -667,7 +667,7 @@ bool CommandSet(string[] args)`
`667`	`667`	`}`
`668`	`668`	`else`
`669`	`669`	`{`
`670`		`- Debug.Log("Invalid variable name : " + args[1]);`
	`670`	`+ Debug.LogError("Invalid variable name : " + args[1]);`
`671`	`671`	`isError = true;`
`672`	`672`	`return false;`
`673`	`673`	`}`
`@@ -701,7 +701,8 @@ bool CommandAdd(string[] args)`
`701`	`701`	`else`
`702`	`702`	`{`
`703`	`703`	`isError = true;`
`704`		`- Debug.Log("Invalid args in add "+ args);`
	`704`	`+ Debug.LogError("Invalid args in add "+ args[1]);`
	`705`	`+ return false;`
`705`	`706`	`}`
`706`	`707`	`return true;`
`707`	`708`	`}`
`@@ -722,6 +723,12 @@ bool CommandSub(string[] args)`
`722`	`723`	`{`
`723`	`724`	`variableFloats[args[1]] -= GetFloat(args[2]) - GetFloat(args[3]);`
`724`	`725`	`}`
	`726`	`+ else`
	`727`	`+ {`
	`728`	`+ isError = true;`
	`729`	`+ Debug.LogError("Invalid args in Sub " + args[1]);`
	`730`	`+ return false;`
	`731`	`+ }`
`725`	`732`	`return true;`
`726`	`733`	`}`
`727`	`734`
`@@ -744,7 +751,7 @@ bool CommandMul(string[] args)`
`744`	`751`	`else`
`745`	`752`	`{`
`746`	`753`	`isError = true;`
`747`		`- Debug.Log("Invalid args in Mul " + args);`
	`754`	`+ Debug.LogError("Invalid args in Mul " + args[1]);`
`748`	`755`	`return false;`
`749`	`756`	`}`
`750`	`757`	`return true;`
`@@ -769,7 +776,7 @@ bool CommandDiv(string[] args)`
`769`	`776`	`else`
`770`	`777`	`{`
`771`	`778`	`isError = true;`
`772`		`- Debug.Log("Invalid args in div " + args);`
	`779`	`+ Debug.LogError("Invalid args in div " + args[1]);`
`773`	`780`	`return false;`
`774`	`781`	`}`
`775`	`782`	`return true;`
`@@ -1182,7 +1189,7 @@ bool SyntaxAnalysis(string[] args)`
`1182`	`1189`	`}`
`1183`	`1190`	`else`
`1184`	`1191`	`{`
`1185`		`- Debug.Log("Invalid Command Name "+ args[0]);`
	`1192`	`+ Debug.LogError("Invalid Command Name "+ args[0]);`
`1186`	`1193`	`isError = true;`
`1187`	`1194`	`return false;`
`1188`	`1195`	`}`
`@@ -1246,6 +1253,11 @@ void Interpreter(float deltaTime)`
`1246`	`1253`
`1247`	`1254`	`// 構文解析`
`1248`	`1255`	`if (SyntaxAnalysis(args) == false) {`
	`1256`	`+ isPlay = false;`
	`1257`	`+ break;`
	`1258`	`+ }`
	`1259`	`+ if (m_waitFrame > 0 \|\| (m_waitTime > 0f) \|\| m_waitSceneChanged > 0)`
	`1260`	`+ {`
`1249`	`1261`	`break;`
`1250`	`1262`	`}`
`1251`	`1263`	`}`
`@@ -1386,10 +1398,11 @@ void PreProcessor()`
`1386`	`1398`
`1387`	`1399`
`1388`	`1400`
`1389`		`-`
	`1401`	`+ var lineNo = 0;`
`1390`	`1402`	`while (!m_textAssetReader.EndOfStream)`
`1391`	`1403`	`{`
`1392`	`1404`	`var line = m_textAssetReader.ReadLine();`
	`1405`	`+ lineNo++;`
`1393`	`1406`	`if (line == null)`
`1394`	`1407`	`{`
`1395`	`1408`	`break;`
`@@ -1398,7 +1411,14 @@ void PreProcessor()`
`1398`	`1411`	`{`
`1399`	`1412`	`//var args = line.Split(new char[] { '#' }, StringSplitOptions.RemoveEmptyEntries);`
`1400`	`1413`	`var args = line.Split(new char[] { ' ' }, StringSplitOptions.RemoveEmptyEntries);`
`1401`		`- labelOfsts.Add(args[1], m_textAssetReader.Position);`
	`1414`	`+ if (args.Length == 2)`
	`1415`	`+ {`
	`1416`	`+ labelOfsts.Add(args[1], m_textAssetReader.Position);`
	`1417`	`+ }`
	`1418`	`+ else`
	`1419`	`+ {`
	`1420`	`+ Debug.LogError($@"<a href=""{m_textAssetReader.Name}"" line=""{lineNo}"">{m_textAssetReader.Name}:{lineNo}</a> Invalid Label format {line}");`
	`1421`	`+ }`
`1402`	`1422`	`}`
`1403`	`1423`	`else if (line.StartsWith("int"))`
`1404`	`1424`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "com.utj.unitybotkun",`
`3`	`3`	`"displayName": "UnityBotKun",`
`4`		`- "version": "1.1.2",`
	`4`	`+ "version": "1.1.3",`
`5`	`5`	`"unity": "2019.3",`
`6`	`6`	`"keywords": [`
`7`	`7`	`"unity",`