[Tutor] Structure of my simulation / monte-carlo
Dave Angel
davea at ieee.org
Mon Nov 2 02:34:56 CET 2009
Kent Johnson wrote:
> On Sun, Nov 1, 2009 at 10:47 AM, Vincent Davis <vincent at vincentdavis.net> wrote:
>
>> Kent Johsnon writes
>> "This class has a lot of responsibilities:
>> - create applicants
>> - create institutions
>> - run a single match
>> - run multiple matches
>> - calculate statistics on the result of multiple matches
>> A principle of object-oriented design is that a class should have a
>> single responsibility. I would break your class up a bit using
>> multiple classes, perhaps."
>> I am trying to do what you recomend, as much as makes sense to me, but that
>> is why I ask the question so I should consider your answer.
>> This is what I hear you saying, (I don't mean to represent them as
>> sub-classes but more how they would operate on each other) Should I consider
>> making Institutions) a subclass of (Make Institutions)? I can't think of
>> anything that would make sense to inherit.
>> class Simulation:
>> class Create Institutions:
>> class Institutions:
>> class create Applicants:
>> class Applicants:
>> class Match:
>> class Multi Match:
>> I add I am thinking
>> class Simulation:
>> def__init__:(self, results, stats.....repeat..)
>> def create_applicants():
>> class Applicants
>> def creat_institutions():
>> class Institutions
>> def one_simulation(): # one or more
>> create_applicants()
>> create_institutions()
>> class Match()
>> class Results
>> def repeat_simulation()
>> repeat one_simulations
>> class Results
>> After writing this out I now think you are right, more classes.
>>
>
> Now you are getting too complicated. You don't need to use inheritance
> or nested classes, and you can use simple methods (not classes) to
> create applicants and institutions. You already have Applicant,
> Institution and Match classes that run a single match. Now make a
> RepeatMatch class that uses the Match class to run multiple
> simulations.
>
> Kent
>
>
I mostly agree with Kent, but I apparently disagree about which classes
are actually needed. Think what things will have actual instances that
will last long enough to be worth formally defining. So you need
Applicant, and Institution, and Simulation. Notice they're all
singular. I'm assuming one simulation is a single set of test data,
with its results. Then you create as many instances of Simulation as
you need for comparison purposes, and perhaps keep a list of them. It's
not clear that list needs any further structure than the built-in list
type provides.
You don't need a class for creating an Applicant, that's just a line or
two in a loop in the Simulation class. Similarly for Institution.
And if I understand it correctly, you don't need very many different
methods in Simulation either. You need the __init__ to save enough
information to "tag" this particular simulation (call it a label, it's
probably just a string). If __init__ is too complex, you may want to
break it into several phases. But they'll always be called in direct
succession, so there may not be any point. Then you need something that
triggers an analysis, and something that queries for particular
results. That last will then be called from plotting or charting routines.
But you probably don't need anything special for a collection of
Simulation objects. A list will probably be fine.
And from what you said earlier, you WILL need function objects, probably
as parameters to the Simulation constructor. So each instance of
Simulation will be given several function objects to specify the
distribution functions for the parameters to be used when creating
Applicants and Institutions.
DaveA
More information about the Tutor
mailing list