Errores, fallos, preguntas - página 1576

Nuevo comentario
 

Me pregunto qué principio se utiliza para poner los productos en esta pancarta

¿y los productos de los vendedores autorizados van allí? He hojeado 30 páginas y no he visto el mío...

 
Renat Fatkhullin:

Yo no me meto. Llevo 26 años programando sin parar.

Las advertencias son esencialmente errores si hablamos del sector financiero. Y todos los miles de informes de "pérdida de señal, pérdida de precisión, pérdida en los fantasmas, etc." son veredictos sobre la calidad del código. Por lo visto, no entiendes bien las implicaciones.

Por favor, facilite de forma suficientemente completa el trozo de código que el compilador ha señalado como error.

Sin ella toda esta discusión parece antiestética e injusta.

De acuerdo, Renat, estos argumentos sobre la "calidad del código" son irrelevantes para el tema de discusión, ya que aquí sólo se habla de la compilabilidad, es decir, de la viabilidad del código. Y la pérdida de precisión, etc. - esto es una cuestión personal del programador, es decir, su responsabilidad. La conversión implícita, por ejemplo, de int a short, no está prohibida por la norma del lenguaje, ¿verdad? Entonces, ¿por qué estamos predicando ahora?

Vale, he encontrado uno de estos fallos:

class CClass
{

};


class CArr
{
  CClass* data[];
 public: 
  CClass* operator[] (int i)  { return data[i]; }
};


template<typename T>
void Func (T* obj)      {  }
 

void OnStart()
{  
  CArr arr;
  Func(arr[0]);
}

lo que me sale en el registro:

'CClass' - declaración sin tipo TestScript.mq5 16 9
'CClass' - coma esperada TestScript.mq5 16 9

En las versiones anteriores todo iba bien.

 
Alexey Navoykov:

De acuerdo, Renat, estos argumentos sobre la "calidad del código" son irrelevantes para el tema de discusión, ya que aquí sólo hablamos de la compilabilidad, es decir, de la viabilidad del código. Y la pérdida de precisión, etc. - esto es una cuestión personal del programador, es decir, su responsabilidad. La conversión implícita, por ejemplo, de int a short, no está prohibida por la norma del lenguaje, ¿verdad? Entonces, ¿por qué estamos predicando ahora?

2600 lugares potencialmente problemáticos, e incluso en las finanzas - esto no es una moral, pero exactamente el indicador de la calidad del código.


Vale, he encontrado uno de estos fallos:

lo que me sale en el registro:

En las versiones anteriores todo iba bien.

Sí, este fallo ya se discutió (probablemente, con A100) y se solucionó el 4 de mayo. Se excedió con el control de tipos, aparentemente.

Adjunto el último MetaEditor build 1329, que no tiene este error. Por favor, compruébelo allí.

El lanzamiento de MT5 será el 12 de mayo.

Archivos adjuntos:
MetaEditor64.exe.zip  10385 kb
 
Alexey Navoykov:

Estaba bien en las construcciones anteriores.

En tu código, no estás devolviendo un puntero const a un objeto privado. Resulta que las funciones de terceros (en términos de visibilidad de las variables) pueden cambiar aparentemente algo que no debería ser arquitectónicamente accesible para ellos, ya que el programador ha especificado privado.

Siempre que quiero devolver un puntero a un objeto privado, necesariamente especifico un modificador const. En tu caso yo le pondría un alabeo.

No soy de los que vuelan alto, por eso pregunto. ¿Hay que usar este código en algún sitio o es que da pereza poner la const?

 
A100:

Dos días desperdiciados prácticamente en su totalidad (a mi edad ya es mucho), y tenía previsto utilizarlos de una manera ligeramente diferente.

Ha citado más de un centenar de fallos. ¡¿Cada vez que una cantidad tan salvaje de su tiempo?! ¿De dónde ha sacado tanta generosidad hacia los promotores?
Alexey Navoykov:

