Merge pull request #2213 from planetarium/improve/raw-market-state

Improve market action exectuion time
planetarium · Nov 9, 2023 · cdbd6ea · cdbd6ea
2 parents a3f1fac + b0604e9
commit cdbd6ea
Show file tree

Hide file tree

Showing 7 changed files with 28 additions and 28 deletions.
diff --git a/Lib9c/Action/CancelProductRegistration.cs b/Lib9c/Action/CancelProductRegistration.cs
@@ -71,11 +71,11 @@ public override IAccount Execute(IActionContext context)
             {
                 // cancel order before product registered case.
                 var marketState = states.TryGetState(Addresses.Market, out List rawMarketList)
-                    ? new MarketState(rawMarketList)
-                    : new MarketState();
+                    ? rawMarketList
+                    : List.Empty;
                 productsState = new ProductsState();
-                marketState.AvatarAddresses.Add(AvatarAddress);
-                states = states.SetState(Addresses.Market, marketState.Serialize());
+                marketState = marketState.Add(AvatarAddress.Serialize());
+                states = states.SetState(Addresses.Market, marketState);
             }
             var addressesHex = GetSignerAndOtherAddressesHex(context, AvatarAddress);
             foreach (var productInfo in ProductInfos)

diff --git a/Lib9c/Action/CancelProductRegistration0.cs b/Lib9c/Action/CancelProductRegistration0.cs
@@ -79,11 +79,11 @@ public override IAccount Execute(IActionContext context)
             {
                 // cancel order before product registered case.
                 var marketState = states.TryGetState(Addresses.Market, out List rawMarketList)
-                    ? new MarketState(rawMarketList)
-                    : new MarketState();
+                    ? rawMarketList
+                    : List.Empty;
                 productsState = new ProductsState();
-                marketState.AvatarAddresses.Add(AvatarAddress);
-                states = states.SetState(Addresses.Market, marketState.Serialize());
+                marketState = marketState.Add(AvatarAddress.Serialize());
+                states = states.SetState(Addresses.Market, marketState);
             }
             var addressesHex = GetSignerAndOtherAddressesHex(context, AvatarAddress);
             foreach (var productInfo in ProductInfos)

diff --git a/Lib9c/Action/ReRegisterProduct.cs b/Lib9c/Action/ReRegisterProduct.cs
@@ -73,11 +73,11 @@ public override IAccount Execute(IActionContext context)
             else
             {
                 var marketState = states.TryGetState(Addresses.Market, out List rawMarketList)
-                    ? new MarketState(rawMarketList)
-                    : new MarketState();
+                    ? rawMarketList
+                    : List.Empty;
                 productsState = new ProductsState();
-                marketState.AvatarAddresses.Add(AvatarAddress);
-                states = states.SetState(Addresses.Market, marketState.Serialize());
+                marketState = marketState.Add(AvatarAddress.Serialize());
+                states = states.SetState(Addresses.Market, marketState);
             }
 
             var random = context.GetRandom();

diff --git a/Lib9c/Action/ReRegisterProduct0.cs b/Lib9c/Action/ReRegisterProduct0.cs
@@ -73,11 +73,11 @@ public override IAccount Execute(IActionContext context)
             else
             {
                 var marketState = states.TryGetState(Addresses.Market, out List rawMarketList)
-                    ? new MarketState(rawMarketList)
-                    : new MarketState();
+                    ? rawMarketList
+                    : List.Empty;
                 productsState = new ProductsState();
-                marketState.AvatarAddresses.Add(AvatarAddress);
-                states = states.SetState(Addresses.Market, marketState.Serialize());
+                marketState = marketState.Add(AvatarAddress.Serialize());
+                states = states.SetState(Addresses.Market, marketState);
             }
 
             var random = context.GetRandom();

diff --git a/Lib9c/Action/RegisterProduct.cs b/Lib9c/Action/RegisterProduct.cs
@@ -84,10 +84,10 @@ public override IAccount Execute(IActionContext context)
             {
                 productsState = new ProductsState();
                 var marketState = states.TryGetState(Addresses.Market, out List rawMarketList)
-                    ? new MarketState(rawMarketList)
-                    : new MarketState();
-                marketState.AvatarAddresses.Add(AvatarAddress);
-                states = states.SetState(Addresses.Market, marketState.Serialize());
+                    ? rawMarketList
+                    : List.Empty;
+                marketState = marketState.Add(AvatarAddress.Serialize());
+                states = states.SetState(Addresses.Market, marketState);
             }
             sw.Stop();
             Log.Debug("{Source} {Process} from #{BlockIndex}: {Elapsed}",

diff --git a/Lib9c/Action/RegisterProduct0.cs b/Lib9c/Action/RegisterProduct0.cs
@@ -80,10 +80,10 @@ public override IAccount Execute(IActionContext context)
             {
                 productsState = new ProductsState();
                 var marketState = states.TryGetState(Addresses.Market, out List rawMarketList)
-                    ? new MarketState(rawMarketList)
-                    : new MarketState();
-                marketState.AvatarAddresses.Add(AvatarAddress);
-                states = states.SetState(Addresses.Market, marketState.Serialize());
+                    ? rawMarketList
+                    : List.Empty;
+                marketState = marketState.Add(AvatarAddress.Serialize());
+                states = states.SetState(Addresses.Market, marketState);
             }
 
             var random = context.GetRandom();

diff --git a/Lib9c/Action/RegisterProduct2.cs b/Lib9c/Action/RegisterProduct2.cs
@@ -81,10 +81,10 @@ public override IAccount Execute(IActionContext context)
             {
                 productsState = new ProductsState();
                 var marketState = states.TryGetState(Addresses.Market, out List rawMarketList)
-                    ? new MarketState(rawMarketList)
-                    : new MarketState();
-                marketState.AvatarAddresses.Add(AvatarAddress);
-                states = states.SetState(Addresses.Market, marketState.Serialize());
+                    ? rawMarketList
+                    : List.Empty;
+                marketState = marketState.Add(AvatarAddress.Serialize());
+                states = states.SetState(Addresses.Market, marketState);
             }
             foreach (var info in RegisterInfos.OrderBy(r => r.Type).ThenBy(r => r.Price))
             {