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

[Rigal]

1 минуту назад, Гладиoлус сказал:

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

Я боюсь, "не работает" - слишком расплывчатая формулировка.

Что именно работает не так, как вы ожидаете?

[Гладиoлус]

#property copyright "Copyright 2021, MetaQuotes Software Corp."
#property link      "https://www.mql5.com"
#property version   "1.00"
#property strict

// CloseDate = Date1
// OpenDate = Date2
extern datetime Date1=D'2021.12.15 19:00';
extern datetime Date2=D'2021.12.16 13:05';

extern int Amount = 19;
   double sumHigh;
    double sumLow;
    double x;

//+------------------------------------------------------------------+
//| Expert initialization function                                   |
//+------------------------------------------------------------------+
int OnInit()
{

   return(INIT_SUCCEEDED);
}
//+------------------------------------------------------------------+
//| Expert deinitialization function                                 |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
{
 
    
}
//+------------------------------------------------------------------+
//| Expert tick function                                             |
//+------------------------------------------------------------------+
void OnTick()
{

   
   GetMiddleSize();
   Comment("\n" + "\n" + "\n" + "\n" "Сред. значение: " + DoubleToString((sumHigh - sumLow)/Amount, 6));
   
}
//+------------------------------------------------------------------+
double GetMiddleSize()
{
    sumHigh = 0.0;
     sumLow = 0.0;
    x = 0.0;
    int OpenShift = iBarShift(Symbol(),0,Date2); 
    int CloseShift = iBarShift(Symbol(),0,Date1);
    
    for (int i = OpenShift; i <= CloseShift; i++)
    {
         //int RangePeriod = i; //считаем по CountBars начиная с StartBar
         sumHigh += iHigh(Symbol(), Period(), i);
         sumLow += iLow(Symbol(), Period(), i);
         x = (sumHigh - sumLow)/Amount;
    }
    return(x);
}
//+------------------------------------------------------------------+

[Гладиoлус]

вот этот индикатор работает на всех валютных парах но к сожалению на сырье он не вычисляет среднее значени

[Гладиoлус]

я сперва подумал что это из того что золото у брокера отображается как GOLD поэтому перешел на другого где золото как XAUUSD но результат тот же

 

[Rigal]

26 минут назад, Гладиoлус сказал:

я сперва подумал что это из того что золото у брокера отображается как GOLD поэтому перешел на другого где золото как XAUUSD но результат тот же

 

меня интересует не код, а результат, который вас не устраивает.

Что значит "не вычисляет средние значения"? Что отображается в комменте?

[usver73]

2 минуты назад, Rigal сказал:

Что отображается в комменте?

А что он может при таком коде отображать? Нифига...

Функция расчета среднего значения возвращает double, но ничему это значение не присваивается.

Зато локальные переменные внесены в коммент

[Rigal]

33 минуты назад, Гладиoлус сказал:

    for (int i = OpenShift; i <= CloseShift; i++)
    {
         //int RangePeriod = i; //считаем по CountBars начиная с StartBar
         sumHigh += iHigh(Symbol(), Period(), i);
         sumLow += iLow(Symbol(), Period(), i);
         x = (sumHigh - sumLow)/Amount;
    }
    return(x);

К слову:

Если вы пытаетесь вычислить среднее значение, есть смысл делить на количество баров, которые вы посчитали. То есть ваш Amount = OpenShift - CloseShift + 1

Кроме того, нет никакого смысла выполнять деление на каждой итерации цикла.

Можно выкинуть переменную и строчку с делением на эту переменную и возвращать результат вот так:

return ((sumHigh-sumLow)/(CloseShift-OpenShift + 1));

При этом полезно убедиться, что CloseShift != (OpenShift + 1) - и если они равны, возвращать пустое значение.

 

[Rigal]

7 минут назад, usver73 сказал:

А что он может при таком коде отображать? Нифига...

Функция расчета среднего значения возвращает double, но ничему это значение не присваивается.

Зато локальные переменные внесены в коммент

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

 

Изменено 24 декабря, 2021 пользователем Rigal

[Rigal]

Подразумевалось, видимо, вот что:

#property copyright "Copyright 2021, MetaQuotes Software Corp."
#property link      "https://www.mql5.com"
#property version   "1.00"
#property strict

