Jump to content

Cable Run from Profitable FX


Recommended Posts

that meant your broker never allow you not to set tp & sl ...

do your own coding pls.

 

Thank you Muruku... i have got this error 2010.09.22 17:02:21 '2006005': order sell limit 0.10 GBPUSD opening at 1.56702 sl: 0.00000 tp: 0.00000 failed [invalid S/L or T/P]

 

2010.09.22 16:46:35 '2006005': order sell limit 0.10 GBPUSD opening at 1.56805 sl: 0.00000 tp: 0.00000 failed [invalid S/L or T/P]

 

 

what can i do to solve it ?

 

and can you please if you have the newest version with 5digits broker can you share ?

Thank you

Link to comment
Share on other sites

  • Replies 921
  • Created
  • Last Reply

Top Posters In This Topic

Hi,

 

After testing some ideas, seems a dynamic Step and Dynamic Profit becomes real good in real market.

 

The ideas behind this is to have different levels for entry orders, for instance on may 2010 we have like 250 pips daily average, so if we stick with our 17 step level, maybe no good for market conditions, and on April we have 100 pips, so dynamic i think is a important key rule here.

 

Define the Step for the ATRpips value of daily or even weekly data.

 

Define the Profit based on the Steps, if we have a step of 30 we don´t want 5 pips, we want more to get the possible range.

-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

 

Instead of running multiple instances of the EA you can code inside the EA for Profitable Hours, Normal Hours, etc...

 

For instance if(Profitable_Hours) Lots = ProfitableLots ;

 

--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Regarding the magic number, even if you put different one the broker can cheat you, the key is the comment, in earlier post i put the code for the comment, i have update it in order to be more easy so people can put it on the codes :

 

Replace in the order send function the comment by this

 

Comment_Code()

 

Put this code at the end of the EA

string Comment_Code() {

 

string Text[550];

 

Text[0]="abc1"; Text[1]="def2 "; Text[2]="ghi3 "; Text[3]="jkl4 ";

Text[4]="mno5 "; Text[5]="bqr6 "; Text[6]="stu7 ";

Text[7]="vwx8 "; Text[8]="yz90 "; Text[9]="nine "; Text[10]="ten0 ";

 

 

Text[11]="eleven "; Text[12]="twelve "; Text[13]="thirteen "; Text[14]="fourteen ";

Text[15]="fifteen "; Text[16]="sixteen "; Text[17]="seventeen "; Text[18]="eighteen "; Text[19]="nineteen ";

 

 

Text[20]="twenty " ; Text[21]="twenty1 "; Text[22]="twenty2 "; Text[23]="twenty3 "; Text[24]="twenty4 ";

Text[25]="twenty5 " ; Text[26]="twenty6 "; Text[27]="twenty7 "; Text[28]="twenty8 "; Text[29]="twenty9 ";

 

 

Text[30]="thirty "; Text[31]="thirty1 "; Text[32]="thirty2"; Text[33]="thirty3"; Text[34]="thirty4 ";

Text[35]="thirty5 "; Text[36]="thirty6 "; Text[37]="thirty7 "; Text[38]="thirty8 "; Text[39]="thirty9 ";

 

 

Text[40]="forty "; Text[41]="forty1 "; Text[42]="forty2 "; Text[43]="forty3 "; Text[44]="forty4 ";

Text[45]="forty5 "; Text[46]="forty6 "; Text[47]="forty7 "; Text[48]="forty8 "; Text[49]="forty9 ";

 

 

 

Text[50]="fifty "; Text[100]= "hundred"; Text[150]="hundred&fifty"; Text[200]="2hundred ";

Text[250]="2hundred&fifty "; Text[300]="3hundred"; Text[350]="3hundred&fifty";

Text[400]="4hundred "; Text[450]="4hundred&fifty "; Text[500]="5hundred";

for (int i=500; i>=50 ; i=i-50)

{

for (int j=49;j>=0;j--)

{

if (Open[j] == Close[j] )

break;

}

}

Text[i+j]=Text + Text[j];

 

return(Text[i+j]);

 

}

 

