Ad
  • Custom User Avatar

    I was going through the solutions and I noticed that this comment feature wasn't being used much, yet I have a question that I was hoping some warriors could answer.

    This solution hardly got any best practices votes, but it does an excellent job of expaining itself. When i look at the most clever solution, I do indeed see an elegant solution but it took me a good stare before I figured it out. This solution, on the other hand, seemed straightforward with very few complicated tricks and lots of comments to walk you through the logic. I asked myself which solution would I want in my code base on a project I was tasked to maintain. You might say that this solution was too bloated with comments and it could have been done in less lines, but it is also not so convoluted and comes with helpful documentation.

    So my qualm over this answer's poor reception is that in some respects it does seem like best practice. Tell me, do others think that it is best practice to have your solution in a couple of lines with few comments or is it better to have some documentation? Thanks and sorry for the lengthy reply.