Acceder a variable extern desde otro archivo

[reethok]

Bueno, tengo estos archivos:

utility.h

Código C++:

extern static unsigned int global_id = 0;
 

item.cpp

Código C++:

#include "item.h"
#include "utility.h"
 
id_t getUniversalId()
{
    return global_id++;
}
 
Item::Item()
{
    // Genera el id Correcto
    id = getUniversalId();
 
    // Necesita nombrarse el item
    name = "UNDEFINED NAME";
 
    // Establece el precio del producto en 0.00
    price = 0;
}
 

Pero me da error en la función getUniversalId(), a qué se debe esto?

O de qué otro modo podría tener una variable general estática que se aumente en 1 cada que se cree un nuevo objeto Item, y al que puedan acceder diversos objetos en diversos archivos fuente?

Saludos.

[amchacon]

Prueba a quitar el extern y dejarlo solo con el static.

[cesar_casla]

Hola! el modificador extern se utiliza cuando se quiere acceder a una variable que fue creado desde otro archivo, pero no cuando se crea la variable. Por lo tanto en utility.h no debias haber usado "extern"
La combinacion "extern static" creo q no es valida(y si lo fuese no tiene mucho sentido). Si solo hubieras usado static para crear la variable global_id, lo que harias es limitar el alcance de dicha variable, haciendola invisible a otros archivos(lo cual no es tu objetivo)...


Resumiendo:
utility.h

Código C:

unsigned int global_id = 0; /* variable global */
 

item.cpp

Código C:

#include "item.h"
    #include "utility.h"
     extern global_id;
    id_t getUniversalId()
    {
        return global_id++;
    }
     
    Item::Item()
    {
        // Genera el id Correcto
        id = getUniversalId();
     
        // Necesita nombrarse el item
        name = "UNDEFINED NAME";
     
        // Establece el precio del producto en 0.00
        price = 0;
    }
 

Saludos

[reethok]

Iniciado por cesar_casla

Hola! el modificador extern se utiliza cuando se quiere acceder a una variable que fue creado desde otro archivo, pero no cuando se crea la variable. Por lo tanto en utility.h no debias haber usado "extern"
La combinacion "extern static" creo q no es valida(y si lo fuese no tiene mucho sentido). Si solo hubieras usado static para crear la variable global_id, lo que harias es limitar el alcance de dicha variable, haciendola invisible a otros archivos(lo cual no es tu objetivo)...


Resumiendo:
utility.h

Código C:

unsigned int global_id = 0; /* variable global */
 

item.cpp

Código C:

#include "item.h"
    #include "utility.h"
     extern global_id;
    id_t getUniversalId()
    {
        return global_id++;
    }
     
    Item::Item()
    {
        // Genera el id Correcto
        id = getUniversalId();
     
        // Necesita nombrarse el item
        name = "UNDEFINED NAME";
     
        // Establece el precio del producto en 0.00
        price = 0;
    }
 

Saludos

Hice esto y me da el siguiente error al compilar con g++ -c item.cpp:

Código:

item.cpp:3:12: error: ‘global_id’ does not name a type

¿A qué se debe esto?

Saludos.

[Fw190]

Debe ser:

Código C:

extern <tipo> global_id;
 

donde <tipo> debe coincidir con el tipo con el que se haya declarado global_id (en este caso, por ejemplo, unsigned int).

Y, en efecto, extern static no tiene sentido. Además, una variable que vaya a verse como extern desde otros módulos de programa no puede haber sido declarada con el atributo static en el módulo que sea su "propietario".

Si lo que quieres es, simplemente, saber el número de objetos creados, en lugar de ir guardando este dato en una variable global para todos los módulos de tu programa (variable que puede modificarse desde cualquier módulo), será más seguro crearla como static en el módulo en el que tienes el constructor, añade también una función que te devuelva el valor de esa variable (que incrementarás/decrementarás cada vez que generes/destruyas un nuevo objeto). Sería una sencilla modificación sobre lo que ya tienes en item.cpp:

Código C:

#include "item.h"
#include "utility.h"
 
static unsigned int globai_id = 0;
 
id_t getUniversalId()
{
    return global_id;
}
 
Item::Item()
{
    // Incremento el contador de objetos generados
    global_id++;
 
    // Necesita nombrarse el item
    name = "UNDEFINED NAME";
 
    // Establece el precio del producto en 0.00
    price = 0;
}
 
// Y, logicamente, hacer el correspondiente decremento en el destructor
 

[cesar_casla]

Hice esto y me da el siguiente error al compilar con g++ -c item.cpp:

Código:

item.cpp:3:12: error: ‘global_id’ does not name a type

me equivoque al tipiar(todo por hacer copy&paste donde no se debe).. El error se debe por lo q marco Fw190, me olvide de poner el tipo de dato de la variable:

item.cpp

Código C:

#include "item.h"
    #include "utility.h"
     extern unsigned int global_id; /* forma correcta xD */
 
/* ......... */
 

Saludos