remove Executor.SetMain and SetBattle (#213)

* remove Executor.SetMain and SetBattle

can use Duel.MainPhase directly

* init default MainPhase for duel to fix calling to this before the first Main Phase

Game/AI/Decks/HorusExecutor.cs

@@ -153,7 +153,7 @@ namespace WindBot.Game.AI.Decks
         {
             // We should summon Horus the Black Flame Dragon LV6 if he can lvlup.
             if (Enemy.GetMonsterCount() != 0 && !Util.IsAllEnemyBetterThanValue(2300 - 1, false))
-                foreach (ClientCard card in Main.SummonableCards)
+                foreach (ClientCard card in Duel.MainPhase.SummonableCards)
                     if (card.IsCode(11224103))
                         return false;
 

Game/AI/Decks/OldSchoolExecutor.cs

@@ -69,10 +69,10 @@ namespace WindBot.Game.AI.Decks
             if (_lastDoubleSummon == Duel.Turn)
                 return false;
 
-            if (Main.SummonableCards.Count == 0)
+            if (Duel.MainPhase.SummonableCards.Count == 0)
                 return false;
 
-            if (Main.SummonableCards.Count == 1 && Main.SummonableCards[0].Level < 5)
+            if (Duel.MainPhase.SummonableCards.Count == 1 && Duel.MainPhase.SummonableCards[0].Level < 5)
             {
                 bool canTribute = false;
                 foreach (ClientCard handCard in Bot.Hand)

Game/AI/Executor.cs

@@ -16,9 +16,6 @@ namespace WindBot.Game.AI
         public GameAI AI { get; private set; }
         public AIUtil Util { get; private set; }
 
-        protected MainPhase Main { get; private set; }
-        protected BattlePhase Battle { get; private set; }
-
         protected ExecutorType Type { get; private set; }
         protected ClientCard Card { get; private set; }
         protected int ActivateDescription { get; private set; }
@@ -247,16 +244,6 @@ namespace WindBot.Game.AI
             return;
         }
 
-        public void SetMain(MainPhase main)
-        {
-            Main = main;
-        }
-
-        public void SetBattle(BattlePhase battle)
-        {
-            Battle = battle;
-        }
-
         /// <summary>
         /// Set global variables Type, Card, ActivateDescription for Executor
         /// </summary>

Game/Duel.cs

@@ -47,6 +47,8 @@ namespace WindBot.Game
             LastSummonedCards = new List<ClientCard>();
             SolvingChainIndex = 0;
             NegatedChainIndexList = new List<int>();
+            MainPhase = new MainPhase();
+            BattlePhase = new BattlePhase();
         }
 
         public ClientCard GetCard(int player, CardLocation loc, int seq)

Game/GameAI.cs

@@ -202,7 +202,6 @@ namespace WindBot.Game
         /// <returns>A new BattlePhaseAction containing the action to do.</returns>
         public BattlePhaseAction OnSelectBattleCmd(BattlePhase battle)
         {
-            Executor.SetBattle(battle);
             foreach (CardExecutor exec in Executor.Executors)
             {
                 if (exec.Type == ExecutorType.GoToMainPhase2 && battle.CanMainPhaseTwo && exec.Func()) // check if should enter main phase 2 directly
@@ -449,7 +448,6 @@ namespace WindBot.Game
         /// <returns>A new MainPhaseAction containing the action to do.</returns>
         public MainPhaseAction OnSelectIdleCmd(MainPhase main)
         {
-            Executor.SetMain(main);
             CheckSurrender();
             foreach (CardExecutor exec in Executor.Executors)
             {