特效回收代码优化

GameClient/Assets/Game/HotUpdate/Views/Bag/GiftBoxSelectorView.cs

@@ -65,6 +65,7 @@ namespace GFGGame
             _ui.m_btnCancel.onClick.Add(OnBtnCancelClick);
             _ui.m_list.itemRenderer = ListItemRender;
             _ui.m_btnSub.onClick.Add(OnBtnSubClick);
+            AddEffect();
         }
 
         protected override void AddEventListener()
@@ -83,8 +84,7 @@ namespace GFGGame
 
             UpdateView();
             UpdateList();
-            SetSelAllNum();
-            AddEffect();
+            SetSelAllNum(); 
         }
 
         protected override void OnHide()

GameClient/Assets/Game/HotUpdate/Views/Card/CardFosterView.cs

@@ -23,7 +23,7 @@ namespace GFGGame
         private List<CardStarCfg> _cardStarRewards = new List<CardStarCfg>();
         private int _selectTab = 0;
         private List<LongPressGesture> _listLongPress = new List<LongPressGesture>();
-        private EffectUI _effectUI;
+        private EffectUI _effectUI1;
 
         private GObject _consumeSelectItem;//选中的材料item
         private int _consumeSelectIndex = 0;//0为减，1为加
@@ -58,8 +58,6 @@ namespace GFGGame
                 _valueBarController.Dispose();
                 _valueBarController = null;
             }
