iOPEN | Forex Factory

Options

Permalinks
Print Thread

Similar Threads

iHigh iLow iOpen iClose... 7 replies

Help with iOpen, iClose, iHigh, iLow 0 replies

Attachments: iOPEN
Exit Attachments

iOPEN

  • #1
  • First Post: Oct 16, 2010 8:02pm Oct 16, 2010 8:02pm
I'm tinkering with some code and have a problem...here is the code in question:

PHP Code
for (i=limit; i>=0; i--)
   {
      int x = iBarShift(NULL,timeFrame,Time[i+1]-HourShift*3600,true);
      int y = iBarShift(NULL,timeFrame,Time[i]  -HourShift*3600,true);
      if (x!=y)
      {
         int k = i;
         if (FixSundays && timeFrame==PERIOD_D1)
         {
            while (k<Bars && TimeDayOfWeek(Time[k+1]-HourShift*3600)==0) k++;
         }
         int z = iBarShift(NULL,lookupTimeFrame,Time[k+1]-timeFrame*60-HourShift*3600);
             x = iBarShift(NULL,lookupTimeFrame,Time[k+1]             -HourShift*3600);
               
             //
             //
             //
             //
             //

             double LastHigh  = iHigh (NULL,lookupTimeFrame,iHighest(NULL,lookupTimeFrame,MODE_HIGH,z-x,x));
             double LastLow   = iLow  (NULL,lookupTimeFrame,iLowest(NULL,lookupTimeFrame,MODE_LOW,z-x,x));
             double LastClose = Close[k+1];
             double LastOpen = iOpen(NULL,lookupTimeFrame,iOpen(NULL,lookupTimeFrame,MODE_OPEN,z-x,x));
             double Range     = (LastHigh-LastLow);
             double atr = iATR(NULL,PERIOD_D1,ATRDays,i)/2.0;
It won't compile....I'm getting a wrong parameter count on this line....
double LastOpen = iOpen(NULL,lookupTimeFrame,iOpen(NULL,lookupTimeFrame,MODE_OPEN,z-x,x));

Twas working perfectly until I introduced this code.
Any ideas?
Thanks
CanuckCT
  • #2
  • Oct 16, 2010 8:17pm Oct 16, 2010 8:17pm
Here is the visual..
Attached Image (click to enlarge)
Click to Enlarge Name: 2010-10-16_2212.png Size: 72 KB

I could just post the indicator...but I want to understand it.
Thanks
CanuckCT
  • #3
  • Oct 16, 2010 8:21pm Oct 16, 2010 8:21pm
hey ct..... perhaps iopen only.......h

Inserted Code
  [color=#0000bb]double LastOpen [/color][color=#007700]= [/color][color=#0000bb]iOpen[/color][color=#007700]([/color][color=#0000bb]NULL[/color][color=#007700],[/color][color=#0000bb]lookupTimeFrame[/color][color=#007700],[/color][color=#0000bb]MODE_OPEN[/color][color=#007700],[/color][color=#0000bb]x[/color][color=#007700]);[/color]
to trade and code, keep both simple... no call to impress....h
  • #4
  • Oct 16, 2010 8:21pm Oct 16, 2010 8:21pm
,MODE_OPEN,z-x,x)) should be ,MODE_OPEN,z-x),x)
  • #5
  • Oct 16, 2010 8:33pm Oct 16, 2010 8:33pm
I get a wrong parameter count error with this....

Quoting hayseed
hey ct..... perhaps iopen only.......h

Inserted Code
  [color=#0000bb]double LastOpen [/color][color=#007700]= [/color][color=#0000bb]iOpen[/color][color=#007700]([/color][color=#0000bb]NULL[/color][color=#007700],[/color][color=#0000bb]lookupTimeFrame[/color][color=#007700],[/color][color=#0000bb]MODE_OPEN[/color][color=#007700],[/color][color=#0000bb]x[/color][color=#007700]);[/color]
  • #6
  • Oct 16, 2010 8:36pm Oct 16, 2010 8:36pm
And I also get a wrong parameter count with this change also...
Quoting CodeMeister
,MODE_OPEN,z-x,x)) should be ,MODE_OPEN,z-x),x)
  • #7
  • Oct 16, 2010 8:44pm Oct 16, 2010 8:44pm
Inserted Code

double LastOpen = iOpen(NULL,lookupTimeFrame,[color=Red]iOpen(NU  LL,lookupTimeFrame,MODE_OPEN,z-x,x)[/color]);

You are using a second iOpen as the shift value for the first iOpen.
  • #8
  • Oct 16, 2010 8:56pm Oct 16, 2010 8:56pm
If I use this....which I have tried...

double LastOpen = iOpen(NULL,lookupTimeFrame,0); (and I've tried 1 as a shift also)

I get this result. The pivot is stuck at the same level for every day...

Attached Image (click to enlarge)
Click to Enlarge Name: 2010-10-16_2255.png Size: 55 KB
  • #9
  • Oct 16, 2010 9:17pm Oct 16, 2010 9:17pm
Here is the indicator....1.2 works great...1.3 does not.
Attached File(s)
File Type: mq4 Pivot points - Barney Dog Breakout v1.2.mq4   11 KB | 396 downloads
File Type: mq4 Pivot points - Barney Dog Breakout v1.3.mq4   11 KB | 342 downloads
  • #10
  • Oct 16, 2010 10:44pm Oct 16, 2010 10:44pm
Inserted Code
 
double LastOpen = iOpen(NULL,lookupTimeFrame,x);

will look closer....h
to trade and code, keep both simple... no call to impress....h
  • #11
  • Last Post: Oct 16, 2010 11:34pm Oct 16, 2010 11:34pm
double LastOpen = Open[k];

this worked....knew it was simple.
Thanks guys.
Cheers
CanuckCT
0 traders viewing now