LoopGearNetworkTestが通るように修正 #340

KurisuJuha · 2024-05-22T05:56:30Z

No description provided.

sakastudio · 2024-05-22T07:16:43Z

moorestech_server/Assets/Scripts/Game.Block/Blocks/BeltConveyor/VanillaBeltConveyorComponent.cs

        public const float DefaultBeltConveyorHeight = 0.3f;
+        private readonly BlockConnectorComponent<IBlockInventory> _blockConnectorComponent;


オートフォーマッタでprivateとpublicの順番ゴチャゴチャになってる？

public → private readonly → private の順番にしたい

このprで修正します？
riderのオートフォーマッタがやったので今後弄ったときのためにも修正するならeditorconfigごと修正しないといけないと思ってます

このPRでの修正は不要です（私が過去やってしまったやつものこっているので

sakastudio · 2024-05-22T07:17:41Z

moorestech_server/Assets/Scripts/Game.Block/Blocks/BeltConveyor/VanillaBeltConveyorComponent.cs

@@ -135,6 +107,41 @@ public void SetItem(int slot, IItemStack itemStack)
            _inventoryItems[slot] = new BeltConveyorInventoryItem(itemStack.Id, TimeOfItemEnterToExit, itemStack.ItemInstanceId);
        }

+        public void Destroy()


ここの順番入れ替えもオートフォーマッタ？

sakastudio · 2024-05-22T07:19:15Z

...Assets/Scripts/Game.Block/Blocks/Service/ConnectingInventoryListPriorityInsertItemService.cs

