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

23 минуты назад, Partorius сказал:

я так понимаю предлагается поставить таймер чтобы он сигналил один раз в таймфрейм, правильно?

нет, я не предлагаю таймер, я проверил работу флага one на похожем коде, Вам нужно просто проверенный код перенести в свой советник:

if (StopBuy < 10000 && Stochastic < 20)
{
	if (one && AlertOn) 
	{
		one = false;
		Alert(_Symbol," сигнал на покупку ");
		if (PushMessagesOn) 
		{ 
			SendNotification(_Symbol + " сигнал на покупку");
			Print ("Алерт на мобильник покупка", Stochastic);
		}
	}
} else one = true;

Здравствуйте! хочу использовать в советнике данное выражение

ArrayCopy(Old_Hay_mass,Old_Hay_mass,1,0,5);

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

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

Есть ли в этом примере подвох?

Будет работать, только нужно быть аккуратным с размерностью, многомерностью и направлением индексирования (не копировать индикаторные буферы)

1 час назад, 0ll сказал:

Будет работать, только нужно быть аккуратным с размерностью, многомерностью и направлением индексирования (не копировать индикаторные буферы)

Oll благодарю! Может поможете еще в одном вопросе?

Занимаюсь отображением уровней поддержки-сопр. на графике. ObjectCreate тип прямоугольник, OBJPROP_TIME1 - вычисляется, OBJPROP_TIME2 - TimeCurrent. При этом одни уровни тянуться за ценой, другие остаются на каких то определенных (или наоборот неопределенных) точках графика. Перерисовка на каждом баре..

@delyaro по идее сами по себе они должны стоять на месте (в смысле относительно баров графика) и двигать их надо принудительно (опять-же двигать относительно баров)

Вы тоже уточните - что имеете в виду, желательно скрин и что хотите в итоге получить. А чего уровни прямоугольником? линии -же есть.

Уровни прямоугольником для охвата зон т.к. у зоны больше возможности для применения (выделенная область идет как критерий для работы торговых функций, ну и визуализация дополнительно). Для визуализации желательно видеть от какого экстремума зона началась(т.е. закрепление за свечой в истории) и двигаться полоса должна вместе с ценой пока условия ее существования действительны. Сейчас действительно прямоугольники двигаются, но не растягиваются а перемещаются с привязкой к последнему бару, но периодически некоторые теряются и остаются позади.(на скрине2 в круге выделено)

15 часов назад, delyaro сказал:

периодически некоторые теряются и остаются позади

ну раз иногда работает правильно - значит ошибка в учёте объектов или Вы по имени все модифицируете? нужно реализацию смотреть.

5 часов назад, 0ll сказал:

ну раз иногда работает правильно - значит ошибка в учёте объектов или Вы по имени все модифицируете? нужно реализацию смотреть.

Вот такая картинка будет в итоге, только без определения силы, числа касаний и т.п. На скрине сова с внедренным кодом из чужого индикатора немного измененного. Данная реализация для меня слишком неповоротливая оказалась и при продолжительной работе весь график в зонах почти без просветов... Пишу сейчас свой блок , обращение по имени к объектам. Есть ощущение что я не понимаю основы в создании этих объектов. Может я и модификации никакой не сделал... Практически вся графическая часть в куске кода представлена)

@delyaro аккуратно делаете, - люблю так работать. Алгоритм такой:

1. Найдена новая зона. Рисуете объект с именем "Zona_", далее что хотите (советую дата/время - имя должно быть уникально!)

2. Здесь можете удалить старую зону (которую цена только-что прошла), если расстояние менее 20 ст.п.(например)

3. На каждом новом баре запускается цикл по всем объектам графика с фильтром по имени "Zona_" (см. справку ObjectsTotal)

3.1. Модифицируем последнюю координату каждого объекта: ObjectSet(s, OBJPROP_TIME2, TimeCurrent());

4. Здесь можно оценить количество объектов и если много, то удалить самый старый или самый дальний

С третьего пункта мой алгоритм пошел в другом направлении... Вернее даже на третьем. Хотя и первый кривой в принципе))

Теперь процесс пойдет веселее)

Здравствуйте, подскажите пожалуйста функцию открытия ордеров от Horizontal Line? То есть , что бы автоматически открывались ордера от выставленной мною Horizontal Line на графике!

@molot_1 такой функции в MQL нет, но есть несколько советников, которые позволяют настроить торговлю от линий, в том числе на нашем форуме.

