Donar $20 Donar $50 Donar $100 Donar mensualmente
 


Enviar respuesta 
 
Calificación:
  • 0 votos - 0 Media
  • 1
  • 2
  • 3
  • 4
  • 5
Buscar en el tema
Alguno me ayuda que esta pasando en este codigo
Autor Mensaje
cincue Sin conexión
Campeon del cubo Rubik
A toda mostaza
****

Ing. en Sistemas
Facultad Regional Buenos Aires

Mensajes: 129
Agradecimientos dados: 20
Agradecimientos: 17 en 4 posts
Registro en: Mar 2013
Mensaje: #1
Alguno me ayuda que esta pasando en este codigo
Me tira segmentation fault cuando hace el FOR para mostrar no entiendo porque, si lo hago manualmente funciona. Y si al FOR le pongo que imprima hasta el 2 elemento lo hace, a partir del 3 se caga.




#include <stdio.h>
#include <stdlib.h>


int *creaVector(int*);

int main()
{
int *vec;
int k;

vec= creaVector(vec);

for(k=0;k<5;k++)
printf("%i",vec[k]);


return 0;
}

int *creaVector(int *vec){

vec[0]= 2;
vec[1]= 5;
vec[2]= 3;
vec[3]= 0;
vec[4]= 6;

return(vec);

}



