Singleton undefined reference

Częśc. Czy mógłby ktoś sprawdzić ten kawałek kodu? Niestety nie mogę znależć rozwiązania a porównując z tymi co znalazłem w necie wydaje się że powinno być dobrze
Błąd to:
undefined reference to `JsonFonts::instance' JsonFonts.h line 32

/*
 * JsonFonts.h
 *
 *  Created on: 16 lut 2021
 *      Author: danie
 */
#include <vector>
#include <cardSD.h>
#include <ArduinoJson-v6.15.2.h>

#ifndef SRC_FILEMANAGER_JSONFONTS_H_
#define SRC_FILEMANAGER_JSONFONTS_H_

class JsonFonts {
private:
	JsonFonts();
	virtual ~JsonFonts();
	static JsonFonts* instance;
	struct FontsSettings{
		std::string font;
		uint8_t height;
		uint8_t width;
		string path;
	};
	FontsSettings actualSettings;
	vector<FontsSettings> fontsSettings;
public:
	static JsonFonts* getInstance(){
		if(instance == nullptr)
			instance = new JsonFonts();
		return instance;
	}
	void findFontToCreate(uint8_t width, uint8_t height);
	string getPath(uint8_t width, uint8_t height);
	uint8_t getWidth(uint8_t width, uint8_t height);
	uint8_t getHeight(uint8_t width, uint8_t height);
};

#endif /* SRC_FILEMANAGER_JSONFONTS_H_ */

Za mocno zakręciłeś.

class JsonFonts
{
	private:
    JsonFonts();
	public:
    virtual ~JsonFonts();
    static JsonFonts &instance()
	{
		static JsonFonts jf;
        return jf;
    }
};

Nie jest to pełnoprawny Singelton, domyślny konstruktor kopiujący i operator przypisania pozwala na jego kopiowanie.
Zamień na:

#include <iostream>

using namespace std;

class JsonFonts {
private:
    JsonFonts() = default;
public:
    JsonFonts(JsonFonts const&) = delete;
    void operator=(JsonFonts const&) = delete;
    virtual ~JsonFonts(){};

    static JsonFonts& getInstance(){
        static JsonFonts instance;
        return instance;
    }
    int i {10};
};

int main()
{
  auto& ref1 {JsonFonts::getInstance()};
  auto& ref2 {JsonFonts::getInstance()};
  ref1.i = 50;
  cout << ref2.i << endl;
}

W sumie to ja wole jeszcze prostsze podejście:

#include <iostream>
using namespace std;

class JsonFontsSingleton
{
	private:
	struct JsonFonts
	{
		void call() { cout<<"call"<<endl; }
		void *check() { return this; }
	};
	public:
	JsonFonts *operator->()
	{
		static JsonFonts jf;
		return &jf;
	}
};

int main()
{
	JsonFontsSingleton()->call();
	if(JsonFontsSingleton()->check()==JsonFontsSingleton()->check()) cout<<"same object"<<endl;
	return 0;
}

Dzięki wszystkim za przedstawienie różnych opcji. Po pracy dokładnie się im przyjrzę i mam nadzieję, że wszystko zrozumiem :D

Offtopic: Nazwa JsonFonts jest strasznie dziwna, bo Json jest związany z komunikacją sieciową, a Fonts z interface użytkownika. Coś jak "wiosło na pustyni".

Sercem problemu jest, że koledze nie wychodzi linkowanie do statycznej składowej klasy, wcześniej czy później będzie musiał się nauczyć.

@danielbr3

Brakuje ci nie w headerze, a w źródle CPP (tylko w jednym, najlepiej stosownym dla klasy JsonFonts.cpp)

JsonFonts::JsonFonts* instance;

Zgadzam się z @MarekR22 - fatalna nazwa. Niestety z Arduino wychodzą w znacznym procencie koszmarki architektonicze

Mam jeszcze jedno pytanie. Sprawdziłem co zostało zastosowane w drugim przedstawionym rozwiązaniu i je napisałem. Chciałbym tylko jeszcze użyć metody ładującej plik Json w konstruktorze i niestety w żaden sposób nie udaje mi się tego zrobić w tym konstruktorze zdefiniowanym jako domyślny. Jest w ogóle taka możliwość?

danielbr3 napisał(a):

Mam jeszcze jedno pytanie. Sprawdziłem co zostało zastosowane w drugim przedstawionym rozwiązaniu i je napisałem. Chciałbym tylko jeszcze użyć metody ładującej plik Json w konstruktorze i niestety w żaden sposób nie udaje mi się tego zrobić w tym konstruktorze zdefiniowanym jako domyślny. Jest w ogóle taka możliwość?

Domyślny to domyślny deklarowany automaigcznie.
Chcesz sam napisać treść to zabierz słówko default

Weź użyj wersję uproszczoną, masz zwykła strukturę ale ze wszelkimi zabezpieczeniami.

@_13th_Dragon: Poczytałem wcześniej o tych domyślnych konstruktorach kopiujących i znalazłem że to default musi się znaleźć przy jakimś konstruktorze, żeby delete przy pozostałych pełniło swoją funkcję. Za chwile sprawdzę tą trzecią wersje ze strukturą

Dzięki wszystkim za pomoc. Całość działa tak jak powinna