Inicializar array de cadenas en constructor

[aguml]

Ya solo me falta un detalle. Necesito tener un array de cadenas que sea accesible en mas de un sitio asi que lo que hago es esto:

Código C++:

private:
   static const AnsiString HintsMessages[18],HintsMessagesTypes[2];
 

Y en el constructor hago esto:

Código C++:

__fastcall TForm1::TForm1(TComponent* Owner)
   : TForm(Owner)
{
   //Inicializo la cadena constante de validacion de entrada del teclado
   GoodKey = "0123456789abcdefABCDEF";
 
   const AnsiString TForm1::HintsMessages[18]={"Este mensaje activa la ventana.",
                    "Este mensaje pone el foco en la ventana.",
                    "Este mensaje quita el foco de la ventana.",
                    "Este mensaje indica si se redibuja la ventana o no.",
                    "Este mensaje se envia para preguntar a una ventana si se puede cerrar al cerrar sesión.",
                    "Este mensaje se envia para indicar a una ventana que se va a cerrar sesión.",
                    "Este mensaje envia un clic izquierdo.",
                    "Este mensaje es el que se da al pulsar el botón izquierdo del ratón.",
                    "Este mensaje es el que se da al soltar el botón izquierdo del ratón.",
                    "Este mensaje es el que se da al hacer doble clic con el botón izquierdo del ratón.",
                    "Este mensaje es el que se da al pulsar el botón derecho del ratón.",
                    "Este mensaje es el que se da al soltar el botón derecho del ratón.",
                    "Este mensaje es el que se da al hacer doble clic con el botón derecho del ratón.",
                    "Este mensaje es el que se da al pulsar el botón central del ratón.",
                    "Este mensaje es el que se da al soltar el botón central del ratón.",
                    "Este mensaje es el que se da al hacer doble clic con el botón central del ratón.",
                    "Este mensaje es el que se da al girar la rueda vertical del ratón.",
                    "Este mensaje es el que se da al girar la rueda horizontal del ratón."};
 
   const AnsiString TForm1::HintsMessagesTypes[2]={"Esta opción espera la respuesta. El valor de retorno será la respuesta obtenida.",
                         "Esta opción no espera respuesta alguna. Retorna 1 si se envió correctamente y 0 si falló."};
}
 

Lo que queria realmente es hacerlo con const char*HintsMessages[] y const char* HintsMessagesTypes[] pero creo que sería aun mas complicado.
Con lo que tengo me da estos errores:

[C++ Error] Main.cpp(26): E2089 Identifier 'HintsMessages' cannot have a type qualifier
[C++ Error] Main.cpp(45): E2089 Identifier 'HintsMessagesTypes' cannot have a type qualifier

Es que no me hace nada de gracia hacerlo con un TStrings y tener que añadir uno a uno. Por eso quiero hacerlo con un array, para inicializarlo de golpe todo.

[eferion]

Un miembro static no debe ser inicializado desde el constructor sino que debe tener su propia "implementación" ya que entonces se estaría incicializando cada vez que se construye un nuevo objeto.

La incialización de miembros estáticos se hace tal que:

Código C++:

class POO
{
  static int var;
};
 
int POO::var = X;
 

Pudiendo estar la inicialización tanto en la cabecera como en el fichero de implementación.

En tu caso además los arreglos son const, luego no vas a poder modificar su valor una vez se hayan inicializado. Dado que los elemento static tienen su propia inicialización y que los has declarado const, no es posible modificar su valor en el constructor.

El siguiente código te debería funcionar:

Código C++:

const AnsiString TForm1::HintsMessages={
  "Este mensaje activa la ventana.",
  "Este mensaje pone el foco en la ventana.",
  // ...
};
 
__fastcall TForm1::TForm1(TComponent* Owner)
   : TForm(Owner)
{
  //...
}
 

También puedes plantearte usar std::array. Se comporta como un std::vector salvo en el detalle de que el tamaño es fijo, no puedes añadir nuevos elementos ni por supuesto eliminarlos... es el equivalente a lo que estás haciendo ahora mismo. Esta opción puede ser especialmente atractiva si necesitas usar iteradores.

Un saludo.

[aguml]

A ver, lo de static lo puse porque buscando información vi un ejemplo que para inicializar un array de cadenas decía que tenia que ser asi pero realmente te lo que necesito es un array de cadenas constantes que nunca se modificarán y que sólo necesito poder leer la cadena correspondiente al índice deseado. Si declaro el array de constantes como global va perfecto, el problema está en ponerlo como privado de la clase.

[eferion]

Con static lo que consigues es que ese array sea común para todos los objetos que crees de tipo TForm. De hecho la forma habitual de acceder a dicho array será TForm::HintsMessages en vez de this->HintsMessages.

Si no tienes ese requisito y únicamente buscas que sean constantes puedes eliminar static sin problemas e inicializar la variable en el constructor.

En cuanto a los problemas de inicialización puedes, como te comenté, usar std::array y acompañarlo de las listas de inicialización propias de C++11.

[aguml]

Tengo otro array pero es algo como:

Código C++:

private:
   const char cadena [];
 

Y si hago en el constructor:

Código C++:

cadena="Hola mundo";
 

Ahí no hay problemas y se inicializa correctamente pero con un array de cadenas no me deja o no veo la manera correcta de hacerlo.

[eferion]

Compilado bajo C++98:

Código C++:

struct POO
{
    const std::string test[10];
 
    POO();
};
 
POO::POO()
 : test( {"1","2","3","4","5","6","7","8","9","10" })
{ }
 
int main()
{
    POO p;
    std::cout << p.test[5];
}
 

Compilado bajo C++11:

Código C++:

struct POO
{
    const std::string test[10];
 
    POO();
};
 
POO::POO()
 : test{"1","2","3","4","5","6","7","8","9","10" }
{ }
 
int main()
{
    POO p;
    std::cout << p.test[5];
}
 

No funciona esto mismo con AnsiString??

[aguml]

Pues no se ni como hacerlo ya que mi constructor es asi:

Código C++:

__fastcall TForm1::TForm1(TComponent* Owner)
   : TForm(Owner)
{}
 

Todo lo que he probado incluso con std::string no lo acepta asi que ni idea y ademas tenemos el problema de que son dos arrays de cadenas con lo que tendria que inicializar asi 2 y que son con textos muy grandes y quedaría refeo jajaja.
Me parece increible lo complicado que pone C++ algo tan sencillo como eso .

[eferion]

Iniciado por aguml

Pues no se ni como hacerlo ya que mi constructor es asi:

Código C++:

__fastcall TForm1::TForm1(TComponent* Owner)
   : TForm(Owner)
{}
 

Y algo tal que...

Código C++:

__fastcall TForm1::TForm1(TComponent* Owner)
       : TForm(Owner),
         HintsMessages({"Este mensaje activa la ventana.",
                        "Este mensaje pone el foco en la ventana.",
                               // ...
                               })
    {}
 

Esto, claro está, asumiendo que has quitado lo de static.

Iniciado por aguml

Todo lo que he probado incluso con std::string no lo acepta asi que ni idea y ademas tenemos el problema de que son dos arrays de cadenas con lo que tendria que inicializar asi 2 y que son con textos muy grandes y quedaría refeo jajaja.

¿Has probado a poner una clase aparte para proporcionar esas cadenas?

No se, se me ocurre quizás algo tal que:

Código C++:

class Messages
{
  static const AnsiString WindowIsActive;
  static const AnsiString AnotherOne;
  // ...
};
 
const AnsiString Messages::WindowIsActive = "Este mensaje activa la ventana.";
// ...
 

O incluso usar enumerados

Código C++:

enum Messages
{
  WindowIsActive,
  AnotherOne,
  // ...
}
 
AnsiString GetMessage(Messages id)
{
  switch( id )
  {
    case WindowIsActive:
      return "Este mensaje activa la ventana.";
    // ...
  }
}
 

Iniciado por aguml

Me parece increible lo complicado que pone C++ algo tan sencillo como eso .

Y no es que C++ ponga esto demasiado complicado, es que te falta base para entender correctamente cómo funcionan los constructores. Quizás sería recomendable dedicar unas horas a entender su funcionamiento.

Un saludo.

[aguml]

Ok, lo primero que pones no funciona.
Lo demas que indicas no lo veo, para eso hago esto:

Código C++:

private:
   static const char *HintsMessages[18],*HintsMessagesTypes[2];
 

Código C++:

const char* TForm1::HintsMessages[18] = {"Este mensaje activa la ventana.",
                                         "Este mensaje pone el foco en la ventana.",
                                         "Este mensaje quita el foco de la ventana.",
                                         "Este mensaje indica si se redibuja la ventana o no.",
                                         "Este mensaje se envia para preguntar a una ventana si se puede cerrar al cerrar sesión.",
                                         "Este mensaje se envia para indicar a una ventana que se va a cerrar sesión.",
                                         "Este mensaje envia un clic izquierdo.",
                                         "Este mensaje es el que se da al pulsar el botón izquierdo del ratón.",
                                         "Este mensaje es el que se da al soltar el botón izquierdo del ratón.",
                                         "Este mensaje es el que se da al hacer doble clic con el botón izquierdo del ratón.",
                                         "Este mensaje es el que se da al pulsar el botón derecho del ratón.",
                                         "Este mensaje es el que se da al soltar el botón derecho del ratón.",
                                         "Este mensaje es el que se da al hacer doble clic con el botón derecho del ratón.",
                                         "Este mensaje es el que se da al pulsar el botón central del ratón.",
                                         "Este mensaje es el que se da al soltar el botón central del ratón.",
                                         "Este mensaje es el que se da al hacer doble clic con el botón central del ratón.",
                                         "Este mensaje es el que se da al girar la rueda vertical del ratón.",
                                         "Este mensaje es el que se da al girar la rueda horizontal del ratón."};
 
