How do I get rid of TP option in EA? | Forex Factory

Options

Permalinks
Print Thread

Similar Threads

How to get rid of info box in MT4 ? 3 replies

How to get rid of HLOC values in charts 1 reply

How do i get rid of alerts? 3 replies

hwo to get rid of teh Buy line and Sell line 2 replies

How do I get rid of TP option in EA?

  • #1
  • First Post: Jun 20, 2014 10:57am Jun 20, 2014 10:57am
I have an EA that closes all trades when equity target is reached. It also has an option for hard TP for each trade. If I select "0" as the setting for TP value, the EA closes that trade as soon as it is opened. Can I just add this line to completely rid the TP option?

input bool UseTP = false;


Or do I have to dive into the modifying orders section of the code, too, and put a "if" command, too? having the EA have to modify the orders with TPs is taking too much time when it's opening the pending orders.
Too close for pips, switching to points.
  • #2
  • Jun 20, 2014 11:21am Jun 20, 2014 11:21am
No, you cannot just insert the line of code because the EA is not aware of such a switch (I persume).
You would also need to make the code that reacts to the switch.
Return To Disorder
  • #3
  • Jun 20, 2014 11:35am Jun 20, 2014 11:35am
darn....what if I delete the TP input setting line entirely, will that cause an error, too?

