|
|
|
|
|
|
by meritt
3119 days ago
|
|
|
The single biggest stereotype held against the PHP community is they are a bunch of shitty amateur programmers whose programming ability barely extends beyond copy-pasting stackoverflow. Very few of the language-centric criticisms are valid these days. Granted a lot of these criticisms are often levied by trendy kids reinventing the wheel with Ruby, MongoDB, NodeJS, Go, Docker, etc (2008-2017 roughly), but unfortunately they tend the tone for tech as a whole, so PHP still has this very negative reputation even though we tend to get more shit done, make more money, and spend less time learning the latest new fad. Adhering to PSRs goes a long way to combat that stereotype by having a consistent look and feel to open source packages. Attention to detail leads to code quality. Initiatives like phptherightway really help out the community by putting people on the right track from day #1. I agree there's a ton of utter shit out there on packagist and I avoid 99.9% of it like the plague for this very reason. |
|
|
I agree. But the PSR's approach things the wrong way IMO.
The style related PSR's should be exactly two rules:
1. Choose an existing/define a formatting style for the project.
2. Ensure the style is enforced, ideally using a code style formatting tool like phpcs, your IDE, etc
Thats it. Claiming that a project isnt up to scratch because it doesnt conform to an abritrary style that was created literally by what was most common in the member projects at the time, is frankly ridiculous.
Their subsequent decisions show a frankly laughable approach to development. Psr-whatever that defines a logging interface. Every man and his dog uses integer levels, usually the syslog 0-7 range.
This makes it extremely simple to ignore/filter out logs above a given priority.
What do they decide to do? Fuck it, make the defined log levels strings. Why? Because an existing project happened to already use integers that didnt adhere to syslog, and they didn't want to deal with it.
So fuck it, lets have a shittier, less sensible "standard" just so fucking monolog can keep using 400 to mean error or whatver it is.
Readability is good, standardisation is good. The latter should be a project level decision, and honestly ive never seen anyone who's picked a standard - any standard - that meant readability was a problem.