Необходимо закрыть позиции при достижении определенного профита, но код не работает корректно

Джони Браво 2013.12.30 13:04 #3831

Фак! Простите!
Как вести учет уникального номера ордера, если его задает т.п.? Как его изменить?

Alekseu Fedotov 2013.12.30 13:16 #3832

Link_x:

В моем примере: случайный номер ордера, объем - 5, по цене закрытия, с проскальзыванием 0, без стрелки.
В документационном примере: расчет номера ордера по order_id, обьем 1, по цене закрытия, с проскальзыванием 3, стрелка имеет красный цвет.

Смышленый блин, но есть одно но, См. выделенное, ты прописываеш Symbol().

И код, "Как?", просто

for(int i=OrdersTotal()-1;i>=0;i--)  //
    {
   if (OrderSelect(i, SELECT_BY_POS, MODE_TRADES))
       {
     if(OrderType()==OP_BUY) price=Bid;
     else                    price=Ask;
     OrderClose(OrderTicket(),OrderLots(),price,3,CLR_NONE);
       }
    }

Как-то так

Джони Браво 2013.12.30 13:57 #3833

r772ra:
Как-то так

Ух ты! Теперь закрывает позиции, но моментально. Сделаем так:

if(AccountProfit() > 50)
{
for(int i=OrdersTotal()-1;i>=0;i--) 
{
if (OrderSelect(i, SELECT_BY_POS, MODE_TRADES))
{
if(OrderType()==OP_BUY) price=Bid;
else                    price=Ask;
OrderClose(OrderTicket(),OrderLots(),price,3,CLR_NONE);
}
}   
}

И получим код, который закрывает все позиции при потенциальной прибыли 50$.
Спасибо вам большое!
Теперь все понятно!
Клас! :)

Alexey Semenov 2013.12.30 16:03 #3834

Всем привет! Умные люди помогите. Ставлю Profit=10 пунктам (т.е. при получении суммарного профита по двум парам равным 10 пунктам) обе открытые сделки должны быть закрыты. Однако, сделки не закрываются. Где ошибка?

extern double lotAU=0.01;
extern double lotEA=0.01;
extern double Profit=10;
extern string Сomment           = "KVAZ_EURAUD_AUDUSD";
extern int Magic                = 1111;

int EASell, EABuy, AUSell, AUBuy;

//+------------------------------------------------------------------+
//| expert initialization function                                   |
//+------------------------------------------------------------------+
int init()
  {
   return(0);
  }
//+------------------------------------------------------------------+
//| expert deinitialization function                                 |
//+------------------------------------------------------------------+
int deinit()
  {
   return(0);
  }
//+------------------------------------------------------------------+
//| expert start function                                            |
//+------------------------------------------------------------------+
int start()
  {
double prof;
double bidEA = MarketInfo("EURAUD",MODE_BID);
double askEA = MarketInfo("EURAUD",MODE_ASK);
double bidAU = MarketInfo("AUDUSD",MODE_BID);
double askAU = MarketInfo("AUDUSD",MODE_ASK);

string symEA = "EURAUD";
string symAU = "AUDUSD";

for(int i=OrdersTotal()-1; i>=0; i--)
{
 OrderSelect(i,SELECT_BY_POS);
 prof=prof+OrderProfit();
 Print("Profit="+prof);
}
if(prof>=Profit)
{
 for(i=OrdersTotal()-1; i>=0; i--)
 {
  OrderSelect(i,SELECT_BY_POS);
  OrderClose(OrderTicket(),OrderLots(),OrderClosePrice(),5,0);
 }
}

double LineEA = iCustom (Symbol(), 0, "Ind_2 Line+1", 0, 1); // Первый инструмент
double LineAU = iCustom (Symbol(), 0, "Ind_2 Line+1", 1, 1); // Второй инструмент

if(NumberOfPositions("EURAUD")==0 && LineEA > 0.1 && LineAU < -0.1)
    {
       EASell = OrderSend(symEA,OP_SELL,lotEA,bidEA,3,0,0,"KVAZ_EURAUD_AUDUSD",Magic,0,Red);
    }
if(NumberOfPositions("AUDUSD")==0 && LineEA > 0.1 && LineAU < -0.1)
    {
       AUSell = OrderSend(symAU,OP_SELL,lotAU,bidAU,3,0,0,"KVAZ_EURAUD_AUDUSD",Magic,0,Red);
    }
if(NumberOfPositions("EURAUD")==0 && LineEA < -0.1 && LineAU > 0.1)
    {
       EABuy = OrderSend(symEA,OP_BUY,lotEA,bidEA,3,0,0,"KVAZ_EURAUD_AUDUSD",Magic,0,Red);
    }
if(NumberOfPositions("AUDUSD")==0 && LineEA < -0.1 && LineAU > 0.1)
    {
       AUBuy = OrderSend(symAU,OP_BUY,lotAU,bidAU,3,0,0,"KVAZ_EURAUD_AUDUSD",Magic,0,Red);
    }
    
  return(0);
  }
