655
fuck you bill (lemy.lol)
submitted 4 months ago by to c/programmer_humor@programming.dev
93 comments fedilink hide all child comments
 
you are viewing a single comment's thread
view the rest of the comments
[–] 8 points 3 months ago (2 children)

Excellent comment and I fully agree with almost everything. Just one tiny nitpick:

For example if I have some sort of complex calculation, but it’s based on a well known algorithm, I might comment the name of that algorithm.

Unless you really need this complex calculation to be inline (e.g. for performance reasons) it would be better to move it into a function or method and include the algorithm in the name instead of adding a comment.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 4 child comments
    •
  • [–] 3 points 3 months ago

    i want it in a function

    fully include the pre implementation long hand in comments and the book / expert reference

    the number of times i’ve found longer algos not doing as advertised is scary

    improperly used statistical algos given the data sets and hand waving results are so common

    gee i wonder why testing shows no improvement

  • permalink
  • fedilink
  • source
  • parent
  • [–] 3 points 3 months ago

    Very good! Your spidey senses are working perfectly. Hey I want to comment this calculation, why don't I move it into a function so the name can explain what it does. Good call!

    Sometimes the algorithm is inlined for performance, sometimes it's a class with a bunch of functions that as a whole is primarily based on an algorithm, so comments might make sense in those cases. Most of the times it's a library, so the name of the library kinda gives it away and hopefully has good documentation as well.

  • permalink
  • fedilink
  • source
  • parent