Erreurs, bugs, questions - page 2794

[Mihail Matkovskij]

Mihail Matkovskij:

Mon indicateur a commencé à perdre des objets graphiques pour une raison quelconque et ne veut pas les supprimer. J'utilise la suppression automatique dans le destructeur de classe. Je l'appelle :

En conséquence, j'obtiens un faux positif. J'appelle GetLastError() et j'obtiens l'erreur 4101 ("Errory chart identifier"). Supposons. Je compare le chartID de l'objet graphique en cours de suppression avec le résultat de la fonction ChartID() et ils sont exactement les mêmes. Quelle peut être la raison pour laquelle un objet n'est pas supprimé du graphique et comment puis-je contourner ce problème ?

J'ai fait un code source où ce bug se produit :

//+------------------------------------------------------------------+
//|                                           DeleteChartObjects.mq5 |
//|                                      Copyright 2020, © Cyberdev. |
//+------------------------------------------------------------------+
#property copyright "Copyright 2020, © Cyberdev."
#property version   "1.00"
#property indicator_chart_window

#property indicator_plots 0

//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+

#include <ChartObjects\ChartObjectsLines.mqh>
#include <Arrays\ArrayObj.mqh>

//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
//--- input parameters
input int      nBars = 100000;

//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
int customN_Bars = 0;

//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
CArrayObj listOfTrendLines;

//+------------------------------------------------------------------+
//| Custom indicator initialization function                         |
//+------------------------------------------------------------------+
int OnInit() {
//--- indicator buffers mapping
  int totalBars = iBars(NULL, PERIOD_CURRENT);
  customN_Bars = (nBars < totalBars) ? nBars : totalBars;
//---
  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[]
) {
  CChartObjectTrend * trend;
  int delta = rates_total - customN_Bars;
  int shift;
  int i;
  for(i = 0; i < customN_Bars; i++) {
    shift = delta + customN_Bars - i - 1;
    trend = new CChartObjectTrend();
    if(trend.Create(0, "trend"+(string)i, 0, time[shift], low[shift], time[shift], high[shift]))
      listOfTrendLines.Add(trend);
  }
  return(rates_total);
}

//+------------------------------------------------------------------+
//| Expert deinitialization function                                 |
//+------------------------------------------------------------------+
void OnDeinit(const int reason) {
  
  CChartObjectTrend * trend;
  
  int i = listOfTrendLines.Total() - 1;
  for(; i >= 0; i--) {
    trend = dynamic_cast <CChartObjectTrend *> (listOfTrendLines.At(i));
    
    if(CheckPointer(trend) == POINTER_INVALID)
      continue;
    
    delete trend;
  }
}
//+------------------------------------------------------------------+

Actions pour démontrer le bug.

1. Placez l'indicateur sur le graphique.

2. Retirez l'indicateur du tableau.

3 Appuyez sur Ctrl+B, puis sur le bouton "Lister tout" dans la fenêtre et voyez les objets qui n'ont pas été supprimés.

J'en ai jusqu'à 294 sur 100000.

Si vous réduisez la valeur du paramètre d'entrée nBars, le bogue n'apparaît pas.

[Anatoliy Lukanin]

Bon après-midi.

J'ai longtemps hésité à passer de Windows 7 à 10, mais j'ai réinstallé le système cette semaine.

Maintenant j'ai un problème, je ne peux pas télécharger les versions démo de mes indicateurs et autres EAs,

J'ai appuyé sur le bouton de téléchargement et rien ne se passe, je l'ai essayé avec un terminal ouvert et fermé!

J'essaie de le télécharger pour le terminal MT4, je ne suis pas encore passé à MT5.

Mais qu'est-ce qui se passe ?

[Rashid Umarov]

Mihail Matkovskij:
Fait un code source où ce bug se produit :

Que se passe-t-il lorsqu'un pointeur est supprimé ?

//+------------------------------------------------------------------+
//| Expert deinitialization function                                 |
//+------------------------------------------------------------------+
void OnDeinit(const int reason) {
  
  CChartObjectTrend * trend;
  
  int i = listOfTrendLines.Total() - 1;
  for(; i >= 0; i--) {
    trend = dynamic_cast <CChartObjectTrend *> (listOfTrendLines.At(i));
    
    if(CheckPointer(trend) == POINTER_INVALID)
      continue;
    
    delete trend;
  }
}

Existe-t-il une commande permettant de supprimer l'objet graphique ? Voir la note sur ObjectDelete

[Alexey Viktorov]

Mihail Matkovskij:

J'en ai jusqu'à 294 sur 100000.

Si nous diminuons la valeur du paramètre d'entrée nBars, le bogue ne se produit pas.

Peut-il arriver que la boucle dure plus longtemps qu'il ne faut pour exécuter OnDeinit ?

Dans votre exemple tous les objets ont le préfixe "trend", pourquoi ne pas l'utiliser et refuser de la boucle ?