//+------------------------------------------------------------------+

int NumberOfPositions(string sy="", int op=-1, int mn=1111) {      //|  Параметры:                                                                |
                                                                  //|    sy - наименование инструмента   (""   - любой символ,                   |
                                                                  //|                                     NULL - текущий символ)                 |
                                                                  //|    op - операция                   (-1   - любая позиция)                  |
                                                                  //|    mn - MagicNumber                (-1   - любой магик)                    
  int i, k=OrdersTotal(), kp=0;

  if (sy=="0") sy=Symbol();
  for (i=0; i<k; i++) {
    if (OrderSelect(i, SELECT_BY_POS, MODE_TRADES)) {
      if (OrderSymbol()==sy || sy=="") {
        if (OrderType()==OP_BUY || OrderType()==OP_SELL) {
          if (op<0 || OrderType()==op) {
            if (mn<0 || OrderMagicNumber()==mn) kp++;
          }
        }
      }
    }
  }
  return(kp);
}

Умом понимаю, что при закрытии советник просто не видит ордера по символу и мейджику, которые ему нужно закрыть. И все............

Да, еще был вариант такой. Сделка закрывалась только одна при этом условие по профиту 10 пунктов не соблюдалось. Короче, полная хрень..

for(int i=OrdersTotal()-1; i>=0; i--)
{
 OrderSelect(i,SELECT_BY_POS);
 prof=prof+OrderProfit();
 Print("Profit="+prof);
}
if(prof>=Profit)
{
 for(i=OrdersTotal()-1; i>=0; i--)
 {
  OrderSelect(i,SELECT_BY_POS);
  {
  if(OrderSymbol()!= Symbol()  || OrderMagicNumber()!=Magic) continue;
  OrderClose(OrderTicket(),OrderLots(),OrderClosePrice(),5,0);
  }
 }
}

Лавина Пишем бесплатные советники Новая версия платформы MetaTrader

[Deleted] 2013.12.30 17:37 #3835

Здравствуйте! Подскажите пожалуйста (лучше с примером кода), как сделать так, чтобы при достижении определенных условий на графике, на текущем баре и по текущей цене, появлялась стрелка и один раз срабатывал алерт выдавая сообщение? Например: при пересечении мувингов снизу вверх появлялась стрелка вверх с сообщением "покупка" и сверху вниз стрелка вниз с сообщением "продажа" соответственно. Заранее спасибо.

Мультитаймфреймовые индикаторы Помощь в кодировании Как на графике поставить

Alekseu Fedotov 2013.12.30 18:42 #3836

alexey1979621:
Всем привет! Умные люди помогите. Ставлю Profit=10 пунктам (т.е. при получении суммарного профита по двум парам равным 10 пунктам) обе открытые сделки должны быть закрыты. Однако, сделки не закрываются. Где ошибка?

Умом понимаю, что при закрытии советник просто не видит ордера по символу и мейджику, которые ему нужно закрыть. И все............

Да, еще был вариант такой. Сделка закрывалась только одна при этом условие по профиту 10 пунктов не соблюдалось. Короче, полная хрень..

1. Позицию закрываем по аску или биду того инструмента по которому она открыта

а не так, как у вас по OrderClosePrice()

пишем так

if(prof>=Profit)
   {
  for(int i=OrdersTotal()-1;i>=0;i--) 
      {
     if (OrderSelect(i, SELECT_BY_POS, MODE_TRADES))
         {
     double AS=MarketInfo(OrderSymbol(), MODE_ASK);
     double BI=MarketInfo(OrderSymbol(), MODE_BID);

       if(OrderType()==OP_BUY) price=BI;
       else                    price=AS;

       OrderClose(OrderTicket(),OrderLots(),price,3,CLR_NONE);
         }
      }   
   }

Как-то так

forexman77 2013.12.30 18:55 #3837

Возникает вероятность срабатывания стопа и после этого нужно сбросить флаги до первоначального состояния. Решается ли данная проблемма удалением советника с графика?

Alexey Semenov 2013.12.30 19:08 #3838

r772ra:

1. Позицию закрываем по аску или биду того инструмента по которому она открыта

а не так, как у вас по OrderClosePrice()

пишем так

Как-то так

Сделал, но это не решает проблему применительно к вышевыложенной сове. Думаю, при закрытии советник просто не видит ордера по символу и мейджику, которые ему нужно закрыть.

TarasBY 2013.12.30 19:23 #3839

alexey1979621:
Всем привет! Умные люди помогите. Ставлю Profit=10 пунктам (т.е. при получении суммарного профита по двум парам равным 10 пунктам) обе открытые сделки должны быть закрыты. Однако, сделки не закрываются. Где ошибка?

