ФорумРазделы

Вопросы от начинающих MQL5 MT5 MetaTrader 5 - страница 638

Новый комментарий
ev85:

Добрый день.

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

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

Вот пример моего кода для обычной скользящей средней. Что добавить или исправить?


добавить datetime mailTime и запоминать время пред.отправки, чтобы повторно не отправлять письма

datetime mailTime=0;
int init()
  {
   return(0);
  }

start()
  {
//-----------------------------------------------------------------------------------------------------
// Сигналы
//-----------------------------------------------------------------------------------------------------
if(Open[1]>ma && Close[1]<ma && Time[1]>mailTime)  {
bool res = SendMail("Сигнал", " Покупай");
if (res) mailTime=Time[1];
 }
if(Open[1]<ma && Close[1]>ma && Time[1]>mailTime) {
bool res=SendMail("Сигна", "Продавай");
if (res) mailTime=Time[1];
 }
//-------------------------------------------------------------------
   return(0);
  }

ps. пора уже переходить от start() к OnTick() в советниках

pps. mailTime надо ещё сохранять в глобальных переменных терминала на случай рестарта советника.

Maxim Kuznetsov:

добавить datetime mailTime и запоминать время пред.отправки, чтобы повторно не отправлять письма

ps. пора уже переходить от start() к OnTick() в советниках

pps. mailTime надо ещё сохранять в глобальных переменных терминала на случай рестарта советника.

Не помогло. Так и шлет бесконечно письма.

ev85:

Добрый день.

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

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

Вот пример моего кода для обычной скользящей средней. Что добавить или исправить?




static datetime TimeN=0;
int init()
  {
   return(0);
  }
start()
  {
//Операции выполняются только при появлении следующего бара
   datetime TimeC=iTime(NULL,TF,0);
   if(TimeN==0)TimeN=TimeC;
   if(TimeN==TimeC) return;
   TimeN=TimeC;

//-----------------------------------------------------------------------------------------------------
// Сигналы
//-----------------------------------------------------------------------------------------------------
if(Open[1]>ma && Close[1]<ma)  {
bool res = SendMail("Сигнал", " Покупай");
}
if(Open[1]<ma && Close[1]>ma) {
SendMail("Сигна", "Продавай");
}
//-------------------------------------------------------------------
   return(0);
  }
-Aleks-:
С этим годом вообще не отправляются емейлы((
ev85:
С этим годом вообще не отправляются емейлы((
Так у вас чему ma равен? Это не машка на нулевом баре случайно?
-Aleks-:
Так у вас чему ma равен? Это не машка на нулевом баре случайно?

 ma=iMA(NULL,0,MovingPeriod,MovingShift,MODE_SMA,PRICE_CLOSE,0);

ev85:

 ma=iMA(NULL,0,MovingPeriod,MovingShift,MODE_SMA,PRICE_CLOSE,0);

Тогда все ясно. Машка на нулевом баре постоянно пересчитывается - поэтому сигналы будут приходить при каждом тике.

Попробуйте так: 

static datetime TimeN=0;

int Signal=0;


int init()

  {

   return(0);

  }

start()

  {

//Операции выполняются только при появлении следующего бара

   datetime TimeC=iTime(NULL,TF,0);

   if(TimeN==0)TimeN=TimeC;

   if(TimeN==TimeC && Signal==1) return;

   TimeN=TimeC;

   Signal=0;


//-----------------------------------------------------------------------------------------------------

// Сигналы

//-----------------------------------------------------------------------------------------------------

if(Open[1]>ma && Close[1]<ma)  

{

bool res = SendMail("Сигнал", " Покупай");

Signal=1;

}

if(Open[1]<ma && Close[1]>ma) 

{

SendMail("Сигнал", "Продавай");

Signal=1;

}

   return(0);

  }


Вообще, обычно делают сравнение

  ma=iMA(NULL,0,MovingPeriod,MovingShift,MODE_SMA,PRICE_CLOSE,1);

ma>Open[0]; 

 

 


Maxim Kuznetsov:
вторая строка - более чем похожа на массив signed char. То есть вся "строка" может быть просто бинарной структурой распечатанной побайтно, причём возможно её сначала зашифровали, поэтому 0 нет
Да, после некоторых действий выяснилось, что строка действительно signed char. Теперь вопрос: как её привести в читабельный вид на сервере?
-Aleks-:

Тогда все ясно. Машка на нулевом баре постоянно пересчитывается - поэтому сигналы будут приходить при каждом тике.

Попробуйте так: 

static datetime TimeN=0;

int Signal=0;


int init()

  {

   return(0);

  }

start()

  {

//Операции выполняются только при появлении следующего бара

   datetime TimeC=iTime(NULL,TF,0);

   if(TimeN==0)TimeN=TimeC;

   if(TimeN==TimeC && Signal==1) return;

   TimeN=TimeC;

   Signal=0;


//-----------------------------------------------------------------------------------------------------

// Сигналы

//-----------------------------------------------------------------------------------------------------

if(Open[1]>ma && Close[1]<ma)  

{

bool res = SendMail("Сигнал", " Покупай");

Signal=1;

}

if(Open[1]<ma && Close[1]>ma) 

{

SendMail("Сигнал", "Продавай");

Signal=1;

}

   return(0);

  }


Вообще, обычно делают сравнение

  ma=iMA(NULL,0,MovingPeriod,MovingShift,MODE_SMA,PRICE_CLOSE,1);

ma>Open[0]; 

 

 


Спасибо большое! Наконец работает как надо))
ev85:
Спасибо большое! Наконец работает как надо))
На здоровье. Только обратите внимание, что сразу два сигнала - на покупку и продажу не придет - первый сигнал перекроет второй.
1...631632633634635636637638639640641642643644645...1599
Новый комментарий