Así que, una vez más, me quito el sombrero ante A100 por su paciencia. Yo mismo estoy cansado de esto, me resulta más fácil sentarme en la vieja build, que funciona bien, que buscar las causas de los errores en la nueva build, trabajando en el servicio de atención al cliente. ¿O alguien me pagará por este trabajo?

Sí, servicedesk es una cosa genial para que los probadores de terceros lo hagan gratis. Obviamente no se hizo para eso, pero de hecho se ha convertido exactamente en el empleador de los probadores de terceros que trabajan gratis. Si no fuera por estos informes de errores, el compilador habría tardado mucho más en compilar.

Todo el mundo argumentará que debería haber una retribución por encontrar un error, como es la práctica en el mundo. A100 debería recibir el sueldo del examinador estatal. Y aparentemente el salario de un año de probador.

 
Renat Fatkhullin:

Sí, este error ya ha sido discutido (quizás con A100) y solucionado desde el 4 de mayo. Al parecer, se han excedido en el control del tipo.

Adjunto la última compilación 1329 de MetaEditor que no contiene este error. Por favor, compruébelo allí.

El lanzamiento de la MT5 será el 12 de mayo.

Comprobado. Ahora casi no hay errores de compilación, salvo algunas maravillas extrañas que no puedo reproducir por separado del programa, pero que consigo evitar por algún medio aleatorio.

A continuación se muestra un código de ejemplo del área problemática que voy a comentar. De nuevo, compila bien por separado pero genera un error en mi programa.

class CBase
{
};


class CClass : public CBase
{
};

 

class CWork
{
 public:
 
  template<typename T>
  void Run(T& arr[])
  {
    struct Struct
    {
      template<typename T1> static void Set(T1& main,  T& arr[], CBase& obj)   { main.Set(arr); } 
    };
    
    Struct structobj;
    
    structobj.Set(this, arr, arr[0]);
  }

  
  template<typename T>
  void Set(T& arr[]) 
  {  
    for (int i=0; i<ArraySize(arr); i++) Set(arr[i]);  // Здесь возникает ошибка: 'arr' - parameter conversion not allowed
  }
  
  template<typename T>
  void Set(T& obj)    { } 
  
  template<typename T>
  void Set(T*& obj)    { } 

};
 
 

void Main()
{
  CWork work;
  CClass arr[];
  work.Run(arr);

  return;
}

Si añade una línea en la función Main en cualquier lugar (por ejemplo, después de return):

work.Set(arr[0]);

se compila normalmente.

El programador parece haber ido demasiado lejos en la optimización.

Y también hay algunos fallos en el tiempo de ejecución. Por ejemplo, asigno un valor a algún miembro de una estructura, pero luego resulta que el valor que hay es antiguo, es decir, no se asignó nada. Si añado una línea con alguna operación arbitraria cerca, todo se vuelve normal. Estos bugs han empezado desde aquella build de otoño en la que se optimizó el compilador. En definitiva, todo sigue en bruto.

Además, la compilación en sí sigue tardando 20 segundos, mientras que la compilación 1159 sólo tardaba 1-2 segundos. Al mismo tiempo, no noto ningún aumento significativo de la velocidad de mis programas, la ganancia está dentro del 10-20%. Así que puedes olvidarte de esas historias sobre el aumento de velocidad de 2 a 10 veces. Puede que ocurra con muestras de prueba especialmente seleccionadas, pero tenemos aplicaciones reales, no falsas.

En total, la compilación es entre 10 y 20 veces más lenta para obtener un poco más de rendimiento. En mi opinión, no merece la pena. El tiempo que pierde el programador es mucho más valioso.

Todavía estoy obligado a permanecer en la construcción 1159.

 
Anton Zverev:

En tu código, no estás devolviendo un puntero const a un objeto privado. Resulta que las funciones de terceros (en términos de visibilidad de las variables) pueden cambiar aparentemente algo que no debería ser arquitectónicamente accesible para ellos, ya que el programador ha especificado privado.

Siempre que quiero devolver un puntero a un objeto privado, necesariamente especifico un modificador const. En tu caso yo le pondría un alabeo.

