Какая конструкция правильная? - страница 5

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

В итоге получился такой скрипт

Вопрос - Почему он не всегда закрывает все ордера? Например я открываю три Sell подряд, потом пытаюсь закрыть скриптом, он может закрыть один или два или все. В чём причина?

Вам нужен скрипт?

тогда вам не нужен оператор цикла while, скрипты и так работают в зацикленном виде.

Для скрипта вот такая конструкция нужна...

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

//+------------------------------------------------------------------+
//|                                               CloseAllOrders.mq4 |
//|                       Copyright © 2008, PRMQuotes Software Corp. |
//|                                           Jedimedic77@gmail.com  |
//+------------------------------------------------------------------+
#property copyright "Copyright © 2008, PRMQuotes Software Corp."
#property link      ""
//+------------------------------------------------------------------+
//| EX4 imports                                                      |
//+------------------------------------------------------------------+
#include <stdlib.mqh>
//+------------------------------------------------------------------+
//| global variables to program:                                     |
//+------------------------------------------------------------------+
double Price[2];
int    giSlippage;
bool   CloseOpenOrders = true;
//+------------------------------------------------------------------+
//| script program start function                                    |
//+------------------------------------------------------------------+
void start() {
  int iOrders=OrdersTotal()-1, i;
  
  if(CloseOpenOrders) {
    for(i=iOrders; i>=0; i--) {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES) && (OrderType()<=OP_SELL) && GetMarketInfo() && 
      !OrderClose(OrderTicket(),OrderLots(),Price[1-OrderType()],giSlippage)) Print(OrderError());
    }
  }
}
//+------------------------------------------------------------------+
//| Function..: OrderError                                           |
//+------------------------------------------------------------------+
string OrderError() {
  int iError=GetLastError();
  return(StringConcatenate("Order:",OrderTicket()," GetLastError()=",iError," ",ErrorDescription(iError)));
}
//+------------------------------------------------------------------+
//| Function..: GetMarketInfo                                        |
//+------------------------------------------------------------------+
bool GetMarketInfo() {
  RefreshRates();
  Price[0]=MarketInfo(OrderSymbol(),MODE_ASK);
  Price[1]=MarketInfo(OrderSymbol(),MODE_BID);
  double dPoint=MarketInfo(OrderSymbol(),MODE_POINT);
  if(dPoint==0) return(false);
  giSlippage=(Price[0]-Price[1])/dPoint;
  return(Price[0]>0.0 && Price[1]>0.0);
}
//+------------------------------------------------------------------+
 
valenok2003:

В итоге получился такой скрипт

Вопрос - Почему он не всегда закрывает все ордера? Например я открываю три Sell подряд, потом пытаюсь закрыть скриптом, он может закрыть один или два или все. В чём причина?
А котировки сколько разрядов?
 
khorosh:
А котировки сколько разрядов?
пять
 
Martingeil:

Вам нужен скрипт?

Спасибо, конечно. Но взять готовый скрипт просто. Я же хочу понять - в чём причина.
 
Martingeil:

Вам нужен скрипт?

тогда вам не нужен оператор цикла while, скрипты и так работают в зацикленном виде.


скрипт работает только 1 раз. Никаких самозацикливаний там нет и никогда не было.

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

 
Martingeil:

Вам нужен скрипт?

тогда вам не нужен оператор цикла while, скрипты и так работают в зацикленном виде.

Для скрипта вот такая конструкция нужна...

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

В коде while только для задержки, пока поток занят.
 
Techno:

скрипт работает только 1 раз. Никаких самозацикливаний там нет и никогда не было.

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

Да вы правы не так выразился, хотел сказать не зависимо от тиков..
 
Techno:

скрипт работает только 1 раз. Никаких самозацикливаний там нет и никогда не было.

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



Только открытые ордера, одного символа. Отложенники не трогаем.

Прикол в том, что раскритикованный в кодобазе код

int start()
{
//----
  while(OrdersTotal()>0)
  {
    for (int i=0; i < OrdersTotal(); i++)                                                        
    {                                                                                          
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==true)
      {
        OrderClose(OrderTicket(),OrderLots(),OrderClosePrice(),5);  
      }
    }
  }                                               
//----
   return(0);
}

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

зы: 

На счёт ошибочности строки 

for (int i=0; i < OrdersTotal(); i++)

я уже всё понял

 
valenok2003:



Только открытые ордера, одного символа. Отложенники не трогаем.

Дак сразу с отложниками и пусть будет. В общем виде.

int start()
{
   for(int i=OrdersTotal()-1;i>=0;i--)
    {
    if(OrderSelect(i,SELECT_BY_POS)&&OrderSymbol()==Symbol())
      switch(OrderType())
         {
         case 0  : OrderClose(OrderTicket(),OrderLots(),NormalizeDouble(Bid,Digits),5); break;
         case 1  : OrderClose(OrderTicket(),OrderLots(),NormalizeDouble(Ask,Digits),5); break;
         default : OrderDelete(OrderTicket()); break;
         }
    }
        
}
 
Martingeil:
Да вы правы не так выразился, хотел сказать не зависимо от тиков..


это я в курсе
12345678
Новый комментарий
Причина обращения: