Czy takie deklarowanie obiektu jest poprawne?

Question

Mam taki kod:

/* Plik myObject1.h

#include "Object2.h"

class myObject1
{
public:
	myObject1(int a);
	~myObject1();
	void clearDatas();
};

*/

#include "myObject1.h"

//Czy to deklarowanie obiektu liczy sie jako globalne, czy jest to zapis poprawny?
//Da sie w jakis sposob stworzyc obiekt w konstruktorze, zeby byl widoczny we wszystkich metodach?
Object2 mySecObject;

myObject1::myObject1(int a)
{
	mySecObject.methodOne(a);
}

myObject1::~myObject1()
{
	clearDatas();
}

void myObject1::clearDatas()
{
	mySecObject.deleteValues();
}

Tak jak w komentarzu: czy deklaracja obiektu mySecObject w tym miejscu liczy się jako globalna (a jak wiadomo lepiej unikać deklaracji globalnych)? A może jest to jak najbardziej poprawny zapis? Da się w jakiś sposób zadeklarować obiekt mySecObject w konstruktorze myObject1 tak, żeby można było go używać w innych metodach tej klasy?

Answer 1

mySecObject jest zmienną globalną w danej jednostce kompilacji (w jednym pliku cpp)
Jeżeli chcesz, żeby ta zmienna była dostępna wewnątrz twojej klasy to możesz ją tam umieścić.
 

#include "Object2.h"
 
class myObject1
{
private:
    Object2 mySecObject;
public:
    myObject1(int a);
    ~myObject1();
    void clearDatas();
};

Każda instancja klasy myObject1 będzie posiadać własną zmienną mySecObject.

Jeżeli chcesz aby zmienna mySecObject była wspólna dla wszystkich instancji klasy myObject1 to musisz ją uczynić statyczną.

//
// plik nagłówkowy
//
#include "Object2.h"
 
class myObject1
{
private:
    static Object2 mySecObject;
public:
    myObject1(int a);
    ~myObject1();
    void clearDatas();
};


//
// plik źródłowy
//
myObject1::Object2 mySecObject;

 

 

 

Comments

Chyba wkradł się mały błąd w pliku źródłowym w drugim bloku kodu.

Zamiast

myObject1::Object2 mySecObject;

Powinno być

Object2 myObject1::mySecObject;

 

---

Święta racja

Answer 2

Jeśli zamierzasz użyć mySecObject tylko i wyłącznie w metodach klasy myObject1 - możesz użyć prywatnego pola statycznego.

Coś w tym stylu:

// Plik myobject1.h
class myObject1
{
    static Object2 mySecObject;  //zostaw jako prywatne; będziemy tego używali tylko w metodach
};

// Plik myobject1.cpp
Object2 myObject1::mySecObject;

Jest to moim zdaniem bardziej poprawne niż deklarowanie zwykłej zmiennej globalnej - nasze mySecObject, mimo że statyczne, to bardziej enkapsulowane - mają do tego dostęp tylko metody myObject1, a więc ciężej coś zepsuć.

Mimo to, unikałbym zmiennych statycznych - nie są thread safe, a poza tym często można się bez nich obejść. Czy dałoby radę po prostu podać referencję do konstruktora myObject1, i to z niej korzystać w metodach? O tak:

// Plik myObject1.h
class myObject1
{
public:
    myObject1(int a, Object2& secObject);
private:
    Object2& SecObjectRef;
};

// Plik myObject1.cpp
myObject1::myObject1(int a, Object2& secObject):
SecObjectRef(secObject)
{
    SecObjectRef.methodOne(a);  //korzystamy z referencji
}

 

Comments

Nagłówki powinny być najczystsze jak to tylko możliwe. Skoro coś będzie używane tylko w jednym pliku źródłowym, to tam to zadeklaruj. 

Mimo to, unikałbym zmiennych statycznych - nie są thread safe

Jeżeli modyfikujemy obiekt.

---

Dodałbym, że statyczne/nie statyczne pole, zmienna globalna albo referencja w obiekcie dokładnie niczym się nie różnią pod względem thread safe. To zupełnie inna para kaloszy - niezależne od siebie czynniki.

Natomiast deklaracja, taka jak w pierwszym, poście powinna być jeszcze uzupełniona słowem static, o ile faktycznie mySecObject nie jest używany poza tym plikiem w którym jest zadeklarowany:

static Object2 mySecObject;