you are viewing a single comment's thread
view the rest of the comments
[–] 13 points 11 months ago (4 children)

I kind of get the idea that code should be self-documenting, but at the same time, there's so many crazy business rules that comments are basically a necessity if nothing else other than to explain why in the hell the crazed mess that provides the required functionality for the business rules exists.

  • source
  • parent
  • hideshow 4 child comments
  • [–] 12 points 11 months ago

    Yeah some comments are not useful

    # returns the value as a string
    return str(user.id)
    

    Some comments are

    # returns the user id as a string because ZenDesk's API throws errors if it gets a number.
    # See ticket RA-1037
    # See ZenDesk docs: https://etc/
    return str(user.id)
    
  • source
  • parent
  • [–] 6 points 11 months ago (1 child)

    That's typically what people who advocate for less/no comments really mean. The code should self explain "what" it does, but if the "why" isn't obvious (i.e. confusing business logic) nobody argues that you shouldn't comment it. That's how I've worked in every company I've been at (and all developers around me) from 50 person start ups to >2k people. It's really common mentality with Ruby developers

  • source
  • parent
  • hideshow 1 child comment