(Este mensaje fue modificado por última vez en: 11-08-2013 01:43 por cincue.)
11-08-2013 01:41
Encuentra todos sus mensajes Agregar agradecimiento Cita este mensaje en tu respuesta
Abend Sin conexión
Campeon del cubo Rubik
Sin estado :(
****

Ing. Electrónica
Facultad Regional Buenos Aires

Mensajes: 180
Agradecimientos dados: 27
Agradecimientos: 35 en 23 posts
Registro en: Feb 2013
Mensaje: #2
RE: Alguno me ayuda que esta pasando en este codigo
Te tira segmentacion fault porque nunca reservaste la memoria de lo que cargaste en el vector

Una solucion sin entrar en memoria dinamica:



#include <stdio.h>
#include <stdlib.h>


int *creaVector(int*);

int main()
{
int vec[5];
int k;

vec= creaVector(vec);

for(k=0;k<5;k++)
printf("%i",vec[k]);


return 0;
}

int *creaVector(int *vec){

vec[0]= 2;
vec[1]= 5;
vec[2]= 3;
vec[3]= 0;
vec[4]= 6;

return(vec);

}



(Este mensaje fue modificado por última vez en: 11-08-2013 02:02 por Abend.)
11-08-2013 01:54
Encuentra todos sus mensajes Agregar agradecimiento Cita este mensaje en tu respuesta
cincue Sin conexión
Campeon del cubo Rubik
A toda mostaza
****

Ing. en Sistemas
Facultad Regional Buenos Aires

Mensajes: 129
Agradecimientos dados: 20
Agradecimientos: 17 en 4 posts
Registro en: Mar 2013
Mensaje: #3
RE: Alguno me ayuda que esta pasando en este codigo
(11-08-2013 01:54)Abend escribió:  Te tira segmentacion fault porque nunca reservaste la memoria de lo que cargaste en el vector

Tendria que hacer un malloc? Hay otra forma que no sea con malloc ?
11-08-2013 01:58
Encuentra todos sus mensajes Agregar agradecimiento Cita este mensaje en tu respuesta
Abend Sin conexión
Campeon del cubo Rubik
Sin estado :(
****

Ing. Electrónica
Facultad Regional Buenos Aires

Mensajes: 180
Agradecimientos dados: 27
Agradecimientos: 35 en 23 posts
Registro en: Feb 2013
Mensaje: #4
RE: Alguno me ayuda que esta pasando en este codigo
(11-08-2013 01:58)cincue escribió:  
(11-08-2013 01:54)Abend escribió:  Te tira segmentacion fault porque nunca reservaste la memoria de lo que cargaste en el vector

Tendria que hacer un malloc? Hay otra forma que no sea con malloc ?

Ahi edite =P sin malloc es como te lo puse arriba
11-08-2013 02:03
Encuentra todos sus mensajes Agregar agradecimiento Cita este mensaje en tu respuesta
cincue Sin conexión
Campeon del cubo Rubik
A toda mostaza
****

Ing. en Sistemas
Facultad Regional Buenos Aires

Mensajes: 129
Agradecimientos dados: 20
Agradecimientos: 17 en 4 posts
Registro en: Mar 2013
Mensaje: #5
RE: Alguno me ayuda que esta pasando en este codigo
(11-08-2013 02:03)Abend escribió:  
(11-08-2013 01:58)cincue escribió:  
(11-08-2013 01:54)Abend escribió:  Te tira segmentacion fault porque nunca reservaste la memoria de lo que cargaste en el vector

Tendria que hacer un malloc? Hay otra forma que no sea con malloc ?

Ahi edite =P sin malloc es como te lo puse arriba

Me parece que esta mal porque la funcion espera un int* y vos le pasas un int, no compila
11-08-2013 02:23
Encuentra todos sus mensajes Agregar agradecimiento Cita este mensaje en tu respuesta
Abend Sin conexión
Campeon del cubo Rubik
Sin estado :(
****

Ing. Electrónica
Facultad Regional Buenos Aires

Mensajes: 180
Agradecimientos dados: 27
Agradecimientos: 35 en 23 posts
Registro en: Feb 2013
Mensaje: #6
RE: Alguno me ayuda que esta pasando en este codigo
si vos declaras int vec[5];

y le pasas vec, vec es un puntero a int. O sino que es lo mismo &vec[0]
(Este mensaje fue modificado por última vez en: 11-08-2013 02:43 por Abend.)
11-08-2013 02:42
Encuentra todos sus mensajes Agregar agradecimiento Cita este mensaje en tu respuesta
cincue Sin conexión
Campeon del cubo Rubik
A toda mostaza
****

Ing. en Sistemas
Facultad Regional Buenos Aires

Mensajes: 129
Agradecimientos dados: 20
Agradecimientos: 17 en 4 posts
Registro en: Mar 2013
Mensaje: #7
RE: Alguno me ayuda que esta pasando en este codigo
(11-08-2013 02:42)Abend escribió:  si vos declaras int vec[5];

y le pasas vec, vec es un puntero a int. O sino que es lo mismo &vec[0]

entiendo pero porque no compila aun?
11-08-2013 03:38
Encuentra todos sus mensajes Agregar agradecimiento Cita este mensaje en tu respuesta
Desert69 Sin conexión
Presidente del CEIT
Sin estado :( / "Anarquia...
********

Ing. en Sistemas
Facultad Regional Buenos Aires

Mensajes: 2.287
Agradecimientos dados: 156
Agradecimientos: 253 en 162 posts
Registro en: Jun 2008
Mensaje: #8
RE: Alguno me ayuda que esta pasando en este codigo
¡Buenas!

Si ya te están pasando el puntero a la zona de memoria que tenés que modificar, no hace falta que la devuelvas y la asignes =)

#include <stdio.h>

void creaVector(int*);

int main() {
int vec[5];
int k;

creaVector(vec);
for(k=0;k<5;k++)
printf("%i",vec[k]);

return 0;
}

void creaVector(int *vec) {
vec[0]= 2;
vec[1]= 5;
vec[2]= 3;
vec[3]= 0;
vec[4]= 6;
}



Si quisieras usar memoria dinámica, harías algo así:


#include <stdio.h>
#include <stdlib.h>

int *creaVector();

int main() {
int *vec;
int k;

vec = creaVector();
for(k=0;k<5;k++)
printf("%i",vec[k]);

free(vec);
return 0;
}

int *creaVector(void) {
// version prolija de malloc(sizeof(int) * 5)
int *numeros = calloc(5, sizeof(int));
numeros[0]= 2;
numeros[1]= 5;
numeros[2]= 3;
numeros[3]= 0;
numeros[4]= 6;
return numeros;
}


La memoria se reserva con calloc() en este caso en lugar de malloc(), pero en sí es lo mismo. En este caso creaVector() **sí** devuelve el puntero, porque vec no apunta a ningún lado (fijate incluso que ni siquiera se la pasamos a creaVector()).

Básicamente, en este último caso, creaVector() crea un nuevo array con esos datos pre-seteados, mientras que en el caso anterior creaVector() **solamente llena los datos** (la memoria ya se la habían reservado de afuera). Por eso la primer versión necesita un puntero al bloque de memoria en que escribir, mientras que en la nueva no necesita nada (el que quiera pensar en un constructor de Objetos, no es eso, pero la idea no es taaaaaaan distinta).

Por otro lado, fijate que como la memoria la estamos pidiendo nosotros con calloc(), en el segundo caso tenemos que **liberar la memoria** con un free(). En la primer versión, como el array es estático, la memoria la reserva el compilador en la entrada de stack de la función main(). Entonces, esa memoria la va a liberar el mismo compilador, cuando retire del stack el bloquecito de variables que corresponden a la función main() cuando esta finalice. Por eso no le hacemos free() en la primer versión.


"Che, ¿y si quisiera que creaVector() de la segunda versión igualmente reciba el puntero y no devuelva nada?"

Buen, podés hacerlo, pero el puntero no va a ser el mismo que esperamos en la primer versión. En la primer versión, el puntero que recibimos apunta a la zona donde escribir los datos. Nosotros *no modificamos el puntero*, sino su contenido. Recordemos que en C todos los pasajes de parámetros son por valor (es decir, recibimos una copia del valor de la variable con que nos llamaron), entonces las modificacionesque hagamos a nuestros parámetros sólo duran el scope de nuestra función: cuando retornemos a la función que nos llamó, las variables que nos mandaron como parámetros van a estar intactas.

Entonces, el truco es el de siempre: pasar punteros a lo que sea que querramos modificar.

Nos encantaría hacer algo como esto, como primer approach:

#include <stdio.h>
#include <stdlib.h>

void creaVector(int*);

/* WARNING: ESTO NO FUNCIONA */
/* WARNING: ESTO NO FUNCIONA */
/* WARNING: ESTO NO FUNCIONA */

int main() {
int *vec;
int k;

creaVector(vec);
for(k=0;k<5;k++)
printf("%i",vec[k]);

free(vec);
return 0;
}

void creaVector(int *numeros) {
numeros = calloc(5, sizeof(int));
numeros[0]= 2;
numeros[1]= 5;
numeros[2]= 3;
numeros[3]= 0;
numeros[4]= 6;
}


PEEEEEEEERO estaríamos modificando numeros sólo en el contexto de creaVector(), y cuando vuelva la ejecución al main, vec sigue valiendo lo mismo de siempre. La solución es pasarle a creaVector un puntero a vec:


#include <stdio.h>
#include <stdlib.h>

void creaVector(int**);

int main() {
int *vec;
int k;

creaVector(&vec); // & = "la dirección de" = "un puntero a"
for(k=0;k<5;k++)
printf("%i",vec[k]);

free(vec);
return 0;
}

void creaVector(int **numeros) { // ** porque es puntero a puntero a int
*numeros = calloc(5, sizeof(int)); // *numeros = "el contenido de numeros"
(*numeros)[0]= 2;
(*numeros)[1]= 5;
(*numeros)[2]= 3;
(*numeros)[3]= 0;
(*numeros)[4]= 6;
}


Como es una paja escribir tantos (*numero)[?]=?;, hacemos alguna manganeta loca para simplificar las cosas:


void creaVector(int **numeros) {
int *auxiliar = calloc(5, sizeof(int)); // un int* común y silvestre
auxiliar[0]= 2;
auxiliar[1]= 5;
auxiliar[2]= 3;
auxiliar[3]= 0;
auxiliar[4]= 6;

*numeros = auxiliar; // al contenido de numeros le asigno el valor de mi puntero auxiliar
}




Si me preguntan, es más feliz la alternativa de creaVector() sin parámetros (porque, vamos, si él mismo va a reservar la zona de memoria, efectivamente no necesita ningún dato externo para funcionar - ¡¿¿quién dijo encapsulamiento??! ¡Te escuché!).


That's it =)

[Imagen: a2.php]
[Imagen: 971aa6599664453c05cb3e42d58bbc0eo.jpg]
11-08-2013 11:58
Visita su sitio web Encuentra todos sus mensajes Agregar agradecimiento Cita este mensaje en tu respuesta
[-] Desert69 recibio 1 Gracias por este post
Jarry (12-08-2013)
Abend Sin conexión
Campeon del cubo Rubik
Sin estado :(
****

Ing. Electrónica
Facultad Regional Buenos Aires

Mensajes: 180
Agradecimientos dados: 27
Agradecimientos: 35 en 23 posts
Registro en: Feb 2013
Mensaje: #9
RE: Alguno me ayuda que esta pasando en este codigo
(11-08-2013 03:38)cincue escribió:  
(11-08-2013 02:42)Abend escribió:  si vos declaras int vec[5];

y le pasas vec, vec es un puntero a int. O sino que es lo mismo &vec[0]

entiendo pero porque no compila aun?

Estaria re copado que minimo pongas el mensaje de error, adivinos no somos.
11-08-2013 13:06
Encuentra todos sus mensajes Agregar agradecimiento Cita este mensaje en tu respuesta
brunodiaz Sin conexión
The Dark Knight
Bla
**********

Ing. en Sistemas
Facultad Regional Buenos Aires

Mensajes: 7.762
Agradecimientos dados: 136
Agradecimientos: 384 en 132 posts
Registro en: May 2008
Mensaje: #10
RE: Alguno me ayuda que esta pasando en este codigo
Si ejecutas el programa en el foro veias el mensaje de error
11-08-2013 15:00
Encuentra todos sus mensajes Agregar agradecimiento Cita este mensaje en tu respuesta
cincue Sin conexión
Campeon del cubo Rubik
A toda mostaza
****

Ing. en Sistemas
Facultad Regional Buenos Aires

Mensajes: 129
Agradecimientos dados: 20
Agradecimientos: 17 en 4 posts
Registro en: Mar 2013
Mensaje: #11
RE: Alguno me ayuda que esta pasando en este codigo
(11-08-2013 11:58)Desert69 escribió:  ¡Buenas!

Si ya te están pasando el puntero a la zona de memoria que tenés que modificar, no hace falta que la devuelvas y la asignes =)

#include <stdio.h>

void creaVector(int*);

int main() {
int vec[5];
int k;

creaVector(vec);
for(k=0;k<5;k++)
printf("%i",vec[k]);

return 0;
}

void creaVector(int *vec) {
vec[0]= 2;
vec[1]= 5;
vec[2]= 3;
vec[3]= 0;
vec[4]= 6;
}



Si quisieras usar memoria dinámica, harías algo así:


#include <stdio.h>
#include <stdlib.h>

int *creaVector();

int main() {
int *vec;
int k;

vec = creaVector();
for(k=0;k<5;k++)
printf("%i",vec[k]);

free(vec);
return 0;
}

int *creaVector(void) {
// version prolija de malloc(sizeof(int) * 5)
int *numeros = calloc(5, sizeof(int));
numeros[0]= 2;
numeros[1]= 5;
numeros[2]= 3;
numeros[3]= 0;
numeros[4]= 6;
return numeros;
}


La memoria se reserva con calloc() en este caso en lugar de malloc(), pero en sí es lo mismo. En este caso creaVector() **sí** devuelve el puntero, porque vec no apunta a ningún lado (fijate incluso que ni siquiera se la pasamos a creaVector()).

Básicamente, en este último caso, creaVector() crea un nuevo array con esos datos pre-seteados, mientras que en el caso anterior creaVector() **solamente llena los datos** (la memoria ya se la habían reservado de afuera). Por eso la primer versión necesita un puntero al bloque de memoria en que escribir, mientras que en la nueva no necesita nada (el que quiera pensar en un constructor de Objetos, no es eso, pero la idea no es taaaaaaan distinta).

Por otro lado, fijate que como la memoria la estamos pidiendo nosotros con calloc(), en el segundo caso tenemos que **liberar la memoria** con un free(). En la primer versión, como el array es estático, la memoria la reserva el compilador en la entrada de stack de la función main(). Entonces, esa memoria la va a liberar el mismo compilador, cuando retire del stack el bloquecito de variables que corresponden a la función main() cuando esta finalice. Por eso no le hacemos free() en la primer versión.


"Che, ¿y si quisiera que creaVector() de la segunda versión igualmente reciba el puntero y no devuelva nada?"

Buen, podés hacerlo, pero el puntero no va a ser el mismo que esperamos en la primer versión. En la primer versión, el puntero que recibimos apunta a la zona donde escribir los datos. Nosotros *no modificamos el puntero*, sino su contenido. Recordemos que en C todos los pasajes de parámetros son por valor (es decir, recibimos una copia del valor de la variable con que nos llamaron), entonces las modificacionesque hagamos a nuestros parámetros sólo duran el scope de nuestra función: cuando retornemos a la función que nos llamó, las variables que nos mandaron como parámetros van a estar intactas.

Entonces, el truco es el de siempre: pasar punteros a lo que sea que querramos modificar.

Nos encantaría hacer algo como esto, como primer approach:

#include <stdio.h>
#include <stdlib.h>

void creaVector(int*);

/* WARNING: ESTO NO FUNCIONA */
/* WARNING: ESTO NO FUNCIONA */
/* WARNING: ESTO NO FUNCIONA */

int main() {
int *vec;
int k;

creaVector(vec);
for(k=0;k<5;k++)
printf("%i",vec[k]);

free(vec);
return 0;
}

void creaVector(int *numeros) {
numeros = calloc(5, sizeof(int));
numeros[0]= 2;
numeros[1]= 5;
numeros[2]= 3;
numeros[3]= 0;
numeros[4]= 6;
}


PEEEEEEEERO estaríamos modificando numeros sólo en el contexto de creaVector(), y cuando vuelva la ejecución al main, vec sigue valiendo lo mismo de siempre. La solución es pasarle a creaVector un puntero a vec:


#include <stdio.h>
#include <stdlib.h>

void creaVector(int**);

int main() {
int *vec;
int k;

creaVector(&vec); // & = "la dirección de" = "un puntero a"
for(k=0;k<5;k++)
printf("%i",vec[k]);

free(vec);
return 0;
}

void creaVector(int **numeros) { // ** porque es puntero a puntero a int
*numeros = calloc(5, sizeof(int)); // *numeros = "el contenido de numeros"
(*numeros)[0]= 2;
(*numeros)[1]= 5;
(*numeros)[2]= 3;
(*numeros)[3]= 0;
(*numeros)[4]= 6;
}


Como es una paja escribir tantos (*numero)[?]=?;, hacemos alguna manganeta loca para simplificar las cosas:


void creaVector(int **numeros) {
int *auxiliar = calloc(5, sizeof(int)); // un int* común y silvestre
auxiliar[0]= 2;
auxiliar[1]= 5;
auxiliar[2]= 3;
auxiliar[3]= 0;
auxiliar[4]= 6;

*numeros = auxiliar; // al contenido de numeros le asigno el valor de mi puntero auxiliar
}




Si me preguntan, es más feliz la alternativa de creaVector() sin parámetros (porque, vamos, si él mismo va a reservar la zona de memoria, efectivamente no necesita ningún dato externo para funcionar - ¡¿¿quién dijo encapsulamiento??! ¡Te escuché!).


That's it =)

Muy buenas las variantes man, mucha magia se agradece.
11-08-2013 17:48
Encuentra todos sus mensajes Agregar agradecimiento Cita este mensaje en tu respuesta
Buscar en el tema
Enviar respuesta 




Usuario(s) navegando en este tema: 1 invitado(s)



    This forum uses Lukasz Tkacz MyBB addons.