Hope this helps in this task

 

regards,

eggzactly

Edited by eggzactly
Link to comment
Share on other sites

Hi,

 

Seems can optimize the EA to takeprofit with other option like single trade profit, by usd, by total profit in pips, etc....

This one i tested and seems works great, can choose different ways to show profit :

 

enjoy

 

 

double TotalProfit(int pmode, int magic)

{

int total = OrdersTotal();

double Pips = 0, profit=0, totalPips = 0, totalProfits = 0;

 

for (int cnt=0;cnt<total;cnt++)

{

OrderSelect(cnt, SELECT_BY_POS);

int mode=OrderType();

bool condition = false;

if ( magic>0 && OrderMagicNumber()==magic && OrderSymbol() == Symbol()) condition = true;

else if ( magic==0 ) condition = true;

if (condition)

{

switch (mode)

{

case OP_BUY:

Pips = (MarketInfo(Symbol(),MODE_BID)-OrderOpenPrice())/_Point;

profit = OrderProfit() + OrderSwap() + OrderCommission();

totalPips += Pips;

totalProfits += profit;

break;

 

case OP_SELL:

Pips = (OrderOpenPrice()-MarketInfo(Symbol(),MODE_ASK))/_Point;

profit = OrderProfit() + OrderSwap() + OrderCommission();

totalPips += Pips;

totalProfits += profit;

break;

}

}

}

if(pmode==0) return(Pips);

else

if(pmode==1) return(profit);

else

if(pmode==2) return(totalPips);

else

if(pmode==3) return(totalProfits);

}

 

regards,

 

eggzactly

Link to comment
Share on other sites

Ok guys, thanks to all the work you put in to this.

But I don't understand a thing about all these codes you post.

I realy want this ea with everything on it but you will not post it so

you leave me no other choise to begin studying MQL language.

Are there any good sites for start learning MQL?

Thanks; Stijn.

Link to comment
Share on other sites

It would be lovely if someone can assist us who don't have the understanding and post the EA with modifications and broker protection.

I am assuming that modifications are made with the Meta Editor but where coding is placed is a mystery. If you feel that we are not worthy then please point us in the direction where we can learn without being so forward requesting assistance.

Edited by prattpot
Link to comment
Share on other sites

eggzactly,

i need a pcs of logic to delete duplication pending orders (entry with same pending buy/sell price) if misfired by EA or broker server turned up to be hair-wired a bit...

 

I placed a logic to prevent this to be happened as below, but i need a backup logic to wipe up unwanted AFTER if the mentioned duplicated entries HAD happened.

 

if ((gd_124 - Bid <= 5.0 * Point) && (LastSellPrice != gd_124)) { NewOrder(OP_SELLLIMIT, g_ord_lots_108 + IncLot,NextSLStep,0,0); LastSellPrice = gd_124;}

if ((Ask - gd_132 <= 5.0 * Point) && (LastBuyPrice != gd_132)) { NewOrder(OP_BUYLIMIT, g_ord_lots_116 + IncLot,NextBLStep,0,0); LastBuyPrice = gd_132;}

Link to comment
Share on other sites

eggzactly,

i need a pcs of logic to delete duplication pending orders (entry with same pending buy/sell price) if misfired by EA or broker server turned up to be hair-wired a bit...

 

I placed a logic to prevent this to be happened as below, but i need a backup logic to wipe up unwanted AFTER if the mentioned duplicated entries HAD happened.

 

if ((gd_124 - Bid <= 5.0 * Point) && (LastSellPrice != gd_124)) { NewOrder(OP_SELLLIMIT, g_ord_lots_108 + IncLot,NextSLStep,0,0); LastSellPrice = gd_124;}

if ((Ask - gd_132 <= 5.0 * Point) && (LastBuyPrice != gd_132)) { NewOrder(OP_BUYLIMIT, g_ord_lots_116 + IncLot,NextBLStep,0,0); LastBuyPrice = gd_132;}

 

Hi,

 

Maybe this works, need more testing :

 

