0
Закоментарить прямо в коде:
/*...*/

Разелить:
if(pattern1) ...; и if(pattern2) ...;


if((MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(High[1]-Open[1])>Ten*Point && Open[1]>Close[1] && MathAbs(High[1]-Open[1])>MathAbs(Close[1]-Open[1]))/*||((MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(High[1]-Open[1])>Ten*Point && Open[1]<Close[1] && MathAbs(High[1]-Close[1])>MathAbs(Close[1]-Open[1])))*/)
avatar

AM2

  • 7 декабря 2016, 12:00
0
Вечером буду доделывать, п.3. тоже добавлю там просто.
avatar

AM2

  • 7 декабря 2016, 11:45
0
Как доберусь, сообщу.
avatar

AM2

  • 6 декабря 2016, 21:47
0
Андрей, я написал в ТЗ (прочтите внимательно):… Логика работы совы остается прежняя..., исходный код менять не нужно, то есть входы по индикатору BBands! с сохранением последующей логики…

Желательно со скринами все делать, чтобы не было подобных недоразумений о чем я и просил в самом начале, поэтому я повторяю свой вопрос?

1) Функция открытия нового ордера, сразу же после закрытия серии ордеров или текущего ордера, должна быть отключаемая.Переменная в настройки- вкл/откл


Как это?
avatar

AM2

  • 6 декабря 2016, 21:40
0
Сделал вариант: www.opentraders.ru/downloads/1415/


//+------------------------------------------------------------------+
//|                                                          FL2.mq4 |
//|                                              Copyright 2016, AM2 |
//|                                      http://www.forexsystems.biz |
//+------------------------------------------------------------------+
#property copyright "Copyright © 2016, AM2"
#property link      "http://www.forexsystems.biz"
#property description "BBands expert advisor"

//--- Inputs
extern double Lots         = 0.1;  // лот
extern double KLot         = 2;    // умножение лота
extern double MaxLot       = 5;    // максимальный лот
extern int    StopLoss     = 5000; // лось
extern int    TakeProfit   = 5000; // язь
extern double Profit       = 50;   // язь в рублях
extern int    StartHour    = 0;    // час начала торговли
extern int    StartMin     = 30;   // минута начала торговли
extern int    EndHour      = 23;   // час окончания торговли
extern int    EndMin       = 30;   // минута окончания торговли
extern int    Shift        = 2;    // сдвиг
extern int    Slip         = 30;   // слипаж
extern int    Magic        = 123;  // магик
//----
extern string IndName      = "FL23_Decompil";
extern double LevelUP      = 0.3;      // уровень индикатора
extern double LevelDN      = -0.3;     // уровень индикатора

datetime t=0;
//+------------------------------------------------------------------+
//| Expert initialization function                                   |
//+------------------------------------------------------------------+
int OnInit()
  {
//---

//---
   return(INIT_SUCCEEDED);
  }
//+------------------------------------------------------------------+
//| Expert deinitialization function                                 |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
  {
//---

  }
//+------------------------------------------------------------------+
//| Check for open order conditions                                  |
//+------------------------------------------------------------------+
void PutOrder(int type,double price)
  {
   int r=0;
   color clr;
   double sl=0,tp=0;

   if(type==1 || type==3 || type==5)
     {
      clr=Red;
      if(StopLoss>0)   sl=NormalizeDouble(price+StopLoss*Point,Digits);
      if(TakeProfit>0) tp=NormalizeDouble(price-TakeProfit*Point,Digits);
     }

   if(type==0 || type==2 || type==4)
     {
      clr=Blue;
      if(StopLoss>0)   sl=NormalizeDouble(price-StopLoss*Point,Digits);
      if(TakeProfit>0) tp=NormalizeDouble(price+TakeProfit*Point,Digits);
     }

   r=OrderSend(NULL,type,Lot(),NormalizeDouble(price,Digits),Slip,sl,tp,"",Magic,0,clr);
   return;
  }
