• @BravoVictor
    link
    English
    1210 months ago

    I mean… you gotta have them start the habit somehow

    • @[email protected]
      link
      fedilink
      2010 months ago

      Let’s face it, such comments usually cause more problems than do good. If someone changes the code and forgets to modify the comment, the reader might favor one or another at random. “Stop sign” example isn’t the best but you get my point.

      Comments at best should explain some non-obvious logic, or some sort of reasons for implementing one way or another. For SDKs and packages overall, public APIs should also be commented. The rest imo should be readable from code.

      • @[email protected]
        link
        fedilink
        1010 months ago

        A form of “self documentation” I like to do is create variables for conditions before using it in an if statement. If you break down a funky conditional into easy to read variables it becomes a lot more clear what it’s trying to do.

        Idk how to write code on sync:

        const isHumid = xxxx;
        const isHot = yyyy;
        const isSunny = zzzzz;
        
        If (isHot && isHumid && isSunny) {
            ...
        }
        
      • @coloredgrayscale
        link
        510 months ago

        If someone changes the code and forgets to modify the comment, the reader might favor one or another at random.

        Hence why you should comment why, not how/what.

        // slow down traffic before crossing busy main road

        Now you can change the stop sign to a yield without touching the comment. Or judge that the comment can be removed if it’s clear the main road does no longer exist.

    • @[email protected]
      link
      fedilink
      1110 months ago

      Bad/wrong documentation is worse than no documentation.

      “Practice makes perfect” is only true if you’re practicing the right stuff. Otherwise you’re just reinforcing bad habits.