MACD-dsy-2.6b1 EA- Problem | Forex Factory

Options

Permalinks
Print Thread

Similar Threads

Anyone Knows how to read this MACD? What type of MACD is this? 6 replies

Traditional MACD histograms vs MT4's MACD Coding 8 replies

Help with MACD & MA of MACD Cross indicator 0 replies

no MACD line on MT4 MACD indicator 1 reply

MACD / Indicator problem 4 replies

Attachments: MACD-dsy-2.6b1 EA- Problem
Exit Attachments

MACD-dsy-2.6b1 EA- Problem

  • #1
  • First Post: Edited 12:55am Jul 25, 2016 12:33am | Edited 12:55am
Hello

I have found this EA with very impressive results,
But it keeps logging "Order send error 130" when back testing, can some body help me fix this issue please
Attached File(s)
File Type: mq4 MACD-dsy-2.6b1.mq4   26 KB | 310 downloads
  • #2
  • Jul 25, 2016 2:10am Jul 25, 2016 2:10am
The stop loss value in the OrderSend() function is wrong. It uses the following:
Inserted Code
NormalizeDouble(DoubleToStr(50,0),Digits)

This results in an absolute value, not the stop loss relative to the ask or bid price, for buy and sell orders respectively.
(why they would use DoubleToStr() instead of just "50" here is also not clear to me, but that's not the issue.)

It could be fixed in the following way:

  1. for buys: [code]NormalizeDouble(Ask - 50 * Point, Digits)[/code]
  2. for sells: [code]NormalizeDouble(Bid + 50 * Point, Digits)[/code]


I've updated the EA to reflect this. Note that I've also seen some other issues with it, don't think you should trust this EA very much

Attached File(s)
File Type: mq4 MACD-dsy-2.6b1.mq4   26 KB | 219 downloads
  • #3
  • Last Post: Jul 25, 2016 6:33am Jul 25, 2016 6:33am
Quoting smartfxlearn
The stop loss value in the OrderSend() function is wrong. It uses the following: NormalizeDouble(DoubleToStr(50,0),Digits) This results in an absolute value, not the stop loss relative to the ask or bid price, for buy and sell orders respectively. (why they would use DoubleToStr() instead of just "50" here is also not clear to me, but that's not the issue.) It could be fixed in the following way: for buys: NormalizeDouble(Ask - 50 * Point, Digits) for sells: NormalizeDouble(Bid + 50 * Point, Digits) I've updated the EA to reflect this. Note that...
Thanks, now Order send error 4051,
I actually was looking for an EA with minimum "max consecutive losers", i think this EA is very good, if would work...
0 traders viewing now