No soy de los que vuelan alto, por eso pregunto. ¿Hay que usar este código en algún sitio o es que da pereza poner la const?

Sólo oculto un array en privado mientras los objetos CClass en sí están disponibles para un usuario para su acceso completo, ese es el propósito. Si lo necesitara sólo para leer, pondría const.

 
Anton Zverev:
Todos apoyarán que haya una retribución por encontrar un error, como es la práctica en el mundo. A100 debería tener un sueldo de probador estatal. Y aparentemente el salario de un año de probador.
Lo apoyo, pero me temo que se limitará a una placa y una carta de reconocimiento :)
 

No sé si se trata de un error o simplemente de una descripción errónea de los métodos CDealInfoPositionId() yTicket(). He escrito el siguiente código

//+------------------------------------------------------------------+
//|                                                      test_01.mq5 |
//|                                                   Sergey Gritsay |
//|                                             https://www.mql5.com |
//+------------------------------------------------------------------+
#property copyright "Sergey Gritsay"
#property link      "https://www.mql5.com"
#property version   "1.00"
#include <Trade\Trade.mqh>
//#include <Trade\DealInfo.mqh>
CTrade trade;
CDealInfo deal;
ulong  ticket_deal;      // тикет сделки
ulong  ticket_position;     // тикет позиции
//+------------------------------------------------------------------+
//| Script program start function                                    |
//+------------------------------------------------------------------+
void OnStart()
  {
//---
   trade.Buy(0.1,_Symbol);
   ticket_deal=trade.ResultDeal();
   deal.Ticket(ticket_deal);
   ticket_position=deal.PositionId();

   Print("ResultDeal() = ",ticket_deal);
   Print("deal.PositionId() = ",ticket_position);
  }
//+------------------------------------------------------------------+

resultado

NM      0       14:06:49.083    test_01 (EURUSD,H1)     ResultDeal() = 69380150
QE      0       14:06:49.083    test_01 (EURUSD,H1)     deal.PositionId() = 0

He añadido una solicitud para el historial de ofertas utilizando la funciónHistorySelect().

//+------------------------------------------------------------------+
//|                                                      test_01.mq5 |
//|                                                   Sergey Gritsay |
//|                                             https://www.mql5.com |
//+------------------------------------------------------------------+
#property copyright "Sergey Gritsay"
#property link      "https://www.mql5.com"
#property version   "1.00"
#include <Trade\Trade.mqh>
//#include <Trade\DealInfo.mqh>
CTrade trade;
CDealInfo deal;
ulong  ticket_deal;      // тикет сделки
ulong  ticket_position;     // тикет позиции
//+------------------------------------------------------------------+
//| Script program start function                                    |
//+------------------------------------------------------------------+
void OnStart()
  {
//---
   trade.Buy(0.1,_Symbol);
   ticket_deal=trade.ResultDeal();
   HistorySelect(0,TimeCurrent());
   deal.Ticket(ticket_deal);
   ticket_position=deal.PositionId();

   Print("ResultDeal() = ",ticket_deal);
   Print("deal.PositionId() = ",ticket_position);
  }
//+------------------------------------------------------------------+

resultado

JM      0       14:16:33.055    test_01 (EURUSD,H1)     ResultDeal() = 69381116
EF      0       14:16:33.055    test_01 (EURUSD,H1)     deal.PositionId() = 83654708
 
Alexey Navoykov:

En private sólo oculto el array, y los objetos CClass en sí están a disposición del usuario para su acceso completo, ese es el propósito. Si necesitara que fuera sólo de lectura, pondría const.

Ya veo. ¿Puede decirme en qué construcciones puede ser útil? Entiendo que con este enfoque no se puede hacer nada con el propio array (redimensionar, intercambiar elementos, etc.). eliminar, sin embargo, puede aplicarse...

Supongo que lo haces en algún lugar con una plantilla, para que haya la misma sintaxis del operador [] para los diferentes tipos de objetos. En general, ¿podría mostrar el uso de esta construcción cuando sea conveniente?

1...156915701571157215731574157515761577157815791580158115821583...3187
Nuevo comentario