Announcement
Collapse
Looking for a User App or Add-On built by the NinjaTrader community?
Visit NinjaTrader EcoSystem and our free User App Share!
Have a question for the NinjaScript developer community? Open a new thread in our NinjaScript File Sharing Discussion Forum!
Have a question for the NinjaScript developer community? Open a new thread in our NinjaScript File Sharing Discussion Forum!
See more
See less
Partner 728x90
Collapse
NinjaTrader
CS1513 Error
Collapse
X
-
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.JesseNinjaTrader Customer Service
-
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?
Comment
-
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:
Code:namespace NinjaTrader.NinjaScript.Strategies { public class SampleMACrossOver : Strategy { //your code inside here } }
JesseNinjaTrader Customer Service
Comment
Latest Posts
Collapse
Topics | Statistics | Last Post | ||
---|---|---|---|---|
Started by fx.practic, 10-15-2013, 12:53 AM
|
5 responses
5,406 views
0 likes
|
Last Post
by Bidder
Today, 12:22 AM
|
||
Started by Shai Samuel, 07-02-2022, 02:46 PM
|
4 responses
98 views
0 likes
|
Last Post
by Bidder
Today, 12:11 AM
|
||
Started by DJ888, Yesterday, 10:57 PM
|
0 responses
8 views
0 likes
|
Last Post
by DJ888
Yesterday, 10:57 PM
|
||
Started by MacDad, 02-25-2024, 11:48 PM
|
7 responses
160 views
0 likes
|
Last Post Yesterday, 10:23 PM | ||
Started by Belfortbucks, Yesterday, 09:29 PM
|
0 responses
9 views
0 likes
|
Last Post
by Belfortbucks
Yesterday, 09:29 PM
|
Comment