Errori, bug, domande

Alexey Viktorov 2022.01.30 20:24 #31421

Wizard #:
Il codice ha bisogno della linea PositionSelectByTicket... Se è così, come faccio a scrivere correttamente il biglietto al suo interno, usando PositionGetTicket(i) o lasciando ilPositionGetInteger(POSITION_TICKET) ? Grazie in anticipo!

Non essere scortese... La documentazione non dice chiaramente

La funzione restituisce un ticket di posizione per l'indice nella lista delle posizioni aperte e seleziona automaticamente questa posizione per lavorare ulteriormente con essa

Se la posizione è già selezionata, perché riselezionarla con la funzione PositionSelectByTicket

[Eliminato] 2022.01.30 20:52 #31422

Alexey Viktorov #:

Non per essere scortese... La documentazione non dice chiaramente

Se la posizione è già selezionata, perché riselezionarla con la funzione PositionSelectByTicket

Grazie per la vostra risposta! Me lo chiedevo, perché avendo guardato nella libreria <Trade/Trade.mqh>, la funzione PositionClose(const ulong ticket,const ulong deviation) usa anche PositionSelectByTicket. E la stessa funzione PositionClose( const ulong ticket,const ulong deviation ) è spesso usata dai codificatori in combinazione con il ciclo for(int i = PositionsTotal()-1; i >= 0; i--) , che si estende su tutte le posizioni. E mi sto chiedendo se ne sto usando uno in più da qualche parte.

Domande da un "manichino" Domande dai principianti MQL5 MT4 BronzeWarrior e SilverWarrior

Roman 2022.01.30 20:57 #31423

Wizard #:
Ciao, la linea PositionSelectByTicket è necessaria nel codice... Se sì, come scrivere correttamente un biglietto al suo interno, attraverso PositionGetTicket(i) o lasciarePositionGetInteger(POSITION_TICKET)? Grazie in anticipo!

int IsPositions()
{
   int pos = 0;     
   
   int total = PositionsTotal();
   for(int i = total-1; i >= 0; i--)
   {
      if(!PositionGetTicket(i))
         continue;        
          
      if(PositionGetString(POSITION_SYMBOL) == Symbol_T && PositionGetInteger(POSITION_MAGIC) == EXPERT_MAGIC)
         pos = (PositionGetInteger(POSITION_TYPE) == POSITION_TYPE_BUY) ? 1 : (PositionGetInteger(POSITION_TYPE) == POSITION_TYPE_SELL) ? -1 : 0;
   }
   
   return(pos);
}

[Eliminato] 2022.01.30 21:06 #31424

Roman #:

Grazie! Quindi PositionGetTicket(i) sarà sufficiente ePositionSelectByTicket non sarà necessario.

Alexey Viktorov 2022.01.31 06:03 #31425

Wizard #:
Grazie per la vostra risposta! Ho iniziato a chiedermelo, perché avendo guardato nella libreria <Trade/Trade.mqh>, la funzione PositionClose(const ulong ticket,const ulong deviation) usa anche PositionSelectByTicket. E la stessa funzione PositionClose( const ulong ticket,const ulong deviation ) è spesso usata dai codificatori in combinazione con il ciclo for(int i = PositionsTotal()-1; i >= 0; i--) , che si estende su tutte le posizioni. E mi sto chiedendo se ne sto usando uno in più da qualche parte.

La funzione PositionClose(const ulong ticket,const ulong deviation) della libreria riceve un ticket della posizione da chiudere, ma nessuno sa come il ticket è stato ricevuto e se la posizione esiste.

Quindi, PositionSelectByTicket controlla soprattutto se c'è qualcosa da chiudere. E perché avete deciso che tutte le posizioni sono spesso chiuse nel ciclo? Non necessariamente...

Non ho mai studiato Domande dai principianti MQL5 Domande di OrderSend()

[Eliminato] 2022.01.31 14:49 #31426

Alexey Viktorov #:

La funzione PositionClose(const ulong ticket,const ulong deviation) della libreria riceve un ticket della posizione da chiudere, ma nessuno sa come il ticket è stato ricevuto e se la posizione esiste.

