Newbie question

[GKonheiser]

Hi,

Just starting out with ninja C#. I have writen the following code,

{
if((High[0] < High[1]) && (High[1] > High[2]));

DrawDot("high", true, 1, High[1], Color.Black);
Print("Condition True");
}

and am getting the following error,


Error on calling 'OnBarUpdate' method for indicator 'HighsTest' on bar 0: Object reference not set to an instance of an object.

[Harry]

Hi,

Just starting out with ninja C#. I have writen the following code,

{
if((High[0] < High[1]) && (High[1] > High[2]));

DrawDot("high", true, 1, High[1], Color.Black);
Print("Condition True");
}

and am getting the following error,


Error on calling 'OnBarUpdate' method for indicator 'HighsTest' on bar 0: Object reference not set to an instance of an object.

You cannot call High[1] for the first bar, because the first bar has no bar that preceeds it. You can only run your code starting with the second bar.

{
     if(CurrentBar == 0)
          return;
     if(High[0] < High[1]) && (High[1] > High[2])
     {   
            DrawDot("high", true, 1, High[1], Color.Black);
            Print("Condition True");
     }
}

Further pay attention what you do with your brackets and semicolons. The line

if((High[0] < High[1]) && (High[1] > High[2]));

is just an empty statement, which translates to "if the condition is true, do nothing".

The code

{
          if((High[0] < High[1]) && (High[1] > High[2]))  // semicolon removed
        
            DrawDot("high", true, 1, High[1], Color.Black);
            Print("Condition True");
}

is incorrect, because you forgot to set the bracket to include the print line. Without the bracket the print line will be executed all the time, as it is not linked to the condition.

[GKonheiser]

Thanks that's a big help. I'm trying to build an indicator form the ground up that effectively plots Highs and Lows and this is the first little step. ; )