//+----------------------------------------------------------------------------+
//|  Автор    : Ким Игорь В. aka KimIV,  http://www.kimiv.ru                   |
//+----------------------------------------------------------------------------+
//|  Версия   : 30.04.2009                                                     |
//|  Описание : Возвращает флаг разрешения торговли по времени.                |
//+----------------------------------------------------------------------------+
//|  Параметры:                                                                |
//|    hb - часы времени начала торговли                                       |
//|    mb - минуты времени начала торговли                                     |
//|    he - часы времени окончания торговли                                    |
//|    me - минуты времени окончания торговли                                  |
//+----------------------------------------------------------------------------+
bool isTradeTimeInt(int hb=0,int mb=0,int he=0,int me=0)
  {
   datetime db, de;           // Время начала и окончания работы
   int      hc;               // Часы текущего времени торгового сервера

   db=StrToTime(TimeToStr(TimeCurrent(), TIME_DATE)+" "+(string)hb+":"+(string)mb);
   de=StrToTime(TimeToStr(TimeCurrent(), TIME_DATE)+" "+(string)he+":"+(string)me);
   hc=TimeHour(TimeCurrent());

   if(db>=de)
     {
      if(hc>=he) de+=24*60*60; else db-=24*60*60;
     }

   if(TimeCurrent()>=db && TimeCurrent()<=de) return(True);
   else return(False);
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
int CountTrades()
  {
   int count=0;
   for(int i=OrdersTotal()-1;i>=0;i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magic)
           {
            if(OrderType()<2) count++;
           }
        }
     }
   return(count);
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
double Lot()
  {
   double lot=Lots;
   if(CountTrades()>0) lot=NormalizeDouble(Lots*MathPow(KLot,CountTrades()),2);
   if(lot>MaxLot)lot=Lots;
   return(lot);
  }
//+------------------------------------------------------------------+
//| Закрытие позиции по типу ордера                                  |
//+------------------------------------------------------------------+
void CloseAll(int ot=-1)
  {
   bool cl;
   for(int i=OrdersTotal()-1;i>=0;i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magic)
           {
            if(OrderType()==0 && (ot==0 || ot==-1))
              {
               RefreshRates();
               cl=OrderClose(OrderTicket(),OrderLots(),NormalizeDouble(Bid,Digits),Slip,White);
              }
            if(OrderType()==1 && (ot==1 || ot==-1))
              {
               RefreshRates();
               cl=OrderClose(OrderTicket(),OrderLots(),NormalizeDouble(Ask,Digits),Slip,White);
              }
           }
        }
     }
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
double AllProfit()
  {
   double profit=0;
   for(int i=OrdersTotal()-1;i>=0;i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magic)
           {
            if(OrderType()<2) profit+=OrderProfit()+OrderSwap()+OrderCommission();
           }
        }
     }
   return (profit);
  }
//+------------------------------------------------------------------+
//| Check for open order conditions                                  |
//+------------------------------------------------------------------+
void OpenPos()
  {
//--- get Ind
   double fl=iCustom(NULL,0,IndName,0,Shift);

//--- sell conditions
   if(fl<LevelDN)
     {
      PutOrder(1,Bid);
     }
//--- buy conditions
   if(fl>LevelUP)
     {
      PutOrder(0,Ask);
     }
//---
  }
//+------------------------------------------------------------------+
//| OnTick function                                                  |
//+------------------------------------------------------------------+
void OnTick()
  {
   double fl=iCustom(NULL,0,IndName,0,Shift);

   if(t!=Time[0] && isTradeTimeInt(StartHour,StartMin,EndHour,EndMin))
     {
      OpenPos();
      t=Time[0];
     }

   if(CountTrades()>1 && AllProfit()>Profit) CloseAll();

   Comment("\n Profit: ",AllProfit(),
           "\n FL: ",fl);
  }
//+------------------------------------------------------------------+



Советник какое то время идет нормально но потом попадает вот в такую полосу:

avatar

AM2

  • 6 декабря 2016, 20:44
0
по такому условия никогда не закроет.

if(CountTrades()>1 && CountTrades()<2 && t!=Time[0])
avatar

AM2

  • 6 декабря 2016, 20:03
0
Покажите на скрине входы?
avatar

AM2

  • 6 декабря 2016, 19:44
0
Я же здесь не делал ничего.
avatar

AM2

  • 6 декабря 2016, 17:18
0
Сегодня или завтра буду смотреть.
avatar

AM2

  • 6 декабря 2016, 17:12
0
Значит ваш код не остановил торговлю, это можно сделать даже не отключая кнопку.

trade=false;
avatar

AM2

  • 6 декабря 2016, 17:10
0
Только дописать закрытие.
avatar

AM2

  • 6 декабря 2016, 17:08
0
Можно разделить эти условия или закоментарить одно из них.
avatar

AM2

  • 6 декабря 2016, 17:06
