Duda con Inicialización de Array de Objetos

[Ñuño Martínez]

Estás mezclando C y C++. Recuerda que son lenguajes diferentes.

Lo siguiente:

//Obtengo memoria para los elementos necesarios
rutas = new(nothrow) AnsiString*[nSeleccionadas];

Sí, estás obteniendo "memoria para los elementos necesarios", pero estás creando nSeleccionadas referencias a objetos de clase AnsiString, que se almacenan en una lista (array) referenciada por el puntero rutas. Ojo, porque no estás creando ningún objeto.

Por eso lo siguiente:

//Limpio todo lo necesario
memset(rutas, 0, sizeof(rutas));

no limpia nada. Lo que hace es asignar el valor 0 a los primeras 8 octetos (asumiendo 64bit) de la lista referenciada por rutas, independientemente del tamaño de la misma.

memset no sabe de objetos, sólo sabe de posiciones de memoria. Y sizeof devuelve el tamaño del tipo, no el número de elementos de una lista (salvo que lo hayan cambiado en la definición del lenguaje C++ 99 o posterior, que si es así me parece una cagada monumental...). Estas dos funciones son funciones C, no C++.

Para limpiar los objetos has de usar un bucle y limpiar cada objeto, uno a uno, de la forma apropiada, similar a tu código de liberación de memoria:

int Ndx;

  for (Ndx = 0; Ndx < nSeleccionadas; Ndx++)
    *(rutas[Ndx]) = ""; // La clase AnsiString sobrecarga el operador "=".

Nota: Hace años que no programo C++, y de hecho me mosquea lo de "new(nothrow)", pero creo que no he cometido ningún error. Aun así, si alguien lo tiene más fresco, que lo diga.

[ecfisa]

Empezado por Ñuño Martínez

...
Nota: Hace años que no programo C++, y de hecho me mosquea lo de "new(nothrow)", pero creo que no he cometido ningún error. Aun así, si alguien lo tiene más fresco, que lo diga.

No, tampoco creo que hayas cometido algún error, de allí la pregunta de mi mensaje anterior . Por que según la declaración de la variable rutas, yo habría echo algo parecido a:

Código PHP:

{
  AnsiString **rutas;

  // asignar memoria
  rutas = new AnsiString*[MAXROWS];
  for(int r = 0; r < MAXROWS; r++)
    rutas[r] = new AnsiString[MAXCOLS];

  // cargar algunos datos
  for(int r = 0; r < MAXROWS; r++)
    for (int c = 0; c < MAXCOLS; c++) rutas[r][c] = "Rta"+IntToStr(r+c)+" ";

  // mostrar
  for(int r = 0; r < MAXROWS; r++) {
    s = "";
    for (int c = 0; c < MAXCOLS; c++) s += rutas[r][c];
    ListBox1->Items->Add(s);
  }

  // liberar memoria
  for(int c = 0; c < MAXROWS; c++) delete [] rutas[c];
  delete [] rutas;
} 


(La constante nothrow, hace que que la función no lance una excepción sino que devuelva un apuntador nulo)

Saludos

[aguml]

a ver si me se explicar. Ya se que no estoy creando ningun objeto, lo que quiero limpiar es la lista de punteros, justo despues los creo y el poner todos a 0 antes es para luego al liberar la memoria con un condicional compruebo si es o no 0 y solo libero los que no sean 0. En teoria no es necesario ya que "new (nothrow)" devuelve 0 si falla con lo que ya estaria a 0. En el caso del array de AnsiString tambien podria utilizar un TListString pero no se que seria mejor ya que lo uso para almacenar las rutas de unas imagenes para luego poder borrarlas al terminar. No se si un array como el que hago es mas eficiente o no que un TListString ni se como se usa el TListString.

[aguml]

solo necesito una lista de AnsiStrings, no necesito mas dimensiones.

[aguml]

