Maximize profit based on the price move

• 12/08/2021 at 7:44 AM #182979
  Amin1233

  Hi , Anyone can help with the below code please??

  the idea is to not exit a position if it is still going your way. So if longonmarket and it is in profit (+15 points) then update the target profit to 50 and only exit when the candle finishes lower than the previous candle.

  if longonmarket and close[0]-positionprice>15 then set target profit 40 if close[0]<lowest[1] then sell countofposition contracts at market endif endif

  1 2 3 4 5 6

  if longonmarket and  close[0]-positionprice>15 then set target profit 40 if close[0]<lowest[1] then sell countofposition contracts at market endif endif

  Not sure what is wrong, but it only changes the target profit and the second condition doesnt work.

   

  

  Amin1233

  Participant

  Topics: 15

  Replies: 32

  Been thanked: 0 times
  12/08/2021 at 9:43 AM #182986
  robertogozzi

  Try this:

  if not OnMarket then flag = 0 endif if longonmarket and close[0]-positionprice>15 then set target profit 40 flag = 1 endif if close[0]<lowest[1] and flag then sell countofposition contracts at market endif

  1 2 3 4 5 6 7 8 9 10

  if not OnMarket then    flag = 0 endif if longonmarket and  close[0]-positionprice>15 then    set target profit 40    flag = 1 endif if close[0]<lowest[1] and flag then    sell countofposition contracts at market endif

  

  robertogozzi

  Moderator

  Topics: 69

  Replies: 16466

  Been thanked: 3641 times
  12/08/2021 at 10:33 AM #182994
  murre87

  Intressting topic. An active TP need an active SL too.
  How do u manage SL in ur strategy Amin1233?

  

  murre87

  Participant

  Topics: 17

  Replies: 162

  Been thanked: 21 times
  12/08/2021 at 12:45 PM #183006
  Amin1233

  Hi Roberto. Still not working. I try to only add the second part to test and it just doesnt make any difference to the result which is impossible.

   

  if not OnMarket then
  flag = 0
  endif
  if longonmarket and close[0]-positionprice>15 then

  flag = 1
  endif
  if close[0]<lowest[1] and flag then
  sell countofposition contracts at market
  endif

  

  Amin1233

  Participant

  Topics: 15

  Replies: 32

  Been thanked: 0 times
  12/08/2021 at 12:46 PM #183007
  Amin1233

  Murre, I dont change the SL.In this strategy you wouldnt need Stop loss because when the price goes below the last bar it should exit.

  

  Amin1233

  Participant

  Topics: 15

  Replies: 32

  Been thanked: 0 times
  12/08/2021 at 3:38 PM #183020
  JC_Bywan

  Hi, “candle finishes lower than previous candle” is close[0]<low[1], not close[0]<lowest[1]

  1 user thanked author for this post.

  robertogozzi

  

  JC_Bywan

  Moderator

  Topics: 20

  Replies: 2546

  Been thanked: 838 times
  12/08/2021 at 11:19 PM #183048
  Amin1233

  thanks

  

  Amin1233

  Participant

  Topics: 15

  Replies: 32

  Been thanked: 0 times
  01/06/2022 at 11:26 AM #184743
  RichardVeen

  Hi,

  The code above is a great idea but it is not working for me.

  Below the code I testen and the target is not moving despite of the fact that the close(0) – positionprice > 5

  What is wrong with the code?

  //profit & loss
  SET TARGET PROFIT 40
  SET STOP pLOSS 20

  if not OnMarket then
  flag = 0
  endif
  if longonmarket and close[0]-positionprice>5 then
  set target profit 50
  flag = 1
  endif

  if close[0]<low[1] and flag then
  sell countofposition contracts at market
  endif

  

  RichardVeen

  Participant

  Topics: 11

  Replies: 33

  Been thanked: 1 time
  01/06/2022 at 1:35 PM #184755
  JS

  Hi @RichardVeen

  Remove the “Set Target Profit 40” at the top of your code.

  (Otherwise it will reset every time to 40)

  

  JS

  Participant

  Topics: 22

  Replies: 1423

  Been thanked: 557 times
  01/06/2022 at 3:44 PM #184766
  RichardVeen

  Yes, you are right, it is working!

  Thanks!

  1 user thanked author for this post.

  JS

  

  RichardVeen

  Participant

  Topics: 11

  Replies: 33

  Been thanked: 1 time
• Author
  Posts