hi Lester, On Fri, Oct 12, 2012 at 10:22 AM, Lester Caine <les...@lsces.co.uk> wrote: > Clint Priest wrote: >> >> Alright, here is the updated RFC as per discussions for the last few days: >> >> https://wiki.php.net/rfc/propertygetsetsyntax-as-implemented >> >> If you could read it over, make sure I have all of your concerns correctly >> addressed and we can leave this open for the two week waiting period while I >> make the final code changes. > > > Now I remember why I did not like this wholesale ... > > There is no easy way to identify that > $time->Hours = 12; is doing something other than a bog standard 'store 12 in > $Hours' while a simple $time->Hours(12); immediately makes it obvious that > there IS some processing involved. And period = $time->Hours(); flags that > the result is calculated from some other value in the class. > > So IS this new layer of magic mirrors really necessary at all? Is it adding > to the complexity at the expense of the inherent readability of the code > later on? > > I suppose the argument is that 'people expect it' but that is only because > they are not yet used to the better way of working that IS PHP ... just > because some people expect cryptic code should that be any reason to weigh > PHP down further with it? >
Please keep the discussion sane without repeating the same arguments over and over again. This RFC is about adding a new cleaner getter/setter than the __get/__set we have now, which forces us to do some more useless work for basic needs. The complexity you are talking about already exist and you already mentioned this argument already, we got it. Cheers, -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
