Tema: First EA Version1 Near Completion- Need Help

y puede pasar algún tiempo arreglando sus códigos con las sangrías apropiadas. los códigos dentro del mismo bloque deben alinearse. los bloques dentro de otro bloque deben sangrar. sus códigos no son los peores que he leído, pero definitivamente hay margen de mejora en lo que respecta a la legibilidad. puedes usar el indicador que publiqué como guía. saludos, Zen

Iniciado por ;

Ok, aquí está la actualización ... Cuando uso el bucle for, continuamente da mensajes de error 138 sin pedidos abiertos

He modificado el código en mi publicación anterior. Revísalo de nuevo.

1 Adjunto (s) Ok, aquí está la actualización ... Cuando uso el bucle for, continuamente da mensajes de error 138 sin órdenes abiertas. Las órdenes de compra funcionaban correctamente (no abrían las 4 al mismo tiempo) excepto no está restableciendo las órdenes de línea en TP Las órdenes de venta estaban abriendo las 4 al mismo tiempo ... así que copié la lógica de compra y la reinicié para vender lógica para ver si eso solucionaría el problema. Además, ¿por qué no es tp? ¿aleatorio? Siempre es la cantidad de ganancia mínima en lugar de un número aleatorio entre los dos rangos
https://www.forosforex.com/attachmen...5811958515.mq4

puede abordar su ur # 1 mediante el uso de esta función Inserted Code void CheckAndReset () {if (line1ticket gt; 0) {if (! OrderSelect (line1ticket, SELECT_BY_TICKET, MODE_TRADES)) {line1ticket = 0; line1trades = 0; }} if (line1ticket gt; 0) {if (! OrderSelect (line2ticket, SELECT_BY_TICKET, MODE_TRADES)) {line2ticket = 0; line2trades = 0; }} if (line3ticket gt; 0) {if (! OrderSelect (line3ticket, SELECT_BY_TICKET, MODE_TRADES)) {line3ticket = 0; line3trades = 0; }} if (line4ticket gt; 0) {if (! OrderSelect (line4ticket, SELECT_BY_TICKET, MODE_TRADES)) {line4ticket = 0; line4trades = 0; }}} respeta, Zen p.s., sólo un pensamiento, esta función habría sido 1/4 de su tamaño actual si hubiera utilizado matrices para almacenar y gestionar los valores de línea # ticket y line # comercios.

No tiene nada que ver con su problema, pero creo que necesita volver a examinar su función CloseMarketOrders () porque lógicamente ... No creo que funcione. Recuerde que el índice posicional de las operaciones en su pool comercial es muy parecido a un índice de matriz con la primera posición comenzando desde 0. decir que tiene 10 órdenes. su índice de posición va a ser de 0 a 9. OrdersTotal () le da un número 10 porque en realidad hay 10 intercambios en su pool de comercio. Pero si arrojas ese número directamente en OrderSelect, te dirá que no hay tal orden o matriz fuera de límite limitado. Incluso por alguna casualidad continúa, tienes tu tiempo (cntgt; 0) en el camino. eso significa que si solo hay 1 orden o queda con 1 orden, que está en la posición 0, no se procesará. ya que no encaja en tu condición. Entonces ... regla general cuando se trata de hacer bucles hacia atrás, use estos en su lugar. Código insertado cnt = OrdersTotal () - 1; while (cnt gt; = 0) {//blah blah blah ... cnt--; } alternativamente, puedes usar un ciclo for, que es lo que prefiero. Código insertado int numberOfOrders = OrdersTotal (); if (numberOfOrders gt; 0) {for (int cnt = numberOfOrders-1; cnt gt; = 0; cnt--) {//blah blah blah ...}} respetos, Zen

Veo lo que está haciendo ahora ... Estas dos cosas necesitan corrección: # 1 Restablecer los pedidos de línea a 0 cuando ocurre TP (ahora mismo se cierra después de una operación) # 2 Hay algunos errores que están abriendo los 4 ordena al mismo tiempo en lugar de seguir las reglas ... solo debe abrir órdenes en línea cruces con la configuración adecuada ... Creo que esto tiene algo que ver con la necesidad de contar barras o algo así, pero no estoy seguro

3 Adjunto (s) Gracias ... Creo que solucioné ese problema ahora ... pero ahora no estoy siguiendo las reglas y abriendo múltiples órdenes por línea. Accidentalmente tuve esto en tres gráficos en mi demo, y se abrió un número de órdenes ... es bueno hacer 10% en una cuenta demo de 10k en segundos, pero no de la manera en que estaba destinado a funcionar :-) ¿Qué debo hacer para tener solo 1 línea por operación? ... volver a adjuntar e incluir también los indicadores Editar: se corrigió el error de aperturas múltiples .... para el código de línea2 ... ahora el probador de eegia solo está abriendo 4 líneas (todas las ganancias, extracción mínima que es el propósito de este EA). ... de todos modos, cuando la tp golpea, no está restableciendo esa línea a 0 ahora, por lo tanto, no abrirá más intercambios ... volverá a adjuntar la versión más actualizada de EA
https://www.forosforex.com/attachmen...0535111063.mq4
https://www.forosforex.com/attachmen...1923463858.mq4
https://www.forosforex.com/attachmen...1005853592.mq4

No he tenido la oportunidad de ver su código, pero los errores son los siguientes: Error 131: es un volumen comercial incorrecto, por lo que debe asegurarse de que este valor no sea menor o mayor de lo que permite su intermediario. 4051 error: es un valor de parámetro de función inválido, verifique sus funciones (probablemente las personalizadas que ha escrito) y asegúrese de que tengan los valores de parámetros correctos ... paréntesis, valores dentro del paréntesis, etc.