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

Question

[Dyikot]

1.

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

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

2.

struct SerializationData {};

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

	virtual void Serialize(const std::filesystem::path& path,
						   const SerializationData* data) const = 0;	
	virtual std::unique_ptr<SerializationData> Deserialize(
		const std::filesystem::path& path) const = 0;
};

Comments

[Adamos]

std::any без причины - признак говнокода.
Ничего страшного в том, что в метод передадут нулевой указатель, еще нет.

[maaGames]

Почти на 100% уверен, что сериализация в "путь к файла" - плохая архитектура. Открывать/закрывать файл для каждого объекта? Уверен, туда надо stream использовать.

[GavriKos]

А это интерфейс для чего? Если для сериализатора - то тогда название странное. Если для данных - то почему данные передаются в метод?

[Dyikot]

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

[maaGames]

Dyikot, не нужен. Ты собираешься открывать/закрывать файл для чтения/записи каждого объекта. А что делать с контейнерами собрался? А если дочерний класс вызывает функцию сериализации родительского класса? А если ты хочешь не в файл сохранить, а в строку(stringstream)? А как ты собираешься десериализировать два объекта, если в функцию не передаётся смещение, относительно которого объект записан, а открыть поток ты сможешь или в начале или в конце файла, раз смещения нет никакого?

[Dyikot]

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

[maaGames]

Dyikot, Хорошо, не два, а контейнер из полутора тысяч объектов. Что мешает передать в функцию полторы тысячи разнотипных аргументов? У тебя в интерфейсе всего один передаётся.
И, как уже спросил GavriKos, с какой целью ты передаёшь данные объекта в функцию сериализации этого же объекта? Или сериализаторы будут внешними объектами? Чаще всего сериализуемый класс наследуется от такого интерфейса и объект сам себя сериализует. А зачем классу самому в себя передавать указатель на себя?

[Dyikot]

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

[Dyikot]

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

[maaGames]

Dyikot, Не, ты пока не понял про что говорю. Давай сериализовывать фрукты. У тебя есть классы Апельсин и Банан. Ты будешь их наследовать от ISerializator? Или у тебя в каждом из этих классов будет объект Данные, унаследованные от ISerializator? Вот у тебя есть массив из фруктов, как ты будешь этот массив сериализовывать? В цикле для каждого вызывать метод сериализации и на каждой итерации будет файл открывать, записыаться и закрываться. А как ты потом этот массив обратно прочитаешь, если на каждой итерации надо файл не просто открывать, но и в определённое смещение смещаться?

[Dyikot]

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]

Dyikot, Понятно, то есть у тебя не сам объект сериализируется, а есть отдельные внешние объекты.
Тогда остаётся только вопрос, как ты будешь делать десериализацию, если в файл сохранено более одного объекта? Вот у тебя есть путь к файлу. Ты можешь открыть файл либо в самом начале, либо в самом конце. Чтобы открыть и перейти на произвольное место в файле, тебе надо знать это произвольное смещение.

[Dyikot]

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