public bool ZeroLine in Parameters

[alexstox]

Hello. What is wrong in this code in indicator
[Description("displaying zeroline")]
[GridCategory("Parameters")]
public bool ZeroLine
{
get { return zeroLine; }
}


If it == true, indicator display zero line.

[NinjaTrader_ChelseaB]

Hi alexstox,

There is no set { } for this public variable.

Why have this variable if it doesn't set anything?

If you are planning to use it just to return a value with the get { }, you will still need a set { } to prevent an error.

Take a look at the Period public property of the SMA indicator for an example.

[alexstox]

#region Variables
private bool zeroLine = true;
...............
protected override void Initialize()
{
if (ZeroLine==true) {Add(new Line(Color.Gray, 0, "zeroline"));}
.................
#region Properties
[Description("displaying zeroline")]
[GridCategory("Parameters")]
public bool ZeroLine
{
set { zeroLine; }
}

Still doesn't work.

[NinjaTrader_ChelseaB]

Hello alexstox,

You have now removed the get { } which will also cause an error.

Do you see errors appearing at the bottom of the NinjaScript editor window when you try and compile this?

What do these errors say?

Please take a look at the Period public property in the SMA indicator. This has a get { } and set { }. You will need both.

Also, you are not setting anything in set.

Improper code:
public bool ZeroLine
{
set { zeroLine; }
}

If you don't want to set zeroLine then do not add zeroLine; to the set function.

Proper code (for not setting the zeroLine)
public bool ZeroLine
{
set { }
}

Proper code (for setting the zeroLine)
public bool ZeroLine
{
set { zeroLine = value; }
}

[alexstox]

#region Variables
private bool zeroLine = true;
...............
protected override void Initialize()
{
if (ZeroLine==true) {Add(new Line(Color.Gray, 0, "zeroline"));}
.................
#region Properties
[Description("displaying zeroline")]
[GridCategory("Parameters")]
public bool ZeroLine
{
get { return zeroLine; }
set { zeroLine; }
}

As you see this variable is only for on/off Zero line in indicator.

This doesn't help.
Error (translation from Russian in Google): As the operator can use an assignment expression, call increment, decrement, and create a new object - CS0201

This error about zeroLine in set { zeroLine; }

As you see this variable is only for on/off Zero line in indicator.

[NinjaTrader_ChelseaB]

alexstox,

Please see post #4.
http://www.ninjatrader.com/support/f....php?t=64768#4

You are not setting anything in set.

Improper code:
public bool ZeroLine
{
set { zeroLine; }
}

If you don't want to set zeroLine then do not add zeroLine; to the set function.

Proper code (for not setting the zeroLine)
public bool ZeroLine
{
set { }
}

Proper code (for setting the zeroLine)
public bool ZeroLine
{
set { zeroLine = value; }
}

[alexstox]

1) What is right in my case: to set or not to set? If I need ZeroLine variable only to ON/OFF displaying zero line in indicator?

2) "get" is not used with bool?

[NinjaTrader_ChelseaB]

Hi alexstox,

1) Yes, you need a set { } and a get { } no matter what.

If you are wanting a parameter that you can set when starting the script you have to set the private variable to the current value of the public variable in set { }.

2) No, public bools also need both a get { } and a set { }.

The code you are looking for is this.

public bool ZeroLine
{
get { return zeroLine; }
set { zeroLine = value; }
}

[koganam]

#region Variables
private bool zeroLine = true;
...............
protected override void Initialize()
{
if (ZeroLine==true) {Add(new Line(Color.Gray, 0, "zeroline"));}
.................
#region Properties
[Description("displaying zeroline")]
[GridCategory("Parameters")]
public bool ZeroLine
{
get { return zeroLine; }
set { zeroLine; }
}

As you see this variable is only for on/off Zero line in indicator.

This doesn't help.
Error (translation from Russian in Google): As the operator can use an assignment expression, call increment, decrement, and create a new object - CS0201

This error about zeroLine in set { zeroLine; }

As you see this variable is only for on/off Zero line in indicator.

set { zeroLine = [SIZE=3][B][COLOR=Red]value[/COLOR][/B][/SIZE]; }