Умом понимаю, что при закрытии советник просто не видит ордера по символу и мейджику, которые ему нужно закрыть. И все............

Да, еще был вариант такой. Сделка закрывалась только одна при этом условие по профиту 10 пунктов не соблюдалось. Короче, полная хрень..

Ошибка на ошибке. Может так что-то заработает:

extern double lotAU=0.01;
extern double lotEA=0.01;
extern double Profit=10;
extern string EXP_Comment           = "KVAZ_EURAUD_AUDUSD";
extern int Magic                = 1111;

int EASell, EABuy, AUSell, AUBuy;
string gsa_SMB[2];
//+------------------------------------------------------------------+
//| expert initialization function                                   |
//+------------------------------------------------------------------+
int init()
  {
  gsa_SMB[0] = "EURAUD";
  gsa_SMB[1] = "AUDUSD";
   return (0);
  }
//+------------------------------------------------------------------+
//| expert deinitialization function                                 |
//+------------------------------------------------------------------+
int deinit()
  {
   return(0);
  }
//+------------------------------------------------------------------+
//| expert start function                                            |
//+------------------------------------------------------------------+
int start()
{
    int    li_N;
    double prof = 0., ld_Price = 0.;

    for (int i = OrdersTotal() - 1; i >= 0; i--)
    {
        OrderSelect (i,SELECT_BY_POS);
        li_N = fGet_NumSymbol (OrderSymbol(), gsa_SMB);
        if (li_N < 0) continue;
        if (OrderMagicNumber() != Magic) continue;
        prof += OrderProfit();
        Print ("Profit=", prof);
    }
    if (prof>=Profit)
    {
        for (i=OrdersTotal()-1; i>=0; i--)
        {
            OrderSelect (i, SELECT_BY_POS);
            li_N = fGet_NumSymbol (OrderSymbol(), gsa_SMB);
            if (li_N < 0) continue;
            if (OrderMagicNumber() != Magic) continue;
            if (OrderType() == OP_BUY) ld_Price = MarketInfo (gsa_SMB[li_N], MODE_BID);
            else if (OrderType() == OP_SELL) ld_Price = MarketInfo (gsa_SMB[li_N], MODE_ASK);
            OrderClose (OrderTicket(), OrderLots(), ld_Price, 5);
        }
    }
    double bidEA = MarketInfo ("EURAUD", MODE_BID),
           askEA = MarketInfo ("EURAUD", MODE_ASK),
           bidAU = MarketInfo ("AUDUSD", MODE_BID),
           askAU = MarketInfo ("AUDUSD", MODE_ASK),
           LineEA = iCustom (Symbol(), 0, "Ind_2 Line+1", 0, 1), // РџРµСЂРІС‹Р№ РёРЅСЃС‚СЂСѓРјРµРЅС‚
           LineAU = iCustom (Symbol(), 0, "Ind_2 Line+1", 1, 1); // Р’С‚РѕСЂРѕР№ РёРЅСЃС‚СЂСѓРјРµРЅС‚

    if (NumberOfPositions (gsa_SMB[0], -1, Magic) == 0)
    {
        if (LineEA > 0.1) if (LineAU < -0.1)
        {EASell = OrderSend (gsa_SMB[0], OP_SELL, lotEA, bidEA, 3, 0, 0, EXP_Comment, Magic, 0, Red);}
        if (LineEA < -0.1) if (LineAU > 0.1)
        {EABuy = OrderSend (gsa_SMB[0], OP_BUY, lotEA, askEA, 3, 0, 0, EXP_Comment, Magic, 0, Blue);}
    }
    if (NumberOfPositions (gsa_SMB[1], -1, Magic) == 0)
    {
        if (LineEA < -0.1) if (LineAU > 0.1)
        {AUBuy = OrderSend (gsa_SMB[1], OP_BUY, lotAU, askAU, 3, 0, 0, EXP_Comment, Magic, 0, Blue);}
        if (LineEA > 0.1) if (LineAU < -0.1)
        {AUSell = OrderSend (gsa_SMB[1], OP_SELL, lotAU, bidAU, 3, 0, 0, EXP_Comment, Magic, 0, Red);}
    }
    return (0);
}
//+------------------------------------------------------------------+
int fGet_NumSymbol (string fs_Symbol, string ar_SMB[])
{
    for (int li_IND = 0; li_IND < 2; li_IND++)
    {if (fs_Symbol == ar_SMB[li_IND]) return (li_IND);}
    return (-1);
}
//+------------------------------------------------------------------+

???

Alexander 2013.12.30 19:25 #3840

alexey1979621:
Сделал, но это не решает проблему применительно к вышевыложенной сове. Думаю, при закрытии советник просто не видит ордера по символу и мейджику, которые ему нужно закрыть.

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

Любой вопрос новичка, чтоб не захламлять форум. Профи, не проходите мимо. Без вас никуда - 6. - страница 384