Quindi PositionSelectByTicket controlla soprattutto se c'è qualcosa da chiudere. E perché avete deciso che tutte le posizioni sono spesso chiuse nel ciclo? Non necessariamente...

Sì, ora ho capito tutto. Recentemente ho avuto un EA, l'ho scritto seguendo il tuo esempio (con le medie) da un forum, combinandolo con il codice di Timur Mashnin. Ora ho deciso di cambiare le condizioni lì e riscriverle un po', sostituendo tutto con funzioni di sistema, senza librerie. Se con un semplice PositionSelect tutto era molto semplice, una posizione - una scelta, allora con il ciclo qui ovviamente ho bisogno di più considerazioni e logica.

[ARCHIVIO]Qualsiasi domanda da principiante, L'apprendimento automatico nel trading: Qualsiasi domanda da principiante,

Mikhail Rudyk 2022.01.31 14:54 #31427

Ciao

Potete per favore aiutarmi

con il codice.

Fatto un indicatore nel tester funziona correttamente

Quando lo metto sul grafico non viene visualizzato correttamente

Non riesco a capire perché sia sbagliato.

//+------------------------------------------------------------------+
//|                                                        Oscil.mq5 |
//|                                  Copyright 2021, MetaQuotes Ltd. |
//|                                             https://www.mql5.com |
//+------------------------------------------------------------------+
#property copyright "Copyright 2021, MetaQuotes Ltd."
#property link      "https://www.mql5.com"
#property version   "1.00"
#property indicator_separate_window
#property  indicator_buffers 5
#property indicator_plots   5

#property  indicator_color1  clrNONE
#property  indicator_color2  clrRoyalBlue
#property  indicator_color3  clrPink
#property  indicator_color4  clrAqua
#property  indicator_color5  clrYellow

#property  indicator_width1 1
#property  indicator_width2 5
#property  indicator_width3 5
#property  indicator_width4 5
#property  indicator_width5 5

double MainLine[];
double UpLine[];
double DnLine[];
double muls[];
double x,y,z;
double price;
double mulSum=0;
double Pi   = 3.1415926535;
bool LastUp = false;
bool GoUp   = false;
input bool otl    = false;
/***********Range***************/
int    Length             = 3;
int    MajorRangeStrength = 4;


double MajorRangeBuy[];
double MajorRangeSell[];


double RangePrice  = 0.0,
       SweepB      = 0.0;
int    Switch2     = 0,
         SwitchB     = 0;
double Price2BuyA  = 0.0;
int    Price2BuyB  = 1.0;
double Price2SellA = 0.0;
int    Price2SellB = 0.0;
bool   BuySwitchB  = false,
       SellSwitchB = false;
       
int hendlMA_1;
double MA_1[];

