[Tutor] Sort of database & "family tree" question

[Alan Gauld]

"spir" <denis.spir at free.fr> wrote

>> > The current way of reading the data is this:
>> > 
>> > def get_info(person)
>> >    infoDic = {}
>> >    infoDic['first']      = parser.get(person, 'firstName')
>> >    infoDic['last']       = parser.get(person, 'lastName')
>> >    infoDic['father']   = parser.get(person, 'father')
>> >    infoDic['mother'] = parser.get(person, 'mother')
>> >    return infoDic
>> 
>> TYhis is almost the same but you are using a dict. 
>> A sligtly more readable version is to define a class Person:
>> 
>> class Person:
>>      def __init__(self, parser):
>>          self.first  = parser.get(person, 'firstName')
> 
> Maybe I don't get the point, but I find it strange to make Person 
> dependant not only of the data storage format, but also of the 
> parsing technique. 

Yes, its not normal and the class probably should be called 
ParsedPerson or somesuch. I ws trying to mimic the current 
code style as much as possible, that was the reason for the
decision. Normally I would agree with you and use:

> class Person:
>    def __init__(self, first, last, father, mother):
>        self.first  = first
>        self.last   = last
>        self.father   = father
>        self.mother = mother
> 

Alan G.