Ranter
Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
Comments
-
hjk10157312yTo be fair that is clearer than isset() or !empty() for non PHP people. I'm assuming $program is some sort of event program not the actual app.
-
omfg.. All in a single branch? Please tell me someone commented-in a chapter of a book there.
the fuck... Even whole Java files aren't that horrible! -
@hjk101 I think nobody complains about the conditional here. The problem is the huge "else" branch. Like, "return early" much?
-
I missed the line numbering too. Without reading the comments. Holy shit. I don't remember ever having one php file with more than 1000 lines. But that else { } alone has 1000 lines.
-
Imagine code review...
- Wait, you can't have the whole program just like that in one single file. You need to structure it.
- ok, hold my beer - [encloses whole prog in one if]
Related Rants
Santa has come early this year...
rant
phptsd
cmsofdoom
php