int hendlMA_2;
double MA_2[];
//+------------------------------------------------------------------+
//| Custom indicator initialization function                         |
//+------------------------------------------------------------------+
int OnInit()
  {
//--- indicator buffers mapping
   SetIndexBuffer(0,MainLine,INDICATOR_DATA);
   PlotIndexSetDouble(0, PLOT_EMPTY_VALUE, 0);
   PlotIndexSetInteger(0,PLOT_DRAW_TYPE,DRAW_HISTOGRAM); 
   ArraySetAsSeries(MainLine, true);
   
   SetIndexBuffer(1,UpLine,INDICATOR_DATA);
   PlotIndexSetDouble(1, PLOT_EMPTY_VALUE, 0);
   PlotIndexSetInteger(1,PLOT_DRAW_TYPE,DRAW_HISTOGRAM); 
   ArraySetAsSeries(UpLine, true);
   
   SetIndexBuffer(2,DnLine,INDICATOR_DATA);
   PlotIndexSetDouble(2, PLOT_EMPTY_VALUE, 0);
   PlotIndexSetInteger(2,PLOT_DRAW_TYPE,DRAW_HISTOGRAM); 
   ArraySetAsSeries(DnLine, true);
   
   SetIndexBuffer(3,MajorRangeBuy,INDICATOR_DATA);
   PlotIndexSetDouble(3, PLOT_EMPTY_VALUE, 0);
   PlotIndexSetInteger(3,PLOT_DRAW_TYPE,DRAW_HISTOGRAM); 
   ArraySetAsSeries(MajorRangeBuy, true);
   
   SetIndexBuffer(4,MajorRangeSell,INDICATOR_DATA);
   PlotIndexSetDouble(4, PLOT_EMPTY_VALUE, 0);
   PlotIndexSetInteger(4,PLOT_DRAW_TYPE,DRAW_HISTOGRAM); 
   ArraySetAsSeries(MajorRangeSell, true);
   
   hendlMA_1=iMA(Symbol(),0,1,0,MODE_LWMA,PRICE_CLOSE);
   ArraySetAsSeries(MA_1,true);
   
   hendlMA_2=iMA(Symbol(),0,1,0,MODE_SMMA,PRICE_CLOSE);
   ArraySetAsSeries(MA_2,true);
   
   ArrayResize(muls, 99);
   
   mulSum = 0;
   
   for (int i0 = 0; i0 < 98; i0++) {//повторяем в цикле 98 раз
      if (i0 <= 18) y = 1.0 * i0 / 18; //если это первые 18 повторений
      else y = (i0 - 18) * 7.0 / 79.0 + 1.0; //иначе
      
      x = MathCos(Pi * y);
      z = 1.0 / (3.0 * Pi * y + 1.0);
      if (y <= 0.5) z = 1;
      
      muls[i0] = z * x;
      mulSum += muls[i0];
   }
   if(otl)Print(" Распределение создано muls[20]=",muls[20]);
//---
   return(INIT_SUCCEEDED);
  }
