Trade limit per day fix my code.

[olingerc]

I am trying to limit my automated trading system to 1 complete trade per day. Meaning it will stop the strategy after a buy and then a sell OR a sell then a buy. (its a basic crossover strategy at the moment) I searched and found some sample code but I do not know where to actually put it in my strategy. Here it is. My strategy does limit the time It makes trades now where do I stick the code to limit the number of trades. Within which brackets is what I'm looking for.

protected override void OnBarUpdate()
{

/* Checks to see if the time is during the busier hours (format is HHMMSS or HMMSS). Only allow trading if current time is during a busy period.
The timezone used here is (GMT-05:00) EST. */

if ((ToTime(Time[0]) >= 093000 && ToTime(Time[0]) < 161400))
{
// Condition set 1
if (CrossBelow(ZLEMA(myInput0), SMA(myInput1), 1))
{
EnterShort(DefaultQuantity, "");
}

// Condition set 2
if (CrossAbove(ZLEMA(myInput0), SMA(myInput1), 1))
{
ExitShort("", "");
}

// At the start of a new session
if (Bars.FirstBarOfSession)
{
// Store the strategy's prior number of trades
priorTradesCount = Performance.AllTrades.Count;

/* NOTE: Using .AllTrades will include both historical virtual trades as well as real-time trades.
If you want to only count profits from real-time trades please use .RealtimeTrades. */
}

/* prevent trading if 1 trades have already been made in this session. */
if (Performance.AllTrades.Count - priorTradesCount > 1)

{
/* TIP FOR EXPERIENCED CODERS: This only prevents trade logic in the context of the OnBarUpdate() method. If you are utilizing
other methods like OnOrderUpdate() or OnMarketData() you will need to insert this code segment there as well. */

// Returns out of the OnBarUpdate() method. This prevents any further evaluation of trade logic in the OnBarUpdate() method.
return;
}


}


}

[NinjaTrader_Austin]

Hi there, unfortunately we are not able to code out strategies. You'll have to use some sort of a bool flag to limit your trades. Please see the following idea for how to accomplish this:

// in variables

private bool hasEnteredLong = false;
private bool hasEnteredShort = false;

protected override void OnBarUpdate()
{
   if (Bars.FirstBarOfSession)
   {
       // this means this is the first bar of the day, reset the flags
       hasEnteredLong = false;
       hasEnteredShort = false;
   }

   if (conditions for long entry == true && hasEnteredLong == false)
   {
       hasEnteredLong = true;
       EnterLong(....);
   }
   
   if (conditions for short entry == true && hasEnteredShort == false)
   {
       hasEnteredShort = true;
       EnterShort(....);
   }
}

[olingerc]

Thanks I will give this a try.