Обучение языку MQL4 под MetaTrader 4📌

В цикле перебора для подсчёта профита, тоже есть ошибка:

if (OrderType() == OP_BUY && OrderType() == OP_SELL)

а должно быть:

if (OrderType() == OP_BUY || OrderType() == OP_SELL) .

Ошибка здесь:
return(count);
avg = NormalizeDouble(avg / count, 2);
нужно:
return(NormalizeDouble(avg / count, 2));

и странности в функции закрытия ордеров:
- где Вы берете тикет (ticketbuystop)?
- где Вы выбираете ордер, чтобы использовать OrderLots()?
- OrderClosePrice() использовать нельзя - там пусто! ордер ещё не закрыт. Справку читаете?

Вы мой пост выше читали?

В цикле перебора для подсчёта профита, тоже есть ошибка:

if (OrderType() == OP_BUY && OrderType() == OP_SELL)

а должно быть:

if (OrderType() == OP_BUY || OrderType() == OP_SELL) .

Прошу Гуру в MQL прокомментировать варианты использования ф-ции iCustom.

Практически всегда встречается в кодах такой вариант записи вызова стороннего индикатора:

//+------------------------------------------------------------------+
Buf_0_bar_0 = iCustom(NULL,0,"Индюк",параметры индюка,0,0);
Buf_0_bar_1 = iCustom(NULL,0,"Индюк",параметры индюка,0,1);
..........................................................
Buf_n_bar_n = iCustom(NULL,0,"Индюк",параметры индюка,n,n);
//+------------------------------------------------------------------+

Но вот недавно встретил иную запись, через вызов как отдельной функции:

//+------------------------------------------------------------------+
double Индюк(int buf, int shift)
{
return(iCustom(NULL,0,"Индюк",параметры индюка,buf,shift));
}
//+------------------------------------------------------------------+
Buf_0_bar_0 = Индюк(0,0);
Buf_0_bar_1 = Индюк(0,1);
..........................................................
Buf_n_bar_n = Индюк(n,n);
//+------------------------------------------------------------------+

Вопрос: есть ли существенная разница в этих подходах к вызову стороннего индюка в плане на качество работы и каковы преимущества/недостатки между этими вариантами, есть ли нюансы, какой вариант предпочтителен?
И еще интересует: в ф-ции Индюк(int buf, int shift) разве не должны формальные параметры передаваться по ссылке?

  void CloseAllOrders()   // закрытие всех ордеров
{
   for(int i = OrdersTotal()-1; i>=0; i--)
   {
      if (OrderSelect(i, SELECT_BY_POS, MODE_TRADES))
      {
         if (OrderSymbol() == Symbol() && OrderMagicNumber() == Magic)
         {
            if (OrderType() == OP_BUY || OrderType() == OP_SELL)
            {
               if (OrderType() == OP_BUY) PriceClose = Bid;
               if (OrderType() == OP_SELL) PriceClose = Ask;
               {
                  if (OrderClose(OrderTicket(),OrderLots(),PriceClose,Slippage,CLR_NONE) == true)
                  {
                     Print ("Функция OrderClose успешно выполнена");              
                  }
                  else
                     Print ("OrderClose завершилась с ошибкой #", GetLastError());
               }
            }   
         }
      }
   }
   for(int ii = OrdersTotal()-1; ii>=0; ii--)
   {
      if (OrderSelect(ii, SELECT_BY_POS, MODE_TRADES))
      {
         if (OrderSymbol() == Symbol() && OrderMagicNumber() == Magic)
         {
            if (OrderType() == OP_BUYLIMIT || OrderType() == OP_BUYSTOP || OrderType() == OP_SELLLIMIT || OrderType() == OP_SELLSTOP)
            {
               if (OrderDelete(OrderTicket()) == true)
               {
                  Print ("Функция OrderDelet успешно выполнена");
               }
               else
                  Print ("OrderDelet завершилась с ошибкой #", GetLastError());
            }
         }
      }
   }                              
}

mobidik Вызовы равнозначны
Второй вариант для экономии строк в исходнике, но вызов функции в откомпилированном коде будет дольше (незначительно).
Что касается передачи по ссылке - зачем? внутри функции передаваемые значения не меняются, да и вызов её скорее всего с константами (как Вы и написали Индюк(0,0);).

