Hi guys, > Many of us don't consider it bad code. I've also always initialized > variables when it was required (and many times when it wasn't) even though > I wasn't forced to do so. A lot of other people do as well. If it's so > important to you, start a program to teach people how you think they should > code.
One problem that needs to be understood is that PHP is used by a lot of users. Because it's easy to pick up and you have fast feedback it probably have a higher percentage of juniors than average. How language was 10 years ago it didn't helped them much in learning and they did lot of mistakes. Some of those mistakes cost companies lots of money and, in time, people got to the conclusion that PHP is bad. This is a very important thing!, Yes you can write working great code in PHP but it's very easy to write working bad code as well. And it's not about you and me or the other persons chatting here, it's about the rest of the world. PHP improved on it's bad image in the later years but this needs to continue. IMO, one thing that we need to also do is to make the language image better. With this in mind, I believe the "undefined variable" error will be a step forward. It's not about if you don't consider it bad code and that apparently the majority consider it good. It's that if language wouldn't allow you to write that code it will benefit the language image and the rest of the PHP comunity. Also, I would also like to remind of this: https://github.com/php/php-src/blob/master/docs/mailinglist-rules.md I think some parts might have been violated multiple time in this thread. Regards, Alex On Thu, Sep 12, 2019 at 10:29 PM Chase Peeler <chasepee...@gmail.com> wrote: > On Thu, Sep 12, 2019 at 3:17 PM Olumide Samson <oludons...@gmail.com> > wrote: > > > On Thu, Sep 12, 2019 at 8:11 PM Michael Babker <michael.bab...@gmail.com > > > > wrote: > > > > > On Thu, Sep 12, 2019 at 2:06 PM Olumide Samson <oludons...@gmail.com> > > > wrote: > > > > > >> On Thu, Sep 12, 2019 at 8:00 PM Michael Babker < > > michael.bab...@gmail.com> > > >> wrote: > > >> > > >>> On Thu, Sep 12, 2019 at 1:51 PM Peter Kokot <peterko...@gmail.com> > > >>> wrote: > > >>> > > >>> > Just a dumb idea, since there clearly is a majority in favor of the > > >>> > change with these warnings and strictness and all that now... Why > not > > >>> > making something like an LTS PHP 7.x where all the legacy code > would > > >>> > work OK as long as practically possible and 8.x+ would be the > future > > >>> > of what the developers want and not what business wants? One who > > won't > > >>> > upgrade due to the BC breaks also won't need the new features > anyway > > >>> > very realistically. > > >>> > > > >>> > > >>> Please don't tie the notion of LTS with the idea that a new major can > > >>> break > > >>> BC at will or create larger scale breaks because the previous major > has > > >>> extended support. Sooner or later that will end up back at the ++ > idea > > >>> and > > >>> fragmentation encouraged by the language is a bad idea. > > >>> > > >> > > >> Not sure you are really seeing the goal... > > >> > > >> Why is LTS not a good idea? > > >> > > > > > > I'm not saying LTS is a bad idea. I'm saying using LTS to justify > > > shipping larger scale BC breaks, such as the changes suggested in the > > last > > > couple of "contentious" RFCs in a major version because "hey, we have a > > LTS > > > version you can use that until you're ready to deal with the backlog of > > BC > > > breaks created" is a bad idea. > > > > > > > > > > For the record, I happen to agree with as these RFCs would have minimal > > > impact on my day-to-day work, but having also been in the role of a > > > maintainer of open source libraries and applications I also grasp why > > these > > > types of changes can be problematic to the ecosystem (both end users of > > > those libraries and applications and the maintainers of them) and > > wouldn't > > > jump the gun to ship them without careful consideration. > > > > > > > Most of these changes wouldn't have been problematic to you if the > language > > has prevented you from writing what we can now consider bad code, so > please > > allow the new PHP developer that newly start using PHP to not follow that > > your path that will/might hunt him later in the future... > > > > > Many of us don't consider it bad code. I've also always initialized > variables when it was required (and many times when it wasn't) even though > I wasn't forced to do so. A lot of other people do as well. If it's so > important to you, start a program to teach people how you think they should > code. > > > > There a notices, warning and errors to inform you that this shouldn't > have > > been the use case of this feature and you chose to ignore it and now, we > > are simplifying things and making those your errors teach you how to > write > > proper codes in the future, you're objecting.. > > > As has been discussed before, notices are not the same as warnings and > errors. Also, if those things are so wonderful, why can't you use them to > catch the issues you are complaining you can't catch right now? Again, you > are telling me there is something out there which will allow you to force > yourself to write "good code" without forcing me to follow the same > restrictions. Yet, you still feel it's necessary to not use those tools, > and instead modify the entire language so that I am forced to follow what > you deem best practices, even if I don't? > > > > > Why not just stay in PHP 7.x? > > > > Because other features that I want to utilize will also be added in PHP > 8. > > Or were you implying you want hitch-free, no-modification upgrade to PHP 8 > > from PHP 7.0? > > > > I never said that. Here we go again with the "I guess you are against all > BC breaks" nonsense. If BC breaks are required to add new functionality, > or, have a very minimal negative impact, then I don't have a problem with > them. This is not one of those cases. It changes a fundamental aspect of > the language, an aspect that many people actually like, and it doesn't add > any new features to the language, nor is it needed to add any new features > to the language. > > > > If yes, follow the best practices and not suppress error notices. > > > > Just My Opinion > > > > > -- > Chase Peeler > chasepee...@gmail.com >
