Phoenix 6 Expert vs Simple - Page 5

ejlamarque · #41 (**permalink**) 03-16-2007, 11:10 PM

I am runing Phoenix 6 Alpha and Phoenix 6 Alpha Easy
I just post Expert and Journal report regard today in Live post.
It is ok or ...? Tell me please.
Good weekend

bertbin · #42 (**permalink**) 03-17-2007, 03:20 AM

Quote:

Originally Posted by ejlamarque

I am runing Phoenix 6 Alpha and Phoenix 6 Alpha Easy
I just post Expert and Journal report regard today in Live post.
It is ok or ...? Tell me please.
Good weekend

Yes and have a good we...

Pcontour · #43 (**permalink**) 03-18-2007, 02:57 AM

Quote:

case 1440:
W_Trade_PERIOD="D1"; break;
case 10080:
W_Trade_PERIOD="D1"; break; //xox D7
case 43200:
W_Trade_PERIOD="D1"; break; //xox D30

Should the last two D1 entries be D7 and D30?

Pcontour · #44 (**permalink**) 03-18-2007, 03:01 AM

Daraknor

This code is correct

if (!disableBuy && !disableSell)

But I would rather not see a double negative. People will have trouble with it.

if (enableBuy || enableSell)

I will send you the code by email

Pcontour · #45 (**permalink**) 03-18-2007, 03:14 AM

Daraknor

The section followed by

//it looks funny, but it above statement is properly closed.

It is properly closed but I think is actually not doing what you want, I added a few brackets and will send you the code.

Pcontour · #46 (**permalink**) 03-18-2007, 03:35 AM

Quote:

bool OrderModify( int ticket, double price, double stoploss, double takeprofit, datetime expiration, color arrow_color=CLR_NONE)
Modification of characteristics for the previously opened position or pending orders. If the function succeeds, the returned value will be TRUE. If the function fails, the returned value will be FALSE. To get the detailed error information, call GetLastError() function.

So

int results= OrderModify ( ticket, OrderOpenPrice(), sl, OrderTakeProfit(), 0, Red );

if(results!=(-1)) return(true);
else return(false);

could be coded

bool result= OrderModify ( ticket, OrderOpenPrice(), sl, OrderTakeProfit(), 0, Red );

return (result)

or the best perhaps - this will reduce a few CPU cycles

return ( OrderModify ( ticket, OrderOpenPrice(), sl, OrderTakeProfit(), 0, Red ) );

Daraknor
Which one will have less instruction?

autumnleaves · #47 (**permalink**) 03-19-2007, 01:56 PM

I figured out why the EASY signal is not working properly. It is currently taking the haClose value from the very first bar and sticking with it. haClose has to be defined so that it updates on every bar. Any suggestions.

daraknor · #48 (**permalink**) 03-19-2007, 10:17 PM

Quote:

Originally Posted by Pcontour

So

int results= OrderModify ( ticket, OrderOpenPrice(), sl, OrderTakeProfit(), 0, Red );

if(results!=(-1)) return(true);
else return(false);

could be coded

bool result= OrderModify ( ticket, OrderOpenPrice(), sl, OrderTakeProfit(), 0, Red );

return (result)

or the best perhaps - this will reduce a few CPU cycles

return ( OrderModify ( ticket, OrderOpenPrice(), sl, OrderTakeProfit(), 0, Red ) );

Daraknor
Which one will have less instruction?

I changed my style for order modify to this:

if(OrderModify(ticket,OrderOpenPrice(),sl,OrderTak eProfit(),0,Red)!=(-1)) LogError();

No variables needed, and the intended consequence is handled in line next to the event, instead of in a different chunk of code. LogError() basically does a GetLastError() and then write it to a log file.

daraknor · #49 (**permalink**) 03-19-2007, 10:32 PM

Quote:

Originally Posted by autumnleaves

I figured out why the EASY signal is not working properly. It is currently taking the haClose value from the very first bar and sticking with it. haClose has to be defined so that it updates on every bar. Any suggestions.

Old way is:

Code:

double haClose1 = (PRICE_OPEN + PRICE_HIGH + PRICE_LOW + PRICE_CLOSE)*(0.25);

But I'd recommend (and I think PContour might have as well):

Code:

double haClose1 = (Open[1] + High[1] + Low[1] + Close[1])*0.25;

The second one will give you the previous bar data. You can use Open[0] but it is the current bar and hasn't been completely written. To save a little on computing time if you are doing a lot of bar related activities is to only compute when a new bar is drawn:

Code:

int barcount; //Global variable at the top of the EA
double haClose1; //Global variable
init()
{ //... all of the normal init information
   barcount=Bars;
}
//Indicator code:
if(Bars>barcount) 
{
  haClose1 = (Open[1] + High[1] + Low[1] + Close[1])*0.25;
  barcount=Bars;
}

autumnleaves · #50 (**permalink**) 03-19-2007, 11:04 PM

Darak or anyone, do you have an idea how to fix the EASY signal? haClose needs to be defined on the current bar so that the result will be meaningful.

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Phoenix - Development+Suggestions - MQ4 in Post#1	daraknor	Phoenix	424	09-09-2008 12:13 AM
Phoenix 2007 (new thread)	Hendrick	Phoenix	1326	03-27-2008 11:34 PM
Phoenix is here!	Hendrick	Phoenix	374	02-06-2008 04:26 PM
Phoenix optimization	Prankie	Phoenix	173	10-17-2007 08:24 AM
Expert Advisor Upgrader - Turn Simple Into Superb	Scorpion	Tools and utilities	7	04-13-2007 02:01 AM

Register in Forex TSD!
Trading Systems Leaders in this forum (automated trading systems) are winning more than 3000 pips in a month (30000$ investing one lot every time). Click here to register and get more information