// CloseDate = Date1
// OpenDate = Date2
extern datetime Date1=D'2021.12.15 19:00';
extern datetime Date2=D'2021.12.16 13:05';

//+------------------------------------------------------------------+
//| Expert initialization function                                   |
//+------------------------------------------------------------------+
int OnInit()
{

   return(INIT_SUCCEEDED);
}
//+------------------------------------------------------------------+
//| Expert deinitialization function                                 |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
{
 
    
}
//+------------------------------------------------------------------+
//| Expert tick function                                             |
//+------------------------------------------------------------------+
void OnTick()
{

   Comment("\n" + "\n" + "\n" + "\n" "Сред. значение: " + DoubleToString(GetMiddleSize(), 6));
   
}
//+------------------------------------------------------------------+
double GetMiddleSize()
{
    double sumHigh = 0.0;
    double sumLow = 0.0;
    int OpenShift = iBarShift(Symbol(),Period(),Date2); 
    int CloseShift = iBarShift(Symbol(),Period(),Date1);
    
    for (int i = OpenShift; i <= CloseShift; i++)
    {
         sumHigh += iHigh(Symbol(), Period(), i);
         sumLow += iLow(Symbol(), Period(), i);
    }
    return(CloseShift >= OpenShift ? (sumHigh - sumLow) / (CloseShift - OpenShift + 1) : 0.0);
}
//+------------------------------------------------------------------+

 

[usver73]

29 минут назад, Rigal сказал:

Ну эти переменные не локальные, глобальные.

А, ну, да- не увидел, что он их зачем то вынес..

[Rigal]

56 минут назад, usver73 сказал:

А, ну, да- не увидел, что он их зачем то вынес..

а иначе бы не скомпилировалось

[usver73]

1 час назад, Rigal сказал:

return(CloseShift >= OpenShift ? (sumHigh - sumLow) / (CloseShift - OpenShift + 1) : 0.0);

Такая конструкция для начинающего очень сложна для восприятия.

И вообще, практичнее считать сумму разницы хай-лоу. Вдруг завтра захочется посчитать ЕМА.. 

ИМХО

[Rigal]

1 час назад, usver73 сказал:

Такая конструкция для начинающего очень сложна для восприятия.

Не спорю, но тут тонкий баланс между научить и сделать.

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

А сделать - тут не очено важно, как, вопрос компактности.

1 час назад, usver73 сказал:

И вообще, практичнее считать сумму разницы хай-лоу. Вдруг завтра захочется посчитать ЕМА.

Согласен целиком. Исходно (пока казалось, что в этом есть смысл) разбил на две суммы исключительно для того, чтобы как можно точнее воспроизвести ТЗ - тогда у человека появляется возможность связать код с предложенным алгоритмом и дальше его развивать на свое усмотрение.

[Гладиoлус]

Спасибо большое, безмерно вам благодарен, еще немного и я закончу свой индикатор. Это мне так облегчит жизнь а то на excel постоянно делать миллион вычислений ужасно утомляет.

[usver73]

1 час назад, Гладиoлус сказал:

, еще немного и я закончу свой индикатор

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

[VitChel]

Помогите, пожалуйста. Есть свой индикатор (назовем его Std_devma), который показывает две линии: первая линия - индикатор  iStdDev, вторая - MA, рассчитанная из массива iStdDev:

 

#property indicator_separate_window
#property indicator_buffers 2
#property indicator_color1 Blue
#property indicator_color2 Red

extern int STD_Period=20;
extern int STD_Shift=0;
extern int MA_Period=7;
extern int MA_Shift=0;
double B1[], B2[], B3[];

//----
int ExtCountedBars=0;
//+------------------------------------------------------------------+
//| Custom indicator initialization function                         |
//+------------------------------------------------------------------+
int init()
  {
//---- indicators
//---- drawing settings
   SetIndexStyle(0, DRAW_LINE);
   SetIndexStyle(1, DRAW_LINE);
   SetIndexBuffer(0,B1);
   SetIndexBuffer(1,B2);
   IndicatorDigits(MarketInfo(Symbol(),MODE_DIGITS));
//---- indicator short name
   IndicatorShortName("STD_DEV+MA");
   ArraySetAsSeries(B1,true);
   ArraySetAsSeries(B2,true);
//----
   return(0);
  }
