Code: Select all

   if (MarketTradesTotal == 0)
      return;//Only need this function if there are market orders   

Code: Select all

   /*Somehow, the hedgeProfitTarget could be left at less than the account equity
   after an equity TP closure. This would result in new positions being opened
   and immediately closed - no idea why. This code block attempts to solve the problem.
   */
   if (OpenTrades == 0)
   {
      if (hedgeProfitTarget <= AccountEquity() )
      {
         hedgeProfitTarget = NormalizeDouble((AccountEquity() + cashToAddToEquity), 2);
         GlobalVariableSet(hedgeProfitTargetGvName, hedgeProfitTarget);
      }//if (hedgeProfitTarget <= AccountEquity() )
      
   }//if (OpenTrades == 0)

Code: Select all

   //SM("    Equity gain so far this week " + AccountCurrency() + "  " 
     // + DoubleToStr(weeklyEquityGain, 2) + "  Weekly target gain is " 
     // + AccountCurrency() + "  " + DoubleToStr(weeklyEquityTarget, 2) + NL);

Code: Select all

   //The next function deals with hedge/basket closures, so comment it out if not needed
   if (ShouldTradesBeClosed() )
   {
      //ShouldTradesBeClosed() deletes the globals if the closure target was hit,
      //so force a recalculation of the target at the next tick
      return;
   }//if (ShouldTradesBeClosed() )

Code: Select all

bool ShouldTradesBeClosed()
{
   //Examine baskets of trades for possible closure
   
   if (OpenTrades == 0)
      return(false);//Nothing to do

   if (MarketTradesTotal == 0)
      return(false);//Only need this function if there are market orders   

   int tries = 0;
   bool closePosition = false;
/*      
   //Global variables to help detect deposits into the account.
   // 1) Detect any manual deposit (positive jump in AccountBalance
   //    that isn't due to closed trades—in demo it's just a deposit)
   double currentBalance = AccountBalance();
   double deltaBalance   = currentBalance - g_lastBalance;
   if(deltaBalance > 0)
   {
      // Assume it's a deposit; accumulate
      g_totalDeposits += deltaBalance;
   }//if(deltaBalance > 0)
   g_lastBalance = currentBalance;

   // 2) Compute "true" profit based on equity minus deposits
   double currentEquity     = AccountEquity();
   double equityNetOfDeposits = currentEquity - g_totalDeposits;
   double profitSoFar       = equityNetOfDeposits - g_initialBalance;//CR's original
   //hedgeProfitTarget = equityNetOfDeposits - g_initialBalance;//To fit with my coding
Alert(profitSoFar);
return;   
*/
   //Hedged position. Has it hit tp?
   if (!closePosition)
   {
      //Have we hit cash tp?
      if (!closePosition)   
         if (!CloseEnough(hedgeProfitTarget, 0) )
            if (AccountEquity() >= hedgeProfitTarget)
               closePosition = true;
         
   }//if (!closePosition)

   //Goldy kept on missing opportunities to close down when the cash upl
   //had shot well past hedgeProfitTarget. The chart feedback would show
   //a negative distance to the next euity target which meant that
   //the equity was well above the target. This next snippet is an attepmt
   //to deal with this.
   if (!closePosition)
      if (hedgeProfitTarget - AccountEquity() < 0)
         closePosition = true;
      
   //Exit the function if the target has not been reached
   if (!closePosition)
      return(false);
         
   if (closePosition)
   {
      tries = 0;
      ForceTradeClosure = true;
      while(ForceTradeClosure)//CloseAllTrades() resets ForceTradeClosure unless the close fails
      {
         ForceTradeClosure = false;
         CloseAllTrades(AllTrades);  
         tries++;
         if (tries >= 100)
         {
            break;
         }//if (tries >= 100)
      }//while(ForceTradeClosure)
      Alert(TimeToStr(TimeCurrent(), TIME_DATE|TIME_MINUTES) + " Equity target reached. All orders should have closed and the position re-started.");
   
      // Check if we should suspend trading due to previous ADX peak detection
      if (adxPeakDetectedAwaitingEquityTarget && SuspendTradingAfterEquityTargetOnADXPeak)
      {
         tradingSuspendedByADX = true;
         adxPeakDetectedAwaitingEquityTarget = false; // Reset the flag
         string suspendMessage = StringConcatenate(
            "Trading Suspended for ", Symbol(), 
            " - Equity target hit after ADX peak reversal. Trading suspended until ADX rises above ", 
            DoubleToStr(AdxTradeAllowedLevel, 0)
         );
         if (IncludeMobileDevicesInAlerts)
            SendNotification(suspendMessage);
         Alert(suspendMessage);
      }
      
      //Delete the gvs
      if (closePosition)
      {
         GlobalVariableDel(cashToAddtGvName);
         GlobalVariableDel(hedgeProfitTargetGvName);
         
         return(true);
      
      }//if (closePosition)
      
   
      /*
      //Remove the expert?
      if (RemoveGoldyAfterWeeklyEquityTpHit)
         if (!ForceTradeClosure)
            RemoveExpert = true;
      */
   
   }//if (closePosition)
   
   //Got here, so no position closure
   return(false);
   
}//bool ShouldTradesBeClosed()