Помогите найти ошибку в советнике

res=OrderSend(Symbol(),OP_SELL,LotsOptimized(),Bid,3,Ask+Stoploss*Point,Bid-TakeProfit,"",MAGICMA,0,0,Red;

Забыл в конце поставить скобку ;-)
MagicNumber нужен, что бы отличить те ордера, которые открыл эксперт от всех остальных (ведь ты можеш открывать ордера вручную на том же акаунте)

Спасибо большое, все глаза просмотрел, а редактор при компилировании ошибку чувствует, а показать не может.

А не подсобите с моими потугами - где-то задвоено, где-то противоречит, где-то можно проще и т.д., и вообще, чтоб не сливал?
ijs-admin@mriya.com. Спасибо.

" и вообще, чтоб не сливал "

это мечта у всех кто тут - что бы был советник который не сливает!

в одной ситуации он будет давать профит - но настанет момент когда правило заложенное в него не будут работать
вопрос как раз в том что бы создать такой советник который будет переключать правила под разные ситуации на рынке
если думается - ставим пару супер - индикаторов и получаем грааль - это шибка

У меня тоже вопрос по этому советнику. В качестве тренировки решила вместо стандартных мувингов вставить J2JMA (by Nikolay Kositsin). Прогоняла в тестере и такое ощущение, что советник этот индикатор не видит – позиции не открываются. Подскажите, в чем ошибка?

//+------------------------------------------------------------------+
//| Moving Average.mq4 |
//| Copyright © 2005, MetaQuotes Software Corp. |
//| https://www.metaquotes.net// |
//+------------------------------------------------------------------+
#define MAGICMA 20050610

extern double Lots = 0.1;
extern double MaximumRisk = 0.02;
extern double DecreaseFactor = 3;
extern double MovingPeriod = 12;
extern double MovingShift = 6;
extern string J2JMA="J2JMA";
extern int fast1=7;
extern int fast2=7;
extern int slow1=21;
extern int slow2=5;
//+------------------------------------------------------------------+
//| Calculate open positions |
//+------------------------------------------------------------------+
int CalculateCurrentOrders(string symbol)
{
int buys=0,sells=0;
//----
for(int i=0;i<OrdersTotal();i++)
{
if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==false) break;
if(OrderSymbol()==Symbol() && OrderMagicNumber()==MAGICMA)
{
if(OrderType()==OP_BUY) buys++;
if(OrderType()==OP_SELL) sells++;
}
}
//---- return orders volume
if(buys>0) return(buys);
else return(-sells);
}
//+------------------------------------------------------------------+
//| Calculate optimal lot size |
//+------------------------------------------------------------------+
double LotsOptimized()
{
double lot=Lots;
int orders=HistoryTotal(); // history orders total
int losses=0; // number of losses orders without a break
//---- select lot size
lot=NormalizeDouble(AccountFreeMargin()*MaximumRisk/1000.0,1);
//---- calcuulate number of losses orders without a break
if(DecreaseFactor>0)
{
for(int i=orders-1;i>=0;i--)
{
if(OrderSelect(i,SELECT_BY_POS,MODE_HISTORY)==false) { Print("Error in history!"); break; }
if(OrderSymbol()!=Symbol() || OrderType()>OP_SELL) continue;
//----
if(OrderProfit()>0) break;
if(OrderProfit()<0) losses++;
}
if(losses>1) lot=NormalizeDouble(lot-lot*losses/DecreaseFactor,1);
}
//---- return lot size
if(lot<0.1) lot=0.1;
return(lot);
}
//+------------------------------------------------------------------+
//| Check for open order conditions |
//+------------------------------------------------------------------+
void CheckForOpen()
{
int res;
double f,fp,s,sp;
//---- go trading only for first tiks of new bar
if(Volume[0]>1) return;
//---- get Moving Average
f=iCustom(Symbol(),0,"J2JMA.mq4",fast1,fast2,5,5,0,0,1);
fp=iCustom(Symbol(),0,"J2JMA.mq4",fast1,fast2,5,5,0,0,2);
s=iCustom(Symbol(),0,"J2JMA.mq4",slow1,slow2,5,5,0,0,1);
sp=iCustom(Symbol(),0,"J2JMA.mq4",slow1,slow2,5,5,0,0,2);
//---- sell conditions
if(fp>=sp && f<s)
{
res=OrderSend(Symbol(),OP_SELL,LotsOptimized(),Bid,3,0,0,"",MAGICMA,0, Red);
return;
}
//---- buy conditions
if(fp<=sp && f>s)
{
res=OrderSend(Symbol(),OP_BUY,LotsOptimized(),Ask,3,0,0,"",MAGICMA,0, Blue);
return;
}
//----
}
//+------------------------------------------------------------------+
//| Check for close order conditions |
//+------------------------------------------------------------------+
void CheckForClose()
{
double f,fp,s,sp;
//---- go trading only for first tiks of new bar
if(Volume[0]>1) return;
//---- get Moving Average
f=iCustom(Symbol(),0,"J2JMA.mq4",fast1,fast2,5,5,0,0,1);
fp=iCustom(Symbol(),0,"J2JMA.mq4",fast1,fast2,5,5,0,0,2);
s=iCustom(Symbol(),0,"J2JMA.mq4",slow1,slow2,5,5,0,0,1);
sp=iCustom(Symbol(),0,"J2JMA.mq4",slow1,slow2,5,5,0,0,2);
//----
for(int i=0;i<OrdersTotal();i++)
{
if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==false) break;
if(OrderMagicNumber()!=MAGICMA || OrderSymbol()!=Symbol()) continue;
//---- check order type
if(OrderType()==OP_BUY)
{
if(fp>=sp && f<s) OrderClose(OrderTicket(),OrderLots(),Bid,3,White);
break;
}
if(OrderType()==OP_SELL)
{
if(fp<=sp && f>s) OrderClose(OrderTicket(),OrderLots(),Ask,3,White);
break;
}
}
//----
}
//+------------------------------------------------------------------+
//| Start function |
//+------------------------------------------------------------------+
void start()
{
//---- check for history and trading
if(Bars<100 || IsTradeAllowed()==false) return;
//---- calculate open orders by current symbol
if(CalculateCurrentOrders(Symbol())==0) CheckForOpen();
else CheckForClose();
//----
}
//+------------------------------------------------------------------+

Все что нужно для работы индикатора установлено куда надо, индикатор рисуется (и в тестере тоже).

С уважением.