//+------------------------------------------------------------------+
//| Custom indicator iteration function                              |
//+------------------------------------------------------------------+
int start()
  {
   
   int    pos=Bars-2;
   int i;   
   if(ExtCountedBars>2) pos=Bars-ExtCountedBars-1;
//---- main calculation loop
   for(i=0;i<Bars;i++)
      {
      B1[i] = iStdDev(Symbol(),Period(),STD_Period,STD_Shift,MODE_LWMA,PRICE_CLOSE,i);
      }
   for(i=0;i<Bars;i++)
      {
      
      B2[i]= iMAOnArray(B1,0,MA_Period,MA_Shift,MODE_SMA,i);
      }  
   return(0);
   }

 

Как мне в советнике вызвать данный индикатор и получить его данные?

[usver73]

1 час назад, VitChel сказал:

Как мне в советнике вызвать данный индикатор и получить его данные?

Посмотрите справку на iCustom()

[Leon28]

Здравствуйте.У меня почемуто в перестал работать тестер стратегий.В журнале пишет тэст генератор файл ерор.Что это значит?

[Гладиoлус]

Здравствуйте, вот опять к вам за помощью обращаюсь. Не получается у меня нарисовать уровни по заданным значениям.

Например по х беру дату а по у цену. Длина трендовой линии не имеет значения. Помогите пожалуйста с кодом.   

[Гладиoлус]

[Гладиoлус]

у по по цене а х по дате. А длина голубой трендовой линии не имеет значения (можно ну примерно такую длину установить как на скриншоте)

Изменено 30 декабря, 2021 пользователем Гладиoлус

[Rigal]

4 часа назад, Гладиoлус сказал:

у по по цене а х по дате. А длина голубой трендовой линии не имеет значения (можно ну примерно такую длину установить как на скриншоте)

а вы уже попробовали что-то?

В метатрейдере есть удобная библиотека для работы с линиями, ChartObjects/ChartObjectsLines.mqh

Используя эту библиотеку можно написать, например, так:

//+------------------------------------------------------------------+
//|                                                    TrendLine.mq4 |
//|                      Copyright 2021, Igor Ryabchikov (aka Rigal) |
//|                     https://tlap.com/forum/profile/109537-rigal/ |
//+------------------------------------------------------------------+
#property copyright "Copyright 2021, Igor Ryabchikov (aka Rigal)"
#property link      "https://tlap.com/forum/profile/109537-rigal/"
#property version   "1.00"
#property strict

input int RangePeriod = 12; //период на графике, на котором считаем High/Low
#include <ChartObjects/ChartObjectsLines.mqh>

CChartObjectTrend highLine, lowLine;

//+------------------------------------------------------------------+
//| Expert initialization function                                   |
//+------------------------------------------------------------------+
int OnInit() {
   //создаем линии 
   //время левой и правой точек выбираем, просто для примера, на три бара назад и текущее
   //цену задаем в ноль - "прячем" внизу до поры до времени (пока не понадобятся они нам)
   highLine.Create(0, "MyHighLine", 0, iTime(Symbol(), Period(), 3), 0.0, TimeCurrent(), 0.0);
   highLine.RayLeft(false);   //выключаем луч влево. Вправо - так же, только RayRight
   highLine.Color(clrRed);
   highLine.Width(2);         //чуть толще

   lowLine.Create(0, "MyLowLine", 0, iTime(Symbol(), Period(), 3), 0.0, TimeCurrent(), 0.0);
   lowLine.RayLeft(false); //выключаем луч влево. Вправо - так же, только RayRight
   lowLine.Color(clrBlue);
   lowLine.Width(2);         //чуть толще
   
   return(INIT_SUCCEEDED);
}
//+------------------------------------------------------------------+
//| Expert tick function                                             |
//+------------------------------------------------------------------+
void OnTick() {
   int highestBarInRange = iHighest(Symbol(), Period(), MODE_HIGH, RangePeriod);
   datetime highestTime = iTime(Symbol(), Period(), highestBarInRange);
   double highestPrice = iHigh(Symbol(), Period(), highestBarInRange);
   //обновняем линию high
   highLine.Time(0, highestTime); //точка 0 (левая)
   highLine.Price(0, highestPrice);   
   highLine.Time(1, TimeCurrent()); //точка 1 (правая)
   highLine.Price(1, highestPrice);   
   
   //повторяем для Low
   int lowestBarInRange = iLowest(Symbol(), Period(), MODE_LOW, RangePeriod);
   datetime lowestTime = iTime(Symbol(), Period(), lowestBarInRange);
   double lowestPrice = iLow(Symbol(), Period(), lowestBarInRange);
   //обновняем линию high
   lowLine.Time(0, lowestTime); //точка 0 (левая)
   lowLine.Price(0, lowestPrice);   
   lowLine.Time(1, TimeCurrent()); //точка 1 (правая)
   lowLine.Price(1, lowestPrice);   
}
//+------------------------------------------------------------------+

