hello support,

i try to manage my risk in my strategy with a sample way, but i have a problem with my cycle.
the scenario is simple : with a winner trade, i increments my risk by one. with a loser trade , i decrease my risk by one.

but the cycle repeats itself because stategy returns to "void onaccount". so i would like that the strategy enters the cycle once.

could you help me please?

   
  @Override
    public void onAccount(IAccount account) throws JFException {
         
         
         double riskmax = 4;
         double riskAddLess = 1;
         double riskmin = 1;
         double baseBalance = account.getBalance();
         double currentBalance = account.getBaseEquity();
         
         for(int ip=0;ip<1;ip++){
            if (order.getState() == IOrder.State.CLOSED){
               if(baseBalance < currentBalance && risk < riskmax){
            
                  risk = risk + riskAddLess;
                  
            }else if(baseBalance > currentBalance && risk > riskmin){
            
         
                  risk = risk - riskAddLess;
         
            }
             }
      }
    print("risk:" +risk);
    }

thank.
eric

Exactly when do you want it to execute?

when the order is closed with profit i want to increase my "double risk" . when the order is closed with loss i want to decrease my "double risk". well, my cycle "for" do that but the cycle repeat itself because the strategy read constantly the "void onAccount" and re enter in the cycle. when the new order is opened the cycle is stopped due at the if statement : if (order.getState() == IOrder.State.CLOSED).

when the order is closed the strategy go to the cycle increase or decrease but only once.

i hope is more clear. sorry for my english

eric

Then you need to move the logic to the IStrategy.onMessage method, e.g.:

   @Override
   public void onMessage(IMessage message) throws JFException {
      if(message.getType() == IMessage.Type.ORDER_CLOSE_OK && message.getOrder().getState() == IOrder.State.CLOSED){
         if(message.getOrder().getProfitLossInUSD() >= 0){
            //...
         } else {
            //...
         }
      }
   }

For more please refer to the article and the full workflow diagrams:
https://www.dukascopy.com/wiki/#Order_state

ok thank for that.