I had already tried with “Not OnMarket” earlier but with no change/improvement, don’t know if that what you meant? Well late now and I will check for answers later tomorrow when I’m back at home. Thanks.

Forgot to mention the clearing of the variables. The code

// Clear entry variables
if time = 085000 then
tradecount = 0 // Reset daily tradecount
firsttarget = 0 // Reset first target reach variable
endif

Vonasi wrote:

Daniel_H wrote:

The code wasn’t intended to open more positions until the previous one was done. I know how CumulateOrders work.

Your description is very difficult to understand.

The code in ProOrder is read from top to bottom at the close of each candle. If CUMULATEORDERS is set to FALSE then no additional positions will be opened at the close of a candle if a position is already open. So you can put any number of BUY instructions in your code within whatever IF THEN ENDIF statements you like but they will all be ignored. Other variable changes will not be ignored unless they are protected by an IF NOT ONMARKET.

Hi again.  Yes the description in my first post wasn’t very good, I didn’t want to write to much text and thought the comments in the code would help as well.

Then in your reply you once again write about opening additional positions which I just explained about in my previous two posts (and where clear about in the part below your nickname). Everything about the problem is in those two posts (06/21/2018 at 8:55 PM and 06/21/2018 at 9:02 PM). I also wrote that I’ve tried with IF NOT ONMARKET but it doesn’t change anything, it’s still the same problem.

Here is the code as of now I’m testing with to get the system to buy the second time AFTER the first trade has been stopped out.

DEFPARAM CumulateOrders = False
DEFPARAM FLATAFTER = 172000

// Prevents the system from creating new orders to enter the market or increase position size before the specified time
noEntryBeforeTime = 092500
timeEnterBefore = time >= noEntryBeforeTime

// Prevents the system from placing new orders to enter the market or increase position size after the specified time
noEntryAfterTime = 170000
timeEnterAfter = time < noEntryAfterTime

set stop ploss 4
targetprofit = 5

// Clear entry variables
if time = 085000 then
tradecount = 0 // Reset daily tradecount
firsttarget = 0 // Reset first target reach variable
//secondtrade = 0
endif

// Conditions to enter long positions
cl1 = close > open

// Long 1
if not onmarket and firsttarget = 0 and tradecount = 0 then
if cl1 and timeEnterBefore and timeEnterAfter then
buy 1 contract at market
tradecount = 1 // the addon works fine here
//tradecount = tradecount + 1
endif
endif

// Long 2
if not onmarket and firsttarget = 0 and tradecount = 1 then
if not onmarket and cl1 and timeEnterBefore and timeEnterAfter then
buy 2 contract at market
tradecount = 2 // the addon does work, but the buy order won't buy before the tradecount addon, but without this line the order buys. So why does the same addon code work the first time but not the second time?
//secondtrade = 1
endif
endif

// Long 3
if not onmarket and firsttarget = 0 and tradecount = 2  then
if not onmarket and cl1 and timeEnterBefore and timeEnterAfter then
buy 4 contract at market
tradecount = tradecount + 1 // the addon does work, but the buy order won't buy before the tradecount addon, but without this line the order buys. So why does the same addon code work the first time but not the second time?
endif
endif

if longonmarket then
if firsttarget = 0 then
if close > tradeprice + targetprofit*pipsize then
sell at market
firsttarget = 1
endif
endif
endif


// End

Late now, time for zzz, will check more tomorrow.

Thanks.

Vonasi.

Thanks for optimizing the code, will definitely be able to use some parts of it (for the tests and in the future).

About complicating things it’s because I want the possibility to use different positionsizes at the different trades. I might want to go with 1 contract in the first trade, then 2 and finally 2 again in the third trade. Or perhaps 1 contract in the first and second trade and 2 or maybe 4 contracts  in the third trade, that’s what I want to test out and play around with. That’s the reason I wanted to separate the trades. Is there a way to be able to do this?

Though I still want to be able to understand why the separate if statements I posted 06/22/2018 at 12:20 AM didn’t work. They would work in any other big programming language, hence I feel there has to be som bugs in the ProOrder language.

Thanks.
/Daniel

Ok I didn’t need to increase the maxtrades, I accidently wrote the wrong tradecount in the second and third statement (0, 2 and 3), should be like this:

// Long 1
if not onmarket and firsttarget = 0 and tradecount < maxtrades and cl1 and timeEnterBefore and timeEnterAfter then
if tradecount = 0 then
buy 1 contracts at market
endif
 
if tradecount = 1 then
buy 4 contracts at market
endif
 
if tradecount = 2 then
buy 2 contracts at market
endif
tradecount = tradecount + 1
//positionsize = positionsize * 2
endif

Thanks again.