Советник будет таскать линии за максимумом и минимумом заданного периода баров

Получится вот так:

Спойлер

 

Я, понятно, совершенно неоптимально считаю это все на каждом тике, ну и все такое, это просто пример.

[Гладиoлус]

16 часов назад, Rigal сказал:

а вы уже попробовали что-то?

В метатрейдере есть удобная библиотека для работы с линиями, ChartObjects/ChartObjectsLines.mqh

Используя эту библиотеку можно написать, например, так:

//+------------------------------------------------------------------+
//|                                                    TrendLine.mq4 |
//|                      Copyright 2021, Igor Ryabchikov (aka Rigal) |
//|                     https://tlap.com/forum/profile/109537-rigal/ |
//+------------------------------------------------------------------+
#property copyright "Copyright 2021, Igor Ryabchikov (aka Rigal)"
#property link      "https://tlap.com/forum/profile/109537-rigal/"
#property version   "1.00"
#property strict

input int RangePeriod = 12; //период на графике, на котором считаем High/Low
#include <ChartObjects/ChartObjectsLines.mqh>

CChartObjectTrend highLine, lowLine;

//+------------------------------------------------------------------+
//| Expert initialization function                                   |
//+------------------------------------------------------------------+
int OnInit() {
   //создаем линии 
   //время левой и правой точек выбираем, просто для примера, на три бара назад и текущее
   //цену задаем в ноль - "прячем" внизу до поры до времени (пока не понадобятся они нам)
   highLine.Create(0, "MyHighLine", 0, iTime(Symbol(), Period(), 3), 0.0, TimeCurrent(), 0.0);
   highLine.RayLeft(false);   //выключаем луч влево. Вправо - так же, только RayRight
   highLine.Color(clrRed);
   highLine.Width(2);         //чуть толще

   lowLine.Create(0, "MyLowLine", 0, iTime(Symbol(), Period(), 3), 0.0, TimeCurrent(), 0.0);
   lowLine.RayLeft(false); //выключаем луч влево. Вправо - так же, только RayRight
   lowLine.Color(clrBlue);
   lowLine.Width(2);         //чуть толще
   
   return(INIT_SUCCEEDED);
}
//+------------------------------------------------------------------+
//| Expert tick function                                             |
//+------------------------------------------------------------------+
void OnTick() {
   int highestBarInRange = iHighest(Symbol(), Period(), MODE_HIGH, RangePeriod);
   datetime highestTime = iTime(Symbol(), Period(), highestBarInRange);
   double highestPrice = iHigh(Symbol(), Period(), highestBarInRange);
   //обновняем линию high
   highLine.Time(0, highestTime); //точка 0 (левая)
   highLine.Price(0, highestPrice);   
   highLine.Time(1, TimeCurrent()); //точка 1 (правая)
   highLine.Price(1, highestPrice);   
   
   //повторяем для Low
   int lowestBarInRange = iLowest(Symbol(), Period(), MODE_LOW, RangePeriod);
   datetime lowestTime = iTime(Symbol(), Period(), lowestBarInRange);
   double lowestPrice = iLow(Symbol(), Period(), lowestBarInRange);
   //обновняем линию high
   lowLine.Time(0, lowestTime); //точка 0 (левая)
   lowLine.Price(0, lowestPrice);   
   lowLine.Time(1, TimeCurrent()); //точка 1 (правая)
   lowLine.Price(1, lowestPrice);   
}
//+------------------------------------------------------------------+

