Забыл операцию присваивания. Попробуйте сделать еще один советник, который компилируется без ошибок 500-м билдом эдитора - MQL4 и MetaTrader 4

Artyom Trishkin 2014.10.30 05:25 #7571

Ozero:

Перес ";" за скобку.

Программа работает, НО периодически ругается на ошибку 4108.

Ozero.

Открываем справку, смотрим ошибки времени выполнения, видим:

4108

ERR_INVALID_TICKET

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

Думаем, ищем ...

MegaHunter 2014.10.30 05:26 #7572

Люди помогите надо чтоб торговал на каждой валюте со своими ордерами различал и отличал + после открытого ордера если цена откатывается ещё дальше мог открыть ещё ордер такой же как и первый.

//+------------------------------------------------------------------+
//|                                                  MACD Sample.mq4 |
//|                      Copyright © 2005, MetaQuotes Software Corp. |
//|                                       http://www.metaquotes.net/ |
//+------------------------------------------------------------------+
 
extern double TakeProfit = 50;
extern double Lots = 0.1;
extern double TrailingStop = 30;
extern double MACDOpenLevel=3;
extern double MACDCloseLevel=2;
extern double MATrendPeriod=26;

//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
int start()
  {
   double MacdCurrent, MacdPrevious, SignalCurrent;
   double SignalPrevious, MaCurrent, MaPrevious;
   int cnt=0, ticket, total;
// первичные проверки данных
// важно удостовериться что эксперт работает на нормальном графике и
// пользователь правильно выставил внешние переменные (Lots, StopLoss,
// TakeProfit, TrailingStop)
// в нашем случае проверяем только TakeProfit
   if(Bars<100)
     {
      Print("bars less than 100");
      return(0);  
     }
   if(TakeProfit<10)
     {
      Print("TakeProfit less than 10");
      return(0);  // проверяем TakeProfit
     }
// ради упрощения и ускорения кода, сохраним необходимые
// данные индикаторов во временных переменных
   MacdCurrent=iMACD(NULL,0,12,26,9,PRICE_CLOSE,MODE_MAIN,0);
   MacdPrevious=iMACD(NULL,0,12,26,9,PRICE_CLOSE,MODE_MAIN,1);
   SignalCurrent=iMACD(NULL,0,12,26,9,PRICE_CLOSE,MODE_SIGNAL,0);
   SignalPrevious=iMACD(NULL,0,12,26,9,PRICE_CLOSE,MODE_SIGNAL,1);
   MaCurrent=iMA(NULL,0,MATrendPeriod,0,MODE_EMA,PRICE_CLOSE,0);
   MaPrevious=iMA(NULL,0,MATrendPeriod,0,MODE_EMA,PRICE_CLOSE,1);
 
   total=OrdersTotal();
   if(total<1) 
     {
      // нет ни одного открытого ордера
      if(AccountFreeMargin()<(1000*Lots))
        {
         Print("We have no money. Free Margin = ", AccountFreeMargin());
         return(0);  
        }
      // проверяем на возможность встать в длинную позицию (BUY)
      if(MacdCurrent<0 && MacdCurrent>SignalCurrent && MacdPrevious<SignalPrevious &&
         MathAbs(MacdCurrent)>(MACDOpenLevel*Point) && MaCurrent>MaPrevious)
        {
         ticket=OrderSend(Symbol(),OP_BUY,Lots,Ask,3,0,Ask+TakeProfit*Point,
                          "macd sample",16384,0,Green);
         if(ticket>0)
           {
            if(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES))
               Print("BUY order opened : ",OrderOpenPrice());
           }
         else Print("Error opening BUY order : ",GetLastError()); 
         return(0); 
        }
      // проверяем на возможность встать в короткую позицию (SELL)
      if(MacdCurrent>0 && MacdCurrent<SignalCurrent && MacdPrevious>SignalPrevious && 
         MacdCurrent>(MACDOpenLevel*Point) && MaCurrent<MaPrevious)
        {
         ticket=OrderSend(Symbol(),OP_SELL,Lots,Bid,3,0,Bid-TakeProfit*Point,
                          "macd sample",16384,0,Red);
         if(ticket>0)
           {
            if(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES))
               Print("SELL order opened : ",OrderOpenPrice());
           }
         else Print("Error opening SELL order : ",GetLastError()); 
         return(0); 
        }
      return(0);
     }
   // переходим к важной части эксперта - контролю открытых позиций
   // 'важно правильно войти в рынок, но выйти - еще важнее...'
   for(cnt=0;cnt<total;cnt++)
     {
      OrderSelect(cnt, SELECT_BY_POS, MODE_TRADES);
      if(OrderType()<=OP_SELL &&   // это открытая позиция? OP_BUY или OP_SELL 
         OrderSymbol()==Symbol())  // инструмент совпадает?
        {
         if(OrderType()==OP_BUY)   // открыта длинная позиция
           {
            // проверим, может уже пора закрываться?
            if(MacdCurrent>0 && MacdCurrent<SignalCurrent && MacdPrevious>SignalPrevious &&
               MacdCurrent>(MACDCloseLevel*Point))
                {
                 OrderClose(OrderTicket(),OrderLots(),Bid,3,Violet); // закрываем позицию
                 return(0); // выходим
                }
            // проверим - может можно/нужно уже трейлинг стоп ставить?
            if(TrailingStop>0)  
              {                 
               if(Bid-OrderOpenPrice()>Point*TrailingStop)
                 {
                  if(OrderStopLoss()<Bid-Point*TrailingStop)
                    {
                     OrderModify(OrderTicket(),OrderOpenPrice(),Bid-Point*TrailingStop,
                                 OrderTakeProfit(),0,Green);
                     return(0);
                    }
                 }
              }
           }
         else // иначе это короткая позиция
           {
            // проверим, может уже пора закрываться?
            if(MacdCurrent<0 && MacdCurrent>SignalCurrent &&
               MacdPrevious<SignalPrevious && MathAbs(MacdCurrent)>(MACDCloseLevel*Point))
              {
               OrderClose(OrderTicket(),OrderLots(),Ask,3,Violet); // закрываем позицию
               return(0); // выходим
              }
            // проверим - может можно/нужно уже трейлинг стоп ставить?
            if(TrailingStop>0)  
              {                 
               if((OrderOpenPrice()-Ask)>(Point*TrailingStop))
                 {
                  if((OrderStopLoss()>(Ask+Point*TrailingStop)) || (OrderStopLoss()==0))
                    {
                     OrderModify(OrderTicket(),OrderOpenPrice(),Ask+Point*TrailingStop,
                                 OrderTakeProfit(),0,Red);
                     return(0);
                    }
                 }
              }
           }
        }
     }
   return(0);
  }