@@ -22,7 +23,7 @@ public ConnectingInventoryListPriorityInsertItemService(BlockConnectorComponent<

        public IItemStack InsertItem(IItemStack itemStack)
        {
-            IReadOnlyList<IBlockInventory> inventories = _blockConnectorComponent.ConnectTargets;
+            IReadOnlyList<IBlockInventory> inventories = _blockConnectorComponent.ConnectTargets.Keys.ToArray();


色んなところでやっちゃってますけど、ToArrayとかはどこかで一掃したいですよね〜〜

僕は必要であれば気にしないタイプですが、確かに見る側からしたら鬱陶しいですよねー

コード的な理由というよりは、GCが増えるので気になりポイントですね
そのうちパフォーマンスが重要になりそうなので、どこかのタイミングで一掃したいなぁ見たいな感じです

sakastudio · 2024-05-22T07:32:05Z

moorestech_server/Assets/Scripts/Game.Block/Config/LoadConfig/OptionLoader/GearConnectOption.cs

+    {
+        public static readonly GearConnectOptionLoader Loader = new();
+
+        public IConnectOption Load(dynamic connectorOption)


GearConnectOptionLoader と GearConnectOption を同じファイルに書いておきたい感ありますが、アセンブリ的に厳しいやつですかね

IConnectOptionLoaderをBlock.Interfaceに置けば解決する説

かもしれないです
やってみます

sakastudio · 2024-05-22T07:32:18Z

moorestech_server/Assets/Scripts/Game.Block/Config/LoadConfig/BlockConfigJsonLoad.cs

+
+                var option = connector.option;
+                if (option != null && loader != null)
+                {


option != nullはいらないです
option == null なのにローダー指定してるのはどっちかが間違っているので、明確に誤りな部分はちゃんとエラーをすろーするようにしておきたいですね

//TODO エラーログ出力するみたいなの付けておいてください

修正しましたー

sakastudio · 2024-05-22T07:33:44Z

moorestech_server/Assets/Scripts/Game.Block/Config/LoadConfig/Param/SimpleGearGeneratorParam.cs

-            var gearConnectSettings = BlockConfigJsonLoad.GetConnectSettings(blockParam, "gearConnects");
-
+
+            var gearConnectSettings = BlockConfigJsonLoad.GetConnectSettings(blockParam, "gearConnects", GearConnectOptionLoader.Loader);


gearConnects、どこかにpublic const stringとして置いておきたいです

"gearConnects"はもとからあったもので追加したわけじゃないので、Issue作って別PRでやりませんか？

sakastudio · 2024-05-22T07:40:18Z

moorestech_server/Assets/Scripts/Tests/CombinedTest/Game/GearNetworkTest.cs

-            var gearPositionC = new Vector3Int(0, 0, 1);
-            var gearPositionD = new Vector3Int(1, 0, 1);
+            var gearPositionC = new Vector3Int(0, 1, 0);
+            var gearPositionD = new Vector3Int(1, 1, 0);


このパラメーター変更はまずいかも
今回の変更のテストをきちんと行えていないのでは

いままでこういう形の歯車NWだったのが

こうなってしまっている

今まではシャフト + 歯の接続のループだったのが、いまは歯での接続のループなので、今回変更した回転反転のフラグの検証ができていない

修正しましたー

sakastudio · 2024-05-22T07:40:50Z

moorestech_server/Assets/Scripts/Tests/CombinedTest/Game/GearNetworkTest.cs


            // smallGearDの回転方向とRPMのテスト
            Assert.AreEqual(rpm, smallGearD.CurrentRpm);
-            Assert.AreEqual(false, smallGearD.IsCurrentClockwise);
+            Assert.AreEqual(true, smallGearD.IsCurrentClockwise);


これも戻す必要ありそう

修正しましたー

sakastudio

LGTM！！

"gearConnects"という文字列を直置きしている問題を修正

KurisuJuha added 22 commits May 21, 2024 16:19

SimpleGearGeneratorのパラメータミスを修正

10000

d3a88a2

一旦コミット

4df7b5d

GearConnectOptionの作成

645d145

BlockConnectorComponentがoptionを保持するように変更

02db940

ConnectionInventoryListPriorityInsertItemServiceのエラーを修正

4cd3f45

GearEnergyTransformerComponentのエラーを修正

900ff30

VanillaBeltConveyorComponentのエラーを修正

b8febae

BlockPlaceToConnectionBlockTestのコンパイルエラーの修正

c8627c5

FullInsertAndChangeConnectorBeltConveyorTestのテストを通るように修正

cdaafd9

FullInsertBeltConveyorTestが通るように修正

225cdb1

InsertBeltConveyorTestが通るように修正

82ad7a9

BeltConveyorInsertChestLogicTestを修正

3682cf5

BeltConveyorOutputChestLogicTestを修正

a941013

MiningTestが通るように修正

14e82f9

ConnectingInventoryListPriorityInsertItemServiceTestが通るように修正

58d6109

MachineConnectToBeltConveyorTestが通るように修正

808141c

MultiBlockConnectTestが通るように修正

33e1fee

NotConnectableBlockTestが通るように修正

2802021

GearNetworkTestのエラーを修正（まだテストは通らない）

3aedf8f

IsReverseをTransformerごとの値ではなく、Transformer同士の接続の値に変更

0adac2c

テスト側のパラメータミスを修正（117行目から123行目まで）

12a202a

simpleGearGeneratorのgearConnectorにoptionを追加

3e811a9

KurisuJuha requested a review from sakastudio May 22, 2024 05:56

キャストミスを修正

2dfe691

sakastudio reviewed May 22, 2024

View reviewed changes

sakastudio requested changes May 22, 2024

View reviewed changes

KurisuJuha added 2 commits May 22, 2024 21:34

option!=nullを削除

ff9a956

テストの内容を修正

2817232

KurisuJuha added 2 commits May 23, 2024 14:28

Debug.Logを削除

f25103a

GearConnectOptionLoaderとGearConnectionOptionを同じアセンブリに移動

755b7f1

KurisuJuha requested a review from sakastudio May 23, 2024 05:48

sakastudio approved these changes May 23, 2024

View reviewed changes

sakastudio merged commit a4761ce into feature/gear-system May 23, 2024

sakastudio deleted the fix/loop-gear-network-test branch May 23, 2024 07:04

KurisuJuha mentioned this pull request May 23, 2024

SimpleGearGeneratorParamなどに"gearConnects"という文字列が直置きされている #341

Closed

sakastudio added a commit that referenced this pull request Jun 5, 2024

Merge pull request #359 from moorestech/feature/#340

6c3723c

"gearConnects"という文字列を直置きしている問題を修正

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

LoopGearNetworkTestが通るように修正 #340

LoopGearNetworkTestが通るように修正 #340

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

		public const float DefaultBeltConveyorHeight = 0.3f;
		private readonly BlockConnectorComponent<IBlockInventory> _blockConnectorComponent;

		var gearConnectSettings = BlockConfigJsonLoad.GetConnectSettings(blockParam, "gearConnects");


		var gearConnectSettings = BlockConfigJsonLoad.GetConnectSettings(blockParam, "gearConnects", GearConnectOptionLoader.Loader);

LoopGearNetworkTestが通るように修正 #340

LoopGearNetworkTestが通るように修正 #340

Uh oh!

Conversation

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!