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

[ArturR]

Всем привет!

Пол ночи просидел не могу понять почему не выполняется простое условие, помогите разобраться пожалуйста:

      macd1 = iMACD (_Symbol,PERIOD_CURRENT,5,34,5,0,1,0);
      macd2 = iMACD (_Symbol,PERIOD_CURRENT,5,34,5,0,1,1);
      highprice1 = High [0];
      highprice2 = High [1];
      lowprice1 = Low [0];
      lowprice2 = Low [1];
              if (macd1 > 0 && macd1 > macd2 && highprice1 > highprice2) 
                 {
                 maxprice1 = highprice1;
                 macdMAXSell = macd1;
                 Print("maxprice1 = highprice1",maxprice1);
                 }
              if (macd1 < 0 && macd1 < macd2 && lowprice1 < lowprice2)
                 {
                 minprice1 = lowprice1;
                 macdMAXBuy = macd1;
                 Print("minprice1 = lowprice1",minprice1);
          	 } 

запускаю на исторических данных, значения переменным macd1,2, lowprice, highprice присваиваются, а условие выполняться не хочет, всю голову сломал не могу понять куда копать... (((

[Artyom Trishkin]

Nikolay Gaylis:

может 1 и 1 ?/т.е. true и код ошибки 1

ERR_NO_RESULT

1

Нет ошибки, но результат неизвестен

Полный бардак и непонимание того, что делаете...

bool takelimit;   // Здесь переменная равна false

//--- Здесь вот что: сначала выбирается ордер по тикету (MODE_TRADES здесь не нужно от слова "совсем", что говорит о том, что вы не представляете что вы делаете), 
//--- далее переменной takelimit присваивается результат работы функции выбора ордера, 
//--- и если ордер есть с таким тикетом, или был когда-то, то функция вернёт true, и takelimit будет присвоено true, соответственно, будет выведен код последней ошибки,
//--- что вовсе не говорит, что ошибка здесь была при выборе ордера по тикету - вы же переменную, хранящую код последней ошибки, не сбросили при помощи ResetLastError()
//--- И далее вообще не понятно к чему у вас фигурные скобки - они висят в пустоте, а это составной оператор, который должен после условия идти.

if(takelimit=OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES))
   Print(string(takelimit)+"   :",GetLastError());//здесь возвращает число 11, получается ошибка код 11, но не нашел расшифровку нигде?
  {
   if(iBarShift(Symbol(),0,OrderOpenTime())==1)
      deletelimit=OrderDelete(ticket);
   if(!deletelimit)
      Comment(GetLastError());
  }
//+------------------------------------------------------------------+
//--- А нужно примерно так (фигурные скобки специально оставил для понимания где какой блок кода):
   ResetLastError();
   if(OrderSelect(ticket,SELECT_BY_TICKET) && OrderCloseTime()==0)
     {
      if(iBarShift(Symbol(),PERIOD_CURRENT,OrderOpenTime())==1)
        {
         if(!OrderDelete(ticket))
            Print("Ошибка удаления ордера #",(string)ticket,": ",GetLastError());
        }
     }
   else
     {
      Print("Ошибка выбора ордера #",(string)ticket,": ",GetLastError());
     }

[[Удален]]

Вадим Мотеюнас:

получается фраза Print(takelimit,GetLastError()) возвращает два значения так как 1-ая переменная типа bool и ей не нужен GetLastError, и т.к. есть запятая ,GetLastError возвращает последнюю ошибку в коде в момент вызова GetLastError?

Да..там два значения принтит.....

Вот почти тоже самое,что Вы написали

int ticket;

void Fn()
  {
   ResetLastError();//Сбрасываем код предыдущей ошибки
   if(!OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES)){Print(GetLastError());}//Если не выбран ордер по указанному тикету-принтим код ошибки
   else//иначе(если ордер нашёлся)
     {
      if(iBarShift(Symbol(),0,OrderOpenTime())==1 && !OrderDelete(ticket)){Comment(GetLastError());}//Если выбранный ордер открылся на предыдущем баре и не удалось его удалить(надеюсь Вы работаете с отложенниками)-тогда выводим комент ошибки
     }
  }

[Artyom Trishkin]

Nikolay Gaylis:

Да..там два значения принтит.....

Вот почти тоже самое,что Вы написали

И вы туда же...

А если ордер уже удалён, то что?

[[Удален]]

Artyom Trishkin:

Полный бардак и непонимание того, что делаете...

Я вообще-то ничего и не делаю,а предположил,что у человека принтит не 11 ,а 1 и1 )))

и всего-то...

[[Удален]]

Artyom Trishkin:

Полный бардак и непонимание того, что делаете...

А..так Вы не тому отвечаете...ясненько)

[Artyom Trishkin]

Nikolay Gaylis:

Я вообще-то ничего и не делаю,а предположил,что у человека принтит не 11 ,а 1 и1 )))

Как вот эта строка:

Print(string(takelimit)+"   :",GetLastError());

Может распечатать "11" ?

Она распечатает тогда уж "1   :1"

При этом код ошибки 1 - это нет ошибки, но результат неизвестен.

[[Удален]]

Artyom Trishkin:

Как вот эта строка:

Может распечатать "11" ?

Она распечатает тогда уж "1   :1"

При этом код ошибки 1 - это нет ошибки, но результат неизвестен.

Да это я её уже переделал!-Выше читайте...

[Artyom Trishkin]

Nikolay Gaylis:

А..так Вы не тому отвечаете...ясненько)

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

[[Удален]]

Artyom Trishkin:

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

MODE_TRADES ещё не смотрел у него-только тот кусок,что 11 принтил...