There were similar discussions in the PHP community, with arguments that were sensible and rather stupid on both sides of the argument. I'm honestly a bit torn

There were similar discussions in the PHP community, with arguments that were sensible and rather stupid on both sides of the argument. I'm honestly a bit torn by it. Yes, your code absolutely should be written in a way that keeps it tidy and easy to understand. But you've always got complex situations which arent always going to be easy or obvious for someone fresh to the codebase. There should be a good middle ground. I dont need to see comments saying "this is a loop that gets all the users". If its got a variables called users, calling a methog called "getUsers()" then thats pretty damn obvious whats happening. However if you then go on to do something weird like loop over each user and calculate a score based on the number of posts they've made then theres undoubtedly going to be some logic in there that even just a simple one sentence comment will help someone understand. It's a fine line, and getting it right is a skill in itself.