//+------------------------------------------------------------------+
//| Custom indicator iteration function                              |
//+------------------------------------------------------------------+
int OnCalculate(const int rates_total,
                const int prev_calculated,
                const datetime &time[],
                const double &open[],
                const double &high[],
                const double &low[],
                const double &close[],
                const long &tick_volume[],
                const long &volume[],
                const int &spread[])
  {
//---
  if (PeriodSeconds() <60*60 || PeriodSeconds() >10080*60) return(0);
   int depth=0;

   int l_ind_counted_8 = prev_calculated; //Возвращает количество баров, не измененных после последнего вызова индикатора.
   
   int bars=Bars(Symbol(),PERIOD_CURRENT);
   if (l_ind_counted_8 < 0) return (0);
   if (l_ind_counted_8 == 0)
      {
         depth = bars - 98;
         for(int a=0;a<bars;a++)
            {
               MainLine[a] = 0;
               UpLine[a] = 0;
               DnLine[a] = 0;
               MajorRangeBuy[a]=0;
               MajorRangeSell[a]=0;
            }
      }
   if (l_ind_counted_8 > 0)  depth = bars - l_ind_counted_8;
   if(otl)Print(" количество баров, не измененных после последнего вызова индикатора= ",l_ind_counted_8,"  Количество баров на текущем графике Bars=",bars,"  depth= ",depth);
   if (l_ind_counted_8 < 1) {
      for (int i2 = 1; i2 < 100; i2++) {
         MainLine[bars - i2] = 0;
         UpLine[bars - i2] = 0;
         DnLine[bars - i2] = 0;
      }
   }
   
   for (int i1 = depth; i1 >= 0; i1--) 
   {
      price = 0;
          
          CopyBuffer(hendlMA_1,0,0,bars,MA_1);
          
      for (int i2 = 0; i2 <= 98; i2++) 
         {
            if(i2 + i1>=bars)break;
            price += muls[i2] * MA_1[i2 + i1];
         }
          
      if (mulSum > 0.0) MainLine[i1] = price / mulSum;

     GoUp=MainLine[i1 + 1] > MainLine[i1] ;
     
      if (GoUp) 
      {
         if (!LastUp) DnLine[i1+1] = MainLine[i1+1];
         DnLine[i1] = MainLine[i1];
         UpLine[i1] = 0;
      }
         else
      {
         if (LastUp) UpLine[i1+1] = MainLine[i1+1];
         UpLine[i1] = MainLine[i1];
         DnLine[i1] = 0;
      }
      LastUp=GoUp; 

   }//  for (int i1

 //  return (0);

/***************** Range **********************/

  int counted_bars=prev_calculated;
  if(otl)Print(" Range counted_bars = ", counted_bars);
   if(counted_bars<0) return(-1);
   int position=bars-counted_bars;
   if (position<0) position=0;
   if (position==0) position=1;
   int rnglength = 250;
   double range = 0.0, srange = 0.0;
   if(otl) Print(" position=",position);
   
   for (int pos = position; pos >=0; pos--)
   {/***************** MAIN Range **********************/
      srange = 0.0;
      int j = 0;
      for (int i=0;i<rnglength;i++)
      {
         j++;
         int posr = pos + i;
         if (posr >= bars) break; 
         srange = srange + (High(posr) - Low(posr));
      }
      range = srange / j * Length;
      int BarNumber = bars-pos; //??????????
      if (BarNumber < 0)  BarNumber = 0;
          
          CopyBuffer(hendlMA_2,0,0,bars,MA_2);
          //Print(bars," - ",pos);
      if(pos<bars)RangePrice = MA_2[pos];  //Moving Average MODE_SMMA
      else RangePrice = MA_2[pos-1];

      if (BarNumber == 1)
      {
         SweepB  = range *  MajorRangeStrength;
         Price2BuyA = RangePrice;
         Price2SellA = RangePrice;
      }     

      if (BarNumber > 1)
      {

         if (Switch2  >  - 1)//проверка цикла на покупку
         {
            if (RangePrice < Price2BuyA) //если средняя цена ниже
            {
if (BuySwitchB ) MajorRangeBuy [pos +BarNumber - Price2BuyB] = 0;                                                                //OUT
                           Price2BuyA = RangePrice;
               Price2BuyB = BarNumber;
               BuySwitchB = true;
            } 
            else if (RangePrice > Price2BuyA)
            {
                            SwitchB = BarNumber - Price2BuyB;
MajorRangeBuy [pos +SwitchB] = MainLine[pos + SwitchB]*1.0005;                                                                                                                          //OUT
                BuySwitchB = true;

                              if (RangePrice - MA_2[pos + SwitchB] >= SweepB && SwitchB >= 1)
                              {
                     Switch2 =  - 1;
                     Price2SellA = RangePrice;
                     Price2SellB = BarNumber;
                     SellSwitchB = false;
                     BuySwitchB = false;
               }
            }
         }
         if (Switch2  < 1)//проверка цикла на продажу
         {
            if (RangePrice  > Price2SellA )
            {
if (pos +BarNumber - Price2SellB<bars&&SellSwitchB ) MajorRangeSell [pos +BarNumber - Price2SellB] = 0;                                                         //OUT
                           Price2SellA = RangePrice;
               Price2SellB = BarNumber;
               SellSwitchB = true;
                    }
                       else if (RangePrice < Price2SellA)
                    {
               SwitchB = BarNumber - Price2SellB ;

         if(pos+ SwitchB<bars)MajorRangeSell[pos + SwitchB] =MainLine[pos + SwitchB]*1.0005;                                                                                                                             //OUT
                SellSwitchB = true;             
        
                              if (pos + SwitchB<bars&&MA_2[pos + SwitchB] - RangePrice >= SweepB && SwitchB >= 1)
                              {
                                     Switch2 = 1;
                     Price2BuyA = RangePrice;
                     Price2BuyB = BarNumber;
                     SellSwitchB = false;
                     BuySwitchB = false;
                                  }
            }
         }
      }

   //   MajorRangeSell[pos] = 0;
    //  MajorRangeBuy[pos]  = 0;  
    }
//--- return value of prev_calculated for next call
   return(rates_total);
  }
//+------------------------------------------------------------------+
//========================================================================================
double High(int index)
{   
   if(index < 0) return(-1);
   double Arr[];
   ENUM_TIMEFRAMES timeframe=PERIOD_CURRENT;
   if(CopyHigh(Symbol(),timeframe, index, 1, Arr)>0) 
        return(Arr[0]);
   else return(-1);
}
//========================================================================================
double Low(int index)
{   
   if(index < 0) return(-1);
   double Arr[];
   ENUM_TIMEFRAMES timeframe=PERIOD_CURRENT;
   if(CopyLow(Symbol(),timeframe, index, 1, Arr)>0) 
        return(Arr[0]);
   else return(-1);
}

