Спасибо, что ответили. Tip=-1; вставил на 86-ю строку. Теперь при обратном пересечении после правильного закрытия ордера тут же, в ту же минуту, постоянно открывает и мгновенно закрывает очередной ордер. И таким образом в тестере за 22 минуты открыл и закрыл более 3-х тысяч ордеров! Ну и естественно слил депозит
.
Вот смотри: как войти в туалет - включаем свет, по выходу - выключаем, делаем это вручную или автоматика.
Это похоже на просьбу написать советник, а не вопросы по языку. https://forexsystemsru.com/threads/predlozhenija-po-avtomatizacii-torgovli.65203/
Это условие всегда будет false. Просто потому что нельзя выбрать открытый ордер с индексом -1.
Правильно. Это простейший вариант. Для тестера потянет.
Спасибо,уже придумала)))
1. Open[0] это цена открытия бара, а не цена открытия ордера.
Нужно помнить что каждая функция, это код, а переменная, это ячейки памяти. Если есть возможность лишний раз не выполнять функцию, надо писать так. Вызвал функцию один раз, поместил результат в переменную и юзай её сколько надо.
Спасибо за ответ. То есть, предположим, вызов SummaryProfit() при открытие новой свечи.
Сколько раз выполняется TimeCurrent() ? А можно выполнить один раз. И такого, что можно сократить много.
double SummaryProfit()
{
double Sum = 0;
//Суммарный профит по всем открытым ордерам
for(int i=0;i < OrdersTotal();i++)
{
if(OrderSelect(i, SELECT_BY_POS, MODE_TRADES))
if(OrderType() < 2 && OrderSymbol() == _Symbol && OrderMagicNumber() == Magik) //Только для рыночных ордеров по текущему символу и с магиком совы - что не нужно просто удалаяем
Sum += OrderProfit()+ OrderSwap() + OrderCommission(); //включая своп и комиссию
}
//Сумма по ордерам, закрытым сегодня:
for( int i=OrdersHistoryTotal()-1;i>=0;i--)
{
if(OrderSelect(i, SELECT_BY_POS, MODE_HISTORY))
{
if(
OrderType() < 2 //
&& TimeYear(OrderCloseTime()) == TimeYear(TimeCurrent()) //Год
&& TimeDayOfYear(OrderCloseTime()) == TimeDayOfYear(TimeCurrent()) //и День соотвествуют текущему
&& OrderSymbol() == _Symbol //По текущему символу
&& OrderMagicNumber() == Magik ) //По заданному магику
Sum += OrderProfit()+ OrderSwap() + OrderCommission();
}
}
return(Sum);
}Нет, TimeCurrent() выполняется количество рыночных ордеров в истории умножить на 2. А можно выполнять 1 раз. TimeYear(), TimeDayOfYear(), OrderCloseTime() можно выполнять в 2 раза меньше раз. Используйте переменные вместо лишних вызовов функций.
Спасибо. Попробую.
datetime day;
double Balanse=0;
if (day!=TimeDay(TimeCurrent()))
{
Balanse=NormalizeDouble( AccountBalance(),Digits);
day=TimeDay(TimeCurrent());
}
if((AccountEquity() < (Balanse-maxLoss)) || (AccountEquity() > (maxProfit+Balanse)))
{CloseOrder();return;}
//+------------------------------------------------------------------+
//| |
//+------------------------------------------------------------------+
void StopTrade(double maxProfit, double maxLoss)
{
static MqlDateTime tdDay= {0};
MqlDateTime tdCurrent;
TimeToStruct(TimeCurrent(),tdCurrent);
if(tdDay.day!=tdCurrent.day)
{
tdDay=tdCurrent;
double Balanse=NormalizeDouble(AccountBalance(),Digits);
if((AccountEquity() < (Balanse-maxLoss)) || (AccountEquity() > (maxProfit+Balanse)))
{
CloseOrder();
return;
}
}
}//+------------------------------------------------------------------+
//| |
//+------------------------------------------------------------------+
void StopTradeByNewDay(double maxProfit, double maxLoss)
{
static datetime dtLastbarD1=0;
datetime dtCurrentBarD1=iTime(_Symbol,PERIOD_D1,0);
if(dtLastbarD1!=dtCurrentBarD1)
{
dtLastbarD1=dtCurrentBarD1;
double Balanse=NormalizeDouble(AccountBalance(),Digits);
if((AccountEquity() < (Balanse-maxLoss)) || (AccountEquity() > (maxProfit+Balanse)))
{
CloseOrder();
return;
}
}
}Здравствуйте. Спасибо за ответ. Вы правы, условия работают. Ошибка была в переменной Balanse. Ваш вариант тоже интересный, завтра попробую.
input double maxLoss = 500; //Максимальный убыток за день
input double maxProfit = 500; //Максимальный профит за день
datetime day;
double Balanse=0;
void OnTick()
{
if (day!=TimeDay(TimeCurrent()))
{
Balanse=NormalizeDouble( AccountBalance(),Digits);
day=TimeDay(TimeCurrent());
}
if((AccountEquity() < (Balanse-maxLoss)) || (AccountEquity() > (maxProfit+Balanse))) {CloseMarketOrder();return;}
//
//
Условия...
//
}