Всем привет. Увлекся массивами, построил вот такую вот функцию:

Профит еще не закрытых ордеров на покупку

Далее хочу посмотреть как выведется у меня определенная ячейка массива, что я для этого делаю, пишу перебор этого массива и вот желаю вывести ячейку к примеру 5:

Вывод ячейки массива с помощью перебора

... и тут тестер останавливается и я не пойму в чем причина. 

Если кто знает прошу помощи. Долго занимаюсь этим вопросом, но результат только в том что пока загнал в массив числа. Где-то надо что-то исправить)

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

Если кто знает прошу помощи.

У Вас есть размер массива, прежде чем обратиться к элементу нужно удостовериться что №элемента < размера массива.

Условия выбора ордеров в цикле должны быть одинаковые, а у Вас магик появляется. И зачем перебор ордеров в основной функции?

что Вы хотели сделать с этим - cBuyProfitOpen[n];? напишите просто:

for(int n=0; n < ArraySize( cBuyProfitOpen ) ; n++ )    Print( "Профит №", n, "=", DoubleToStr(cBuyProfitOpen[n],2) );

@diodio 

1. Когда тестер останавливается от должен писать причину остановки, смотри логи. В твоем случае, я предполагаю, это буде неправильное обращение к массиву, к не существующему элементу.
2. по первому коду я понял, что хотим хранить профит каждого ордера отдельно. Итак:

• для чего тут int counter=0 и  в конце ArrayResize(cBuyProfitOpen,counter);
• не правильно так работать с массивами cBuyProfitOpen[counter++]=OrderProfit();

Я не знаю ваших намерений поэтому ваш код, только правильно написаный:

void OrderProfitBuy(double &cBuyProfitOpen[])
  {
//use counter and size of the array or CArrayInt* and do not resize every time
   int counter=0,size=OrdersTotal();
   ArrayResize(cBuyProfitOpen,size);
   for(int i=OrdersTotal()-1; i>=0; i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderType()==OP_BUY)
           {
            cBuyProfitOpen[i]=OrderProfit();			//Вот тут исправление !!!!!!!!!!!!!!!
            counter++;
           }
        }
     }
   ArrayResize(cBuyProfitOpen,counter);			// но в вашем случае этот кусок портит все сделанное вверху, зачем это?
   return;
  }

1. по в второму коду

•  что вы хотели сделать вот этим "cBuyProfitOpen[n];"

Массив это как банк с депозитными ячейками или вокзал с камерой хранения. Сейф или камера хранения это массив, а вот что мы пишем в скобках [№] это как раз ячейка. С этими ячейками взаимодействие такие, либо положить, либо взять. Пример: с записью в ячейку cBuyProfitOpen[n]=3 или возьмем с ячейки int q=cBuyProfitOpen[n]. В вашем случае пришли глазками хлопать, от этого ни чего не изменится.

• что вы хотели сделать вот этим "!cBuyProfitOpen[5]" в принципе тут все выражение под вопросом.
• откуда взялась цифра "5" в "!cBuyProfitOpen[5]" и cBuyProfitOpen[5], что она обозначает?

В 26.09.2019 в 21:37, diodio сказал:

 if(!cBuyProfitOpen[5] && (cBuyProfitOpen[5]>=0 || cBuyProfitOpen[5]<=0))

Чтобы обратится к элементу динамического массива нужно быть уверенным что элемент существует, вернее запрос не выходит за границу массива, т.е. Перед подобной строкой нужно проверить соответствие размера массива 6 или более.

Здравствуйте!

Есть в советнике функция:

