Обучение языку MQL4 под MetaTrader 4📌

Лажа какая-то получилась. @-)
Пришел к выводу что все надо переписывать.

oleguitar, не совсем понятно что вы имеете ввиду, если нужно подсчитать ордера Sell и SellStop, то в последнем условии нужно вместо и - (&&), поставить или - (||).

Ребят, прохожу 3й урок Сергея из курса и на данный момент столкнулся с проблемой:
Советник должен модифицировать открытые ордера после их заключения, но он модифицирует только первый ордер. В чем проблема - понять не смог. Найдите ошибку, кто может, пожалуйста=)

Вот код:

extern double Vol = 0.1;
extern int SL = 50;
extern int TP = 300;
extern int Slip = 5;
extern int Magic = 12345;
extern bool KanalClose = true;

extern string Par = "Параметры TMA";
extern string TimeFrame = "current time frame";
extern int HalfLength = 56;
extern int Price = PRICE_CLOSE;
extern double ATRMultiplier = 2.0;
extern int ATRPeriod = 100;
extern bool Interpolate = true;

double PriceHigh, PriceLow, Sl, Tp;
int ticket;

//+------------------------------------------------------------------+
//| Expert initialization function |
//+------------------------------------------------------------------+
int OnInit()
{
if (Digits == 3 || Digits == 5)
{
SL *= 10;
TP *= 10;
Slip *= 10;
}
return(INIT_SUCCEEDED);
}
//+------------------------------------------------------------------+
//| Expert deinitialization function |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
{


}
//+------------------------------------------------------------------+
//| Expert tick function |
//+------------------------------------------------------------------+
void OnTick()
{

PriceHigh = iCustom (Symbol(), 0, "TMA with Distances", TimeFrame, HalfLength, Price, ATRMultiplier, ATRPeriod, Interpolate, 1, 0);
PriceLow = iCustom (Symbol(), 0, "TMA with Distances", TimeFrame, HalfLength, Price, ATRMultiplier, ATRPeriod, Interpolate, 2, 0);


if (Bid >= PriceHigh && CountOrder() == 0)
{
Sl = NormalizeDouble(Bid + SL*Point, Digits);
Tp = NormalizeDouble(Bid - TP*Point, Digits);
bool Send1 = OrderSend (Symbol(), OP_SELL, Vol, Bid, Slip, 0, 0, "", Magic, 0, clrDarkBlue);
ticket = Send1;
if (ticket > 0)
{
ModOrder();
}
}

if (Bid {
Sl = NormalizeDouble(Ask - SL*Point, Digits);
Tp = NormalizeDouble(Ask + TP*Point, Digits);
bool Send2 = OrderSend (Symbol(), OP_BUY, Vol, Ask, Slip, 0, 0, "", Magic, 0, clrRed);
ticket = Send2;
if (ticket > 0)
{
ModOrder();
}
}
if (KanalClose == true)
{
if (Bid 0)
{
CloseOrderSell();
}
if (Bid >= PriceHigh && CountBuy()>0)
{
CloseOrderBuy();
}
}
}
//+------------------------------------------------------------------+
int CountBuy()
{
int count = 0;
for (int trade = OrdersTotal()-1; trade>=0; trade --)
{
bool Select3 = OrderSelect (trade, SELECT_BY_POS, MODE_TRADES);
if (OrderSymbol() == Symbol() && OrderMagicNumber() == Magic)
{
if (OrderType() == OP_BUY)
count++;
}
}
return(count);

}


//+------------------------------------------------------------------+
int CountSell()
{
int count = 0;
for(int trade = OrdersTotal() - 1; trade >=0; trade--)
{
bool Select4 = OrderSelect (trade, SELECT_BY_POS, MODE_TRADES);
if (OrderSymbol() == Symbol() && OrderMagicNumber() == Magic)
{
if (OrderType() == OP_SELL)
count++;
}

}

return(count);
}
//+------------------------------------------------------------------+
void CloseOrderSell()
{
for (int trade = OrdersTotal() - 1; trade >=0; trade--)
{
bool Select5 = OrderSelect(trade, SELECT_BY_POS, MODE_TRADES);
if (Select5 == true)
{
if (OrderSymbol() == Symbol() && OrderMagicNumber() == Magic && OrderType() == OP_SELL)
{
bool Close1 = OrderClose (OrderTicket(), OrderLots(), Ask, Slip, clrDarkMagenta);
}
}
}
}
//+------------------------------------------------------------------+
void CloseOrderBuy()
{
for (int trade = OrdersTotal() - 1; trade >=0; trade--)
{
bool Select6 = OrderSelect(trade, SELECT_BY_POS, MODE_TRADES);
if (Select6 == true)
{
if (OrderSymbol() == Symbol() && OrderMagicNumber() == Magic && OrderType() == OP_BUY)
{
bool Close2 = OrderClose (OrderTicket(), OrderLots(), Bid, Slip, clrDarkMagenta);
}
}
}
}
//+------------------------------------------------------------------+
int CountOrder()
{
int count = 0;
for(int trade = OrdersTotal() - 1; trade >=0; trade--)
{
bool Select7 = OrderSelect (trade, SELECT_BY_POS, MODE_TRADES);
if (OrderSymbol() == Symbol() && OrderMagicNumber() == Magic)
{
count++;
}

}

return(count);
}
//+------------------------------------------------------------------+
void ModOrder()
{
bool Select8 = OrderSelect(ticket, SELECT_BY_TICKET, MODE_TRADES);
if(Select8 == true)
{
bool Modify = OrderModify (ticket, OrderOpenPrice(), Sl, Tp, 0, clrAqua);
}
}
//+------------------------------------------------------------------+

В чем проблема - понять не смог. Найдите ошибку, кто может, пожалуйста=)

