Why is sell being triggered at point not called in code?

[CSharpTrader]

Hey there... I'm working with a strategy, and I see in the backtest chart, some of the positions are closing when I don't expect... I have one condition check that can call ExitLong(), and that is if(Close[0]<SMA(5)[0]... my positions are exiting with the close over the sma line. The whole method is below, and an image is attached(The 5per sma is the blue dashed line). Any help would be great...

protected override void OnBarUpdate()
        {
            if(CurrentBar<85)
            {
                Print("Not enough bars. Current bar count: " + CurrentBar);
                return;
            }
            else
            {
                if(Close[0]>SMA(5)[0])
                    if(Volume[0]>=(SMA(Volume,10)[1])*2)
                        if(Close[0]>(MAX(High,10)[1]))
                            if(Slope(SMA(50),11,1)<0)
                                if(CrossAbove(SMA(10),SMA(20),20))
                                    if(CountIf(delegate {return Close[0]>SMA(20)[0];},10)>4)
                                            if(CountIf(delegate {return (SMA(10)[0]<SMA(20)[0])&&(SMA(20)[0]<SMA(50)[0]);},30)>9)
                                                EnterLong();
                
                if(Close[0]<SMA(5)[0])
                    ExitLong();
            }
        }

[NinjaTrader_Matthew]

Hello,

If you're checking that all these conditions are true, please put them all in the same statement. You can separate the conditions using the && operator.

			 if(CurrentBar<85)
            {
                Print("Not enough bars. Current bar count: " + CurrentBar);
                return;
            }
            else
            {
                if(Close[0]>SMA(5)[0] &&
                    Volume[0]>=(SMA(Volume,10)[1])*2 &&
                        Close[0]>(MAX(High,10)[1]) &&
                            Slope(SMA(50),11,1)<0 &&
                                CrossAbove(SMA(10),SMA(20),20) &&
                                    CountIf(delegate {return Close[0]>SMA(20)[0];},10)>4 &&
                                            CountIf(delegate {return (SMA(10)[0]<SMA(20)[0])&&(SMA(20)[0]<SMA(50)[0]);},30)>9)
                                                EnterLong();
                
                if(Close[0]<SMA(5)[0])
                    ExitLong();
			}

[CSharpTrader]

Syntax issue?

Thanks Matthew. Is the syntax incorrect? I did that for speed of execution to not have to check all conditions if an early condition is false. Is the code structure related to my sell issue?

[NinjaTrader_Matthew]

I believe it is related to the structure. Current EnterLong is only called when the last condition is true.

[CSharpTrader]

Thanks..

Thanks again. My understanding of C# is that when you eliminate curly braces, only one statement immediately following the conditional check can be executed. One for the .NET gurus...