hm, I forgot to mention another suggestion:
Re-Entries are nice to boost profits - but the problem is they can boost your losses the same way 
In most of my eas I am using the following features now:
- check if there are at least int MinBarsBetweenTradesOfSameType = 4; (0:disable // could be implemented as external input) bars of the current TF gone since a) the last orderclose and b) the last entry (for orders of the same ordertype only!)
this is the purpose of the function bool OCTimeLastOrderSameTypeOk
- check if the open trades (of the same ordertype) are at least at BreakEven (BE). It is no guarantee for profits, but imho it does not hurt. the function bool SLLastOrderSameTypeOk should do this (requires the (external) variable bool NewOrderOnlyIfOldIsAtBE=TRUE;)
how to implement these two functions:
insert this:
Code: Select all if (!OCTimeLastOrderSameTypeOk(Symbol(),type,MagicNumber)) SendTrade=FALSE;
if (NewOrderOnlyIfOldIsAtBE && !SLLastOrderSameTypeOk(Symbol(),type,MagicNumber)) SendTrade=FALSE;
Code: Select all if (SendTrade)
{
if (!SendAlertNotTrade)
{
result = SendSingleTrade(Symbol(), type, TradeComment, SendLots, price, stop, take);
Code: Select all //check if bar difference between current time and orderopentime/orderclosetime is >= MinBarsBetweenTradesOfSameType
bool OCTimeLastOrderSameTypeOk(string symbol,int type,int magicnumber) {//fxdaytrader
if (MinBarsBetweenTradesOfSameType==0) return(true);
//Open Trades:
for (int cnt=OrdersTotal()-1; cnt>=0; cnt--) {
if (!OrderSelect(cnt,SELECT_BY_POS,MODE_TRADES)) continue;
if (OrderSymbol()!=symbol) continue;
if (OrderMagicNumber()!=magicnumber) continue;
if (OrderType()!=type) continue;
{
if (OrderOpenTime()>=iTime(OrderSymbol(),0,0)) return(false);
if (TimeCurrent() - OrderOpenTime() < (MinBarsBetweenTradesOfSameType * Period() * 60)) return(false);
}
}//for (int cnt=OrdersTotal()-1; cnt>=0; cnt--) {
//History
for (cnt=OrdersHistoryTotal()-1; cnt>=0; cnt--) {
if (!OrderSelect(cnt,SELECT_BY_POS,MODE_HISTORY)) continue;
if (OrderSymbol()!=symbol) continue;
if (OrderMagicNumber()!=magicnumber) continue;
if (OrderType()!=type) continue;
{
if (TimeCurrent() - OrderCloseTime() < (MinBarsBetweenTradesOfSameType * Period() * 60)) return(false);
}
}//for (cnt=OrdersHistoryTotal()-1; cnt>=0; cnt--) {
return(true);
}
//checks if all open orders are at least at BE
bool SLLastOrderSameTypeOk(string symbol,int type,int magicnumber) {//fxdaytrader
for (int cnt=OrdersTotal()-1; cnt>=0; cnt--) {
if (!OrderSelect(cnt,SELECT_BY_POS,MODE_TRADES)) continue;
if (OrderSymbol()!=symbol) continue;
if (OrderMagicNumber()!=magicnumber) continue;
{
if (OrderType()==type && type==OP_BUY) {
if (OrderStopLoss()==0||(OrderStopLoss()<OrderOpenPrice())) return(false);
}//if (OrderType()==type && type==OP_BUY) {
if (OrderType()==type && type==OP_SELL) {
if (OrderStopLoss()==0||(OrderStopLoss()>OrderOpenPrice())) return(false);
}//if (OrderType()==type && type==OP_SELL) {
}
}//for (int cnt=OrdersTotal()-1; cnt>=0; cnt--) {
return(true);
}
 |
