Error lógico cuando compilo ejercicio - CONSULTA

Iniciado por Cero++, 10 Diciembre 2018, 15:06 PM

0 Miembros y 1 Visitante están viendo este tema.

Cero++

Buenas a todos, les adjunto aquí el código que hice, tenía que re-dimensionar un puntero, tal que, al usar esa función mi puntero se modifique con su nuevo tamaño y siga manteniendo los valores antiguos más los nuevos.
Si me pueden explicar lo que estoy haciendo mal, sería de gran ayuda, así puedo entenderlo  ;D
Aquí el código:

Código (cpp) [Seleccionar]
#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
int *Redimensionar(int *p,int c_adicionales, int _tam_de_p);
int main(int argc, char *argv[]) {
srand(time(0));
int *x=nullptr,n;
cout<<"ingrese el tamanio del arreglo"<<endl;
cin>>n;
x=new int [n];
for(int i=0;i<n;i++) {
x[i]=rand()%200;
}
for(int i=0;i<n;i++) {
cout<<x[i]<<" ";
}
cout<<endl;
for(int i=0;i<n+2;i++) {
delete []x;
}
int ceros;
cout<<"Ingrese la nueva cantidad: "; cin>>ceros;
x=Redimensionar(x,ceros,n); //Por que me da basura en los dos primeros valores?
for(int i=0;i<n+ceros;i++) {
cout<<x[i]<<" ";
}
return 0;
}


int *Redimensionar(int *p, int c_adicionales, int _tam_de_p){
int *aux=nullptr; int _tam=_tam_de_p+c_adicionales;
aux=new int [_tam];
for(int i=0;i<_tam_de_p;i++) {
aux[i]=p[i];
}
for(int i=_tam_de_p+1;i<_tam;i++) {
aux[i]=0;
}
return aux;
}
Ser diligente es lo ÚNICO que importa para lograr el éxito[c/ode]

CalgaryCorpus

Aqui mi perfil en LinkedIn, invitame un cafe aqui

Cero++

Buenas colega, me podrías explicar por qué funciona al hacer eso?
Gracias por responder!!!
Ser diligente es lo ÚNICO que importa para lograr el éxito[c/ode]

K-YreX

Línea 8: Declaras un puntero. Un puntero simplemente apunta, no es más.
Línea 11: Reservas memoria dinámica para n elementos del array. Ese espacio de memoria está apuntado por x. Cuando tienes memoria dinámica siempre tienes que tener un puntero apuntando ahí. Si el puntero apunta a otro sitio, pierdes esa memoria.
Línea 12-14: Guardas n valores aleatorios [0,199] en el array.

La función <Redimensionar()> coge el puntero que tenías reserva memoria adicional y añade 0 a ese espacio adicional. Antes de eso copias los valores de <x> (<p> en la función) en <aux> (el nuevo puntero) esto se hace en líneas 35-37. Si antes de llamar a la función borras la memoria de <x> cuando llegas a la función, <p> apunta a basura porque has borrado a lo que apuntaba originalmente.

Sin embargo aunque el programa ya funciona no estás liberando la memoria. Tienes que liberar la memoria en dos ocasiones:
- Primero, en la función. <x> apunta a un espacio de memoria pero después de llamar a la función va a apuntar a otro, va a apuntar al mismo sitio que <aux>. Entonces donde apuntaba <x> tienes que liberarlo antes de que acabe la función, pero después de que se ejecuten las líneas 35-37 para que no se copie basura.
- Segundo, al finalizar el programa. En la función has reservado memoria para <aux>. Luego has hecho que <x> apunte a ese espacio de memoria. Pero antes de que acabe el programa tienes que liberar el espacio a donde apunta <x>.

PD: Un array unidimensional se libera sin bucle <for>:
Código (cpp) [Seleccionar]

delete [] array;


Código (cpp) [Seleccionar]

cout << "Todos tenemos un defecto, un error en nuestro código" << endl;

CalgaryCorpus

Cita de: Cero++ en 11 Diciembre 2018, 00:18 AM
Buenas colega, me podrías explicar por qué funciona al hacer eso?
Gracias por responder!!!

Para mí resulta interesante saber por qué pusiste esas líneas ahí en primer lugar.
Aqui mi perfil en LinkedIn, invitame un cafe aqui

Cero++

Cita de: CalgaryCorpus en 11 Diciembre 2018, 06:03 AM
Para mí resulta interesante saber por qué pusiste esas líneas ahí en primer lugar.

Bueno, desde mi lógica fue así: puse el delete del puntero antes de la llamada de la función, porque pensaba que antes de hacer que apunte a un nuevo arreglo, que además, esta re-dimensionado, iba a dejar al puntero como si estuviese nulo y que así apuntara a la nueva dirección, ya que pensaba que mi puntero viejo, al tener un tamaño más pequeño no funcionaria, que solo tomaría la misma cantidad de valores y que por ende, al eliminarlo y hacer que apuntara a la nueva dirección re-dimensionada no habría inconvenientes. Por eso puse el delete antes de la invocación de la función.
Como veras, recién estoy aprendiendo esto, así que bueno, trato de razonar lo que estudio pero no siempre me resulta, pero con la ayuda de ustedes, que me dan la explicación de por qué cada cosa me termina de cerrar lo que estudié!  ;D
Ser diligente es lo ÚNICO que importa para lograr el éxito[c/ode]

Cero++

Cita de: YreX-DwX en 11 Diciembre 2018, 00:50 AM

Sin embargo aunque el programa ya funciona no estás liberando la memoria. Tienes que liberar la memoria en dos ocasiones....
PD: Un array unidimensional se libera sin bucle <for>
Código (cpp) [Seleccionar]

delete [] array;



Colega, me ayudó un montón su explicación y ya con leer lo que puso Calgarys pude darme cuenta de dónde colocar los dos deletes incluso antes de que respondieras a mi consulta!!  ;D
Gracias!
Ser diligente es lo ÚNICO que importa para lograr el éxito[c/ode]