на 1-е

замените этот блок:

//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
double FindLastBuyPrice()
  {
   int oticket,ticketNumber=0;
   double oprice=0;
   for(int i=OrdersTotal()-1;i>=0;i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderType()==OP_BUY && OrderMagicNumber()==Magic)
           {
            oticket=OrderTicket();
            if(oticket>ticketNumber)
              {
               ticketNumber=oticket;
               oprice=OrderOpenPrice();
              }
           }
        }
     }
   return(oprice);
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
double FindLastSellPrice()
  {
   int oticket,ticketNumber=0;
   double oprice=0;
   for(int i=OrdersTotal()-1;i>=0;i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderType()==OP_SELL && OrderMagicNumber()==Magic)
           {
            oticket=OrderTicket();
            if(oticket>ticketNumber)
              {
               ticketNumber=oticket;
               oprice=OrderOpenPrice();
              }
           }
        }
     }
   return(oprice);
  }

Быстро… Сейчас попробую.

Заменил указанную часть кода. Проблема осталась. Видит локирующий ордер и не считает как надо.

Возможно тут причина…

double AllProfit2()
  {
   double pr=0;
   int second=OrdersTotal()-1-Second;
   for(int i=OrdersTotal()-1;i>second;i--)

тогда опишите полностью какие локи ордера в рынке, когда неверно работает. буду смотреть 1-го. и скрины приведите

Пошагово.
Запускаем сов., он открывает ордер согласно настройкам. Непонятно, почему в строке «Second» -1, хотя по логике должно быть -2.

Скриптом открываем второй ордер на этой же паре, с тем же магиком. В строке «Second» значение не верно.

На любой другой паре открываем ордер вручную без разницы в каком направлении. Строка «Profit2» обнуляется, сов перестает его считать и не закроет ордер «Second» при наступлении заданных условий. Хотя усреднение будет происходить согласно заданному шагу и свой «Ptrofit» считает верно.

Тоже самое будет если вручную или каким либо другим совом открыть ордер на этой паре.

Если обобщить — работу советника нарушает любой, открытый не советником, ордер в терминале, в не зависимости от магика. Вот эту накладочку и прошу исправить.

В подсчёте ордеров ошибка

Заменить части кода:

//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
void CloseAll2()
  {
   bool cl;
   int second=CountTrades()-1-Second;
   for(int i=CountTrades()-1;i>second;i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magic)
           {
            if(OrderType()==0)
              {
               RefreshRates();
               cl=OrderClose(OrderTicket(),OrderLots(),NormalizeDouble(Bid,Digits),Slip,White);
              }
            if(OrderType()==1)
              {
               RefreshRates();
               cl=OrderClose(OrderTicket(),OrderLots(),NormalizeDouble(Ask,Digits),Slip,White);
              }
           }
        }
     }
  }

//+------------------------------------------------------------------+
//| Профит всех ордеров по типу ордера                               |
//+------------------------------------------------------------------+
double AllProfit2()
  {
   double pr=0;
   int second=CountTrades()-1-Second;
   for(int i=CountTrades()-1;i>second;i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magic)
           {
            if(OrderType()==0)
              {
               pr+=OrderProfit()+OrderCommission()+OrderSwap();
              }

            if(OrderType()==1)
              {
               pr+=OrderProfit()+OrderCommission()+OrderSwap();
              }
           }
        }
     }
   return(pr);
  }

Comment("\n Lot: ",Lot(),
           "\n Trades: ",CountTrades(),
           "\n Profit: ",AllProfit(),
           "\n Profit2: ",AllProfit2(),
           "\n Second: ",CountTrades()-Count);

К сожалению при тестировании выявлены те же косяки. Без помощи Андрея не обойтись.

дайте на него ссылку в базе, чтобы обновлять

Он видит чужие ордера. Никаких действий с чужими ордерами он не производит, но он их считает и начинает работать как обычный усреднитель

проверил, не считает.

советник открыл позу:


я открыл вручную, все тоже:

переписал некоторые функции. по 2 штуки с краю также откусывает:



www.opentraders.ru/downloads/2186/

Откусывать по 2 ордера не очень целесообразно. При резком однонаправленном движении цены получаются пропуски. Логичнее откусывать крайние ордера, по одному. Переписанный буду смотреть. Спасибо.