Check Limit Orders

// --- Check Limit Positions

void CheckLimitPositions()

{

SelllimitOrderTicket=0;

BuylimitOrderTicket=0;

SelllimitOrders=0;

BuylimitOrders=0;

 

LastSellPrice = FindLastSellPrice(magic);

LastBuyPrice = FindLastBuyPrice (magic);

 

for (int cnt = OrdersTotal() - 1; cnt >= 0; cnt--)

{

if (OrderSelect(cnt, SELECT_BY_POS,MODE_TRADES))

{

if (OrderSymbol() == Symbol() && OrderMagicNumber() == magic)

{

switch (OrderType())

{

case OP_SELLLIMIT:

SelllimitOrders++;

SelllimitOrderTicket = OrderTicket();

SelllimitOrderOpenPrice = OrderOpenPrice();

 

if(HasOpenPosition && Trades <= MaxCountOrders)

{

if ( SelllimitOrderOpenPrice != LastSellPrice )

{

int tmp1= OrderModify(SelllimitOrderTicket,SelllimitOrderOpenPrice + StepLevels * _Point,0,0,0,CLR_NONE);

if (tmp1<0) Print("Modify SELLLimit: " + GetLastError());

}

}

break;

 

case OP_BUYLIMIT:

BuylimitOrders++;

BuylimitOrderTicket = OrderTicket();

BuylimitOrderOpenPrice = OrderOpenPrice();

 

if(HasOpenPosition && Trades <= MaxCountOrders)

{

if ( BuylimitOrderOpenPrice != LastBuyPrice )

{

int tmp3= OrderModify(BuylimitOrderTicket,BuylimitOrderOpenPrice - StepLevels * _Point,0,0,0,CLR_NONE);

if (tmp3<0) Print("Modify BuyLimit: " + GetLastError());

}

}

}//end switch

}//end if order symbol

}//end if select orders

}// end for

}//end main

 

 

Check Lot

Seems This Works

 

I have in my EA 2 levels of trading, this is :

 

5 Trades with Step - First Lot = 1stLevelLots

3 Trades with Step * 3 - IncLot = 2stLevelLots_x - this is the multiply of FirstLots

 

If Lot is not correct, then Delete the Limit Order

 

// --- Lot Filter Delete Limit Order

void LotFilterOnDeleteLimit()

{

double newlot;

for (int i = OrdersTotal() - 1; i >= 0; i--) {

OrderSelect(i, SELECT_BY_POS, MODE_TRADES);

if (OrderSymbol() == Symbol() && OrderMagicNumber() == magic && OrderCloseTime() == 0) {

if (OrderType() == OP_BUYLIMIT)

{

if(Trades <= _1stLevel_MaxOrders)

{

newlot = _1stLevelLots;

if (newlot != _1stLevelLots) OrderDelete (OrderTicket());

}

else

if(Trades > _1stLevel_MaxOrders && Trades <= _2stLevel_MaxOrders)

{

newlot = _1stLevelLots * _2stLevelLots_X;

if (newlot == _1stLevelLots) OrderDelete (OrderTicket());

}

}

if (OrderType() == OP_SELLLIMIT)

{

if(Trades <= _1stLevel_MaxOrders)

{

newlot = _1stLevelLots;

if (newlot != _1stLevelLots) OrderDelete (OrderTicket());

}

else

if(Trades > _1stLevel_MaxOrders && Trades <= _2stLevel_MaxOrders)

{

newlot = _1stLevelLots * _2stLevelLots_X;

if (newlot == _1stLevelLots) OrderDelete (OrderTicket());

}

}

}

}

}

 

regards,

 

eggzactly

Edited by eggzactly
Need Further Test Or Improvements for Some GURU Coder Please
Link to comment
Share on other sites

Hi muruku,

 

I have tried your function

 

double Pips = TotalProfit(Magic);

if (Pips >= MinProfit){

CloseAllTrades(0,Magic);

DeletePendingOrders(Magic);

}

 

which called the function below.

 

int TotalProfit(int Magic)

