antonpug

Seems to be broken. Keeps spinning.

How often do you actually read through the code, trying to understand what's happening and you need english language? Once you read an english description - do you still often have to look at the code to understand what is really happening? //complete purchase process - what does that mean? When you look at the method, I see exactly what's being called and with what parameters.

Documenting business logic works much better via Spec files - if you have BDD test files Given/When/Then format, that'll go a long way in describing functionality, in my opinion.

There must be an insane amount of cognitive overload when having to parse through 2x the number of lines because every line of code is accompanied by a line of comment.

Some commenting is okay and necessary. The point is that generally, if you are adding heavy commenting, you can probably refactor the code in such a way that commenting is not necessary. Even in a large codebase.

It's not hard to understand what this.completePurchaseProcess(paymentInfo) does.

Sarcasm? Or am I misunderstanding your comment? :) You annotate "every line"?

Could you be a little bit more constructive? It's okay if you disagree - but I'd be curious to hear your side.