my first strategy – beginner level

• 07/30/2017 at 11:23 PM #42041
  mmichael

  Hello,

  I am starting my first strategy, just to understand the logic of proorder and how things work.

  At this stage I am facing 2 issues:

  • how to take partiel profit (50%) at a certain level
  • have a dynamic stop loss:
    • close 100% if the bar after the entry does not confirm (red bar in case of a long trade)
    • else: stop moves below each bar low

   

  DEFPARAM CumulateOrders mm20 = Average[20](close) mm50 = Average[50](close) openLong= mm20 crosses over mm50 if not longonmarket and openLong then buy 1 lot at market // buy 1 contracts at market amplitude = close - low endif set stop loss amplitude set target profit amplitude // partial profit - to be completed // to do // stop move - to be completed if longonmarket then newStop = low[1] endif sell at newStop STOP

  1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

  DEFPARAM CumulateOrders mm20 = Average[20](close) mm50 = Average[50](close)   openLong= mm20 crosses over mm50   if not longonmarket and openLong then buy 1 lot at market // buy 1 contracts at market amplitude = close - low endif   set stop loss amplitude set target profit amplitude   // partial profit - to be completed // to do   // stop move - to be completed if longonmarket then newStop = low[1] endif   sell at newStop STOP

   

  

  mmichael

  Participant

  Topics: 4

  Replies: 36

  Been thanked: 4 times
  07/31/2017 at 6:38 AM #42050
  Nicolas

  Partial closure of position is still not supported by ProOrder.

  About your stoploss, the exit at each new bar low should operate nicely the way you have coded it. In the code below I implemented the exit on red candlestick:

  DEFPARAM CumulateOrders mm20 = Average[20](close) mm50 = Average[50](close) openLong= mm20 crosses over mm50 if not longonmarket and openLong then buy 1 lot at market // buy 1 contracts at market amplitude = close - low endif set stop loss amplitude set target profit amplitude // partial profit - to be completed // to do // stop move - to be completed if longonmarket then if tradeindex=barindex[1] and close<open then sell at market else newStop = low[1] sell at newStop STOP endif endif

  1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26

  DEFPARAM CumulateOrders mm20 = Average[20](close) mm50 = Average[50](close)   openLong= mm20 crosses over mm50   if not longonmarket and openLong then buy 1 lot at market // buy 1 contracts at market amplitude = close - low endif   set stop loss amplitude set target profit amplitude   // partial profit - to be completed // to do   // stop move - to be completed if longonmarket then if tradeindex=barindex[1] and close<open then   sell at market else   newStop = low[1]   sell at newStop STOP endif endif

  Not tested, please confirm.

  1 user thanked author for this post.

  mmichael

  

  Nicolas

  Keymaster

  Topics: 81

  Replies: 20866

  Been thanked: 4781 times
  07/31/2017 at 10:48 PM #42142
  mmichael

  Hello,

  thx a lot Nicolas,

  I tested the proposed code and there still have an issue with the stop execution.

  normally the stop is set to the previous low (low[1]) but as you can see in the screenshot, it executes one bar later than expected. please advice if you have an idea to fix this.

  I tried both “sell at newStop STOP” between the if / endif and outside, but behaviour is the same.

  Below my code:

  DEFPARAM CumulateOrders = False // Cumul des positions désactivé mm50 = Average[50](close) openLong= close > mm50 and close[1] < mm50 openShort= close < mm50 and close[1] > mm50 graph openLong graph openShort if not longonmarket and openLong then buy 1 lot at market // buy 1 contracts at market endif if not shortonmarket and openShort then sellShort 1 lot at market // buy 1 contracts at market endif if longonmarket then if tradeindex=barindex and close<open then sell at market else newStop = low[1] sell at newStop STOP endif endif if shortonmarket then if tradeindex=barindex and close>open then buy at market else newStop = high[1] buy at newStop STOP endif endif graph newStop //debugging

  1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35

  DEFPARAM CumulateOrders = False // Cumul des positions désactivé   mm50 = Average[50](close) openLong= close > mm50 and close[1] < mm50 openShort= close < mm50 and close[1] > mm50 graph openLong graph openShort   if not longonmarket and openLong then buy 1 lot at market // buy 1 contracts at market endif   if not shortonmarket and openShort then sellShort 1 lot at market // buy 1 contracts at market endif   if longonmarket then if tradeindex=barindex and close<open then sell at market else newStop = low[1] sell at newStop STOP endif endif   if shortonmarket then if tradeindex=barindex and close>open then buy at market else newStop = high[1] buy at newStop STOP endif endif   graph newStop                              //debugging

   

  

  mmichael

  Participant

  Topics: 4

  Replies: 36

  Been thanked: 4 times
  08/01/2017 at 5:41 AM #42149
  Madrosat

  il  me semble qu’avec shortonmarket il faut mettre exitshort et non buy

  

  Madrosat

  Participant

  Topics: 24

  Replies: 230

  Been thanked: 9 times
  08/01/2017 at 11:09 AM #42184
  Nicolas

  Madrosat is right, if you want to exit a short position you should use “EXITSHORT” instead of buy (errors are contained in line 28 and 32 of your last code).

  

  Nicolas

  Keymaster

  Topics: 81

  Replies: 20866

  Been thanked: 4781 times
  08/02/2017 at 4:29 AM #42276
  mmichael

  Thx gentlemen,

  I have re-written my code, but still struggling with the stop.

  the stop executes one bar late and at close price (not at the defined stop level)

   

  Below my code and I also attached the ITF file

  // ================================================================================ Params defparam cumulateorders = false // Cumul des positions désactivé positionSize = 1 // ================================================================================ calc openlong openShort closeLong closeShort moveStopLong moveStopShort mm50 = Average[50](close) openLong= close > mm50 and close[1] < mm50 openShort= close < mm50 and close[1] > mm50 graph openLong graph openShort closeLong = barindex = tradeindex and close < open closeShort = barindex = tradeindex and close > open // ================================================================================ position opening if not onmarket and openLong then buy positionSize lot at market endif if not onmarket and openShort then sellShort positionSize lot at market endif // ================================================================================ set stop & target if longonmarket then if closeLong then sell at market else stopLevel = low[1] sell at stopLevel STOP endif endif if shortonmarket then if closeShort then exitshort at market else stopLevel = high[1] exitshort at stopLevel STOP endif endif graph stopLevel if not onmarket then stopLevel=0 endif

  1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52

  // ================================================================================ Params   defparam cumulateorders = false // Cumul des positions désactivé positionSize = 1   // ================================================================================ calc openlong openShort closeLong closeShort moveStopLong moveStopShort   mm50 = Average[50](close) openLong= close > mm50 and close[1] < mm50 openShort= close < mm50 and close[1] > mm50 graph openLong graph openShort   closeLong = barindex = tradeindex and close < open closeShort = barindex = tradeindex and close > open     // ================================================================================ position opening   if not onmarket and openLong then buy positionSize lot at market endif   if not onmarket and openShort then sellShort positionSize lot at market endif   // ================================================================================ set stop & target   if longonmarket then if closeLong then sell at market else stopLevel = low[1] sell at stopLevel STOP endif endif   if shortonmarket then if closeShort then exitshort at market else stopLevel = high[1] exitshort at stopLevel STOP endif endif   graph stopLevel   if not onmarket then stopLevel=0 endif

   

  

  mmichael

  Participant

  Topics: 4

  Replies: 36

  Been thanked: 4 times
  08/02/2017 at 9:02 PM #42369
  Nicolas

  Are you sure your “stoplevel” is correctly calculated? If the answer is yes, then the problem should come from the way you are putting your pending stop orders, because they are included in a conditional block, they may be put on market too late.

  

  Nicolas

  Keymaster

  Topics: 81

  Replies: 20866

  Been thanked: 4781 times
  08/04/2017 at 11:20 PM #42490
  mmichael

  Hello Nicolas,

  thanks a lot for your help, I did several tests and it does not work correctly.

  could you please tell me simply how to set a stop at the previous low (for a long)?

  illustration in the screenshot.

  My current code (not ok):

  defparam cumulateorders = false if not onmarket and close > Average[50](close) and close[1] < Average[50](close) then buy 1 lot at market endif if not onmarket then lastStop=0 endif if longonmarket then lastStop = low[1] sell at lastStop STOP endif

  1 2 3 4 5 6 7 8 9 10 11 12 13 14

  defparam cumulateorders = false   if not onmarket and close > Average[50](close) and close[1] < Average[50](close) then buy 1 lot at market endif   if not onmarket then lastStop=0 endif   if longonmarket then lastStop = low[1] sell at lastStop STOP endif

   

  

  mmichael

  Participant

  Topics: 4

  Replies: 36

  Been thanked: 4 times
  08/05/2017 at 12:21 AM #42492
  mmichael

  my solution have been found here:

  https://www.prorealcode.com/topic/bullish-engulfing-breakout-bearish-engulfing/

  below the corrected code:

  defparam cumulateorders = false if not onmarket and close > Average[50](close) and close[1] < Average[50](close) then buy 1 lot at market // initial stop lastStop = low sell at lastStop STOP endif if longonmarket then lastStop = low sell at lastStop STOP endif

  1 2 3 4 5 6 7 8 9 10 11 12 13

  defparam cumulateorders = false if not onmarket and close > Average[50](close) and close[1] < Average[50](close) then buy 1 lot at market // initial stop lastStop = low sell at lastStop STOP endif if longonmarket then lastStop = low sell at lastStop STOP endif

  PS.: some strategies that I have checked during my research for the stop are wrong, meaning the “lastStop = low[1]” that we can see in several strategies is not ok!!

  

  mmichael

  Participant

  Topics: 4

  Replies: 36

  Been thanked: 4 times
  08/05/2017 at 9:53 AM #42496
  Nicolas

  Because the code is read at Close and position opened at the next candle open, you need to refer to Low[0]. When you find a variable not calculated correctly, use GRAPH! and the solution will come to you quickly 🙂

  1 user thanked author for this post.

  mmichael

  

  Nicolas

  Keymaster

  Topics: 81

  Replies: 20866

  Been thanked: 4781 times
• Author
  Posts