{

int total=OrdersTotal();

int totalPips = 0;

for (int cnt=0;cnt<total;cnt++)

{

OrderSelect(cnt, SELECT_BY_POS);

int mode=OrderType();

bool condition = false;

if (OrderMagicNumber()== Magic)condition = true;

if (condition)

{

switch (mode)

{

case OP_BUY:

totalPips += MathRound((MarketInfo(OrderSymbol(),MODE_BID)-OrderOpenPrice())/MarketInfo(OrderSymbol(),MODE_POINT));

break;

 

case OP_SELL:

totalPips += MathRound((OrderOpenPrice()-MarketInfo(OrderSymbol(),MODE_ASK))/MarketInfo(OrderSymbol(),MODE_POINT));

break;

}

}

}

return(totalPips);

}

 

I observe my account which is floating more than $30 in profits, but the EA still did not closed all the trades?

 

By the way, my broker is a 5 digit no, hence, I change the value for step and MinProfit to be 170 and 150

 

int Step = 170; // org 17 .. still stick to 17

extern double FirstLot = 0.01; // org 0.10 .. risky .. so just put 0.1

double IncLot = 0.0; // org 0.0

double MinProfit = 150; // org 15

 

Would that be the cause to all orders not closed?

Link to comment
Share on other sites

HI EGGZACTLY,

 

this is my idea for dynamics STEP

string ATRCONTROL = "=======FIBO/ATR PIP CONTROL Entry=======";

 

bool VolatilityBasedSTEP= FALSE;

double StepFactor = 0.8;

double ATRIncreaseDecreaseFactor = 1.0;

int ATR_Period = 20;

int ATR_Timeframe = 240;

 

 

 

start()

 

double ld_0 = iATR(NULL, ATR_Timeframe, ATR_Period2, 0);

if (Poin == 0.01) ld_0 = ATRIncreaseDecreaseFactor * (100.0 * ld_0);

if (Poin == 0.0001) ld_0 = ATRIncreaseDecreaseFactor * (10000.0 * ld_0);

if (VolatilityBasedSTEP) {

Step= NormalizeDouble(ld_0 * StepFactor, 0);

ratio = NormalizeDouble(1, 0);

}

 

what do you think ?

Link to comment
Share on other sites

HI EGGZACTLY,

 

this is my idea for dynamics STEP

 

 

what do you think ?

 

Maybe one idea,i am testing a more simple one, just dividing the ATR from Daily, and then i get the step.

 

Other possible idea is using fibo from last swing high/low, calculating from a non-repainting ZigZag - other is BB Range, there is too many ideas, but for now i just want to compile the system, have more then 1000 code lines, and need to be optimized and correct some bugs, then i can test more ideas regarding the dynamic levels and profit.

 

Is important check the maximum trades too, and that can be dynamic too, due to the volatility of market - for now i´m using 5 trades with step level distance, then 3 trades with 3 X the step level distance. It could handle the major drops and almost catch the tops and bottoms of sudden news and breakout of market.

 

regards,

 

eggzactly

Link to comment
Share on other sites

Hi muruku,

 

I have tried your function

 

double Pips = TotalProfit(Magic);

if (Pips >= MinProfit){

CloseAllTrades(0,Magic);

DeletePendingOrders(Magic);

}

 

which called the function below.

 

int TotalProfit(int Magic)

{

int total=OrdersTotal();

int totalPips = 0;

for (int cnt=0;cnt<total;cnt++)

{

OrderSelect(cnt, SELECT_BY_POS);

int mode=OrderType();

bool condition = false;

if (OrderMagicNumber()== Magic)condition = true;

if (condition)

{

switch (mode)

{

case OP_BUY:

totalPips += MathRound((MarketInfo(OrderSymbol(),MODE_BID)-OrderOpenPrice())/MarketInfo(OrderSymbol(),MODE_POINT));

break;

 

case OP_SELL:

totalPips += MathRound((OrderOpenPrice()-MarketInfo(OrderSymbol(),MODE_ASK))/MarketInfo(OrderSymbol(),MODE_POINT));

break;

}

}

}

return(totalPips);

}

 

