CS1513 Error

[Ken7787]

I've added an attachment of the error. Trying to figure out what may be the issue with the expected }. any advice would be greatly appreciated.

[NinjaTrader_Jesse]

Hello Ken7787,

The error in this case lists the exact problem, the code you have is incomplete and the compiler is expecting that you finish the code. It is expecting a closing curly brace. By the looks of the code you may be missing multiple closing curly braces.

If you take a look at one of the sample scripts like SampleMACrossOver near the bottom of the file you will see multiple closing braces }. Your code needs to be formatted exactly like that.

NinjaScript uses the C# language which each section of code is contained within a body or opening and closing brace { }

You need a closing brace for the class and another for the namespace.

[Ken7787]

private bool IsTradingTime()
{
DateTime currentTime = Cbi.Globals.Now;
DateTime start = DateTime.ParseExact(startTradingTime, "HH:mm", null);
DateTime stop = DateTime.ParseExact(stopTradingTime, "HH:mm", null);

return currentTime >= start && currentTime <= stop;
}

So, with this I just need to add more curly braces? Also, it needs to look exactly like how your example is?

[NinjaTrader_Jesse]

Hello Ken7787,

Yes every script needs to follow the correct structure, as mentioned we are using C# language which is structured in a specific way. The code you provided is just a single method, that method has an opening and closing brace to contain the code inside the method. If you look in the SampleMACrossOver strategy it has a method inside of the class called OnBarUpdate. Your strategy would need to be structured in a similar way where the methods you create go inside the class which resides inside of the curly braces.

A script needs multiple opening and closing braces, the default structure of a class looks like the following:

namespace NinjaTrader.NinjaScript.Strategies
{
    public class SampleMACrossOver : Strategy
    {​
          //your code inside here

    }
}