Доработка ботов (советников, индикаторов) vol. 2

1_Lexa · 12.04.2020

Что тут надо изменить?
/////////////////////////////////////////////////
// Close the order but only if it is in profit //
/////////////////////////////////////////////////

void CloseIfInProfit(string symb , int index)
{
for (int i = 0;i <= OrdersTotal();i++)
{
if( OrderSelect(i,SELECT_BY_POS,MODE_TRADES) )
{
if( OrderSymbol() == symb )
{
double profit = ( OrderProfit() - OrderCommission () - OrderSwap() );
if( ( OrderType() == OP_BUY || OrderType() == OP_SELL) && ( profit >= 0 ) && (OrderMagicNumber() == MAGIC) )
{
OrderClose(OrderTicket(),OrderLots(),MarketInfo(symb,MODE_BID),100,Red); // Close any open orders
ObjectSetText("Txt2"+symb,"",TextSize,"Arial"); // Gets rid of entry criteria message.
}
if (OrderType() == OP_BUYSTOP || OrderType() == OP_SELLSTOP)
{
OrderDelete(OrderTicket()); // Delete any pending orders
ObjectSetText("Txt2"+symb,"",TextSize,"Arial"); // Gets rid of entry criteria message.
}
}
}
}
partial[index] = false;
}

1_Lexa · 12.04.2020

Пытаюсь редактировать, но выходит ещё хуже

AlexeNP · 12.04.2020

1_Lexa сказал(а):
Что тут надо изменить?

вроде был же разговор о правильных ценах открытия и закрытия ордеров... даже ссылка была вроде по этой теме... стоит ли тратить время и силы на советник в котором допущены такие грубые ошибки?

Код:

/////////////////////////////////////////////////
// Close the order but only if it is in profit //
/////////////////////////////////////////////////
void CloseIfInProfit(string symb, int index)
  {
   for(int i = OrdersTotal()-1; i >= 0; i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol() == symb && OrderMagicNumber() == MAGIC)
           {
            double profit = OrderProfit() - OrderCommission() + OrderSwap();
            if(OrderType() == OP_BUY  && profit >= 0)
              {
               OrderClose(OrderTicket(),OrderLots(),MarketInfo(symb,MODE_BID),100,Red); // Close any open orders !!! правильная цена закрытия
               ObjectSetText("Txt2"+symb,"",TextSize,"Arial"); // Gets rid of entry criteria message. Вот вся эта переписка какую нагрузку несет?
              }

            if(OrderType() == OP_SELL && profit >= 0)
              {
               OrderClose(OrderTicket(),OrderLots(),MarketInfo(symb,MODE_ASK),100,Red); // Close any open orders !!! правильная цена закрытия
               ObjectSetText("Txt2"+symb,"",TextSize,"Arial"); // Gets rid of entry criteria message.
              }
            if(OrderType() == OP_BUYSTOP || OrderType() == OP_SELLSTOP)
              {
               OrderDelete(OrderTicket()); // Delete any pending orders
               ObjectSetText("Txt2"+symb,"",TextSize,"Arial"); // Gets rid of entry criteria message.
              }
           }
        }
     }
   partial[index] = false;//это что, куда, зачем?
  }

1_Lexa · 12.04.2020

Не знаю что с ним делать

1_Lexa · 12.04.2020

AlexeNP сказал(а):

вроде был же разговор о правильных ценах открытия и закрытия ордеров... даже ссылка была вроде по этой теме... стоит ли тратить время и силы на советник в котором допущены такие грубые ошибки?

Код:

/////////////////////////////////////////////////
// Close the order but only if it is in profit //
/////////////////////////////////////////////////
void CloseIfInProfit(string symb, int index)
  {
   for(int i = OrdersTotal()-1; i >= 0; i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol() == symb && OrderMagicNumber() == MAGIC)
           {
            double profit = OrderProfit() - OrderCommission() + OrderSwap();
            if(OrderType() == OP_BUY  && profit >= 0)
              {
               OrderClose(OrderTicket(),OrderLots(),MarketInfo(symb,MODE_BID),100,Red); // Close any open orders !!! правильная цена закрытия
               ObjectSetText("Txt2"+symb,"",TextSize,"Arial"); // Gets rid of entry criteria message. Вот вся эта переписка какую нагрузку несет?
              }

            if(OrderType() == OP_SELL && profit >= 0)
              {
               OrderClose(OrderTicket(),OrderLots(),MarketInfo(symb,MODE_ASK),100,Red); // Close any open orders !!! правильная цена закрытия
               ObjectSetText("Txt2"+symb,"",TextSize,"Arial"); // Gets rid of entry criteria message.
              }
            if(OrderType() == OP_BUYSTOP || OrderType() == OP_SELLSTOP)
              {
               OrderDelete(OrderTicket()); // Delete any pending orders
               ObjectSetText("Txt2"+symb,"",TextSize,"Arial"); // Gets rid of entry criteria message.
              }
           }
        }
     }
   partial[index] = false;//это что, куда, зачем?
  }

partial[index] = false;//это что, куда, зачем?- Я его удаляю, или пишу что-то другое, появляется куча ошибок

AlexeNP · 12.04.2020

1_Lexa сказал(а):
Не знаю что с ним делать

и об этом речь была... Функция типа void должна возвращать ничего, даже 0... то есть запись типа return(0); - это ошибка

