Acceder un array por direccion

[lucas_max2002]

Hola a todos

Estoy tratando de solucionar un problema en mi codigo, pero no he podido.

Tengo una clase que se llama Red.

La clase red tiene una propiedad que se llama Tag, donde guardo cualquier valor de tipo int.

Ahora, tengo un arreglo declarado de esta manera

Código C++:

Red Redes**;
 
Redes = (Red**)malloc(sizeof(Red*)*1024);
 
for (int i = 0; i < 1024; i++)
   Redes[i] = new Red(...);
 

La cuestion es que ahora en un trozo de codigo necesito acceder a cada elemento de Redes, pero no a traves del indizador sino de la direccion. Por ello he echo esta funcion para obtener cada elemento

Código C++:

inline Red* ACCESO_INDIVIDUAL(void* buffer, int index)
{
    return (Red*)buffer+id;
}
 

pero cuando la llamo y hago

Código C++:

int tag_value = (*ACCESO_INDIVIDUAL(Redes, 5)).Tag;
 

no obtengo el valor que deberia, sino un valor aleatorio

Entonces mi pregunta es, estoy accediendo mal, o debo de hacerlo de otra manera??

Espero me puedan colaborar, gracias

[cesar_casla]

Hola! nose mucho de C++, pero esta forma de declarar:

Código C++:

Red Redes**;
 

no la conocia. No deberia ser Red **Redes???
Tambien en la funcion usas "id" cuando deberia ser bufer+"index" ....

Sacando esos detalles, te marco algunas observaciones:

1º Me parece que no es conveniente mezclar funciones como malloc y new. Si estas en C++ usa new

Código C++:

Redes= new Red*[1024];
 

2º En la funcion vos casteas bufer+"index" a (Red *), pero bufer+index sigue siendo un puntero a puntero a Red por lo tanto el casteo esta demas, y deberias retornar Red**

Código C++:

inline Red** ACCESO_INDIVIDUAL(void* buffer, int index)
    {
        return (Red**)buffer+id;
    }
 

3º Otra observacion es que no le encuentro mucho sentido declarar Redes como puntero a puntero, no es mas conveniente hacer

Código C++:

Red *Redes;
     
    Redes = new Red[1024];
     
    for (int i = 0; i < 1024; i++)
       Red[i].Tag= xNumero;
 

Saludos

[lucas_max2002]

Hola cesar_casla

Gracias por tus observaciones

Hola! nose mucho de C++, pero esta forma de declarar:
Red Redes**;
no la conocia. No deberia ser Red **Redes???

Tambien en la funcion usas "id" cuando deberia ser bufer+"index" ....

Tienes razon, me he equivocado al copiar el codigo aqui en el foro. Tambien en lo de id e index. Lo siento.

1º Me parece que no es conveniente mezclar funciones como malloc y new. Si estas en C++ usa new
Redes= new Red*[1024];

Esto lo hago asi, para poder llamar el constructor de cada elemento del array de manera independiente. Si trato de hacerlo, con la declaracion que tu me dices, me da unos errores.

2º En la funcion vos casteas bufer+"index" a (Red *), pero bufer+index sigue siendo un puntero a puntero a Red por lo tanto el casteo esta demas, y deberias retornar Red**
inline Red** ACCESO_INDIVIDUAL(void* buffer, int index)
{
return (Red**)buffer+id;
}

En este caso retorno solo *, ya que es un elemento del array. El ** es para su conjunto

3º Otra observacion es que no le encuentro mucho sentido declarar Redes como puntero a puntero, no es mas conveniente hacer

Red *Redes;

Redes = new Red[1024];

for (int i = 0; i < 1024; i++)
Red[i].Tag= xNumero;

(Puntero)(Puntero) **. Variable cuyos posibles valores son direcciones de otras variables puntero. O al menos asi lo entiendo.
Tambien he probado esta definicion que me dices, y asi no me permite llamar al constructor del objeto.


De todas formas, he probado todos los cambios que me has sugerido, y asi se me genera una serie de errores.

Muchas gracias

[cesar_casla]

Lo del constructor tiene mas sentido, ahora entiendo xq puntero a puntero. Igual subi todo el codigo asi t ayudo mejor

[lucas_max2002]

Hola cesar_casla

He echo otro codigo, mas cortito, pero con el mismo planteamiento de antes.
Podes copiar todo en un solo archivo.

para compilar estoy usando el g++

Código C++:

#include <stdio.h>
#include <stdlib.h>
 
class Network
{
    public:
        int Tag;
        
        Network(int tag);
};
 
Network::Network(int tag)
{
    Tag = tag;
}
 
int main(void)
{
    Network** net = (Network**)malloc(sizeof(Network)*100);
    
    for (int i=0; i < 100; i++)
        net[i] = new Network(i);
                
    printf ("The value of net[3]->Tag = %d\n", ((Network*)net+3).Tag);
 
    return 0;
}
 

la salida del printf me da

The value of net[3]->Tag = 134521760

cuando deberia ser

The value of net[3]->Tag = 3

sigo sin enterder porque no me da el resultado esperado.

muchas gracias por tu colaboracion

[cesar_casla]

Gracias por hacer un miniprograma de tu duda y no postiar 100 lineas como hacen muchos...


Primero que nada es raro que el g++ no te haya marcado un error del codigo que pusiste(use devC++ y a mi si me marco error).
Como ya te habia dicho antes el cast que haces en:

Código C++:

(Network*)net+3).Tag
 

esta mal. Aclaracion:
net: es un puntero a puntero a Network
*net: es un puntero a Network
**net: es el objeto Network

Sabiendo esto si vos haces net+3 estas modificando el puntero a puntero, pero para poder acceder a Tag tenias que hacer: *(net+3) ó net[3]. Usando cualquiera de esas dos formas en ves del "." debes usar "->" ya que cada subindice del arreglo net es un puntero por lo tanto para acceder a un miembro del objeto tenes que usar -> (tema muy tratado cuando se ve estructuras).

Código C++:

(*(net+3))->Tag;
 

Tambien podes usar el "." pero tenes que acceder al objeto, osea

Código C++:

(**(net+3)).Tag
 

Basta de palabras, te dejo el codigo sin usar malloc y funcionando con 3 formas de imprimir Tag:

Código C++:

#include <stdio.h>
    #include <stdlib.h>
     
    class Network
    {
        public:
            int Tag;
           
            Network(int tag);
    };
     
    Network::Network(int tag)
    {
        Tag = tag;
    }
     
    int main(void)
    {
        Network** net = new Network*[100];
       
        for (int i=0; i < 100; i++)
            net[i] = new Network(i);
                   
        printf ("The value of net[3]->Tag = %d\n", (*(net+3))->Tag); // usando la aritmetica de punteros con "->"
         printf ("The value of net[3]->Tag = %d\n", net[3]->Tag); // usando subindices con "->"
        printf ("The value of net[3]->Tag = %d\n", (**(net+3)).Tag); //usando la aritmetica de punteros con "."
     system("PAUSE");
        return 0;
    }
 

Saludos

PD: no es un tema facil comprender el uso de puntero a puntero sin un grafico; si encuentro algo lo subo

[lucas_max2002]

Hola cesar_casla

Muchisimas gracias por tu colaboracion.

He podido resolver la duda y el problema que tenia.

de nuevo, muchas gracias