I'm about to give up on this language. Need help building correctly...
underworlddjb: I've racked my brain around this language. Nothing makes sense to me.
You are frustrated because you don't know what is going on. Find out.-
OrderSend(NULL,OP_BUY,Lots,Ask,3,SL,TP,"Buy",420,0,clrLime);
You would know why, if you Check your return codes for errors, and report them including GLE/LE. Don't look at it unless you have an error. Don't just silence the compiler, it is trying to help you.
What are Function return values ? How do I use them ? - MQL4 programming forum
Common Errors in MQL4 Programs and How to Avoid Them - MQL4 Articles - Don't use NULL.
- You can use NULL in place of _Symbol only in those calls that the documentation specially says you can. iHigh does, iCustom does, MarketInfo does not. OrderSend does not.
- Don't use NULL (except for pointers where you explicitly check for it.) Use _Symbol and _Period, that is minimalist as possible and more efficient.
- Zero is the same as PERIOD_CURRENT which means _Period. Don't hard code numbers.
- MT4: No need for a function call with iHigh(NULL,0,s) just use the predefined arrays, i.e. High[].
-
if((HMA_Fast<HMA_Slow)==true)
You should be able to read your code out loud and have it make sense. You would never write if( (2+2 == 4) == true) would you? if(2+2 == 4) is sufficient. So don't write if(bool == true), just use if(bool) or if(!bool). Code becomes self documenting when you use meaningful variable names, like bool isLongEnabled where as Long_Entry sounds like a trigger price or a ticket number and "if long entry" is an incomplete sentence. -
double TP=(Ask+ATR_TP*Point); double SL=(Ask-ATR_TP*Point); OrderSend(NULL,OP_SELL,Lots,Ask,3,SL,TP,"Sell",420,0,clrRed);
A SL must be above the market, TP below, for a sell order. - What is the purpose of ATR_SL?
- You buy at the Ask and sell at the Bid. So for buy orders you pay the spread on open. For sell orders you pay the
spread on close.
- Your buy order's TP/SL (or Sell Stop's/Sell Limit's entry) are triggered when the Bid reaches it. Not the Ask. Your SL is shorter by the spread and your TP is longer. Don't you want the same/specified amount for either direction?
- Your sell order's TP/SL (or Buy Stop's/Buy Limit's entry) will be triggered when the Ask reaches it. To
trigger at a specific
Bid price, add the average spread.
MODE_SPREAD (Paul) - MQL4 programming forum - Page 3 #25 - The charts show Bid prices only. Turn on the Ask line to see how big the spread is (Tools → Options (Control-O) → charts → Show ask line.)
- Never risk more than a small percentage of your account, certainly less than 2% per trade, 6% total.
-
In code (MT4): Risk depends
on your initial stop loss, lot size, and the value of the pair. It does not depend on margin and leverage.
- You place the stop where it needs to be - where the reason for the trade is no longer valid. E.g. trading a support bounce the stop goes below the support.
- AccountBalance * percent/100 = RISK = OrderLots * (|OrderOpenPrice - OrderStopLoss| * DeltaPerLot + CommissionPerLot) (Note OOP-OSL includes the spread, and DeltaPerLot is usually around $10/pip but it takes account of the exchange rates of the pair vs. your account currency.)
- Do NOT use TickValue by itself - DeltaPerLot
and verify that MODE_TICKVALUE is returning a value in your deposit currency, as promised by the documentation, or
whether it is returning a value in the instrument's base currency.
MODE_TICKVALUE is not reliable on non-fx instruments with many brokers - MQL4 programming forum 2017.10.10
Lot value calculation off by a factor of 100 - MQL5 programming forum 2019.07.19 - You must normalize lots properly and check against min and max.
- You must also check FreeMargin to avoid stop out
Most pairs are worth about $10 per PIP. A $5 risk with a (very small) 5 PIP SL is $5/$10/5 or 0.1 Lots maximum.
-
In code (MT4): Risk depends
on your initial stop loss, lot size, and the value of the pair. It does not depend on margin and leverage.
William Roeder:
You are frustrated because you don't know what is going on. Find out.
As far as GLE, I was going to work on that later. I'm just trying to get basic functionality at this point. I've been working on this for six months
and getting nowhere. Thanx for the NULL correction. I'll get that changed. This is all good information. I'll impliment changes.
You are frustrated because you don't know what is going on. Find out.
- You would know why, if you Check your return codes for errors, and report them including GLE/LE.
Don't look at it unless you have an error. Don't just silence the compiler, it is trying to help you.
What are Function return values ? How do I use them ? - MQL4 programming forum
Common Errors in MQL4 Programs and How to Avoid Them - MQL4 Articles - Don't use NULL.
- You can use NULL in place of _Symbol only in those calls that the documentation specially says you can. iHigh does, iCustom does, MarketInfo does not. OrderSend does not.
- Don't use NULL (except for pointers where you explicitly check for it.) Use _Symbol and _Period, that is minimalist as possible and more efficient.
- Zero is the same as PERIOD_CURRENT which means _Period. Don't hard code numbers.
- MT4: No need for a function call with iHigh(NULL,0,s) just use the predefined arrays, i.e. High[].
- You should be able to read your code out loud and have it make sense. You would never write if( (2+2 == 4) == true) would you? if(2+2 == 4) is sufficient. So don't write if(bool == true), just use if(bool) or if(!bool). Code becomes self documenting when you use meaningful variable names, like bool isLongEnabled where as Long_Entry sounds like a trigger price or a ticket number and "if long entry" is an incomplete sentence.
William Roeder:
What is the purpose of ATR_SL?
What is the purpose of ATR_SL?
It's a stoploss based on the ATR of the previous candle. Best used on the daily chart. It's also used as a TP with a scale out.
underworlddjb: It's a stoploss based on the ATR of the
previous candle. Best used on the daily chart. It's also used as a TP with a scale out.
double ATR_SL=iCustom(NULL,0,"ATR in SL-TP_2",14,1.5,1,0,0,1,0,1); double ATR_TP=iCustom(NULL,0,"ATR in SL-TP_2",14,1.5,1,0,0,1,1,1);
The variable is not used.
William Roeder:
It's used. It's in the "OnTick"
The variable is not used.
double TP=(Ask+ATR_TP*Point); double SL=(Ask-ATR_TP*Point);
As far as the SL, I wanted to calculate 1%-2% of my current account. As to scale up or down with the balance. The more profit the larger the lot size
and inverse. Did I not do that correctly? Where as the ATR was the determination of SL and TP.
double lots = AccountBalance()*(RiskPercent/100.0)/(StopLoss*pipValue);
It is advisable to use
double TP = (Ask + ATR_TP*Point); double SL = (Bid - ATR_TP*Point);
instead of
double TP = (Ask + ATR_TP*Point); double SL = (Ask - ATR_TP*Point);
After that, you need to check those TP and SL for minimum allowable pip by using MarketInfo(Symbol,
MODE_STOPLEVEL) or else you will face an error. I'm pretty sure you don't want that.
then, for.. ATR_TP and ATR_SL from iCustom i'm not sure what indicator you are using, but if it is this ATR, i don't think it will work.
If you are asking a question publicly, please make sure the information is complete.
If you are trying to hide something, try freelance instead.
underworlddjb: It's used. It's in the "OnTick"
double TP=(Ask+ATR_TP*Point); double SL=(Ask-ATR_TP*Point);
Look at your code, ATR_SL is not used.
Ahmad Zuhairdi Noh: It is advisable to use
No it is not.
You buy at the Ask and sell at
the
Bid. So for buy orders you pay the spread on open. For sell orders you pay the spread on close.
- Your buy order's TP/SL (or Sell Stop's/Sell Limit's entry) are triggered when the Bid reaches it. Not the Ask. Your SL is shorter by the spread and your TP is longer. Don't you want the same/specified amount for either direction?
- Your sell order's TP/SL (or Buy Stop's/Buy Limit's entry) will be triggered when the Ask reaches it. To
trigger at a specific
Bid price, add the average spread.
MODE_SPREAD (Paul) - MQL4 programming forum - Page 3 #25 - The charts show Bid prices only. Turn on the Ask line to see how big the spread is (Tools → Options (Control-O) → charts → Show ask line.)
William Roeder:
well, yes it is.. using this TP and SL for BUY trade (long);No it is not.
double TP = (Ask + ATR_TP*Point); double SL = (Bid - ATR_TP*Point);and this TP and SL for SELL trade (short);
double TP = (Bid - ATR_TP*Point); double SL = (Ask + ATR_TP*Point);
This is to prevent error 130.
You are missing trading opportunities:
- Free trading apps
- Over 8,000 signals for copying
- Economic news for exploring financial markets
Registration
Log in
You agree to website policy and terms of use
If you do not have an account, please register
I've racked my brain around this language. Nothing makes sense to me. All I want is to get in and out of trades based on a couple of indies. ANY help would be appreciated.