Login | Register
My pages Projects Community openCollabNet

Discussions > dev > Re: [propel-dev] roadmap

propel
Discussion topic

Back to topic list

Re: [propel-dev] roadmap

Reply

Author Alan Pinstein <apinstein at mac dot com>
Full name Alan Pinstein <apinstein at mac dot com>
Date 2006-03-28 12:29:31 PST
Message I have a few things that need to be considered before using __call/
__get/__set

1) For those of us that want API documentation for our codebase, I
don't think we can get it unless the functions are actually there.
2) How do __call/__get/__set work with subclasses? How does
overriding work?
3) If this behavior is added, shouldn't it be TOGGLE-ABLE?
4) Why do you want to do this? Before we go do this level of
changing, where are the performance results that show these changes
would make more than a negligible improvement? For instance, most of
the getters are one-liners; and most of the setters are 3-liners.

I am not sure that the overhead required by __call etc would be LESS
than the direct function call itself... don't forget that call has to
check to see if the function exists anyway, then you have to have a
bunch of code in the __call routine to "do the right thing".

So let's see some evidence that a suggested change will actually
improve performance before making a change that causes the code to be
less readable, less documented, and of unknown performance benefit!

5) Maybe there is some level of abstraction that could be used to
reduce the code?

For instance, the dts-style functions have a lot of code that could
probably be factored out into the base class to reduce these
generated functions from 20 lines of code to a single line. Same is
true of the related object fetching, setters, etc.

Alan


On Mar 28, 2006, at 10:28 AM, Hans Lellelid wrote:

>> * Utilizing dynamic __call/__get/__set in BaseObject/BasePeer classes
>> and kill much generated code from the Object/Peer classes for
>> get*/set*/add*
>>
>> If noone dislikes this I'll add two dev discussion pages to the wiki.
>>
>
> Adding the pages sounds good to me. The only concern I have (which I
> can mention in comments on wiki too) is that certain methods (temporal
> methods) take additional parameters -- which might present a problem
> __call/__get/__set, unless we just standardize on an API that does not
> support these. Just something to discuss.

« Previous message in topic | 9 of 14 | Next message in topic »

Messages

Show all messages in topic

RE: [propel-dev] roadmap Soenke Ruempler <ruempler at topconcepts dot com> Soenke Ruempler <ruempler at topconcepts dot com> 2006-03-28 07:22:41 PST
     Re: [propel-dev] roadmap hlellelid Hans Lellelid 2006-03-28 07:28:05 PST
         Re: [propel-dev] roadmap =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> 2006-03-28 08:15:03 PST
             Re: [propel-dev] roadmap =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> 2006-03-28 08:18:18 PST
                 Re: [propel-dev] roadmap hlellelid Hans Lellelid 2006-03-28 09:24:14 PST
                     Re: [propel-dev] roadmap =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> 2006-03-28 09:35:18 PST
                         Re: [propel-dev] roadmap hlellelid Hans Lellelid 2006-03-28 09:39:47 PST
             Re: [propel-dev] roadmap hlellelid Hans Lellelid 2006-03-28 09:25:10 PST
         Re: [propel-dev] roadmap Alan Pinstein <apinstein at mac dot com> Alan Pinstein <apinstein at mac dot com> 2006-03-28 12:29:31 PST
             Re: [propel-dev] roadmap hlellelid Hans Lellelid 2006-03-28 12:43:57 PST
                 Re: [propel-dev] roadmap =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> 2006-03-28 16:07:17 PST
             Re: [propel-dev] roadmap Oliver Schonrock <oliver dot schonrock at realtsp dot com> Oliver Schonrock <oliver dot schonrock at realtsp dot com> 2006-03-28 15:09:45 PST
                 Re: [propel-dev] roadmap Cameron Brunner <cameron dot brunner at gmail dot com> Cameron Brunner <cameron dot brunner at gmail dot com> 2006-03-28 15:47:22 PST
             Re: [propel-dev] roadmap =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> =?ISO-8859-1?Q?David_Z=FClke?= <dz at bitxtender dot com> 2006-03-28 16:03:39 PST
Messages per page: