Какой вариант на ваш взгляд лучше для интерфейса?

Евгений Шатунов, maaGames, Я немного изменил. Надеюсь стало чуть лучше. Или все еще не очень? Можно еще добавить варианты передачи массива(наверно std::span) IStringConvertable объектов с дополнительным параметром в качестве разделителя.

ISerializer

class IStringConvertable
{
public:
	virtual ~IStringConvertable() = default;

	virtual std::string Convert() const = 0;
	virtual void ConvertBack(const std::string& data) = 0;
};

class ISerializer
{
public:
	virtual ~ISerializer() = default;

	virtual void Serialize(const std::filesystem::path& path,
						   const IStringConvertable& data) const = 0;
	virtual void Deserialize(const std::filesystem::path& path,
							 IStringConvertable& data) const = 0;
};

class Serializer: public ISerializer
{
public:
	void Serialize(const std::filesystem::path& path, 
				   const IStringConvertable& data) const override;
	void Deserialize(const std::filesystem::path& path, 
					 IStringConvertable& data) const override;

	template<typename T> requires std::derived_from<T, IStringConvertable>
	T Deserialize(const std::filesystem::path& path) const
	{
		T data = {};
		Deserialize(path, data);
		return data;
	}
};

void Serializer::Serialize(const std::filesystem::path& path,
						   const IStringConvertable& data) const
{
	if(std::ofstream stream(path); stream.is_open())
	{
		stream << data.Convert();
	}
}

void Serializer::Deserialize(const std::filesystem::path& path,
							 IStringConvertable& data) const
{
	if(std::ifstream stream(path); stream.is_open())
	{
		std::string buffer;
		stream >> buffer;
		data.ConvertBack(buffer);
	}
}

UPD: Добавил еще

IRangeSerializer

class IRangeSerializer
{
public:
	virtual ~IRangeSerializer() = default;

	virtual void Serialize(const std::filesystem::path& path,
						   const std::span<IStringConvertable*> range,
						   const std::string& delimetr) const = 0;
	virtual void Deserialize(const std::filesystem::path& path,
							 std::span<IStringConvertable*> range,
							 const std::string& delimetr) const = 0;
};

Какой вариант на ваш взгляд лучше для интерфейса?

maaGames, буду решать проблемы по мере их поступления

Какой вариант на ваш взгляд лучше для интерфейса?

maaGames, Вот пример из сделанного проекта:

struct SettingsData: SerializationData
{
	size_t WindowWidth = 400;
	size_t WindowHeight = 800;
	size_t ActiveResolutionIndex = 1;
	size_t SoundEffectVolume = 100;
	size_t MusicVolume = 100;

	std::string ToString() const;

	void Convert(const std::vector<std::string>& values);
};

class SettinsDataSerializer: public ISerializable
{
public:
	static constexpr char Delimetr = '\n';
public:
	std::filesystem::path GetDefaultFilePath() const;

	std::filesystem::path GetFileExtension() const;

	void Serialize(const std::filesystem::path& path, 
				   const SerializationData* data) const override;

	void Serialize(const SerializationData* data) const;

	std::unique_ptr<SerializationData> Deserialize(
		const std::filesystem::path& path) const override;

	std::unique_ptr<SerializationData> Deserialize() const;
private:
	bool IsPathRelevant(const std::filesystem::path& path) const;
};

Какой вариант на ваш взгляд лучше для интерфейса?

maaGames, я исправил название класса на сериализатор. Ну можно наследоватся от SerializationData и сделать там необьходимые поля. Если это контейнер то с const &.

Какой вариант на ваш взгляд лучше для интерфейса?

GavriKos, поправил название на "ISerializator".

Какой вариант на ваш взгляд лучше для интерфейса?

maaGames, что мешает передать сразу два объекта и там уже как надо записать?

Какой вариант на ваш взгляд лучше для интерфейса?

maaGames, для stream нужен path. Вот я и передаю.

Данные в таком случае будут хранится в стеке?

почему это то нельзя?

Данные в таком случае будут хранится в стеке?

Василий Банников, почему

Насколько больше будет занимать памяти Свойство (get + set)?

Василий Банников, Спасибо за то что написал слово сниппет... Раньше я не знал что это именно так называется. Теперь сделал сниппет для создания сигнатур сеттера и геттера.

Насколько больше будет занимать памяти Свойство (get + set)?

Василий Банников, Ну чтобы можно было заменить реализацию по умолчанию. Если делать как методы то надо делать их виртуальными и лишь в наследовании переопределять.

Насколько больше будет занимать памяти Свойство (get + set)?

res2001, Ну или например с помощью среды разработки. По типу rfor. Чтобы самостоятельно сигнатуры не писать, а лишь в теле нужное добавить.

Насколько больше будет занимать памяти Свойство (get + set)?

res2001, Почему не сделали(можно ли сделать) чтобы компилятор при каком нибудь ключевой слове бы сам генерировал эти методы?

Баг при подсчете времени? Или как это объяснить?

Adamos, Замена переменной каким образом должно помочь? Зачем выводить time_point?
так?

auto start = high_resolution_clock::now();
auto lastStart = high_resolution_clock::now();
auto time = 0us;
do
{
	std::this_thread::yield();
	lastStart = high_resolution_clock::now();
	time += duration_cast<microseconds>(lastStart - start);
	start = lastStart;
	//std::cout << duration_cast<milliseconds>(time) << std::endl;
}
while(time < Duration && !_isPaused);

Баг при подсчете времени? Или как это объяснить?

Adamos, Ну а как тогда посчитать duration, если не переписывать start? Про Elapsed ничего не понял

Нормально ли создавать пустую вирутуальную функцию?

pfemidi, класс активной фигуры тогда будет абстрактный.

Почему тут требуется cast double -> int?

Без каста в первом примере ошибка Error C2397 conversion from 'double' to 'int' requires a narrowing

Почему в истории начал появляться коммит слияния?

Сергей Кузнецов, Вернутся к любой точке это Create brunch from commit или Reset to commit?

Почему в истории начал появляться коммит слияния?

MVV, она у меня идентична с main. Она мне нужна если что-то пойдет не так вернутся на main.

Почему в истории начал появляться коммит слияния?

MVV, ну если удалю update и сделаю новую на основе main. Не поможет чтобы merge commit не было?