// конец.

Alexey Kravchenko 2014.10.30 05:33 #7573

Всем привет !

Только сейчас заметил странную работу NormalizeDouble.

То есть советник то работает хорошо и ошибок не выдает а вот логи в тестере очень странные.

Поясню.

Вычисляю функцией профит. Делаю как и положено NormalizeDouble(Profit,Digits);

Далее Print(" profit=",DoubleToStrMorePrecision(Profit,8));

В результате я вижу в логах такое : "profit=0.88881912". По идее должно быть 0.88881000 верно ?

Или я туплю и делаю что то не так ?

Помощь Copilot - Разработка Гибкая торговая система MetaTrader Особенности тестирования - Алгоритмический

Artyom Trishkin 2014.10.30 05:36 #7574

goodearth:

Всем привет !

Только сейчас заметил странную работу NormalizeDouble.

То есть советник то работает хорошо и ошибок не выдает а вот логи в тестере очень странные.

Поясню.

Вычисляю функцией профит. Делаю как и положено NormalizeDouble(Profit,Digits);

Далее Print(" profit=",DoubleToStrMorePrecision(Profit,8));

В результате я вижу в логах такое : "profit=0.88881912". По идее должно быть 0.88881000 верно ?

Или я туплю и делаю что то не так ?

У вас значение Profit участвует в торговых приказах?

Alexey Kravchenko 2014.10.30 05:43 #7575

artmedia70:
У вас значение Profit участвует в торговых приказах?

Да конечно. Причем без ошибок :-). Я поэтому и удивился.

У меня Build 735

Торговля идет на демке.

Аааааааа всё понял ! Я забыл операцию присваивания :-). "Profit=NormalizeDouble(Profit,Digits)".

Правда все равно не понятно почему советник не выдает ошибок ))

Может на сервере автоматом скругляют.....

Мне почему то казалось что NormalizeDouble это процедура а не функция :-).

Групповые операции - Торговля Немедленное исполнение - Открытие Немедленное исполнение - Открытие

Boris 2014.10.30 06:17 #7576

Ozero:

Перес ";" за скобку.

Программа работает, НО периодически ругается на ошибку 4108.

Ozero.

Вообще, удалить надо!

Artyom Trishkin 2014.10.30 06:35 #7577

borilunad:

Вообще, удалить надо!

Борис, лучше б ты ему прямо кодом показал, а то я с недосыпу забыл... ;)

ResetLastError();
if(!OrderClose(OrderTicket(),OrderLots(),_Bid,15)) Print("Чё-та не закрылася позиция. Фигня вот такая происходит: "+GetLastError());

Boris 2014.10.30 07:05 #7578

artmedia70:

Борис, лучше б ты ему прямо кодом показал, а то я с недосыпу забыл... ;)

Не хотел злоупотреблять твоим авторским правом! ;)

А серьёзно, что-то не вставлялось! Но сейчас попробую!

ResetLastError();
if(!OrderClose(OrderTicket(),OrderLots(),_Bid,15)) Print("");

А сейчас получилось! Странно!

visnu 2014.10.30 11:35 #7579

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

Подскажите пожалуйста решение.

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

Но в новой версии выдает ошибки. Вот такие:

bool TP?=true;   // ошибка  '?' - semicolon expected

а вот про эту аж дважды

if(TP?)    // ')' - unexpected token    
           // ')' - ':' colon sign expected

Я посмотрел по справочнику по новой версии и по старой, но про вопросительный знак ничего похожего не нашел.

Вопрос: что значат эти операции и как их поправить, чтобы код нормально компилировался?

Заранее спасибо

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

[Удален] 2014.10.30 14:26 #7580

Здравствуйте! Подскажите пожалуйста, почему у меня иногда не срабатывает оператор if esle.
Вот пример:

double H3=ObjectGet ("H3 line", OBJPROP_PRICE1);
int start()
{
double price = Bid;
if (price==H3)
{
 Alert ("Урааааа  ", Symbol());
}
return(0);
}

Где:
H3 - это ценовое значение уровня
price - текущая цена

В итоге выходит, что цена доходит до этого уровня (иногда даже останавливается на нем), а Алерт не срабатывает.
Подскажите в чем ошибка?

Моментальные уведомления (алерты, Alert) Relative Strength Index - Relative Strength Index -

Любой вопрос новичка, чтоб не захламлять форум. Профи, не проходите мимо. Без вас никуда - 6. - страница 758