//+-------------------------------------------------------------------------------------------------------------------------------------------------+
//| Функция закрывает последнюю позицию buy                                                                                                         |
//+-------------------------------------------------------------------------------------------------------------------------------------------------+
void CloseBuyLast(void)
  {
   while(true)
      if(OrderClose(gv.grid.last.ticket,gv.grid.last.volume,Bid,0))
        {
         if(OrderSelect(gv.grid.last.ticket,SELECT_BY_TICKET))
            gv.grid.profit+=OrderProfit()+OrderSwap();

         break;
        }
      else
        {
         Sleep(1000);
         RefreshRates();
        }

   gv.grid.number--;
   gv.grid.volume      -= gv.grid.last.volume;
   gv.grid.nextOpVolume = gv.grid.last.volume;
   gv.grid.nextOpPrice  = gv.grid.last.openPrice;

   for(int i=MAX-1; i>=0; i--)
      if(gv.grid.pos.isOpened)
        {
         gv.grid.pos.isOpened   = false;
         gv.grid.pos.allowClose = false;
         gv.grid.pos.ticket     = 0;
         gv.grid.pos.type       = -1;
         gv.grid.pos.volume     = 0;
         gv.grid.pos.openPrice  = 0;
         gv.grid.pos.takeProfit = 0;
         break;
        }

   for(int i=0; i<MAX; i++)
      if(gv.grid.pos.isOpened)
        {
         gv.grid.last.isOpened   = gv.grid.pos.isOpened;
         gv.grid.last.allowClose = gv.grid.pos.allowClose;
         gv.grid.last.ticket     = gv.grid.pos.ticket;
         gv.grid.last.type       = gv.grid.pos.type;
         gv.grid.last.volume     = gv.grid.pos.volume;
         gv.grid.last.openPrice  = gv.grid.pos.openPrice;
         gv.grid.last.takeProfit = gv.grid.pos.takeProfit;
        }

   CheckPanel();
  }

Можно ли видя эту функцию написать аналогичную функцию, которая закрывала бы ПЕРВЫЙ ордер.

Если возможно, то прошу помочь.

Спасибо, заранее!!! 

39 минут назад, Владимир Стариков сказал:

Можно ли видя эту функцию написать аналогичную функцию, которая закрывала бы ПЕРВЫЙ ордер.

Функция основана на классах, нужно видеть его устройство. В советнике сделан учёт ордеров сетки и после закрытия идёт обновление учёта. Задачка не для слабонервных даже при наличии полного кода.

50 минут назад, 0ll сказал:

Функция основана на классах, нужно видеть его устройство. В советнике сделан учёт ордеров сетки и после закрытия идёт обновление учёта. Задачка не для слабонервных даже при наличии полного кода.

В любом случае, спасибо!!!

Хотя бы утвердился в мысли. что классы - это не просто так....

Всем привет!  MikR0ReR спасибо, вставил ваше исправление в код, это действительно правильный цикл перебора:

void OrderProfitBuy(double &cBuyProfitOpen[])
  {
   int counter=0,size=OrdersTotal();
   ArrayResize(cBuyProfitOpen,size);
   for(int i=0; i<OrdersTotal(); i++)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderType()==OP_BUY)
           {
            cBuyProfitOpen[i]=OrderProfit();
            counter++;
           }
        }
     }
   ArrayResize(cBuyProfitOpen,counter); // здесь мы уже получаем итоговый размер массива, т.к. в счетчик  counter++ стикается эта цифра
   return;
  }

   OrderProfitBuy(cBuyProfitOpen);
   for(int n=0; n < ArraySize(cBuyProfitOpen); n++)
     {
     if(ArraySize(cBuyProfitOpen)>5)
     Print("Профит ячейки №3= ", DoubleToStr(cBuyProfitOpen[3],2));
     break;
     }

@diodio я думаю при наличии "других" ордеров этот код будет глючить (здесь я не согласен с MikR0ReR):

cBuyProfitOpen[i]=OrderProfit();
counter++;

думаю надо вместо i использовать counter, ибо i может быть значительно больше counter (числа "наших" ордеров).

59 минут назад, diodio сказал:

Ну и тут возникает вопрос, а куда делись те предыдущие параметры этого массива?

Не понял про параметры, может значения? значения перезаписываются, как в обычной переменной.

@VAS35 а в чем проблема поменять тип данных с double на long

double S_1 = iVolume(NULL, 0, 1);
на
long S_1 = iVolume(NULL, 0, 1);

@VAS35 считает он только первых 2 свечи, вместо 4

не нашел ни каких проблем с этим считает все бары

Но заметил функцию с передачей параметра signal = Signal(i-1);

int Signal(int i)

{

хотя здесь этот параметр не используется.

}

По конкретней опишите проблему. Может чего то я не понял.

Спасибо, что откликнулись! Проблема в том, что в условиях прописываются размеры свечей. Но индикатор их не рассчитывает, или рассчитывает неправильно. При тестировании на истории чаще всего неправильно считает размер 3 свечи. 

@VAS35 почему решили что проблема на 3 свече. У вас Выдает ошибку?? так дайте скрин! Я например проверил ошибок нет. И я так понимаю вы дали не весь код, урезанный?? мне нужно конкретика.