Re: [Modeling-users] ModelMasons refactored

[Mario R. <ma...@ru...>]

Just a minor comment, w.r.t. naming of the generated files/classes.
I feel that:

a) "Base" is probably not the best name extension to use, as it has
a well-known generic meaning, and in addition it says nothing about
the "volatility" of these generated files/classes. Better names would
be something like "Auto" or "Gen" or "mdl" or "pom" or "morfy" ...

b) It is unnecessarily repeated -- what is the point of naming all the
classes in the "Base" sub-package also with "Base"? Wouldn't it be
more convenient to name only the sub-package? E.g.

mortal
|-- Objects
|   |-- Article.py
|   |-- Auto
|   |   |-- Article.py
|   |   |-- Folder.py
|   |   |-- RssFeed.py
|   |   |-- __init__.py
|   |-- Folder.py
|   |-- RssFeed.py
|   |-- __init__.py
|   |-- model_Mortal.py
|   |-- model_Mortal.xml
|   `-- setup.py
`-- __init__.py

which implies the import statements in the "working: classes:

==============================================
# Modeling Automatically-generated  Objects
import Auto

class Folder(Auto.Folder):
     ...
==============================================

Regards, mario



> On Sun, Apr 20, 2003 at 06:24:35PM +0200, Sebastien Bigaret wrote:

[snip]

> mortal
> |-- Objects
> |   |-- Article.py
> |   |-- Base
> |   |   |-- ArticleBase.py
> |   |   |-- FolderBase.py
> |   |   |-- FolderBase.pyc
> |   |   |-- RssFeedBase.py
> |   |   |-- __init__.py
> |   |   `-- __init__.pyc
> |   |-- Folder.py
> |   |-- Folder.pyc
> |   |-- RssFeed.py
> |   |-- __init__.py
> |   |-- model_Mortal.py
> |   |-- model_Mortal.xml
> |   `-- setup.py
> `-- __init__.py

[snip]

>
> 2) I look at the code i generated:
> ==============================================
> # Modeling Base Objects
> from Base.FolderBase import FolderBase
>
>
> class Folder(FolderBase):
>   """
>   Folders are objects ...
>   """
>
>   def __init__(self):
>     "Initializer"
>     FolderBase.__init__(self)
> ===============================================

[snip]