Translate For Loop in Easy Language to NT C#

February 22nd, 2011, 09:26 PM

I'm working on some John Ehler's Easy Language code, and I have reached this section that is so far over my head I'll never be able to translate it.

{Sum DeltaPhases to reach 360 degrees. The sum is the instantaneous period.}

InstPeriod = 0;
Value4 = 0;

For count = 0 to 50 begin
Value4 = Value4 + DeltaPhase[count];
If Value4 > 360 and InstPeriod = 0 then begin
InstPeriod = count;
end;
end;

Will you please translate that into C#?

February 22nd, 2011, 09:49 PM

give this a try...

Code

 
int InstPeriod = 0;
int Value4 = 0;
for (int count = 0; count <= 50; count++)
{
     Value4 =+ DeltaPhase[count];
     if (Value4 > 360 && InstPeriod == 0)
     {
          InstPeriod = count;
     }
}

February 22nd, 2011, 11:11 PM

Thanks for taking a stab at it Sunpost... who wrote..

int InstPeriod = 0;
int Value4 = 0;
for (int count = 0; count <= 50; count++)
{
Value4 =+ DeltaPhase[count];
if (Value4 > 360 && InstPeriod == 0)
{
InstPeriod = count;
}
}

DeltaPhase is just a double, so I dropped the [count] from DeltaPhase[count] in your answeras that would not compile... looks like the compiler was thinking the [count] was addressing an array... so I have this now...

//Sum DeltaPhases to reach 360 degrees. The sum is the instantaneous period.
InstPeriod = 0;
Value4 = 0;

for (int count = 0; count < 50; count++)
{
Value4 =+ DeltaPhase;
DrawTextFixed("Tag1",Value4.ToString(),TextPosition.BottomRight); //for debugging
if (Value4 >360 && InstPeriod == 0)
{
InstPeriod = count;
DsInstPeriod[0] = InstPeriod;
//DrawTextFixed("Tag1",InstPeriod.ToString(),TextPosition.BottomRight);
}
}

The DrawTexts I'm using for debugging... What's happening is that DeltaPhase gets added to Value4 once, and gets drawn, but then the if statement never executes... I believe because the loop doesn't run, doesn't keep adding DeltaPhase to Value4, so Value 4 never gets greater than 360 for the if statement to evaluate to true.

Any help much appreciated.

February 22nd, 2011, 11:30 PM

The issue with the statements in the if not being reached after the first time is that InstPeriod gets set to a value other than 0 making the condition (InstPeriod == 0) false.

Code

 
if (Value4 >360 &&InstPeriod == 0)
{
    InstPeriod = count;
     DsInstPeriod[0] = InstPeriod;
     //DrawTextFixed("Tag1",InstPeriod.ToString(),TextPos ition.BottomRight);
}

February 23rd, 2011, 12:40 AM

Thanks for the help sunpost...

I got the loop running.... indicator doesn't look right, but at least its plotting something other than 0. I probably have something wrong up in the ArcTangent math to debug...

what was wrong with the loop was I had

Value4 =+ DeltaPhase;

instead of +=

Progress has been made.... however. Thanks.

February 23rd, 2011, 07:34 AM

great progress @Crassius!

You might be interested in a webinar today by @shodson...
Introduction to C# and NinjaScript Programming
Trading Webinar's - [AUTOLINK]Big Mike Trading[/AUTOLINK] Forum

Translate For Loop in Easy Language to NT C#

Discussion in NinjaTrader

Translate For Loop in Easy Language to NT C#