On Jan 30, 2007, at 7:48 PM, John Siracusa wrote:
> Given that this method will always be called with arguments, and
> that those
> arguments can change per-call, how can there be any expectation
> that the
> results will be saved internally? How would such a feature even work?
it wouldn't -- i can't imagine any circumstance in which it could.
> The whole point of the find method is to allow ad hoc queries.
> Think of it
> as a Manager call with the query parameters that link one object to
> another
> being implied, rather than explicitly visible in the args.
i'm in 100% agreement. but the manager class is that 'special'
place where all sorts of conditional stuff happens.
it just seems counter-intuitive to everything else in the base rose
class to me.
> If someone wants to keep the results of a find method call around
> for a
> while, they certainly have the means to do so by...keeping the results
> around for a while! :)
>
> # Keep @prices around until it's no longer needed
> @prices = $p->find_prices(...);
>
> If there exists a well-defined set of related objects that you want
> to be
> able to keep inside the parent object, just create a relationship
> with all
> the parameters that define this set, then use the usual
> get_set_on_save
> method.
i have no qualms with that.
i just think that the nature of this new function requires that the
user be made painfully aware of this departure from the standard
workings.
the consensus is clearly otherwise, but let me expand my suggestions to:
a- the name semantically notes that it doesn't alter the object
like other rose functions
b- the function require a flag that the user know whats going on
c- this be placed not in rose::db::object, but in some other
class which extends rose::db::object ( rose::db::objectalt )
i think my issue is that it functions so much like a manager call and
not like a rose object , yet the point of it is to have the
functionality of a manager call in the object itself.
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Rose-db-object mailing list
Rose-db-object@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/rose-db-object