en realidad todos son listas, unos de objetos, otros de strings. Supongo que para las strings podria utilizar TStringList y para el resto TList pero no se usarlos ni como gestionar cuando deuna excepcion por falta de memoria. Supongo que seria "try[ lista->Add("ruta"); ]catch(...)[ //mensaje de error ]". Perdonen por todo pero es que en el movil no me deja ni poner corchetes, ni saltos de lineas ni nada. Para un TList ¿Como seria? ¿Como creo cada objeto y como lo destruyo y libero memoria en ambos casos?

[ecfisa]

Hola aguml.

Si deseas trabajar con un array dinámico de elementos tipo AnsiString la declaración,

Código PHP:

AnsiString **rutas = 0; 


es incorrecta, estas declarando una matriz de elementos tipo AnsiString, la declaración debería ser:

Código PHP:

  AnsiString *rutas; 


De todos modos creo que tal vez usando TStrings te resulte mas simple... Un ejemplo (sin mucho ornamento):

Código PHP:

TStrings *ts;

// Crear lista
void __fastcall TForm1::FormCreate(TObject *Sender)
{
  ts = new TStringList;
}

// Cargar texto y grafico en TStrings
void __fastcall TForm1::Button1Click(TObject *Sender)
{
  for(int i = 0; i < ImageList1->Count; i++) {
    Graphics::TBitmap* bm = new Graphics::TBitmap;
    ImageList1->GetBitmap(i, bm); 
        ts->AddObject(IntToStr(i),   // cadena
                     (TObject*)bm);  // imágen
  }
  ListBox1->Items->Assign(ts);
}

// Mostrar en un ListBox
void __fastcall TForm1::ListBox1DrawItem(TWinControl *Control, int Index,
      TRect &Rect, TOwnerDrawState State)
{
  TListBox *lb = static_cast<TListBox*>(Control);
  Graphics::TBitmap *bm = static_cast<Graphics::TBitmap*>(lb->Items->Objects[Index]);

  lb->Canvas->FillRect(Rect);
  lb->Canvas->Draw(Rect.left,Rect.top, bm);  // mostrar bitmap
  lb->Canvas->TextOut(Rect.left+bm->Width+10, // mostrar cadena
    Rect.top, lb->Items->Strings[Index]);
}

// Liberar memoria
void __fastcall TForm1::FormDestroy(TObject *Sender)
{
  delete ts;
} 


(En el ejemplo los gráficos son extraidos de un TImageList)

Resultado:



Saludos

[aguml]

en tu codigo veo cosas que no entiendo. Por ejemplo, creas un puntero de tipo TStrings y luego le asignas un TStringList ¿Por que no creas un simple AnsiString y luego haces un Add con el TStringList? Por cierto, si creo AnsiString *rutas, cuando quiera guardar una ruta supongo que seria rutas[i] pero ¿Y si quiero por ejemplo leer un caracter de una ruta en concreto? ¿Seria rutas[i][j]?

[ecfisa]

Hola aguml.

Empezado por aguml

en tu codigo veo cosas que no entiendo. Por ejemplo, creas un puntero de tipo TStrings y luego le asignas un TStringList..

La explicación es polimorfismo.

Empezado por aguml

¿Por que no creas un simple AnsiString y luego haces un Add con el TStringList?

Quise ponerte un ejemplo mas completo para mostrar el potencial de la clase, pero podría haber sido:

Código PHP:

{
  TStrings *rutas = new TStringList;

  for(int i = 0; i < 15; i++) rutas->Add("Ruta "+String(i));
  ListBox1->Items->Assign(rutas);

  delete rutas;
} 


Empezado por aguml

cuando quiera guardar una ruta supongo que seria rutas[i] pero ¿Y si quiero por ejemplo leer un caracter de una ruta en concreto? ¿Seria rutas[i][j]?

No, sería:

Código PHP:

  rutas->Strings[i][j]; 


Saludos