0
Массив Lsa_0 далее нигде не используется. Зачем он здесь? Ведь вся серия индюков FL содержит ее. В чем может быть дело?


Не используется, убирайте.
avatar

AM2

  • 6 декабря 2016, 12:19
0
Можно убрать обратное условие на закрытие, главное чтоб по времени закрывалось.


Закрытие по минутам есть, по сигналу можно отключить. Сейчас у меня в таком виде:

//+------------------------------------------------------------------+
//|                                                    TenSvechi.mq4 |
//|                                              Copyright 2016, AM2 |
//|                                      http://www.forexsystems.biz |
//+------------------------------------------------------------------+
#property copyright "Copyright 2016, AM2"
#property link      "http://www.forexsystems.biz"
#property version   "1.00"
#property strict

//--- Inputs
extern double Lots       = 0.1;      // лот
extern double KLot       = 1;        // умножение лота
extern double MaxLot     = 5;        // максимальный лот
extern int StopLoss      = 2000;     // лось
extern int TakeProfit    = 3000;     // язь
extern int Slip          = 30;       // реквот
extern int Telo          = 30;       // тело
extern int Ten           = 30;       // тень
extern int CloseTime     = 30;       // >0-закрытие по времени
extern int CloseSig      = 0;        // 1-закрытие по сигналу
extern int Magic         = 123;      // магик
//+------------------------------------------------------------------+
//| Expert initialization function                                   |
//+------------------------------------------------------------------+
int OnInit()
  {
   Comment("");
   return(INIT_SUCCEEDED);
  }
//+------------------------------------------------------------------+
//| Expert deinitialization function                                 |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
  {
   Comment("");
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
void PutOrder(int type,double price)
  {
   int r=0;
   color clr=Green;
   double sl=0,tp=0;

   if(type==1 || type==3 || type==5)
     {
      clr=Red;
      if(StopLoss>0) sl=NormalizeDouble(price+StopLoss*Point,Digits);
      if(TakeProfit>0) tp=NormalizeDouble(price-TakeProfit*Point,Digits);
     }

   if(type==0 || type==2 || type==4)
     {
      clr=Blue;
      if(StopLoss>0) sl=NormalizeDouble(price-StopLoss*Point,Digits);
      if(TakeProfit>0) tp=NormalizeDouble(price+TakeProfit*Point,Digits);
     }

   r=OrderSend(NULL,type,Lot(),NormalizeDouble(price,Digits),Slip,sl,tp,"",Magic,0,clr);
   return;
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
int CountTrades()
  {
   int count=0;
   for(int i=OrdersTotal()-1;i>=0;i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magic)
           {
            if(OrderType()<2) count++;
           }
        }
     }
   return(count);
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
void OpenPos()
  {
   if((MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(Open[1]-Low[1])>Ten*Point && Open[1]<Close[1] && MathAbs(Open[1]-Low[1])>MathAbs(Close[1]-Open[1]))||(MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(Open[1]-Low[1])>Ten*Point && Open[1]>Close[1] && MathAbs(Close[1]-Low[1])>MathAbs(Close[1]-Open[1])))
     {
      PutOrder(0,Ask);
     }

   if((MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(High[1]-Open[1])>Ten*Point && Open[1]>Close[1] && MathAbs(High[1]-Open[1])>MathAbs(Close[1]-Open[1]))||((MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(High[1]-Open[1])>Ten*Point && Open[1]<Close[1] && MathAbs(High[1]-Close[1])>MathAbs(Close[1]-Open[1]))))
     {
      PutOrder(1,Bid);
     }
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
double Lot()
  {
   double lot=Lots;

   if(OrderSelect(OrdersHistoryTotal()-1,SELECT_BY_POS,MODE_HISTORY))
     {
      if(OrderProfit()<0)
        {
         lot=OrderLots()*KLot;
        }
     }
   if(lot>MaxLot)lot=Lots;
   return(lot);
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
void CloseAll(int ot=-1)
  {
   bool cl;
   for(int i=OrdersTotal()-1;i>=0;i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magic)
           {
            if(OrderType()==0 && (ot==0 || ot==-1))
              {
               RefreshRates();
               cl=OrderClose(OrderTicket(),OrderLots(),NormalizeDouble(Bid,Digits),Slip,White);
              }
            if(OrderType()==1 && (ot==1 || ot==-1))
              {
               RefreshRates();
               cl=OrderClose(OrderTicket(),OrderLots(),NormalizeDouble(Ask,Digits),Slip,White);
              }
           }
        }
     }
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
void ClosePos()
  {
   for(int i=OrdersTotal()-1;i>=0;i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magic)
           {
            if(OrderType()==OP_BUY)
              {
               if((MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(High[1]-Open[1])>Ten*Point && Open[1]>Close[1] && MathAbs(High[1]-Open[1])>MathAbs(Close[1]-Open[1]))||((MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(High[1]-Open[1])>Ten*Point && Open[1]<Close[1] && MathAbs(High[1]-Close[1])>MathAbs(Close[1]-Open[1]))))
                 {
                  CloseAll(0);
                 }
              }
            if(OrderType()==OP_SELL)
              {
               if((MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(Open[1]-Low[1])>Ten*Point && Open[1]<Close[1] && MathAbs(Open[1]-Low[1])>MathAbs(Close[1]-Open[1]))||(MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(Open[1]-Low[1])>Ten*Point && Open[1]>Close[1] && MathAbs(Close[1]-Low[1])>MathAbs(Close[1]-Open[1])))
                 {
                  CloseAll(1);
                 }
              }
           }
        }
     }
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
datetime OrderOpen()
  {
   datetime t=0;
   for(int i=OrdersTotal()-1;i>=0;i--)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magic)
           {
            if(OrderType()<2) t=OrderOpenTime();
           }
        }
     }
   return(t);
  }