Oll, спасибо, помогло. Если можешь, объясни, что ему не понравилось в прошлом варианте. Важно понять это.

chistaia-rodina, открывает ордер, модифицирует(Sl, Tp), закрывает, открывает новый - больше не модифицирует, остальные также. Надеюсь, понятно=)
Если не ошибаюсь, то функция указана правильно, и пересчет ордеров там не нужен потому, что мы ссылаемся к ордеру по тикету. Вызвав функцию в цикле она ссылается на Sl и Tp, характерные для этого цикла. Проблемы быть не должно. К тому же изначально модификация одера вызывалась не отдельной функцией, проблема была та же=)
Если не прав, поправьте. Только начал обучение, многого не понимаю.

Подумал еще, понял, что пересчет ордеров не помешал бы, если функция будет использоваться еще, в других обстоятельствах, а в данном случае и без него можно=)

Oll, спасибо, помогло. Если можешь, объясни, что ему не понравилось в прошлом варианте. Важно понять это.

Ясно. Спасибо. Это я понял.

chistaia-rodina, если я вас правильно понял, для этого в функции нужно указать не void тип данных, а, например, int. Далее произвести поиск ордеров через цикл for и сделать выборку по параметрам. Если не прав, поправьте.
На счет Sl и Tp, в циклах для Buy и для Sell сделок они рассчитываются немного по разному, и, как по мне, разницы большой в количестве текста не будет, будем ли мы ее ставить в расчете функции (тогда понадобится 2 функции для каждого типа сделок) или в цикле старт.

Перечитал еще раз ваше сообщение. Если мы сохраняем тип void, то нам эту функцию необходимо будет прописать не внутри цикла по открытию сделки, а вне оного. Я вас правильно понял?

Ну вот так:
_________________________________________________
void OrderModify()
{
for и так далее, вся обработка внутри скобок

}
_________________________________________________

Прописываем всё отдельно в виде пользовательской функции.
В основном коде, напишем просто:

if (CountBuy() > 0 || CountSell() > 0)
OrderModify();

Да, это я понял. Я имел ввиду, вызов функции мы делаем уже не из цикла с открытием ордера(как это было раньше), а просто из старта?! Ну, в общем, я понял. Спасибо!=)

Попробуй этот вариант, пошаманил маненько, авось поможет

FractalGrid_Expert.mq411 скач.

огромное человеческое спасибо! с понедельника начну.

Попробуй этот вариант, пошаманил маненько, авось поможет

Советник открывает ордера при каждом тике, как запретить?

Советник открывает ордера при каждом тике, как запретить?

Всем привет!

Есть такой вопрос:
Я хочу, чтобы при выполнении условия на покупку или продажу открывался не один ордер, а два. Мне продублировать условия с разными мэджиками или как-то по-другому сделать можно?

Всем привет!

Есть такой вопрос:
Я хочу, чтобы при выполнении условия на покупку или продажу открывался не один ордер, а два. Мне продублировать условия с разными мэджиками или как-то по-другому сделать можно?

Сделал с разными мэджиками, работает как нужно