chistaia-rodina, Спасибо, большое! И подсказали и объяснили!

Ну так как заставить это работать:
Функцию вычисления суммарного профита, описанную выше, я поправил (и переименовал);
Функцией закрытия ордеров любезно поделился г-н chistaia-rodina, спасибо ему;
В теле совы пишу:
if (TotalProfit() >= Profit)
{
CloseAllOrders();
}

А в ответ - тишина!

Выкладывайте Ваш TotalProfit(). или сами пишите Print(TotalProfit()); и смотрите, что функция возвращает.

Здравствуйте, коллеги. С MQL я не дружу, сам дельфин. Помогите немножко пожалуйста.

Хочу узнать максимальное значение ATR за период.

int start()
{
  int i;
  double MaxValue=0;
  double Value;

  for(i=0;i  {
    Value = iCustom(Symbol(),Period(),"ATR",ATR_Period,0,i);
  if (Value>MaxValue)
      MaxValue=Value;
  }

  CreateLabel (IndName+"_Value",MaxValue, 0,0,10,10,"",20,LabelColor);
  return(0);
}

Совершенно неизвестно что за кастомный индюк у тебя по имени ATR (он вообще есть?)
iATR (NULL, 0, ATR_Period, i) должен тебя спасти.

//+------------------------------------------------------------------+
//| Expert tick function                                             |
//+------------------------------------------------------------------+
void OnTick()
  {
//---
  int i;
  double MaxValue=0;
  double Value;

  for(i=0;i  {
    Value = iCustom(Symbol(),Period(),"ATR",14,0,i);
  if (Value>MaxValue)
      MaxValue=Value;
  }

  Comment("MaxValue = "+MaxValue);

  }
//+------------------------------------------------------------------+

А подскажите пожалуйста, какой объект нужен, чтобы, скажем, отметить все бары на истории, которые меньше 30 пунктов, стрелочками?

Сделал наскоро скрипт, но уже ответили выше - надо брать через встроенную функцию iATR()

MaxATR.mq410 скач.

А подскажите пожалуйста, какой объект нужен, чтобы, скажем, отметить все бары на истории, которые меньше 30 пунктов, стрелочками?

Большое спасибо, сам протупил. Помогите еще протупить чуть-чуть!

Надо нарисовать Arrow в том месте Subwindow=1, где получили значение MaxValue. Номер свечи знаем, значение по высоте знаем. Нарисовать не умеем :-&

Надо нарисовать Arrow в том месте Subwindow=1, где получили значение MaxValue. Номер свечи знаем, значение по высоте знаем. Нарисовать не умеем :-&

Другое дело: обновление максимума. Вот нашли вы мах на 60 баре, а через 10 бар АТР выдал новый мах. - что делать? - старый затирать, новый рисовать? или пусть остаётся?, а если махимумы обновляются на каждом новом баре?

if (BarChanged)
{
  CalculateMaxValue; 
  ObjectDelete("MaxArrow");
  ObjectCreate("MaxArrow", OBJ_ARROW, ?????????)
}

Набросал такой цикл, не знаю, правильно ли
Только вот куда его?

for (int i = 1; i {
double ExtMapBuffer[ ];
double a = NormalizeDouble((Open[ i ]-Close[ i ]),Digits);

if (MathAbs(a) {
SetIndexBuffer(0,ExtMapBuffer);
SetIndexStyle(0,DRAW_ARROW,1,5);
SetIndexArrow(0,233);
}
}

oleguitar, на мой взгляд использовать константы "200" в вашем случае - немного недальновидно.
Такие вещи лучше сразу делать через определение констант в заголовке советника и использовать уже символьные имена.
Например,
#define OURMAGIC 137433
И спустя какое-то время не придется судорожно вспоминать что же это такое 137433 или иное другое число.

Господа асы, помогите решить проблему с совой. не работает функция закрытия ордеров с суммарным профитом, значение профит задается во внешнем параметре Profit. Сова прилагается. Придумывал и писал сам, по видео урокам, так что сильно не пинайте. В общем, там все работает, кроме закрытия с профитом. Подскажите что исправить, ну или исправьте, если не сложно.

SyperMartin.mq416 скач.

Я mql изучаю месяца два только, не судите строго)
индикаторы не умею писать конечно же
найти бы такой, где поменьше кода, чтобы разобраться что к чему