Implementacja Singletona w c++

pytanie zadane 15 grudnia 2018 w C i C++ przez dawid2002 Mądrala (5,190 p.)
zamknięte 15 grudnia 2018 przez dawid2002

Witam, mam ostatnio taki problem, że chciałem stworzyć klasę, która miała być Singletonem, ale kod mi niestety nie działa!

#include <iostream>

using namespace std;

class C_Singleton
{

    static C_Singleton *instancja;

    C_Singleton() {}
    C_Singleton(const C_Singleton &A);

public:

    string name;

    static C_Singleton& get_instance()
    {

        if(instancja == NULL)
            instancja = new C_Singleton();

        return *instancja;
    }

    ~C_Singleton()
    {
        delete this;
    }
};

int main()
{
   C_Singleton s1 = C_Singleton::get_instance();

   s1.name = "ABC";

   return 0;
}

Nie wiem co jest tutaj nie tak i nie rozumiem czemu kompilator pokazuje błąd!

Jeśli ktoś może mi pomóc z implementacją tego kodu to bardzo dziękuje!

komentarz zamknięcia: znam już odpowiedź na moje pytanie

3 odpowiedzi

odpowiedź 15 grudnia 2018 przez monika90 Pasjonat (22,940 p.)
wybrane 15 grudnia 2018 przez dawid2002

Najlepsza

Po pierwsze, nie powinno się używać singletonów.

s1 powinno być referencją, czyli tak:

 C_Singleton& s1 = C_Singleton::get_instance();

Brakuje też definicji składowej instancja, dodaj to po definicji klasy:

C_Singleton *C_Singleton::instancja;

A destruktor jest nonsensowny z powodu nieskończonej rekursji.

I oczywiście, jak zwykle, brakuje #include <string>

komentarz 15 grudnia 2018 przez Ehlert Ekspert (212,670 p.)

Z tego co kojarzę to delete this może mieć sens ale łatwo o undefined behaviour.

komentarz 15 grudnia 2018 przez monika90 Pasjonat (22,940 p.)

delete this w destruktorze nie ma sensu , bo samo wywołuje destruktor.

komentarz 15 grudnia 2018 przez dawid2002 Mądrala (5,190 p.)

@monika90,

Dzięki za pomoc. Aczkolwiek nie rozumiem czemu trzeba wstawiać ponownie definicje tego wskaźnika "instancja" skoro już ona jest w linijce 8 ?

komentarz 15 grudnia 2018 przez monika90 Pasjonat (22,940 p.)

W linii 8 masz tylko deklarację, statyczne dane składowe wymagają też definicji poza klasą w pliku *.cpp, nie w *.h (choć nie zawsze, ale zostawmy to na razie).

komentarz 15 grudnia 2018 przez dawid2002 Mądrala (5,190 p.)

dzięki za pomoc!

odpowiedź 15 grudnia 2018 przez jankustosz1 Nałogowiec (35,880 p.)

This nie jest tworzone dynamicznie wiec nie możesz robić na nim delete. Albo możesz juz sam nie wiem.

instancja lepiej jakby byla typem shared_ptr

Nie możesz przypisać do s1 bo nie stworzyłeś odpowiedniego operatora, ani konstruktora

komentarz 15 grudnia 2018 przez dawid2002 Mądrala (5,190 p.)

słusznie! dzięki , będę miał to na uwadze.

odpowiedź 15 grudnia 2018 przez mokrowski Mędrzec (155,460 p.)

Od C++11 wystarczy zwykły static by mieć atomową inicjalizację:

https://godbolt.org/z/z9c_-c

Poza tym, przychylam się. Singleton to nieodpowiedni wzorzec.