int  ObjectsDeleteAll(
   long           chart_id,   // идентификатор графика
   const string     prefix,   // префикс имени объекта
   int       sub_window=-1,   // индекс окна
   int      object_type=-1    // тип объекта для удаления
   );

[Mihail Matkovskij]

Rashid Umarov:

Que se passe-t-il lorsqu'un pointeur est supprimé ?

Existe-t-il une commande permettant de supprimer l'objet graphique ? Voir la note sur ObjectDelete

J'ai certainement déjà pensé à cette note. Mais ensuite, j'ai décidé pour une raison quelconque que c'était un bug du terminal. Je m'en excuse. Donc ça doit être ma faute.

Mais je ne sais pas quoi faire dans ce cas. Dois-je appeler ObjectDelete, puis ObjectFind et attendre qu'il échoue pour savoir si l'objet a été effectivement supprimé ? A en juger par la même note, cela prend trop de temps. Je ne sais pas si une deuxième suppression fonctionnera (dans mon indicateur, j'ai essayé d'appeler ObjectDelete deux fois de suite, mais cela n'a pas fonctionné). Et la fonction Sleep des indicateurs ne fonctionne pas pour donner au graphique le temps de supprimer chaque objet. Peut-être existe-t-il quelque part des exemples permettant de contourner ce problème ?

[Mihail Matkovskij]

Alexey Viktorov:

Ne peut-il pas arriver que la boucle dure plus longtemps que le temps imparti pour exécuter OnDeinit ?

Après tout, tous les objets de votre exemple ont le préfixe "trend", pourquoi ne pas l'utiliser et refuser la boucle ?

C'est seulement dans l'exemple. L'indicateur avec lequel je travaille a des noms complexes. Et j'ai donné un exemple juste pour la démonstration.

[Artyom Trishkin]

Mihail Matkovskij:

J'ai certainement déjà pensé à cette note. Mais ensuite, pour une raison quelconque, j'ai pensé que c'était un bug du terminal. Je m'en excuse. Il s'avère que c'est ma faute.

Mais je ne sais pas quoi faire dans ce cas. Dois-je appeler ObjectDelete, puis ObjectFind et attendre qu'il échoue pour savoir si l'objet a été effectivement supprimé ? A en juger par la même note, cela prend trop de temps. Je ne sais pas si une deuxième suppression fonctionnera (dans mon indicateur, j'ai essayé d'appeler ObjectDelete deux fois de suite, mais cela n'a pas fonctionné). Et la fonction Sleep des indicateurs ne fonctionne pas pour donner au graphique le temps de supprimer chaque objet. Peut-être existe-t-il quelque part des exemples permettant de contourner ce problème ?

Je pense que vous ajoutez tous les pointeurs à la liste. Le sous-système terminal supprime lui-même ces objets, dont les pointeurs se trouvent dans la liste CArrayObj.

trend = new CChartObjectTrend();
    if(trend.Create(0, "trend"+(string)i, 0, time[shift], low[shift], time[shift], high[shift]))
      listOfTrendLines.Add(trend);

Ici, vous avez une fuite de mémoire potentielle. Et une invocation potentielle d'un objet non valide.

[Mihail Matkovskij]

Artyom Trishkin:
Je pense que vous ajoutez tous les pointeurs à la liste. Le sous-système terminal supprime lui-même ces objets, dont les pointeurs se trouvent dans la liste CArrayObj.

Après l'arrêt du terminal, les objets restent sur le graphique après l'indicateur. Et lorsque je redémarre le terminal, ces objets sont visibles et gâchent toute la vue.

[Artyom Trishkin]

Mihail Matkovskij:

Après l'arrêt du terminal, les objets restent sur le graphique après l'indicateur. Et lorsque je redémarre le terminal, ces objets sont visibles et gâchent toute la vue.

Je ne suis pas à mon PC pour vérifier votre exemple. J'ai ajouté à mon message ci-dessus ce que je vois dans les erreurs.

[Mihail Matkovskij]

Artyom Trishkin:
Je pense que vous ajoutez tous les pointeurs à la liste. Le sous-système terminal supprime lui-même ces objets, dont les pointeurs se trouvent dans la liste CArrayObj
.

Ici, vous avez une fuite de mémoire potentielle.

S'il y a une fuite de mémoire (par pointeur, l'objet n'a pas été supprimé), le journal indique, à la fin, quels objets n'ont pas été supprimés et combien de mémoire n'a pas été libérée. Mais je n'avais rien de tout ça. Les objets sont supprimés, mais leurs objets graphiques ne sont pas tous supprimés.

Rashid Umarov a remarqué que le problème est que le graphe n'a pas le temps de traiter un si grand nombre de commandes pour supprimer des objets de la liste ! Quel est le rapport avec les fuites de mémoire ?