При использовании с функцией закрытие ордеров метатрейдер вылетает - MQL4 и MetaTrader 4

Lowech 2017.08.31 10:01 #2861

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

double ClosProfit()
{
double summa=0;
int orders=OrdersHistoryTotal();
{
for(int i=orders-1;i>=0;i--)
{
if(OrderSelect(i,SELECT_BY_POS,MODE_HISTORY)==false)
{
Print("Error in history!");
break;
}
if(OrderSymbol()!=Symbol() || OrderType()>OP_SELL)
continue;
//---
if(OrderProfit()>0) break;

If(OrderProfit()<0)i++;

}

If(i<0)
summa=OrderProfit()+summa;
}

If(OrderProfit()>summa)

summa=OrderProfit()+summa;

return(summa);
}

100 пунктов Необходимо прописать цикл Как кодировать?

Nauris Zukas 2017.08.31 12:07 #2862

Lowech:
Добрый день. Помоги с функцией пожалуйста, суммирует убыток закрытых ордеров, при использование с функцией закрытие ордеров метатрейдер вылетает, думаю какая та ошибка в этой функции.

Лучше уже готовые посмотреть и если надо то под себя подкорректировать. GetProfitFromDateInCurrency()

Konstantin Erin 2017.08.31 13:09 #2863

Lowech:
Добрый день. Помоги с функцией пожалуйста, суммирует убыток закрытых ордеров, при использование с функцией закрытие ордеров метатрейдер вылетает, думаю какая та ошибка в этой функции.

что значит метатрейдер вылетает? Программа компилируется с ошибками!!!

в трех местах вместо if с маленькой строчной буквы написано If с большой прописной буквы

пара фигурных скобок явно лишняя: первая после int orders=OrdersHistoryTotal(); и парная ей после summa=OrderProfit()+summa; - но это маловажно

ну и еще есть...

помогите с функцией Перерисовка стандартных индикаторов?! Unhandled exception 0xC0000005

Ivan Ivanov 2017.08.31 13:32 #2864

Lowech:
Добрый день. Помоги с функцией пожалуйста, суммирует убыток закрытых ордеров, при использование с функцией закрытие ордеров метатрейдер вылетает, думаю какая та ошибка в этой функции.

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

double ClosProfit()
{ double summa=0;
  int orders=OrdersHistoryTotal(); 
  for(int i=orders-1;i>=0;i--)
  {
    if(OrderSelect(i,SELECT_BY_POS,MODE_HISTORY)==false) { Print("Error in history!"); break; }
    if(OrderSymbol()!=Symbol() || OrderType()>OP_SELL) continue;
    if(OrderProfit()<0.0) summa=OrderProfit()+summa;
  }
  return(summa);
}

Konstantin Erin 2017.08.31 13:34 #2865

Опоздал на 2 минуты пока писал ... Давайте рассмотрим внимательно эту часть:

  for(int i=orders-1;i>=0;i--)
  {
    if(OrderSelect(i,SELECT_BY_POS,MODE_HISTORY)==false)
    {
      Print("Error in history!");
      break;
    }
    if(OrderSymbol()!=Symbol() || OrderType()>OP_SELL) continue;
    if(OrderProfit()>0) break;
    if(OrderProfit()<0)i++;
  }
  if(i<0)

Скомпилировал ее с директивой #property strict - ругает последнюю строку: 'i' - undeclared identifier. Дело в том, что переменная объявлена в цикле и действительна только в нем. Без директивы #property strict скомпилировалось без ошибок, но это плохо. Директиву надо использовать.

Вместо if(OrderSelect(i,SELECT_BY_POS,MODE_HISTORY)==false) проще написать if(!OrderSelect(i,SELECT_BY_POS,MODE_HISTORY)). Теперь далее у Вас стоит break - это выход из цикла. А надо обработать другие ордера. Впрочем на истории это лишнее. Ошибка случается на рыночных и отложенных ордерах, если к моменту обработки ордер закрыли и он отсутствует. В общем, правильнее if(!OrderSelect(i,SELECT_BY_POS,MODE_HISTORY)) continue; - продолжить обработку следующих ордеров

Из тех же соображений if(OrderProfit()>0) break; заменить на if(OrderProfit()>0) continue;

А вот и причина ошибки: if(OrderProfit()<0)i++; - если встретится убыточный номер, то индекс увеличивается. А заголовок цикла for(int i=orders-1;i>=0;i--) его уменьшает. Повторяется обработка того же ордера - программа зацикливается. Наверное, надо так:

#property   strict
double ClosProfit()
{
  double summa=0;
  for(int i=OrdersHistoryTotal()-1; i>=0; i--)
  {
    if(!OrderSelect(i,SELECT_BY_POS,MODE_HISTORY)) continue;
    if(OrderSymbol()!=Symbol() || OrderType()>OP_SELL) continue;
    if(OrderProfit()<0) summa+=OrderProfit();
  }
  return(summa);
}

Lowech 2017.08.31 15:20 #2866

STARIJ:
Опоздал на 2 минуты пока писал ... Давайте рассмотрим внимательно эту часть:

Скомпилировал ее с директивой #property strict - ругает последнюю строку: 'i' - undeclared identifier. Дело в том, что переменная объявлена в цикле и действительна только в нем. Без директивы #property strict скомпилировалось без ошибок, но это плохо. Директиву надо использовать.

Вместо if(OrderSelect(i,SELECT_BY_POS,MODE_HISTORY)==false) проще написать if(!OrderSelect(i,SELECT_BY_POS,MODE_HISTORY)). Теперь далее у Вас стоит break - это выход из цикла. А надо обработать другие ордера. Впрочем на истории это лишнее. Ошибка случается на рыночных и отложенных ордерах, если к моменту обработки ордер закрыли и он отсутствует. В общем, правильнее if(!OrderSelect(i,SELECT_BY_POS,MODE_HISTORY)) continue; - продолжить обработку следующих ордеров

Из тех же соображений if(OrderProfit()>0) break; заменить на if(OrderProfit()>0) continue;

А вот и причина ошибки: if(OrderProfit()<0)i++; - если встретится убыточный номер, то индекс увеличивается. А заголовок цикла for(int i=orders-1;i>=0;i--) его уменьшает. Повторяется обработка того же ордера - программа зацикливается. Наверное, надо так:

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

if(OrderProfit()>0) break; заменить на if(OrderProfit()>0) continue;

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

if(OrderProfit()>0) break;

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

Советники: iK_TF - TrendFinder Помогите новичку. ( Закрытие Получить сумму убытка последних

Konstantin Erin 2017.08.31 15:27 #2867

Lowech:

Спасибо, больше метатрейдер не вылетает) По поводу мне нужно чтобы если ордер закрылся профитом, этот профит функция не учитывала, только сумму убытка и до первого ордера закрытого в профит. Т.е.
должна останавливать счетчик если ордер закрылся в профит, я правильно понимаю?

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

Lowech 2017.08.31 17:10 #2868

STARIJ:
Рассуждения Ваши почти правильны. Тут отсутствует гарантия какого-либо порядка расположения ордеров в истории. Для себя - можно, на продажу - плохо

А разве цикл по ордерам не попорядку ордера считает?

Vitaly Muzichenko 2017.08.31 18:13 #2869

STARIJ:

Если есть исходник - DROW_NONE - отсутствие какого-либо рисования

Не помогло, всё-равно отображает циферки в углу

Konstantin Erin 2017.08.31 19:27 #2870

Lowech: А разве цикл по ордерам не попорядку ордера считает?

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

Любые вопросы новичков по MQL4 и MQL5, помощь и обсуждение по алгоритмам и кодам - страница 287