Советник будет таскать линии за максимумом и минимумом заданного периода баров

Получится вот так:

  Показать контент

 

Я, понятно, совершенно не оптимально считаю это все на каждом тике, ну и все такое, это просто пример.

Понял, а вот можно что бы линия была отрезком?

[Гладиoлус]

16 часов назад, Rigal сказал:

а вы уже попробовали что-то?

В метатрейдере есть удобная библиотека для работы с линиями, ChartObjects/ChartObjectsLines.mqh

Используя эту библиотеку можно написать, например, так:

//+------------------------------------------------------------------+
//|                                                    TrendLine.mq4 |
//|                      Copyright 2021, Igor Ryabchikov (aka Rigal) |
//|                     https://tlap.com/forum/profile/109537-rigal/ |
//+------------------------------------------------------------------+
#property copyright "Copyright 2021, Igor Ryabchikov (aka Rigal)"
#property link      "https://tlap.com/forum/profile/109537-rigal/"
#property version   "1.00"
#property strict

input int RangePeriod = 12; //период на графике, на котором считаем High/Low
#include <ChartObjects/ChartObjectsLines.mqh>

CChartObjectTrend highLine, lowLine;

//+------------------------------------------------------------------+
//| Expert initialization function                                   |
//+------------------------------------------------------------------+
int OnInit() {
   //создаем линии 
   //время левой и правой точек выбираем, просто для примера, на три бара назад и текущее
   //цену задаем в ноль - "прячем" внизу до поры до времени (пока не понадобятся они нам)
   highLine.Create(0, "MyHighLine", 0, iTime(Symbol(), Period(), 3), 0.0, TimeCurrent(), 0.0);
   highLine.RayLeft(false);   //выключаем луч влево. Вправо - так же, только RayRight
   highLine.Color(clrRed);
   highLine.Width(2);         //чуть толще

   lowLine.Create(0, "MyLowLine", 0, iTime(Symbol(), Period(), 3), 0.0, TimeCurrent(), 0.0);
   lowLine.RayLeft(false); //выключаем луч влево. Вправо - так же, только RayRight
   lowLine.Color(clrBlue);
   lowLine.Width(2);         //чуть толще
   
   return(INIT_SUCCEEDED);
}
//+------------------------------------------------------------------+
//| Expert tick function                                             |
//+------------------------------------------------------------------+
void OnTick() {
   int highestBarInRange = iHighest(Symbol(), Period(), MODE_HIGH, RangePeriod);
   datetime highestTime = iTime(Symbol(), Period(), highestBarInRange);
   double highestPrice = iHigh(Symbol(), Period(), highestBarInRange);
   //обновняем линию high
   highLine.Time(0, highestTime); //точка 0 (левая)
   highLine.Price(0, highestPrice);   
   highLine.Time(1, TimeCurrent()); //точка 1 (правая)
   highLine.Price(1, highestPrice);   
   
   //повторяем для Low
   int lowestBarInRange = iLowest(Symbol(), Period(), MODE_LOW, RangePeriod);
   datetime lowestTime = iTime(Symbol(), Period(), lowestBarInRange);
   double lowestPrice = iLow(Symbol(), Period(), lowestBarInRange);
   //обновняем линию high
   lowLine.Time(0, lowestTime); //точка 0 (левая)
   lowLine.Price(0, lowestPrice);   
   lowLine.Time(1, TimeCurrent()); //точка 1 (правая)
   lowLine.Price(1, lowestPrice);   
}
//+------------------------------------------------------------------+

Советник будет таскать линии за максимумом и минимумом заданного периода баров

Получится вот так:

  Показать контент

 

Я, понятно, совершенно неоптимально считаю это все на каждом тике, ну и все такое, это просто пример.

Мне просто надо сделать так что бы я мог редактировать его потом. 

[Гладиoлус]

3 минуты назад, Гладиoлус сказал:

Мне просто надо сделать так что бы я мог редактировать его потом. 

А то я два раза кликнул на отрезок а он не выделился следовательно невозможно редактировать.