(sorry I have to ask, I don't have access to plateform where I am at, or I would just try it, trial and error.)
Too close for pips, switching to points.
  • #4
  • Jun 20, 2014 11:42am Jun 20, 2014 11:42am
Deleting the input will cause a compile error.
You can comment out the TP set code. But for that you'd need to understand the code logic (without seeing the part of code nobody can help you out with this either).
Return To Disorder
  • #5
  • Jun 20, 2014 11:45am Jun 20, 2014 11:45am
Does this help?

//+------------------------------------------------------------------+
//| Open Buy Pending Orders function |
//+------------------------------------------------------------------+
bool OpenBuyPendingOrder(double BOP)
{
int TicketNumberBuy;
double TProfit, SLoss;
double Spread = MarketInfo(Pair,MODE_SPREAD)*Point;
bool PlacedTrade = false;
bool ModifiedTrade = false;
double Free = AccountBalance();
double OneLot = MarketInfo(Pair,MODE_MARGINREQUIRED);
double Step = MarketInfo(Pair,MODE_LOTSTEP);

TicketNumberBuy = OrderSend(Pair,OP_BUYSTOP,LS,BOP,Slip,0,0,"Gold Mine Buy");

if(TicketNumberBuy == -1)
{
Alert("Buy Order failed, Error NO: "+GetLastError());
PlacedTrade = false;
}
else
{
PlacedTrade = true;
}
if(PlacedTrade)
{
if(OrderSelect(TicketNumberBuy,SELECT_BY_TICKET)==true)
{
TProfit = OrderOpenPrice() + TP*Point;
if(UseSL)
{
SLoss = OrderOpenPrice() - Spread - SL*Point;
}
else
{
SLoss = OrderStopLoss();
}
if(OrderModify(OrderTicket(),OrderOpenPrice(),SLoss,TProfit,0,0)==true)
{
ModifiedTrade = true;
}
else
Alert("Error modifying BuyTrade= "+GetLastError());
}
}
return(true);
}
Too close for pips, switching to points.
  • #6
  • Jun 20, 2014 11:53am Jun 20, 2014 11:53am
Yes.

Inserted Code
bool OpenBuyPendingOrder(double BOP)
{
 int TicketNumberBuy;
 double Spread = MarketInfo(Pair,MODE_SPREAD)*Point;
 double Free = AccountBalance();
 double OneLot = MarketInfo(Pair,MODE_MARGINREQUIRED );
 double Step = MarketInfo(Pair,MODE_LOTSTEP);
 TicketNumberBuy = OrderSend(Pair,OP_BUYSTOP,LS,BOP,Sl ip,0,0,"Gold Mine Buy");
 if(TicketNumberBuy == -1)
 {
    Alert("Buy Order failed, Error NO: "+GetLastError());
    PlacedTrade = false;
 }
 return(true);
}
Return To Disorder
  • #7
  • Jun 20, 2014 11:56am Jun 20, 2014 11:56am
Quoting Thepipster
I have an EA that closes all trades when equity target is reached. It also has an option for hard TP for each trade. If I select "0" as the setting for TP value, the EA closes that trade as soon as it is opened. Can I just add this line to completely rid the TP option? input bool UseTP = false; Or do I have to dive into the modifying orders section of the code, too, and put a "if" command, too? having the EA have to modify the orders with TPs is taking too much time when it's opening the pending orders.

The Easy way...

Set the TP to a profit that will never be reached in accordance to your lot size. TP=Eleventy Billion!!

Adding lines of code isn't necessarily the necessity...removing the close on tp might do the trick...

Unless code is posted, all anybody can do is guess. Go with the first suggestion if you don't want to post your code.
Money Can't Buy Happiness. Poverty Can't Buy SHIT! You Choose!
  • #8
  • Jun 20, 2014 12:13pm Jun 20, 2014 12:13pm
Quoting Magix
{quote} The Easy way... Set the TP to a profit that will never be reached in accordance to your lot size. TP=Eleventy Billion!! Adding lines of code isn't necessarily the necessity...removing the close on tp might do the trick... Unless code is posted, all anybody can do is guess. Go with the first suggestion if you don't want to post your code.
This is what I have been currently doing, but I need the EA to place 20 orders in 1 second...MT4 is a joke getting that job done, so I need the order placing process to go ASAP! Since I will never use hard TP, I can save 1/2 per order by not having the EA modify the order before placing the next one. Thanks for the idea, though.
Too close for pips, switching to points.
  • #9
  • Jun 20, 2014 1:44pm Jun 20, 2014 1:44pm
Quoting Thepipster
{quote} This is what I have been currently doing, but I need the EA to place 20 orders in 1 second...MT4 is a joke getting that job done, so I need the order placing process to go ASAP! Since I will never use hard TP, I can save 1/2 per order by not having the EA modify the order before placing the next one. Thanks for the idea, though.

I don't believe it is MT4 you would be having an issue with in getting the job done...

I would say that broker fills on 20 orders in one minute is where the issue is.

Given latency and liquidity, your expectations, as far as I'd be aware, are not exactly what is available.
Money Can't Buy Happiness. Poverty Can't Buy SHIT! You Choose!
  • #10
  • Jun 20, 2014 1:55pm Jun 20, 2014 1:55pm
I am working on bettering the latency problem, too, like getting a VPS right next to the broker's server..got total time to liquidity provider down to 80ms...but eliminating extra steps helps. My next battle is closing a bunch of opened trades ASAP.....arrg
Too close for pips, switching to points.
  • #11
  • Jun 20, 2014 2:17pm Jun 20, 2014 2:17pm
Quoting Thepipster
I am working on bettering the latency problem, too, like getting a VPS right next to the broker's server..got total time to liquidity provider down to 80ms...but eliminating extra steps helps. My next battle is closing a bunch of opened trades ASAP.....arrg

Been there before too...

Just because you want out and have the EA hammer closing all your trades, does not necessarily mean they will close as required.

Additionally...

I know that a part of the reason for the upgrades to build 6+, was so that brokers wouldn't use the same counter measure software on HFTs...but between you and I...how fucking stupid do they really think we are?
Money Can't Buy Happiness. Poverty Can't Buy SHIT! You Choose!
  • #12
  • Jun 20, 2014 4:44pm Jun 20, 2014 4:44pm
The problem is in this line of code

Inserted Code
TProfit = OrderOpenPrice() + TP*Point;

If you use TP=0 then this returns OrderOpenPrice and the subsequent OrderModify will close the order.

Change to

Inserted Code
if (TP!=0)
   TProfit = OrderOpenPrice() + TP*Point;
else
  TProfit=0;
  • #13
  • Jun 20, 2014 7:39pm Jun 20, 2014 7:39pm
insert an input like double TakeProfitPips = 0.0;

then check

double tp = 0.0;
if (TakeProfitPips!=0.00000) double tp=TakeProfitPips*pips2dbl+Ask/Bid


if (tp!=0.00000) OrderModify(xyz,tp,xyz);
PM me with coding requests and I'll probably put you on my ignore list
  • #14
  • Jun 23, 2014 4:53pm Jun 23, 2014 4:53pm
Thanks for all your suggestions, guys, ....I updated the code and tried a trial last night at market open....it didn't change a thing on opening the orders any faster.......It took 2 seconds per order for 10 pending orders to be placed....tried it a few times in a row...20 seconds...I know I've seen grid EAs put up 50 orders in 4-5 seconds...??? I used rokits coding idea.
Too close for pips, switching to points.
  • #15
  • Jun 23, 2014 5:03pm Jun 23, 2014 5:03pm
you could attach the ea (mql4-file) ...
PM me with coding requests and I'll probably put you on my ignore list
  • #16
  • Last Post: Edited 3:47am Jun 24, 2014 2:33am | Edited 3:47am
Quoting Thepipster
I updated the code and tried a trial last night at market open....it didn't change a thing on opening the orders any faster... It took 2 seconds per order for 10 pending orders to be placed....tried it a few times in a row...20 seconds...I know I've seen grid EAs put up 50 orders in 4-5 seconds...???
I made this test script to use with my broker (live); it took ~1900ms (1.9 sec) to place 10 pending orders (buy limit) on E/U. My ping time is <50ms and it took ~80-90ms per order (and ~100ms inbetween).
If you get a higher number here then check your ping time to trade server(s) and also execution time, meaning it is an issue with your broker rather than with the EA code.
Check it!
Inserted Code
// TEST script
 
void OnStart()
{
  int i = 0;
  int tm = GetTickCount();
  for(; i < 10;) {
    if(OrderSend(NULL, OP_BUYLIMIT, 0.01, 0.00001, 0, 0, 0) != -1) i++;
  }
  Print("time (ms): ", GetTickCount() - tm);
}
Return To Disorder
0 traders viewing now