const char* TForm1::HintsMessagesTypes[2] = {"Esta opción espera la respuesta. El valor de retorno será la respuesta obtenida.",
                                             "Esta opción no espera respuesta alguna. Retorna 1 si se envió correctamente y 0 si falló."};
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
   : TForm(Owner)
{
   //Inicializo la cadena constante de validacion de entrada del teclado
   GoodKey = "0123456789abcdefABCDEF";
}
 

Esto funciona pero no se que tan correcto será.
Otra opcion antes de crearme mi propia clase para eso seria usar por ejemplo TStrings de C++Builder e ir añadiendo uno a uno todos los miembros y seria algo asi:

Código C++:

TStrings *HintsMessagesTypes;
HintsMessagesTypes=new TListStrings;
HintsMessagesTypes->Add("Esta opción espera la respuesta. El valor de retorno será la respuesta obtenida.");
HintsMessagesTypes->Add("Esta opción no espera respuesta alguna. Retorna 1 si se envió correctamente y 0 si falló.");
 

Eso lo hago en el evento OnCreate del form y en el OnClose o en el OnDestroy hago:

Código C++:

HintsMessagesTypes->Clear();
delete HintsMessageTypes;
 

No lo hago asi por lo engorroso que es el tener que estar gestionando la memoria dinamica y los problemas que puede dar al correr en un pc que no tenga suficiente memoria libre y tenga que crear items en tiempo de ejecucion mientras que lo que yo intento ya ocupen una memoria y no soliciten mas al arrancar. Creo que la unica forma es como lo he hecho con static o al menos la unica que me ha funcionado.

[eferion]

Iniciado por aguml

Ok, lo primero que pones no funciona.

Sin saber cómo lo estás escribiendo tu y sin conocer la versión de C++ sobre la que estás compilando es complicado dar respuestas más concretas.

Iniciado por aguml

Lo demas que indicas no lo veo, para eso hago esto:

...

Esto funciona pero no se que tan correcto será.

Yo lo denominaría como algo tan engorroso o incluso más que las soluciones que te propongo.

Iniciado por aguml

Otra opcion antes de crearme mi propia clase para eso seria usar por ejemplo TStrings de C++Builder e ir añadiendo uno a uno todos los miembros y seria algo asi:

...

No lo hago asi por lo engorroso que es el tener que estar gestionando la memoria dinamica y los problemas que puede dar al correr en un pc que no tenga suficiente memoria libre y tenga que crear items en tiempo de ejecucion mientras que lo que yo intento ya ocupen una memoria y no soliciten mas al arrancar.

La gestión de textos en una aplicación siempre es una tarea engorrosa. No todo en la programación iba a ser bonito y agradable de hacer.

En cuanto al tema de la memoria... no se, salvo que trabajes con un 386 con 32MB de RAM no se me ocurre cómo puedes llenar la memoria dinámica a base de textos. Las opciones que te he propuesto cuidan de no repetir cadenas, de tal forma que el consumo de memoria se reduce tanto como usando static.

Iniciado por aguml

Creo que la unica forma es como lo he hecho con static o al menos la unica que me ha funcionado.

Si tienes una forma que te funciona y no hay razones de peso para cambiarla usa esa. Para cambiarla por algo mejor siempre tendrás tiempo en el futuro (sobretodo si el diseño es bueno).

Volviendo al tema de las soluciones que te he propuesto has de tener en cuenta que crear clases es prácticamente gratis en C++. ¿Que empiezas a tener muchos archivos en el proyecto? Lo mismo empieza a tener sentido montarse un sistema de documentación con doxygen para no tener que abrir cada dos por tres los ficheros a ver qué tienen dentro... pero vamos que no te de pereza crear clases para lo que necesites.

Por otro lado, tener clases separadas te puede ayudar a evitar duplicados, en tu caso, textos que se repitan en varias ventanas.

Un saludo.

[aguml]

Pensándolo, para cadenas constantes ¿es una mala práctica que sean globales?