I observe my account which is floating more than $30 in profits, but the EA still did not closed all the trades?

 

By the way, my broker is a 5 digit no, hence, I change the value for step and MinProfit to be 170 and 150

 

int Step = 170; // org 17 .. still stick to 17

extern double FirstLot = 0.01; // org 0.10 .. risky .. so just put 0.1

double IncLot = 0.0; // org 0.0

double MinProfit = 150; // org 15

 

Would that be the cause to all orders not closed?

 

 

Hi eggzactly,

 

Do u think u can help with this?

Link to comment
Share on other sites

Hi eggzactly,

 

Do u think u can help with this?

 

I have post earlier a function for calculating the profit wich working for me with no problems, and i check with real and backtest and at predefined level it closes the trades.

for not put again, just please do a search in my previous post and you can use that function.

 

#post 204

 

regards,

Link to comment
Share on other sites

 

Is important check the maximum trades too, and that can be dynamic too, due to the volatility of market - for now i´m using 5 trades with step level distance, then 3 trades with 3 X the step level distance. It could handle the major drops and almost catch the tops and bottoms of sudden news and breakout of market.

 

could you share the code ?

Link to comment
Share on other sites

could you share the code ?

Global Variables

 

extern string M3 = " Trade Levels & Lots";

extern int StepLevels = 17; // Step Amount Levels Dynamic

extern string Level1 = " 1st Level Options";

extern double _1stLevelLots = 0.1; // 1st Level Volume Multiply

extern int _1stLevel_MaxOrders = 5; // Max Orders Number

extern string Level2 = " 2st Level Options";

extern double _2stLevelLots_X = 5.0; // 2st Level Volume Multiply

extern int _2stLevel_MaxOrders = 10; // Max Orders Number

extern double _2stLevel_Distance_X = 3.0; //Multiply Level from 1st Level Steps

 

void DoTrading ()

{

RefreshRates();

// --- 1st Trades

if (timecond && Trades == 0 && ATR_Signal() == 1)

{

double Ask_ = Ask;

double Bid_ = Bid;

 

OpenPendingOrder(0, _1stLevelLots , Ask_ - StepLevels * _Point, Slippage, Ask_ - StepLevels * _Point, StopLoss, 0, Comment_Code(), magic, 0, CLR_NONE);

OpenPendingOrder(1, _1stLevelLots , Bid_ + StepLevels * _Point, Slippage, Bid_ + StepLevels * _Point, StopLoss, 0, Comment_Code(), magic, 0, CLR_NONE);

}

 

// --- 2st Trades

if (Trades > 0 && Trades <= _1stLevel_MaxOrders && timecond || !timecond )

{

LastSellPrice = FindLastSellPrice(magic);

LastBuyPrice = FindLastBuyPrice (magic);

 

if (Ask - LastBuyPrice <= 5.0 * Point )

{

OpenPendingOrder(0, _1stLevelLots, LastBuyPrice - StepLevels * _Point, Slippage, LastBuyPrice - StepLevels * _Point, StopLoss, 0, Comment_Code(), magic, 0, CLR_NONE);

}

if (LastSellPrice - Bid <= 5.0 * Point )

{

OpenPendingOrder(1, _1stLevelLots, LastSellPrice + StepLevels * _Point, Slippage, LastSellPrice + StepLevels * _Point, StopLoss, 0, Comment_Code(), magic, 0, CLR_NONE);

}

}

 

// --- 3st Trades

if (Trades > _1stLevel_MaxOrders && Trades <= _2stLevel_MaxOrders && timecond || !timecond )

{

LastSellPrice = FindLastSellPrice(magic);

LastBuyPrice = FindLastBuyPrice (magic);

 

if (Ask - LastBuyPrice <= 5.0 * Point )

{

OpenPendingOrder(0, _2stLevelLots_X * _1stLevelLots, LastBuyPrice - (StepLevels * _2stLevel_Distance_X) * _Point, Slippage, LastBuyPrice - (StepLevels * _2stLevel_Distance_X) * _Point, StopLoss, 0, Comment_Code(), magic, 0, CLR_NONE);

}

if (LastSellPrice - Bid <= 5.0 * Point )

{

OpenPendingOrder(1, _2stLevelLots_X * _1stLevelLots, LastSellPrice + (StepLevels * _2stLevel_Distance_X) * _Point, Slippage, LastSellPrice + (StepLevels * _2stLevel_Distance_X) * _Point, StopLoss, 0, Comment_Code(), magic, 0, CLR_NONE);

}

}

 

 

}

 

