On Tue, 23 Aug 2011 17:03:12 +0200, Jacob Carlborg <d...@me.com> wrote:
On 2011-08-23 16:38, Andrei Alexandrescu wrote:
On 8/23/11 12:55 AM, Jacob Carlborg wrote:
On 2011-08-23 08:52, Andrei Alexandrescu wrote:
On 8/22/11 11:30 PM, Jacob Carlborg wrote:
Ok, then I just change "register" to a static method.
A static method of whom?
Andrei
Well, "register" currently an instance method of Serializer so I would
change it to be a static method of Serializer.
I think the ability of a class to be serialized would be independent of
the notion of a serializer. To me, "this class is serializable" really
means "this class has metadata associated with it that allows interested
parties to serialize it". But perhaps this is splitting hairs.
Andrei
You don't want to have it in the class and don't want it in the
serializer. I mean, it needs to be stored somewhere and I thought that a
static method in Serializer would better than a completely global
function.
Are you thinking about having another serialization library that can use
this information as well? I'm not sure if that's good idea, different
serialization implementations might need to do very different things
with the information.
I think the idea is to register something that provides the needed
information at runtime and separate that from the serializer because it's
useful on it's own.
Could be as simple as:
interface Serializable {
string[] allMembers();
Member getMember(string name);
}
interface Member {
final void set(T)(T t) {
setRaw(typeid(t), cast(void*)&t);
}
void setRaw(TypeInfo ti, void* p);
final T get(T)() {
return *cast(T*)enforce(getRaw(typeid(T)));
}
void* getRaw();
}
It's clearly interesting to make this available if you implement something
along that line.
But I would not overstress it for there are dozen of different
requirements profiles (performance, versioning, pre/post-serialize hooks).
So even if I am interested to use something similar for undo/redo or delta
creation, I'm not sure one can find a one-for-all solution here
(i.e. the above approach seems way to dynamic for my needs).
Maybe Variant could help in setting/getting values.
martin
