Autor Tema: Error reservar memoria dentro funcion (Leído 2521 veces)

krnl64 · « **en:** Miércoles 10 de Febrero de 2010, 22:17 »

Hola amigos.
Estoy intentando reservar memoria para una variable externa a la función que la reserva.
Dentro de la función la reserva y la usa,pero al volver el codigo a donde hizo la llamada
falla.
Concretamente falla en la linea 17 de main.c
Por más que miro no sé donde está el fallo.
Pueden hecharme una mano ?
Gracias.

aleatorio.h

Código: C

 
void alsr(int **vector, int inicio, int fin, int cuantos);

aleatorio.c

Código: C

 
#include<stdio.h>
#include<stdlib.h>
#include "aleatorio.h"
 
void alsr(int **vector, int inicio, int fin, int cuantos)
{
    int *temporal=NULL;
    int i;
    int j;
    int pos;
    int lon;
 
 
    lon=fin-inicio; 
 
    /*Pedir memoria para temporal.*/
    temporal=(int*)malloc(lon*sizeof(int));
 
    if(temporal==NULL)
        return;
 
    for(i=0;i<lon;i++)
    {
        *(temporal+i)=inicio+i; 
    }
 
    /*Pedir memoria array externo*/
    *vector=(int*)malloc(cuantos*sizeof(int));
    if(vector==NULL)
        return;
 
    /*Llenamos el vector de aleatorios*/
    for(i=0;i<cuantos;i++)
    {
    
        /* Si tengo mas de un elemento */
        if(cuantos > 1)
        {
            pos=rand()%lon;
            /*Copio el elemento de esa posicion al original*/
            *(vector+i)=*(temporal+pos);
        
            /*Realmente atrasamos el elemento del array*/
            for(j=pos;j<lon-1;j++)
                *(temporal+j)=*(temporal+j+1);
 
            /*Vamos eliminando el ultimo elemento*/
            lon--;      
            temporal=(int*)realloc(temporal,lon*sizeof(int));
        }
        else
        {       
            *(vector+i)=*(temporal);
        }
 
    }
 
 
    free(temporal);
    temporal=0;
 
    return;
}
 
 

main.c

Código: C

 
#include <stdio.h>
#include <stdlib.h>
#include "aleatorio.h"
 
#define cantidad 2
 
int main(int argc,char *argv[])
{
    int *tmp=0;
    int i;
 
    alsr(&tmp,1,5,cantidad);
 
    for(i=0;i<cantidad;i++)
    {
        printf(" %d ",*(tmp+i));
    }
getchar();
}
 
 
 

Eternal Idol

No mire mucho pero el compilador da dos warnings y esas lineas estan mal (C++ no compilaria esto

):
aleatorio.c(42) : warning C4047: '=' : 'int *' differs in levels of indirection from 'int'
aleatorio.c(54) : warning C4047: '=' : 'int *' differs in levels of indirection from 'int'

Lo depure rapidamente y tmp era un puntero hasta que llegue a la linea 42 y al ejecutarla se transformo en 2, al volver al main seguia teniendo ese valor.

PD. Esa manera se acceder a los arrays ya habras visto que no es la mas adecuada, cuesta leer el codigo y es facil equivocarse.

krnl64

Cierto. No mire los warnings.

He sustituido el prototipo y el malloc por esto

Código: C

 
void alsr(int *vector, int inicio, int fin, int cuantos);

Linea 29 en aleatorio.c

Código: C

 
vector=(int*)malloc(cuantos*sizeof(int));

Entiendo que se queja de la indirección, pero ahora sigue fallando y no sé por qué ni cómo arreglarlo.
Ya no es un doble puntero.

Gracias

Eternal Idol

En el codigo original en vector TODO lo que tenes es 1 la direccion de memoria de tmp y 2, en *vector, su contenido (0). Lo que pasa es que estas accediendo a vector[0], es decir el *tmp, tal cual lo dice el warning. Estas haciendo vector[0] = numero que es lo mismo que *tmp = numero.

Esto si pone el numero donde debe ir:

Código: C

(*(*(vector+i)))=*(temporal+pos);
//legible:
*vector[i] = temporal[pos];
 

El problema es que no tenes mas lugar donde seguir poniendolos despues ... en cuanto i sea uno intentara acceder justo despues de la direccion de memoria de tmp como si fuera un puntero y boom.

Si no pasas la direccion de tmp (con **) esta seguira siendo 0 en main siempre y tal como estaba originalmente solo habia UN puntero.

krnl64

Pude leer la explicación antes de que editaras.
Gracias Eternal.
Salu2

Eternal Idol

El problema realmente esta al intentar acceder en alsr a los elementos como si fuera un array de punteros a int cuando no lo es.

Antes o despues de:

Código: Text

int *temporal=NULL;
 

Declaramos otro puntero comun tal como tmp:

Código: Text

int *nvector = NULL;
 

Despues de reservar memoria para vector hacemos:

Código: Text

nvector = (int*)*vector;
 

Accediendo finalmente de esta manera funcionara:

Código: Text

nvector[i] = temporal[pos];
...
//para el else
nvector[i] = temporal[0];
 
 

PD. Usando nvector en lugar de vector podes usar tu sintaxis original pero no se entiende bien a simple vista

PD2. Cuidado con lon que puede volverse 0, con cantidad siendo 5 por ejemplo, y al calcular pos dar una excepcion por divir por 0.

krnl64

Cita de: "Eternal Idol"

El problema realmente esta al intentar acceder en alsr a los elementos como si fuera un array de punteros a int cuando no lo es.
Despues de reservar memoria para vector hacemos:
Código: Text
nvector = (int*)*vector;

Si, te entendi al ver

Código: C

 
 (*(*(vector+i)))=*(temporal+pos);

ya que en el contexto de la función es un doble puntero aunque realmente en su contexto original no lo era.
Me costo deducir el cast y luego lo he visto aqui escrito.

Hasta ahora solo habia reservado memoria para una variable externa a una función si usar esa memoria dentro de la función que la reserva.
Me acuesto habiendo aprendido algo más

Gracias de nuevo.
Salu2

Eternal Idol

De nadas

SoloCodigo

Autor Tema: Error reservar memoria dentro funcion (Leído 2521 veces)

krnl64

Error reservar memoria dentro funcion

Eternal Idol

Re: Error reservar memoria dentro funcion

krnl64

Re: Error reservar memoria dentro funcion

Eternal Idol

Re: Error reservar memoria dentro funcion

krnl64

Re: Error reservar memoria dentro funcion

Eternal Idol

Re: Error reservar memoria dentro funcion

krnl64

Re: Error reservar memoria dentro funcion

Eternal Idol

Re: Error reservar memoria dentro funcion