One Order Per Bar

[cre8it8]

Is there code that I can put in my strategy that makes it only enter a new order on a bar that hasn't just closed on a trade without it being set on calculate on bar close? I want the setting of calculate on bar close = false because I get better entries but I don't want it to put in a new order right after it closes a trade on the same bar because that can really screw things up sometimes. I want it to wait until the next bar to enter the new order after the close of the previous order.

I hope that makes sense. Thanks for any help.

[NinjaTrader_Bertrand]

cre8it8, you can work with BarsSinceExit() in your entry conditions to avoid reentering on the same bar then - http://www.ninjatrader-support.com/H...SinceExit.html

[cre8it8]

That looks great. Thanks for the fast reply and help Bertrand.

[cre8it8]

Is there a reason this is not running correctly? Is there something I need to add in order to use BarsSinceExit? This ran fine before I added it. Thanks.

if (SMA(9)[0] -0 * TickSize > SMA(20)[0])
{
if (Position.MarketPosition == MarketPosition.Flat
&& BarsSinceExit() >0 )
{
EnterLong(1, "Long 1a");

}

[NinjaTrader_Bertrand]

This will not be picked up, as your signal names do not match unfortunately.

[cre8it8]

I tried changing BarsSinceExit() >0 to BarsSinceExit("Long 1a") >0 and it didn't seem to work. Something I am missing?

[NinjaTrader_Bertrand]

cre8it8, please try just with emtpy "" signal names for a test.

You can also work with a bool flag here and reset it for example then on FirstTickOfBar so you can take trades again.

[cre8it8]

Odd, I created a strategy with the example from the link you gave me and it didn't work when I ran it. Is there something that needs to be put into the variables section or somewhere to make the below work.



protected override void OnBarUpdate()
{
if (CurrentBar < 20)
return;

// Only enter if at least 10 bars has passed since our last exit
if (BarsSinceExit() > 10 && CrossAbove(SMA(10), SMA(20), 1))
EnterLong();
}

[NinjaTrader_JoshP]

cre8it8,

If your strategy has never traded yet, BarsSinceExit() would never be greater than 10. This would mean you would also never trade so you have a catch-22 situation there. You will need to allow for the first trade before using that type of condition.

[cre8it8]

I see. I think I can work with that info. I will give a few things a try and see what happens. Thanks Josh.

[cre8it8]

So I tried a couple of things like have the code without the BarsSinceExit and then have the code below that with it but to no result. Can you give me an example of how you would get it to allow the first trade and then after that trade go to the code with the BarsSinceExit. Thanks Josh

[lookOutBelow]

Not that you asked me... But, here is what I've done.

In variables...
private bool OneBarSinceExit = true;



Then inside the OnBarUpdate:
if (BarsSinceExit() > 1)
OneBarSinceExit = true;


Prior to initiating a trade and along with your other checks, make sure OneBarSinceExit is true.


After a trade is sent, reset OneBarSinceExit back to false. It is important that OneBarSinceExit is set to true in the variables. Or else it never makes that first trade.

[cre8it8]

Thanks for jumping in "look out below" your opinion is valued. I tried to do what you suggested on some easy code but I couldn't get it to work. I am fairly amateur at programming and most of what I am able to do is from looking at others sample codes and figuring out how to create what I need. Here is what I tried to do with what you told me below. Am I even close? Thanks for your help.

#region Variables
private bool OneBarSinceExit = true;
#endregion

protected override void Initialize()
{
EntriesPerDirection = 1;
CalculateOnBarClose = false;
SetProfitTarget("Long 1a", CalculationMode.Ticks, 5);
ExitOnClose = false;
}

protected override void OnBarUpdate()
{
if (BarsSinceExit() > 1)
OneBarSinceExit = true;

if (SMA(9)[0] > SMA(20)[0]&& OneBarSinceExit == true)
{
EnterLong(1, "Long 1a");
OneBarSinceExit == false;
}
}
}

#region Properties
#endregion
}
}

[lookOutBelow]

I think if you left everything you did the same, but change:

EnterLong(1, "Long 1a");
OneBarSinceExit == false;

to

EnterLong(1, "Long 1a");
OneBarSinceExit = false;


You have an extra "=" in there. I haven't had much sleep lately, so I may be overlooking something. But, that should work.

[NinjaTrader_Austin]

cre8it8, please try what lookOutBelow has suggested and if it doesn't work, let us know.