-            EffectUIPool.Recycle(_effectUI);
-            _effectUI = null;
 
             if (_ui != null)
             {
@@ -228,20 +226,20 @@ namespace GFGGame
         }
         private void UpdateCardRes()
         {
-            if (_effectUI != null)
+            if (_effectUI1 != null)
             {
-                EffectUIPool.Recycle(_effectUI);
-                _effectUI = null;
+                EffectUIPool.Recycle(_effectUI1);
+                _effectUI1 = null;
             }
             string resName = _cardData.resources[_cardData.resIndex];
             string resPath = ResPathUtil.GetCardAnimationPath(_cardData.resources[_cardData.resIndex]);
             if (_cardData.itemCfg.animationCardStar > 0 && _cardData.star >= _cardData.itemCfg.animationCardStar && Versions.Contains(resPath))
             {
-                _effectUI = new EffectUI();
-                _effectUI.Reset(_ui.m_comCard.m_comCardHolder.m_holder, resName, resName, 100, EffectUIType.Card);
+                _effectUI1 = new EffectUI();
+                _effectUI1.Reset(_ui.m_comCard.m_comCardHolder.m_holder, resName, resName, 100, EffectUIType.Card);
             }
 
-            if (_effectUI != null)
+            if (_effectUI1 != null)
             {
                 _ui.m_c1.selectedIndex = 1;
                 _ui.m_comCard.target.scrollPane.SetPercX(0.5f, false);
@@ -938,6 +936,9 @@ namespace GFGGame
         {
             base.OnHide();
 
+            EffectUIPool.Recycle(_effectUI1);
+            _effectUI1 = null;
+
             Timers.inst.Remove(CheckGuide);
             Timers.inst.Remove(OnTimedEvent);
             UI_ComCostCurrency.ProxyEnd();

GameClient/Assets/Game/HotUpdate/Views/ClothingFoster/ClothingView.cs

@@ -45,10 +45,8 @@ namespace GFGGame
             _effectUI1 = null;
             _propertyList.Clear();
             _addPropertyList.Clear();
-
         }
 
-
         protected override void OnInit()
         {
             base.OnInit();

GameClient/Assets/Game/HotUpdate/Views/Common/ModalStatusView.cs

@@ -32,12 +32,12 @@ namespace GFGGame
 
         protected override void OnHide()
         {
+            EffectUIPool.Recycle(_effectUI1);
+            _effectUI1 = null;
             base.OnHide();
         }
         public override void Dispose()
         {
-            EffectUIPool.Recycle(_effectUI1);
-            _effectUI1 = null;
             if (_ui != null)
             {
                 _ui.Dispose();

GameClient/Assets/Game/HotUpdate/Views/CommonGame/RewardView.cs

@@ -72,9 +72,13 @@ namespace GFGGame
             }
             _listItemDatas.AddRange(suitPart);
 
-            if (_listItemDatas.Count > 4) { 
+            if (_listItemDatas.Count > 4)
+            {
                 _ui.m_listReward.SetVirtual();
             }
+            else if(_ui.m_listReward.isVirtual) {
+                _ui.m_listReward.RefreshVirtualList();
+            }
             _ui.m_listReward.numItems = _listItemDatas.Count;
 
             _ui.m_listReward.ResizeToFit();

GameClient/Assets/Game/HotUpdate/Views/EnduringGiftBox/EnduringGiftBoxView.cs

@@ -44,14 +44,6 @@ namespace GFGGame
 
         public override void Dispose()
         {
-            EffectUIPool.Recycle(_effectUI1);
-            _effectUI1 = null;
-            EffectUIPool.Recycle(_effectUI2);
-            _effectUI2 = null;
-            EffectUIPool.Recycle(_effectUI3);
-            _effectUI3 = null;
-            EffectUIPool.Recycle(_effectUI4);
-            _effectUI4 = null;
             if (_ui != null)
             {
                 _ui.Dispose();
@@ -530,6 +522,15 @@ namespace GFGGame
 
         protected override void OnHide()
         {
+            EffectUIPool.Recycle(_effectUI1);
+            _effectUI1 = null;
+            EffectUIPool.Recycle(_effectUI2);
+            _effectUI2 = null;
+            EffectUIPool.Recycle(_effectUI3);
+            _effectUI3 = null;
+            EffectUIPool.Recycle(_effectUI4);
+            _effectUI4 = null;
+
             this.RemoveEventListener();
             base.Hide();
             _onSuccess = null;

GameClient/Assets/Game/HotUpdate/Views/EnduringGiftBox/SpecialOfferGiftBoxView.cs

@@ -41,14 +41,7 @@ namespace GFGGame
         public override void Dispose()
         {
             base.Dispose();
-            EffectUIPool.Recycle(_effectUI1);
-            _effectUI1 = null;
-            EffectUIPool.Recycle(_effectUI2);
-            _effectUI2 = null;
-            EffectUIPool.Recycle(_effectUI3);
-            _effectUI3 = null;
-      
-            DestroyObjectFromView();
+
             if (_ui != null)
             {
                 _ui.Dispose();
@@ -396,6 +389,15 @@ namespace GFGGame
 
         protected override void OnHide()
         {
+            EffectUIPool.Recycle(_effectUI1);
+            _effectUI1 = null;
+            EffectUIPool.Recycle(_effectUI2);
+            _effectUI2 = null;
+            EffectUIPool.Recycle(_effectUI3);
+            _effectUI3 = null;
+
+            DestroyObjectFromView();
+
             this.RemoveEventListener();
             base.Hide();
         }

GameClient/Assets/Game/HotUpdate/Views/LuckyBox/LuckyBoxBonusShowView.cs

@@ -24,6 +24,13 @@ namespace GFGGame
         public override void Dispose()
         {
             base.Dispose();
+            EffectUIPool.Recycle(_effectUI1);
+            _effectUI1 = null;
+            EffectUIPool.Recycle(_effectUI2);
+            _effectUI2 = null;
+            EffectUIPool.Recycle(_effectUI3);
+            _effectUI3 = null;
+
             // for (int i = 0; i < _gameobjects.Count; i++)
             // {
             //     SceneController.DestroyObjectFromView(_gameobjects[i], _wrappers[i]);
@@ -85,12 +92,7 @@ namespace GFGGame
             }
             countEff = 0;
             _effList.Clear();
-            EffectUIPool.Recycle(_effectUI1);
-            _effectUI1 = null;
-            EffectUIPool.Recycle(_effectUI2);
-            _effectUI2 = null;
-            EffectUIPool.Recycle(_effectUI3);
-            _effectUI3 = null;
+        
             EffectUIPool.Recycle(_effectUI4);
             _effectUI4 = null;
 

GameClient/Assets/Game/HotUpdate/Views/LuckyBox/LuckyBoxNewDressView.cs

@@ -7,7 +7,6 @@ using System.Linq;
 
 namespace GFGGame
 {
-
     public class LuckyBoxNewDressView : BaseWindow
     {
         private UI_LuckyBoxNewDressUI _ui;
@@ -36,10 +35,6 @@ namespace GFGGame
             _effectUI3 = null;
             EffectUIPool.Recycle(_effectUI4);
             _effectUI4 = null;
-            EffectUIPool.Recycle(_effectUI5);
-            _effectUI5 = null;
-            EffectUIPool.Recycle(_effectUI6);
-            _effectUI6 = null;
             if (_ui != null)
             {
                 _ui.Dispose();
@@ -99,14 +94,22 @@ namespace GFGGame
         }
         private void UpdateView()
         {
+            if (_effectUI5 != null)
+            {
+                EffectUIPool.Recycle(_effectUI5);
+                _effectUI5 = null;
+            }
+            if (_effectUI6 != null)
+            {
+                EffectUIPool.Recycle(_effectUI6);
+                _effectUI6 = null;
+            }
             _ui.m_btnPass.visible = _newRewardList.Count <= 1;
             _rewardList = _newRewardList.Count > 0 ? _newRewardList : _oldRewardList;
 
             if (_rewardList.Count - 1 < 0) return;
             ItemData itemdata = _rewardList[_rewardList.Count - 1];
             ItemCfg itemCfg = ItemCfgArray.Instance.GetCfg(itemdata.id);
-            string resPath = "";
-            string resPath1 = "";
             switch (itemCfg.rarity)
             {
                 case ConstDressRarity.Rarity_TIANYI:
@@ -196,6 +199,10 @@ namespace GFGGame
 
         protected override void OnHide()
         {
+            EffectUIPool.Recycle(_effectUI5);
+            _effectUI5 = null;
+            EffectUIPool.Recycle(_effectUI6);
+            _effectUI6 = null;
             _ui.m_t_close.Stop(true, false);
             _ui.m_t_open.Stop(true, false);
             _rewardList.Clear();

GameClient/Assets/Game/HotUpdate/Views/Mail/MailView.cs

@@ -27,6 +27,11 @@ namespace GFGGame
 
         public override void Dispose()
         {
+            EffectUIPool.Recycle(_effectUI1);
+            _effectUI1 = null;
+            EffectUIPool.Recycle(_effectUI2);
+            _effectUI2 = null;
+
             if (_ui != null)
             {
                 _ui.Dispose();
@@ -60,6 +65,7 @@ namespace GFGGame
             _ui.m_btnDelete.onClick.Add(OnClickBtnDelete);
 
             mailDataMgr = MailDataManager.Instance;
+            AddEffect();
         }
         protected override void AddEventListener()
         {
@@ -76,7 +82,6 @@ namespace GFGGame
             base.OnShown();
             mailDataMgr.CurPage = 0;
             UpdateNormal();
-            AddEffect();
             _ui.m_openViewAction.Play();
         }
         private void UpdateNormal()
@@ -131,7 +136,6 @@ namespace GFGGame
             }
         }
 
-
         private async void RefreshMailInfo(bool needSort)
         {
             if (mailDataMgr.TotolCount == 0) return;
@@ -199,11 +203,6 @@ namespace GFGGame
         }
         protected override void OnHide()
         {
-            EffectUIPool.Recycle(_effectUI1);
-            _effectUI1 = null;
-            EffectUIPool.Recycle(_effectUI2);
-            _effectUI2 = null;
-
             base.OnHide();
         }
         protected override void RemoveEventListener()

GameClient/Assets/Game/HotUpdate/Views/Notice/NoticeView.cs

@@ -46,7 +46,7 @@ namespace GFGGame
             _ui.m_listNotice.itemRenderer = ListNoticeItemRender;
 
             _ui.m_c1.onChanged.Add(OnCtrlChange);
-
+            AddEffect();
         }
         protected override void AddEventListener()
         {
@@ -62,7 +62,6 @@ namespace GFGGame
             _ui.m_listNotice.visible = true;
             _ui.m_buttonCom.visible = true;
             OnCtrlChange();
-            AddEffect();
         }
 
         protected override void OnHide()

GameClient/Assets/Game/HotUpdate/Views/RoleInfo/ChangeNameView.cs

@@ -38,7 +38,7 @@ namespace GFGGame
 
             _ui.m_btnSure.onClick.Add(OnClickBtnSureAsync);
             // _ui.m_btnDice.onClick.Add(RandomRoleName);
-
+            AddEffect();
         }
 
         protected override void OnShown()
@@ -48,8 +48,6 @@ namespace GFGGame
             _ui.m_txtNeed.text = string.Format("改名卡：{0}/{1}", ItemDataManager.GetItemNum(GlobalCfgArray.globalCfg.changeNameCostArr[0]), GlobalCfgArray.globalCfg.changeNameCostArr[1]);
             ItemCfg cfg = ItemCfgArray.Instance.GetCfg(GlobalCfgArray.globalCfg.changeNameCostArr[0]);
             _ui.m_loaIcon.url = ResPathUtil.GetCommonGameResPath(cfg.res); 
-
-            AddEffect();
         }
 
         protected override void OnHide()

GameClient/Assets/Game/HotUpdate/Views/RoleInfo/SettingView.cs

@@ -39,6 +39,7 @@ namespace GFGGame
             _ui.m_btnLogout.onClick.Add(OnClickBtnLogout);
             _ui.m_btnSound.onClick.Add(OnClickBtnSound);
             _ui.m_btnMusic.onClick.Add(OnClickBtnMusic);
+            AddEffect();
         }
 
         protected override void OnShown()