//+------------------------------------------------------------------+
//| Expert tick function                                             |
//+------------------------------------------------------------------+
void OnTick()
  {
   if(CountTrades()<1)
     {
      OpenPos();
     }
   if(CloseSig>0) ClosePos();
   if(TimeCurrent()-OrderOpen()>CloseTime*60 && CloseTime>0) CloseAll();
  }
//+------------------------------------------------------------------+
avatar

AM2

  • 6 декабря 2016, 11:55
0
А еще когда 2 длинных хвоста обратное условие на закрытие приводит к множественным переоткрытиям:


avatar

AM2

  • 5 декабря 2016, 22:08
0
если делать без цвета свечи совсем огроменные условия получаются и у меня где то ошибка закралась и ее не выловишь больно то теперь:


   if((MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(Open[1]-Low[1])>Ten*Point && Open[1]<Close[1] && MathAbs(Open[1]-Low[1])>MathAbs(Close[1]-Open[1]))||(MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(Open[1]-Low[1])>Ten*Point && Open[1]>Close[1] && MathAbs(Close[1]-Low[1])>MathAbs(Close[1]-Open[1])))
     {
      PutOrder(0,Ask);
     }

   if((MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(High[1]-Open[1])>Ten*Point && Open[1]>Close[1] && MathAbs(High[1]-Open[1])>MathAbs(Close[1]-Open[1]))||((MathAbs(Close[1]-Open[1])>Telo*Point && MathAbs(High[1]-Open[1])>Ten*Point && Open[1]<Close[1] && MathAbs(High[1]-Close[1])>MathAbs(Close[1]-Open[1]))))
     {
      PutOrder(1,Bid);
     }
avatar

AM2

  • 5 декабря 2016, 22:05
0
Обычно проще с нуля по ТЗ написать чем разбирать чужие каракули :) 
avatar

AM2

  • 5 декабря 2016, 21:39
0
Вот функция по процентам:


   if(EquityRisk>0)
     {
      double BALANCE=AccountInfoDouble(ACCOUNT_BALANCE);
      double EQUITY=AccountInfoDouble(ACCOUNT_EQUITY);
      Prosadka=100*(BALANCE-EQUITY)/BALANCE;
        {
         if(Prosadka>=EquityRisk)
           {
            CloseAll();
            Print("Closed All due to Stop Out");
           }
        }
     }

   if(EquityProfit>0)
     {
      BALANCE=AccountInfoDouble(ACCOUNT_BALANCE);
      EQUITY=AccountInfoDouble(ACCOUNT_EQUITY);
      Prosadka=100*(EQUITY-BALANCE)/BALANCE;
        {
         if(Prosadka>=EquityProfit)
           {
            CloseAll();
            Print("Closed All due to All Profit");
           }
        }
     }


Только вам вместо закрытия нужно флаг торговли переключать.

Здесь отключение кнопки: zakaz.opentraders.ru/33007.html
avatar

AM2

  • 5 декабря 2016, 21:27