¿Qué diseño es el correcto? - página 5

Nuevo comentario
 
valenok2003:

El resultado es este script

Pregunta - ¿Por qué no se cierran siempre todos los pedidos? Por ejemplo, abro tres órdenes de venta una tras otra e intento cerrarlas, pero el script puede cerrar una o dos o todas. ¿Cuál es la razón?

¿Necesita un guión?

Entonces no necesitas una sentencia while, los scripts funcionan en bucles tal y como están.

Para el guión se necesita esta construcción...

No recuerdo el enlace, aquí está el script que cierra todas las órdenes abiertas.

//+------------------------------------------------------------------+
//|                                               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:

El resultado es este script

Pregunta - ¿Por qué no se cierran siempre todos los pedidos? Por ejemplo, si abro tres órdenes de venta seguidas e intento cerrarlas con el script, puede cerrar una, dos o todas. ¿Cuál es la razón?
¿Y las comillas cuántos dígitos tienen?
 
khorosh:
¿Y las comillas cuántos dígitos tienen?
cinco
 
Martingeil:

¿Necesita un guión?

Gracias, por supuesto. Pero es fácil conseguir un guión estándar. Quiero averiguar cuál es la razón.
 
Martingeil:

¿Necesita un guión?

entonces no necesitas la sentencia while, los scripts funcionan en bucles tal y como están.


El script sólo funciona una vez. Allí no hay ni ha habido nunca ningún tipo de bucle.

Valenok, ¿necesitas que el script cierre todas las órdenes o sólo este símbolo?

 
Martingeil:

¿Necesita un guión?

Entonces no necesitas la sentencia while, los scripts funcionan en bucles tal y como están.

Para un guión se necesita esta construcción...

No recuerdo el enlace, aquí hay un script que cierra todas las órdenes abiertas.

En el código while , sólo para retrasar mientras el hilo está ocupado.
 
Techno:

el script sólo funciona una vez. No hay autodisparo allí y nunca lo ha habido.

Valenok, ¿quieres que el script cierre todas las órdenes o sólo este símbolo?

Sí, tienes razón, me refería a que no depende de las garrapatas...
 
Techno:

el script sólo funciona una vez. Ahí no hay ni ha habido nunca autodisparo.

Valenok, ¿necesitas que el script cierre todas las órdenes, o sólo este símbolo?



Sólo se abren órdenes de un símbolo. Los pendientes no se ven afectados.

Lo curioso es que el código criticado en kodobase

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);
}

funciona sin problemas, pero cuando empecé a mejorarlo, empecé a tener problemas.

zy:

Sobre el tema de la cadena equivocada 

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

Ya me lo he imaginado.

 
valenok2003:



Sólo se abren órdenes de un símbolo. No tocamos los pendientes.

Así que, vamos a ir directamente a los aplazamientos y a tenerlo. En forma general.

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:
Tienes razón, no quería decir eso, independientemente de los tics...


Soy consciente de ello.
12345678
Nuevo comentario