Alexey Viktorov 2022.02.01 07:32 #31428

Cosa viene restituito alla struttura MqlTradeCheckResult?

È scritto nella documentazione che"L'importo del margine richiesto per l' operazione commerciale richiesta".

Campo	Descrizione
retcode	Codice di ritorno
equilibrio	Valore del saldo che sarà dopo l'esecuzione dell'operazione
equity	Valore delpatrimonio netto, che sarà dopo l'esecuzione della negoziazione
profitto	Valore del profitto fluttuante che sarà dopo l'esecuzione dell'operazione
margine	L'importo del margine richiesto per l'operazione commerciale richiesta
margine_libero	La quantità di capitale che rimarrà dopo l'esecuzione dell'operazione richiesta
livello del margine	Il livello di margine da impostare dopo l'esecuzione della compravendita richiesta
commento	Commento al codice di risposta, descrizione dell'errore

Ma ciò che si ottiene effettivamente è la dimensione del margine totale, attuale e più quello che sarà preso dopo l'esecuzione di questa operazione.

Ecco lo script

#include <Trade\Trade.mqh>
CTrade trade;
  MqlTradeRequest           my_request;
  MqlTradeCheckResult       my_check_result;
  MqlTick                   mqlTick;
//+------------------------------------------------------------------+
//| Script program start function                                    |
//+------------------------------------------------------------------+
void OnStart()
 {
  SymbolInfoTick(_Symbol, mqlTick);
  CheckOrder(0.01, ORDER_TYPE_SELL);
  Print(my_check_result.margin);
//---
  trade.PositionOpen(_Symbol, ORDER_TYPE_SELL, 0.01, mqlTick.bid, 0.0, 0.0);
//---
  CheckOrder(0.01, ORDER_TYPE_SELL);
  Print(my_check_result.margin);
 }/******************************************************************/

/********************************************************************\
||
\********************************************************************/
bool CheckOrder(double volume, ENUM_ORDER_TYPE order_type)
 {
  if(order_type != ORDER_TYPE_BUY && order_type != ORDER_TYPE_SELL)
    return false;
  ZeroMemory(my_request);
  ZeroMemory(my_check_result);
  bool checkOrder = false;
//--- setting my_request
  my_request.action = TRADE_ACTION_DEAL;
  my_request.symbol = _Symbol;
  my_request.volume = volume;
  my_request.type   = order_type;
  my_request.price  = order_type == ORDER_TYPE_BUY ? mqlTick.ask : mqlTick.bid;
  checkOrder = ::OrderCheck(my_request, my_check_result);
  if(my_check_result.retcode != 0)
    return(false);
//---
  return(true);
 }/******************************************************************/

e il risultato dell'esecuzione

2022.02.01 10:11:28.002 Test bag (EURUSD,H1)    2.25
2022.02.01 10:11:28.108 Test bag (EURUSD,H1)    4.5

Mentre non ci sono posizioni aperte nel conto my_check_result.margin è uguale al margine per la posizione aperta con lotto 0,01, e quando c'è già 0,01 nel conto il margine per il lotto 0,02

File:

Test_bag.mq5 5 kb

Funzione Trailing funds (equity) Domande dai principianti MQL5 Qualsiasi domanda da principiante,

fxsaber 2022.02.01 08:21 #31429

Alexey Viktorov #:

Ma quello che si ottiene in realtà è la dimensione del margine totale, attuale e più quello che si prenderà dopo aver fatto questa operazione.

Giusto.

MarginOpen = CheckResult.margin - ::AccountInfoDouble(ACCOUNT_MARGIN); // Маржа для открытия.

Alexey Viktorov 2022.02.01 08:34 #31430

fxsaber #:

Giusto.

Che sia così, ma allora dovrebbe essere lo stesso nella documentazione.

Se balans è il valore del saldo che sarà dopo che lo scambio è fatto

poi margine - dovrebbe essere. Valore del margine, che sarà dopo l'esecuzione dell'operazione commerciale

Errori, bug, domande - pagina 3143