AlexeNP · 12.04.2020

1_Lexa сказал(а):
partial[index] = false;//это что, куда, зачем?- Я его удаляю, или пишу что-то другое, появляется куча ошибок

да весь этот советник и состоит из кучи мелких ошибок собранных в одну большую... брось ты это гиблое дело - проще новый написать, чем этот до ума довести

BonGo71 · 12.04.2020

Доброго! Выкладываю самый лучший индикатор из всех, что видел и тестировал, но он компилируется с ошибками, не выводит алерты( Возможно можно еще что-то выжать из него, модифицировать.

1_Lexa · 12.04.2020

Он по этим индикаторам работает, к чему их можно прикрепить?

Sapper · 12.04.2020

BonGo71 сказал(а):
Доброго! Выкладываю самый лучший индикатор из всех, что видел и тестировал, но он компилируется с ошибками, не выводит алерты( Возможно можно еще что-то выжать из него, модифицировать.

Попробуйте эти.

BonGo71 · 12.04.2020

Sapper сказал(а):
Попробуйте эти.

сенк ) 2й годный, без ошибок компилится )

вячеслав1946 · 12.04.2020

AlexeNP
Благодарю. Как раз то что надо.

vladradon · 12.04.2020

1_Lexa сказал(а):
Он по этим индикаторам работает, к чему их можно прикрепить?

Он не работает по ним - у него все расчеты внутри. Он работает в оба направления - просто у него расчеты на вход такие, что некоторые пары торгуют только в одном направлении в основном. Если EURUSD он в этом году только в селл торгует, то на USDCHF, которая отрицательно коррелирует с EURUSD, он торгует только в бай (см. скрин). Код немного подремонтировал.

1_Lexa · 12.04.2020

vladradon сказал(а):
Он не работает по ним - у него все расчеты внутри. Он работает в оба направления - просто у него расчеты на вход такие, что некоторые пары торгуют только в одном направлении в основном. Если EURUSD он в этом году только в селл торгует, то на USDCHF, которая отрицательно коррелирует с EURUSD, он торгует только в бай (см. скрин). Код немного подремонтировал.

Спасибо, что обратили на этот советник внимание. Он на селл начал торговать, после небольших изменений. До этого торговал только в бай

1_Lexa · 12.04.2020

Просто у меня есть некоторые вопросы, что и как исправить, но похоже от меня уже устали

1_Lexa · 12.04.2020

vladradon сказал(а):
Он не работает по ним - у него все расчеты внутри. Он работает в оба направления - просто у него расчеты на вход такие, что некоторые пары торгуют только в одном направлении в основном. Если EURUSD он в этом году только в селл торгует, то на USDCHF, которая отрицательно коррелирует с EURUSD, он торгует только в бай (см. скрин). Код немного подремонтировал.

Ещё раз спасибо, что исправили ошибки

1_Lexa · 12.04.2020

Снова ошибки

vladradon · 12.04.2020

1_Lexa сказал(а):
Снова ошибки

Это ошибки истории котировок - к сову не имеют отношения. Те изменения, которые вы для селла делали я не делал - я взял первоначальный вариант. Просто скомпилировал его с жесткими условиями и в нем появилась куча ошибок, из-за которых он просто вылетал на старте.

Sapper · 12.04.2020

1_Lexa сказал(а):
Что тут надо изменить?
/////////////////////////////////////////////////
// Close the order but only if it is in profit //
/////////////////////////////////////////////////

void CloseIfInProfit(string symb , int index)
{
for (int i = 0;i <= OrdersTotal();i++)
{
if( OrderSelect(i,SELECT_BY_POS,MODE_TRADES) )
{
if( OrderSymbol() == symb )
{
double profit = ( OrderProfit() - OrderCommission () - OrderSwap() );
if( ( OrderType() == OP_BUY || OrderType() == OP_SELL) && ( profit >= 0 ) && (OrderMagicNumber() == MAGIC) )
{
OrderClose(OrderTicket(),OrderLots(),MarketInfo(symb,MODE_BID),100,Red); // Close any open orders
ObjectSetText("Txt2"+symb,"",TextSize,"Arial"); // Gets rid of entry criteria message.
}
if (OrderType() == OP_BUYSTOP || OrderType() == OP_SELLSTOP)
{
OrderDelete(OrderTicket()); // Delete any pending orders
ObjectSetText("Txt2"+symb,"",TextSize,"Arial"); // Gets rid of entry criteria message.
}
}
}
}
partial[index] = false;
}

Изменить надо тему.
Вам сюда:
https://forexsystemsru.com/threads/izuchaem-jazyk-programmirovanija-mql4.87869/page-16
или сюда:

https://forexsystemsru.com/threads/vashi-voprosy-po-jazyku-mql4.65289/page-376

1_Lexa · 13.04.2020

До изменения торговал в плюс, и после редактирования стал торговать в минус

Доработка ботов (советников, индикаторов) vol. 2

Активный участник

Активный участник

Вложения

Гуру форума

Активный участник

Вложения

Активный участник

Гуру форума

Гуру форума

Активный участник

Вложения

Активный участник

Вложения

......

Вложения

Активный участник

Активный участник

Программист

Вложения

Активный участник

Вложения

Активный участник

Активный участник

Активный участник

Вложения

Программист

......

Активный участник

Вложения