Duda con Inicialización de Array de Objetos

[aguml]

Hola amigos, estoy usando memoria dinamica y la obtengo bien pero lo que intento es poner toda esa memoria a 0 y lo hago usando memset.

Código PHP:

TAbsPicture *CurrentImage;
TVRLabel **pVRLabel = 0;
TVRBand **pVRBand = 0;
TVRPage **pVRPage = 0;
AnsiString **rutas = 0;
AnsiString **rutasDivisiones = 0;

//Obtengo memoria para los elementos necesarios
rutas = new(nothrow) AnsiString*[nSeleccionadas];
rutasDivisiones = new(nothrow) AnsiString*[nTotalDivisiones];
pVRLabel = new(nothrow) TVRLabel*[nSeleccionadas];
pVRBand = new(nothrow) TVRBand*[nSeleccionadas];
pVRPage = new(nothrow) TVRPage*[nSeleccionadas];

//Limpio todo lo necesario
memset(rutas, 0, sizeof(rutas));
memset(rutasDivisiones, 0, sizeof(rutasDivisiones));
memset(pVRLabel, 0, sizeof(pVRLabel));
memset(pVRBand, 0, sizeof(pVRBand));
memset(pVRPage, 0, sizeof(pVRPage)); 


El caso es que no se como hacerlo ya que como lo tengo solo limpio 4 bytes y no es lo que quiero.
Luego he probado ha hacer esto:

Código PHP:

memset(rutas, 0, sizeof(rutas)*nSeleccionadas);
memset(rutasDivisiones, 0, sizeof(rutasDivisiones)*nTotalDivisiones);
memset(pVRLabel, 0, sizeof(pVRLabel)*nSeleccionadas);
memset(pVRBand, 0, sizeof(pVRBand)*nSeleccionadas);
memset(pVRPage, 0, sizeof(pVRPage)*nSeleccionadas); 


Pero no se si lo hago bien o estoy poniendo a 0 lo que no debo ¿podeis indicarme como hacerlo?

[aguml]

Se me olvidó poner la funcion donde libero toda esa memoria para que la veais y me digais si veis algo mal:

Código PHP:

void LiberarMemoria()
{
   if(rutas != 0)
   {
      for(int i = 0; i < nSeleccionadas; i++)
         if(rutas[i] != 0)
            delete rutas[i];
      delete[] rutas;
   }
   if(rutasDivisiones != 0)
   {
      for(int i = 0; i < nTotalDivisiones; i++)
      {
         DeleteFile(*rutasDivisiones[i]);
         if(rutasDivisiones[i] != 0)
            delete rutasDivisiones[i];
      }
      delete[] rutasDivisiones;
   }
   if(pVRLabel != 0)
   {
      for(int i = 0; i < nSeleccionadas; i++)
         if(pVRLabel[i] != 0)
            delete pVRLabel[i];
      delete[] pVRLabel;
   }
   if(pVRBand != 0)
   {
      for(int i = 0; i < nSeleccionadas; i++)
         if(pVRBand[i] != 0)
            delete pVRBand[i];
      delete[] pVRBand;
   }
   if(pVRPage != 0)
   {
      for(int i = 0; i < nSeleccionadas; i++)
         if(pVRPage[i] != 0)
            delete pVRPage[i];
      delete[] pVRPage;
   }
   if(CurrentImage != 0)
      delete CurrentImage;
} 


[ecfisa]

Hola aguml.

Una consulta, ¿ La variable rutas es un arreglo lineal o bidimensional ?

Saludos

[Ñ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.