SoloCodigo

Hola a todos

Tengo un problema relacionado con esta pregunta que hice:

http://foros.solocodigo.com/index.php?showtopic=25725&hl=

Resulta que ahora al usar el metodo de esa libreria (dll) con el siguiente codigo de llamada


Me salta el siguiente error de ejecucion:



en el error salta cuando esta dentro de la tercera llamada a AgregarModulo, cuando va a borrar los modulos de la clase SMRentorno para luego copiarle el ptro aux con el modulo mod3 añadido

Lo raro es que esto solo sucede cuando se trata de la libreria enlazada dinamicamente, si lo hago estaticamente no sucede ningun error

Alguien tiene idea de lo que puede suceder ???

Gracias

PD: el codigo del metodo AgregarModulo esta aqui

http://foros.solocodigo.com/index.php?showtopic=25725&hl=

Ok gracias

Eso es lo que hice debugear y ver la linea de codigo que lanza el error, que es un delete de una reserva de memoria de un ptro a vectores que se hace en una funcion que agrega objetos modulos a un contenedor de modulos **modulosF , para vaciar lo que hay en ese contenedor y asignarle la posicion de memoria de un ptro aux donde ya se agrego el nuevo modulo


Lo que no entiendo es por que me salta este error al introducir el tercer modulo y por que solamente me pasa cuando el enlace de la libreria es dinamico. Si fuera cosa del codigo tambien saltaria el error si estuviera la libreria estatica, no?

Quizas sea cosa de la dll?? el manejo de memoria dinamica es diferente para una dll?

He simplificado el problema lo maximo posible, dos clases: una contenedora y otra para meter en el contenedor (las dos con 1 solo atributo, el nombre)

He puesto ordenes de copia para los archivos dll y el .h en el postbuild del proyecto de la dll para que vayan a la carpeta "C:\...\Microsoft Visual Studio\MyProjects" y el archivo .lib al directorio "...\VC98\Lib"

El zip esta adjunto al mensaje.

Gracias por el interes

Gracias ¡¡


Pues maneras de hacerlo solo se me ocurre crear un metodo estatico de la clase al que se le pase el puntero y en el hacer la reserva de memoria. Mañana lo probare ....

Si pero hay algo que tambien me parecio raro:


Estoy de acuerdo con que this->modulos lo creas con new[]() pero this->modulo lo creas con new(). El primero es un array pero el segundo no, proba ese cambio  

PD. Despues proba ese cambio con el new/delete usado antes  

El primero borra la reserva de memoria sin llamar al destructor y el otro simplemente llama al destructor de la clase??

No, delete hace lo mismo siempre (llamada al destructor + liberacion de memoria) PERO uno es para un elemento y el otro para arrays.

delete[]: The function called by a delete expression to deallocate storage for an array of objects.

delete: The function called by a delete expression to deallocate storage for individual of objects.

Por cierto, el otro arreglo es necesario, no se si lo habras probado como antes pero yo si y seguiria fallando si no estuvieran los dos arreglos juntos  

Ok, el otro delete si lo he mantenido

Ahora si que funciona creando los objetos en la aplicacion y no en la dll.

Asi que era cosa de ese maldito delete[]    

De todas maneras conviene mantener todo en un heap?? podria dar algun problema  mas?

Si, como te mostraba antes:


El primero estaba mal y el segundo estaba bien ya que era un array.

Las dos cosas saltan, tal vez como usas el VC 6.0 no lo veas pero con el .NET 2003 si que se ve, calcula que son asserts de la RTL ...

Si, conviene siempre tener todo en un heap (a menos que quieras hacerte cargo de la gestion de memoria a un nivel inferior). Yo creo que no te dara mas problemas que tal vez un miserable rendimiento debido a que no libere la memoria