OpenPendingOrder function

 

// --- Open Pending Order

int OpenPendingOrder(int order_code, double Lots, double entry_price, int slippage, double price_stoploss, int stoploss, int takeprofit, string _comment, int magic, int expiration, color clr)

{

int order_ticket = 0;

int error_ = 0;

int cnt = 0;

int numords = 100;

 

switch (order_code) {

case 0:

for (cnt = 0; cnt < numords; cnt++) {

order_ticket = OrderSend(Symbol(), OP_BUYLIMIT, Lots, entry_price, slippage, StopLong(price_stoploss, stoploss), TakeLong(entry_price, takeprofit), _comment, magic, expiration, clr);

error_ = GetLastError();

if (error_ == 0/* NO_ERROR */) break;

if (!((error_ == 4/* SERVER_BUSY */ || error_ == 137/* BROKER_BUSY */ || error_ == 146/* TRADE_CONTEXT_BUSY */ || error_ == 136/* OFF_QUOTES */))) break;

Sleep(1000);

}

break;

case 1:

for (cnt = 0; cnt < numords; cnt++) {

order_ticket = OrderSend(Symbol(), OP_SELLLIMIT, Lots, entry_price, slippage, StopShort(price_stoploss, stoploss), TakeShort(entry_price, takeprofit), _comment, magic, expiration, clr);

error_ = GetLastError();

if (error_ == 0/* NO_ERROR */) break;

if (!((error_ == 4/* SERVER_BUSY */ || error_ == 137/* BROKER_BUSY */ || error_ == 146/* TRADE_CONTEXT_BUSY */ || error_ == 136/* OFF_QUOTES */))) break;

Sleep(1000);

}

}

return (order_ticket);

}

 

// --- Stop Long

double StopLong(double price, int StopLoss_Amount) {

if (StopLoss_Amount == 0) return (0);

return (price - StopLoss_Amount * _Point);

}

 

// --- Stop Short

double StopShort(double price, int StopLoss_Amount) {

if (StopLoss_Amount == 0) return (0);

return (price + StopLoss_Amount * _Point);

}

 

// --- TakeProfit Long

double TakeLong(double price, int StopLoss_Amount) {

if (StopLoss_Amount == 0) return (0);

return (price + StopLoss_Amount * _Point);

}

 

// --- TakeProfit Short

double TakeShort(double price, int StopLoss_Amount) {

if (StopLoss_Amount == 0) return (0);

return (price - StopLoss_Amount * _Point);

}

 

hope this help

 

regards,

Link to comment
Share on other sites

I have post earlier a function for calculating the profit wich working for me with no problems, and i check with real and backtest and at predefined level it closes the trades.

for not put again, just please do a search in my previous post and you can use that function.

 

#post 204

 

regards,

 

Hi eggzactly,

 

I have insert your codes inside the EA. Just want to check with you if I am correct in the EA variables declaration and the function

 

int pmode;

 

double Pips = TotalProfit(pmode,Magic);

if (Pips >= MinProfit){

CloseAllTrades(0,Magic);

DeletePendingOrders(Magic);

}

 

 

Do I need to put

 

CloseAllTrades(1, Magic);

CloseAllTrades(2, Magic);

 

Since this is declare on top inside the init start()

 

What does CloseAllTrades(1, Magic); , CloseAllTrades(2, Magic); and CloseAllTrades(0, Magic); means?

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.




×
×
  • Create New...