Don't Write Comments | Prime Reacts

Поделиться
HTML-код
  • Опубликовано: 23 май 2023
  • Recorded live on twitch, GET IN
    / theprimeagen
    Original: • Don't Write Comments
    Author: / @codeaesthetic
    MY MAIN YT CHANNEL: Has well edited engineering videos
    / theprimeagen
    Discord
    / discord
    Have something for me to read or react to?: / theprimeagenreact
  • НаукаНаука

Комментарии • 775

  • @farqueueman
    @farqueueman Год назад +960

    I go a step further than not writing comments, I run it through an obfuscator before committing to the repository ♥

    • @cmelgarejo
      @cmelgarejo Год назад +51

      Username checks out

    • @farqueueman
      @farqueueman Год назад +71

      @@cmelgarejo how dare you!

    • @QuantumImperfections
      @QuantumImperfections Год назад +19

      It's not often I actually laugh when I "lol" but this was one of those times. Thanks o7

    • @MrLowbob
      @MrLowbob Год назад +16

      minifying is decent too. who doesnt like variables a..z, aa..zz, etc

    • @ShrirajHegde
      @ShrirajHegde Год назад +59

      I just gitignore the source files and track binaries because at the end of the day, that's what matters most

  • @XxDarkCinisterxX
    @XxDarkCinisterxX Год назад +744

    No wonder Tom didn't add comment support to JDSL. It's self explanatory. Tom is a genius.

    • @whig01
      @whig01 Год назад +26

      This is another reason he should consider using TOML. No comment support.

    • @MrHords
      @MrHords Год назад +8

      Fuckin lol 😂 Tom is a genius

    • @uchennaofoma4624
      @uchennaofoma4624 Год назад +5

      Of course, Tom's a genius 😂😂😂

    • @me-low-key
      @me-low-key Год назад

      Tom is a genius

    • @user-gt2th3wz9c
      @user-gt2th3wz9c Год назад +5

      @@whig01 TOML is TOM's language?

  • @thewizardsofthezoo5376
    @thewizardsofthezoo5376 Год назад +437

    I blame Scott for not explaining that comments weren't implemented in JDSL.

    • @ThePrimeTimeagen
      @ThePrimeTimeagen  Год назад +112

      Scott, the true villain in all of this...

    • @fulconandroadcone9488
      @fulconandroadcone9488 Год назад +18

      @@ThePrimeTimeagen He was a visionary. A world where you don't need comments to navigate code. He invented JDSL to accomplish this goal, a true Genius.

    • @Eric-vh4qg
      @Eric-vh4qg Год назад +9

      I agree that senior developers definitely should have made him aware of such a huge vulnerability. I also wonder how something like this was pushed to production. I thought the json metadata was calling for specific revisions of these methods, so you can always maintain backwards compatible dependencies; his changes shouldn't have effected production unless he updated all the metadata to point to his new revisions as well. If he changed so much stuff, why wasn't any of it reviewed? Seems like this company had really bad organizational issues, which allowed for a junior developer to make sweeping changes without oversight.

    • @nachiketagrawal5154
      @nachiketagrawal5154 Год назад +14

      He should have written a comment in JDSL, explaining that comments are not supported
      But unfortunately he could not, since comments weren't supported

    • @TheNewton
      @TheNewton Год назад +7

      He didn't have to implement them, it's a first class feature in the format he selected.
      Because he's a genius.
      { "comment": "this is a JSON comment ~Genius Scott" }
      The noob that didn't understand JDSL put the comments in the javascript.

  • @tordjarv3802
    @tordjarv3802 Год назад +140

    The best comment I have ever seen in a code were:
    ! fiddle with variables and print stuff
    It was in a very old Fortran code for doing configuration interaction calculations for atomic nuclei.

    • @ThePrimeTimeagen
      @ThePrimeTimeagen  Год назад +28

      respect

    • @anmoon
      @anmoon Год назад +7

      ​@@ThePrimeTimeagen yes, it didn't blow up, respect. bet it was JDSL 😂

    • @jayshartzer844
      @jayshartzer844 Год назад +9

      A comment that will remain accurate until the heat death of the universe
      10/10 would compile again

  • @notuxnobux
    @notuxnobux Год назад +86

    Linus torvalds said it well: Dont describe your code, describe why you are doing something (if it's not obvious).
    It's because otherwise the comment does the same thing as the code, so your code should be readable so you dont need comments. But sometimes you need to write code in a specific way for uncontrollable reasons (external issues such as driver issues, some external service behaving in a weird way, some library requiring something that is not obvious, etc).

    • @mattmurphy7030
      @mattmurphy7030 Год назад +16

      My favorite is //hackity hack because it removes all doubt that what you’re seeing is bizarre

    • @geekzombie8795
      @geekzombie8795 6 месяцев назад

      @@mattmurphy7030Lel true

    • @doughxDude87
      @doughxDude87 5 месяцев назад +3

      Sometimes the code is just complicated and needs a comment saying what it does

    • @nerdycatgamer
      @nerdycatgamer 5 месяцев назад +1

      @@doughxDude87 No. Even if you're doing the most complex computations imaginable, you can explain WHAT the code is doing 2000x better with the actual code itself, rather than some comment saying it in English (just like everything said in the video). Everything said in the video applies to any level of complexity.

    • @doughxDude87
      @doughxDude87 5 месяцев назад

      @@nerdycatgamer sure there probably is a way to structure the solution to be better. But is it really the worth the time investment to refactor something that can be easily explained with a comment? All depends on the problem and time resources

  • @TheTrienco
    @TheTrienco Год назад +169

    With the qualifier "most of the time", I absolutely agree. Just yesterday I was cursing my heart out, because code was changed to do the complete opposite of what the comments said. Nobody maintains them, half the time people don't even read them and it takes on average 2 weeks before "redundantly repeating what the code is telling me already" turns into "blatantly lying to my face".

    • @herrpez
      @herrpez Год назад +39

      Why don't code reviews catch that? Odd.
      "Your comment here is useless. Report to the flogging station immediately."
      "This comment has nothing to do with this code. Flogging station."
      People would catch on eventually.

    • @TheTrienco
      @TheTrienco Год назад +28

      @@herrpez I wonder if some of those useless comments can be blamed on certain teachers. I remember at university they told us "for each line of code, there should be one line of comment". You can guess what the result of that was...

    • @grzegorzdomagala9929
      @grzegorzdomagala9929 Год назад +14

      This problem arises when people write in comments what the code does instead of what it should do. "Setting timeout to 5" is useless and potentially misleading "waiting till work process completes first cycle" is usefull - even if you change timeout to 15 or replace it with conditinal wait.
      Or even if someone "corrects" the code to oblivion - feature maintainer will know what you WANTED to achieve here.
      And personally - I just like to mark "difficult" parts of code with prayers to Omnissiah or Cthulhu :)

    • @wetfloo
      @wetfloo 8 месяцев назад +3

      ​@@herrpezI guess because people don't look at lines that weren't changed, and if comments stay the same, that means there's nothing to look at
      At least that's how my brain works

    • @Mclfarm2
      @Mclfarm2 7 месяцев назад +2

      Maintain them then?

  • @horriblyterrible1121
    @horriblyterrible1121 Год назад +75

    Is this video made by Tom the Genius?

    • @ThePrimeTimeagen
      @ThePrimeTimeagen  Год назад +27

      Tom the gineous is such a genious

    • @aurniox
      @aurniox Год назад +1

      ​@@ThePrimeTimeagenwow thats really geanious

  • @QuantumImperfections
    @QuantumImperfections Год назад +51

    Real talk: I usually only put comments in code (be it JS, R, Python or SQL) when i am referencing a business rule that needs to be understood. Preferably simply listing the business document link so if code needs to be updated they can go in-depth as to 'why' something is the way it is. I've found this is a big help in research settings where government & sponsor rules and regulations shift ever so slightly quite often.

    • @mattmurphy7030
      @mattmurphy7030 Год назад +3

      Extending that, I include references that I used if they were important to figuring out some code.

    • @eyondev
      @eyondev Год назад +5

      @@mattmurphy7030 Another extension: Linking to Github Issues when you need a workaround for something that's currently not up to the documentation spec.

    • @antdok9573
      @antdok9573 Год назад +1

      This is why BDD test cases exist.

  • @omega_no_commentary
    @omega_no_commentary Год назад +64

    If you can make your code readable enough and good enough, don't write comments.
    If you can't, Id rather have a commented steaming pile of garbage than a steaming pile of garbage with no comments at all.

    • @longlostwraith5106
      @longlostwraith5106 5 месяцев назад +5

      Late reply, I know, but what makes you think that someone who can't properly convey their ideas through code would be able to convey said ideas through words?

    • @2OmgLol
      @2OmgLol 4 месяца назад

      ​@@longlostwraith5106sometimes it is that way because you have to work with legacy code and structures. You cant reasonably alter everything so its readable by code only. You dont have enough time and resources to be able to do that a lot ofnthe time.

    • @Nocare89
      @Nocare89 Месяц назад +1

      @@longlostwraith5106 I've seen this and was able to understand what was supposed to happen, delete their code, and fix it. Talking to people and to computers are two different things.

  • @zactron1997
    @zactron1997 Год назад +65

    Rust is the first language where I felt like comments were right. The language is descriptive enough that I dont often need to write comments explaining what's happening. The built in testing system means I don't need to write comments explaining edge cases or intended use. Docstrings with tested examples are probably the best documentation I've ever seen in normal library code.
    Obviously other languages can avoid comments too, but in the looser or harder langauges I need comments more.

    • @davidlanda2324
      @davidlanda2324 Год назад

      Go!

    • @KManAbout
      @KManAbout Год назад +2

      Doc strings are just comments

    • @aoeu256
      @aoeu256 Год назад

      docstrings come from LISP, types as comments comes from ML.

    • @aoeu256
      @aoeu256 Год назад +1

      @@KManAbout Doctests will execute the tests in your doc strings. Tests can be nice as you can get chatGPT to implement your function for you from the tests, you could program your IDE to show you values flowing through your program from the tests, you can convert tests into types. However, make sure you are testing the properties of your function as they pertain to your business needs not the implementation details.

    • @homelessrobot
      @homelessrobot 10 месяцев назад +1

      I need comments in any environment where i might be 'in the process' of developing a solution to a problem, rather having already solved the whole thing in my head, and I am now just fleshing it out with the obvious and perfectly well behaved and well formed code structure that I have somehow worked out. Or in other words; code is a process. Bad code and bad non-code descriptions need to be possible to represent while you don't have perfect alternatives at hand.
      If I didn't have comments, I'd just have documents in my source tree referring to specific spans of coude in the source files that the document is talking about. Essentially, comments with a bunch of stupid extra steps.
      I do get the motivation behind this conclusion; you don't want your codebase to be littered with crippled code that needs comment crutches to make any sense. But at some point in the development process, you need to set this aside and write bad code. So you can get around to writing bettter code later.

  • @sutsuj6437
    @sutsuj6437 Год назад +221

    I think using comments as headers to segment areas of a large code blocks, is something useful. It allows quickly skimming through code, without having to read the whole thing.

    • @Yotanido
      @Yotanido Год назад +66

      I will admit, I do sometimes do this. But this typically indicates that your function is doing too many things and should be split.

    • @ybabts
      @ybabts Год назад +18

      If you have to write headers to segment your code, you either have functions that are much too l large or your file is way too large.

    • @PeterAuto1
      @PeterAuto1 Год назад +3

      why not moving the sections into their own functions?

    • @ScibbieGames
      @ScibbieGames Год назад +52

      ​@@Yotanido if it's a bunch of things you only have to do once with no repetition, splitting it up is likely a waste of time.

    • @MrLowbob
      @MrLowbob Год назад +3

      ​@@ScibbieGames if you have such a big thing that you need to write segment comments, usually you could instead use the time to comment it to just split it into different files/classes in the first place and then wouldnt need the comment any more.

  • @SimGunther
    @SimGunther Год назад +33

    I love how "Tom" the megamind genius is a recurring character in these stream highlights

    • @Jabberwockybird
      @Jabberwockybird 4 месяца назад

      I love how he's so ingrained that Prime's ADHD tourette outburst names him.

  • @MilesFlavel
    @MilesFlavel Год назад +87

    I honestly don't care that comments can lie
    I'll take 75% accurate with some context over no comments any day
    The real issue is that comments need to be reviewed with the same scrutiny as the code they're describing

    • @ThePrimeTimeagen
      @ThePrimeTimeagen  Год назад +32

      ^-- this is actually really good observation

    • @Ignas_
      @Ignas_ Год назад +43

      I think that comments should be written to summarize rather than to explain. The "write readable code" argument doesn't really work when you still have to read the entire codebase to see what's going on. And it's usually easier to summarize in a few sentences than aLongAndVagueFunctionNameThatsHardToRead.

    • @RigelOrionBeta
      @RigelOrionBeta Год назад +3

      Yes! I am admittedly very bad at updating comments. But at my last job, part of the review process was making sure your comments are updated. I would pretty consistently be told to update my comments, but the code looked good.
      Comments, to me, are incredibly useful, even if they're just directed at yourself. The problem with comments, just like code, is maintaining them, and making sure they stay relavent, just like code!
      The problem is not comments. The problem is a review process that does not see comments as useful to review. People just kind of glaze over comments. They should be reviewed just like code.

    • @handsome_man69
      @handsome_man69 Год назад +2

      thankyou. Some sane comment at last

    • @antdok9573
      @antdok9573 Год назад

      But how do you test comments? Nothing comes to mind as far as having some automated system judge comments. Unless we start using some framework that now reads all comments?
      If I can't test it, I'm way less likely to use it in my code. Comments fall under this as well. I hardly use them, save for a very reluctant to-do (don't-do) that I try to immediately get rid of ASAP.
      I'll read them, but at some point, I can get away with re-factoring large functions without needing to know precisely what they do.

  • @Jschmuck8987
    @Jschmuck8987 Год назад +55

    This guy’s voice is awesome. Like, I never thought I’d be able to tolerate a voice like this but he’s even more hilarious because of it. I can’t get enough of his videos.

    • @stefanms8803
      @stefanms8803 Год назад +10

      It's because he sounds like a cartoon character.

    • @desertfish74
      @desertfish74 11 месяцев назад +1

      @@stefanms8803 this is exactly it

  • @ernststravoblofeld
    @ernststravoblofeld 10 месяцев назад +14

    All these guys have a good idea, and then try to apply it to all situations everywhere.
    Sometimes you need a comment, so write a damn comment.

  • @StdDev99
    @StdDev99 Год назад +33

    That only applies to those who are just calling APIs and barely have any advanced logic in their code. When you have advanced mathematics and complex algorithms and equations that are based on geometry, you'll need to explain what are you doing and why and why didn't you go for the other obvious more intuitive way. That's what comments are for. It's not to explain what's happening, it's to tell you why it's happening and what was the developer thinking.

    • @NathanHedglin
      @NathanHedglin Год назад +1

      Exactly, well said.

    • @mattmurphy7030
      @mattmurphy7030 Год назад +1

      Nailed it. The only real comment in my game is an outline of the pathfinding algorithm. The rest are just “this is here because ___”

    • @krozareq
      @krozareq Год назад +7

      He states it himself he's not into math. But math is important for a lot of advanced uses. Scalars, vectors and matrices for example are prominent in a lot of areas where transformations are used. never hurts for a programmer to know how to use advanced algorithms. Who knows, he may realize he can heavily optimize something with it instead of just looping through arrays like a novice. That's fine at the small level, but with large stores of data the processing goes through the roof.

    • @marshallsweatherhiking1820
      @marshallsweatherhiking1820 Год назад

      I mostly explain the algorithm in a paragraph inside the function but at the very top of the block. Most of the comments inside the algorithm are just saying what if conditionals mean where it isn’t obvious. The most common example being at the top of an “else”. Like if I have an if block “ if ( size < max ){…” I put “max

    • @BusinessWolf1
      @BusinessWolf1 Год назад +3

      good job, you just repeated exactly what the primogen said, including his disclaimer statement

  • @sonofabippi
    @sonofabippi Год назад +8

    As an embedded engineer, even if your comment is "this code is f'd-up" and at that point, even if it's a lie --- it's still a warning that it is not going to work like what you will work like what is epxected. That's huge. That's helpful. I worked with this stuff that was only very loosely embedded-to-JS, and copying it immediately via clone seemed silly --- but the comment saying, "this is f'd-up, I get it"... GOOD STUFF.

  • @geoffreyzziwambazza7862
    @geoffreyzziwambazza7862 9 месяцев назад +11

    The math in his if statement example had me laughing 😂. And he did it so smoothly, that it doesn’t come off as inappropriate.

  • @julianelischer6961
    @julianelischer6961 Год назад +41

    50 year veteran here. If you have ever come across code you wrote 30 years before and thought "WTF was I doing this for?" because you know you had a really good reason but you can't remember what it was, then you know why comments are crucial. As a team lead I would refuse to OK code that doesn't tell me what a piece of code is supposed to do without having to read the code. It should also tell me the reasoning if anything is "non obvious".

    • @assetaden6662
      @assetaden6662 7 месяцев назад +3

      Lets say, hypothetically, if there was a way to somehow let people know how the piece of code works without writing comments. I mean, what if there was a document, somewhere in confluence, that told specifically how the code works, which type of response to wait for. I can't think of something like that existing.

    • @doughxDude87
      @doughxDude87 5 месяцев назад

      @@assetaden6662I believe that is called rotting documentation

  • @rsdyeahh
    @rsdyeahh Год назад +16

    Very nice, agree in most points.
    Now, try to write an assembly code with no comments and come back 6 months later

    • @mattmurphy7030
      @mattmurphy7030 Год назад

      That’s cheating

    • @okuno54
      @okuno54 5 месяцев назад +3

      Yeah, obvs this only works in languages that are reasonably able to express high-level concepts... unlike notoriously low-level assembly.
      Some of the techniques suggested don't even work in C because its type system is so simplistic. You don't have to (weirdly) flex that you write assembly.

  • @aonodensetsu
    @aonodensetsu Год назад +3

    HTML is not fully structural, you do not need to close since they aren't allowed to be nested, the language will auto-close

  • @turush4575
    @turush4575 Год назад +11

    I'm a beginner, this kind of videos frustrated me a lot because of the titles "don't write comments", "don't do this", "dont do that"...
    And, a beginner life is like:
    -> Starts to learn about comments
    -> Start learning how to comment well
    -> Start commenting well
    =-=-= 1 Week Later =-=-=
    *RUclips video telling you to not write comments*
    Beginner Conclusion: All you've learned is wrong.
    =-=-= Sometime Later =-=-=
    -> Starts to learn about OOP
    -> Got used to OOP
    -> Start writing good OOP code
    =-=-= 1 Week Later =-=-=
    *RUclips video: "OOP IS WRONG", "OOP IS THE ROOT OF ALL EVIL"*
    Beginner Conclusion: All you've learned is wrong.
    And this applies for all of the things I've done. ALWAYS, sometime later, a video will show trying to discourage me or prove that what I'm learning is wrong and it shoudn't exist.
    I bet if I start learning Rust, 1 week later, a video will show to me telling that Rust is "bad".
    But I don't fall for this anymore. I learn by myself now.

    • @mattmurphy7030
      @mattmurphy7030 Год назад +8

      Yeah, that’s because nothing is 100% the absolutely correct thing to do. There are debates and tradeoffs to most things. Writing good comments is good, writing bad comments is bad. Using OO can be good for modeling, using OO can have bad results for program structure and performance.
      Just accept that nothing you’re learning is the absolute end-all-be-all and understand you will eventually learn a flip side to what you’re doing.
      Eventually you’ll know enough that you’ll have your own opinions and you won’t worry too much about what some guy on RUclips says. Like this video, you’ll hear the useful parts and shake your head at the dumb parts.
      You’re ahead of the curve realizing that anyone claiming to have THE answer is going to be contradicted later. Every new framework and paradigm that gets introduced as god’s own work is eventually replaced.
      The problem solving and learning skills you build are what matters. The tools always change.

    • @turush4575
      @turush4575 Год назад

      ​@@mattmurphy7030 Totally agree! Thanks 👑

    • @Cellidor
      @Cellidor Месяц назад

      While I'm not a beginner, I recently started getting back into coding after several years away and I feel this from videos I've seen.
      My method has just been, 'appreciate what's being discussed, and then continue doing what you're doing, perhaps applying this new information to it'.
      In my current code for example, I'm commenting _every_ function, specifically so that I have reminders as I warm back up to coding. Once I'm more on a role, I'll feel more confident removing comments or narrowing them down to just pointing out why sections appear weird.

  • @salvadorroibon
    @salvadorroibon Год назад +7

    Just watching this channel the last year made me a better programmer. Love you Prime, thank you

  • @Anri17_
    @Anri17_ Год назад +30

    Comments are useful for writing down my thought process in the context of a piece of code. I find that it's better to just get things done as fast as possible, leave some comments describing and explaining the function and purpose of the code, then return at a later date with a fresh mind and refactor everything.
    It's kind of like writing an essay. Your first draft will usually be ugly, but it has all of the ideas that you want to deliver. It's only when you return to it later that you are able to edit it with better phrases, more appropriate words, a cleaner and more clear structure, add, or even restructure some of the ideas.

    • @RigelOrionBeta
      @RigelOrionBeta Год назад +1

      This is exactly how half my comments are written for. Much of the time they help me quickly understand what I wrote a set of code to do, and honestly, if it's good enough for me to read in a month of not looking at it, it's probably good enough for others too.

    • @davidp.7620
      @davidp.7620 8 месяцев назад

      The problem comes when you never actually go back to it and you leave module after module of unmaintainable code

    • @Anri17_
      @Anri17_ 8 месяцев назад +7

      @@davidp.7620 What are you talking about. Any change that needs to be made to the code becomes easier, because now you will have comments explaining purpose of the code and outlining other crucial details. If anything it makes code more maintainable.

    • @davidp.7620
      @davidp.7620 8 месяцев назад

      @@Anri17_ as I said, assuming you ever have the time to go back to them

    • @spaceowl5957
      @spaceowl5957 5 месяцев назад +1

      The code is maintainable like this even if you don’t go and clean it up and make it pretty. If the code works and I can understand and therefore change it later it serves its purpose and imo it’s good, functional code, even if it’s somewhat messy.
      Yes it would be nice to make everything nice and polished in the code base but I think that time is honestly better spent making things nice and polished for the end user. Which is what matters. All that polished code really gets is good for is for wanking of to it as a programmer.

  • @kahnfatman
    @kahnfatman 7 месяцев назад +9

    Here to the clean code camp who say the code should document itself: This is what your function names end up look like:
    - setupPageSizeA
    - navigateToSectionFourteenParagraphThirtyTo
    - handleTopLeftCornerButtonClickWhenUserIsUpsetAndFrustrated
    ... The names are so meaningful that you don't need to write comment at all. And Shakespeare shall be proud of ya.

    • @shinobuoshino5066
      @shinobuoshino5066 5 месяцев назад

      Lmao you're retarded, in reality what you wrote looks like this:
      setup_page(page_size::A4)
      navigate_to(section(14), paragraph(32))
      on_feedback_form_button(callback parameters...)
      And yes it's perfectly readable and you're a braindead moron, your forced use of camelCase is proof enough but your attempt at naming is just as bad.

    • @SNOZ562
      @SNOZ562 26 дней назад +1

      this is hyperbole. You can easily create concise names that clue in to what the code is doing. you just sound like youre bad at naming.
      Names should be clear and concise to function. Comments should be added when that isn't sufficient. It's not hard.

  • @Qewbicle
    @Qewbicle 9 месяцев назад +3

    Me writing comments for my future self has saved me, sometimes just the why, sometimes some what's to make a piece less dense (sometimes I write dense code when I'm in a flow, but I won't be in that same mindset a year later).
    The best thing to do is to write comments to someone out of the loop just to bring them to speed.
    Put it where it matters, so if that specific thing changes, so does the adjacent note/comment.
    Those that don't give themselves/others future notes have not ran into that issue.
    After they have the experience of going into different projects and having to revisit the past and realize they can't put together what they thought was obvious, they have to read a bunch of code to catch up to that mindset for that moment, then they'll have wised up to what to put, where to put it, and smart enough to keep those important bits up to date.

  • @woolfel
    @woolfel Год назад +8

    the comments should be about the design rational, the trade offs, business requirements and context. If you're writing a library that calls some other service, the comments should be about the quirks of the service and any odd stuff that you found. The comment should be about "how do I help someone maintain the code."

  • @luis10barbo
    @luis10barbo Год назад +113

    Nothing to add here, just let me state a fact...
    Tom is a Genius.

    • @jaredmistretta3018
      @jaredmistretta3018 Год назад +6

      Oh is this gonna become a thing? I'm down.

    • @kebien6020
      @kebien6020 Год назад +1

      @@jaredmistretta3018 Hi down, I'm up.

  • @TheNewton
    @TheNewton Год назад +21

    Comment now perfect later. Comments are context.
    Beginners need context and have to learn to communicate it.
    Technical writers need context to document.
    Senior devs create context and have to clarify it.
    Managers and stakeholders need explanations from everyone else depending on that context.
    The idea of not commenting is arguing for a perfect ideal code that will never exists.
    Just comment.

    • @okuno54
      @okuno54 5 месяцев назад

      You know there were these kinds of exceptions listed in the video, right? You actually watched it, and didn't just comment after reading only the title, right?

  • @conorstewart2214
    @conorstewart2214 Год назад +4

    Maybe it’s just because I mainly do embedded programming, microcontrollers, FPGAs and similar, but comments are very useful sometimes and I would rather commented code than non commented code, if the comments don’t make sense you can always fall back to just reading the code.

    • @Mclfarm2
      @Mclfarm2 7 месяцев назад

      Probably the more complex your domain the more comments can be helpful.

  • @thom1218
    @thom1218 7 месяцев назад +6

    Programmers that can't be bothered to update a comment to reflect their change don't deserve to meddling in the code to begin with.

    • @shinobuoshino5066
      @shinobuoshino5066 5 месяцев назад

      Programmers that give code terrible names and then bloat their programming time with pointless documentation writing should be fired.

    • @thom1218
      @thom1218 5 месяцев назад +4

      @@shinobuoshino5066 "pointless" documentation is an oxymoron, if it's pointless... then by definition it's not documentation. So yes, we're in complete agreement.

    • @shinobuoshino5066
      @shinobuoshino5066 5 месяцев назад

      @@thom1218 No, all documentation is pointless. There's code, then there's user manual for software that was already compiled, everything else is WORTHLESS. Most useful libraries I ever used had little to no documentation, all it took was one look at the headers.

  • @incremental_failure
    @incremental_failure Год назад +12

    I've been struggling with a relatively short function as of late regarding datetime conversion and various adjustments with a dynamic axis. The code is commented a lot and it has taken me over a week to understand it. Functions and classes are intertwined, most variables are kept short for whatever reason, lists and dicts are unpacked into objects, functions are passed in init, of course no type hints (yes, it's python). A simple thing like using longer, easier to follow variable names would solve so much and dynamic typing is the absolute worst thing about python.

    • @ThePrimeTimeagen
      @ThePrimeTimeagen  Год назад +6

      me hates dynamic typing

    • @incremental_failure
      @incremental_failure Год назад +1

      @@ThePrimeTimeagen Timing is perfect. Part of Cython compiled production code just failed for wrong typing as Cython has discipline regarding types. Tests written in Python let it slide.

    • @PamellaCardoso-pp5tr
      @PamellaCardoso-pp5tr Год назад

      @@incremental_failure why do people don't stick with a rule on adding the types of stuff into the name of the variables/functions in python?
      like if it's a integer variable
      Int_x = 3
      def Void_foo(Int_x):
      String_y = "something"
      wouldn't that make python usage less... shitty? i dunno cuz i abandoned writting in that mess of slow language and moved to zig and haskell instead.
      Monads go booooom babyyy

    • @incremental_failure
      @incremental_failure Год назад +1

      @@PamellaCardoso-pp5tr And complex types? Python isn't that slow either since well, all the complicated stuff has Python only as a layer. If you depend on a dozen or more critical modules, switching languages fast is out of the question.

    • @PamellaCardoso-pp5tr
      @PamellaCardoso-pp5tr Год назад

      ​@@incremental_failure you can try to define interfaces using comments... i dunno man. i would do anything to keep some sort of sanity in my code, specially now that i learned the beauty of types through the category theory point of view and how types relate to each other.
      and yes, python is slow. you aren't using python, you're using C in disguise, and in that regard Mojo seems to be the thing that will make python less shitty to use, but it will still suck without having damn brackets

  • @jl6723
    @jl6723 Год назад +15

    I am a fan of triple-slash comments in C#, my thought is that if you need a comment inline, it probably should be it’s own function and then inside the triple-slash I can detail what is being done, what are the concerns for the comments, and why did I decide to do this. It also allows me to explain concepts.
    It also can be used by auto-documentation tools to make documentation. And then there is a team effort to update and scrutinize the comments when we change things up such that people can better understand.
    Self-commenting code has been a nightmare to work with in the past, because evaluating decisions for refactors suck.

    • @envo2199
      @envo2199 Год назад +4

      ++ comments for methods are truly helping, and every library should use it. If a method doesnt have this intellisense comment, i always get a little angry.

    • @Temari_Virus
      @Temari_Virus Год назад +7

      Honestly C#'s /// comments are less like comments, and more like in-source documentation. Heck, they evem appear in your tooltips!

    • @jaysistar2711
      @jaysistar2711 Год назад +1

      These are called "doc comments", and I think that they originated in Java, which is a language that, like C#, I don't want to use for anything if I can help it, but the doc comments are a good idea, which is why Rust has them as well.

  • @ismaelgoldsteck5974
    @ismaelgoldsteck5974 Год назад +2

    I like putting a "project philosophy" file into the documantation which gives a general idea of how the projects structure is intended to be. Explaining which services there are, how the code can be extended, etc.

  • @dies200
    @dies200 4 месяца назад +1

    At my job we often end up going over the code in a pair of reviewer and author. If at any point we need to stop at a piece of code to explain why it's the way it is or we stop because even the author needs to think about it it's always a three step process
    1) what does the code do exactly?
    2) Can it be simplified or cleaned up?
    3) If no, put a commend explaining why the code is the way it is
    It's a rare occourence, oftentimes to adapt some weirdness from a third party library or incomming data. But when it happens the comments do actually help.
    Basically: If you read a line of code and your first thought is "WTF is this?". It's probably time for a comment

  • @OhsoLosoo
    @OhsoLosoo 9 месяцев назад +3

    My first computer science teacher made us write out what our functions were going to do before we could code.
    We also had to put in the comments which function was calling which.
    It is actually easier to run my code through an anti comment program (just deletes any thing in a ‘’’ or after #) than to break this habit unfortunately.

    • @shinobuoshino5066
      @shinobuoshino5066 5 месяцев назад +1

      Lmao, imagine seriously requiring to put in comments what function is being called, when the function body already shows all functions being called...

    • @OhsoLosoo
      @OhsoLosoo 5 месяцев назад

      @@shinobuoshino5066 I know this might seem stupid. But they wouldn't even let us use IDE's. The teacher was from a country where they were programming using Notepad++ as the IDE.
      Imagine all of this and then on top of it, they exclusively taught OOP in Java at the time, too.
      Safe to say, a lot of ppl were scared away from CS by this entire experience.

  • @iFireender
    @iFireender Год назад +2

    One point: in embedded programming, sometimes comments are almost necessary. Writing some random number to some random register or similar can be very confusing otherwise.
    For example: Accessing the DRW register over SWD after setting a new address in the TAR register requires one dummy read first. So, you set the register you want to read from, and then you read twice. Without a comment in there, that seems like a bug. A simple //dummy read after the first read explains what happens.

    • @HeyPumpkin
      @HeyPumpkin 6 месяцев назад +1

      I agreed right up until the example comment you suggested. Instead of:
      //dummy read after the first read
      What you explained right before that suggestion is much better and is exactly the kind of thing you should have in the comment:
      // Accessing the DRW register over SWD after setting a new address in the TAR register requires one dummy read first.
      That is just about a perfect comment.
      I don't know why you'd have such a clear explanation of exactly why you're needing to do something strange, even down to having decent capitalisation and punctuation, and then throw all of that tidiness and information away for a lazy comment that just repeats what the line of code is doing without explaining why.

  • @PieterJacob
    @PieterJacob 6 месяцев назад +1

    I agree! I don't write comments that explain code. I write comments if my code is weird or violate a basic principle, like a query inside a loop, that explains why in that particular situation I had no other choice.

  • @epotnwarlock
    @epotnwarlock Год назад +13

    you feel forced to comment old/legacy code to hopefully guide the next sucker to greener pastures, also python is epic

  • @digitalnorth
    @digitalnorth Год назад +4

    Thats why Tom said f u to comments in his JDSL , Tom is a genius

  • @dan-bz7dz
    @dan-bz7dz 8 месяцев назад

    I worked for a company once were we had a new junior hire that quit the first day, citing the lack of comments as the number one reason

  • @fabricehategekimana5350
    @fabricehategekimana5350 Год назад

    As always, your videos are incredible!
    00:06:52 I think optionals came from the type theory (I think it's related to monads like those in haskell)

  • @arjunsahlot
    @arjunsahlot 5 месяцев назад

    Prime, I gotta say, you react in the best way possible. Your commentary and support for the creator are top notch.

  • @wade__
    @wade__ Год назад +1

    I like using comments to jot down notes or to organize my thoughts tied to the surrounding code, but as I implement the notes/ideas, those comments get deleted because the code basically becomes the comment

  • @jacksparr0w300
    @jacksparr0w300 Год назад

    Is not similar to in that it cannot be nested anyway?

  • @kattankarl
    @kattankarl 9 месяцев назад

    I'd say the big exception to this if when you write code to integrate with an external system like a ERP system or something. Having a few comments instead of being forced to spelunk ERP documentation to understand why some seemingly pointless details are included / required saves so much time.

  • @fulconandroadcone9488
    @fulconandroadcone9488 Год назад

    I like the idea of commit message as a comment defining why that line was changed and when, and how it looked before and potentially linking to issue that defines a problem and how it was discovered ....

  • @leandroamarillo3262
    @leandroamarillo3262 Год назад

    Man what a worth while watch was Tom's Jdsl video. Probably they were the same stream but as someone who just watched clips is funny AF getting a reference to it

  • @JoshuaWise1994
    @JoshuaWise1994 Год назад +5

    I find comments that explain the code useful in two circumstances:
    1. As a block comment at the top of a file, summarizing what the file does
    2. Comments breaking up sections of a large procedural function that does several complicated steps.
    Many people will say "if you have several complicated steps, you should just split them into multiple named functions". But I disagree with this. When you split them into functions, you're implicitly saying "these functions can be called Independently, and perhaps in any order". But in the case of the large procedural function, each step must happen in a very specific order, and the steps shouldn't be invoked in isolation. Having them in order within a single encapsulated function is much better for expressing how it should be actually used, and comments can just help summarize what each complicated step is doing, and provides a little break between reading each section.

    • @okuno54
      @okuno54 5 месяцев назад

      Wait... even if you're using C, you can still introduce a block. Declare variables that later portions need before the block, then have a block of statements that do some related work and initialize your variables, and boom: a section without comments. Repeat until your big function has enough sections to be comprehensible. Maybe the variables from each block will say why the block exists... or else you add a "why" comment, so #2 is just a special case of #1.

    • @JoshuaWise1994
      @JoshuaWise1994 5 месяцев назад

      @@okuno54 yeah, sometimes you don't need comments in each section. Sometimes you do, because the variables don't explain it well enough. I use blocks in C all the time

  • @isodoubIet
    @isodoubIet Год назад +1

    "The compiler does not read comments, and neither do I."

    - Bjarne Stroustrup

  • @Muaahaa
    @Muaahaa Год назад +4

    I was repurposing an old computer the other week. It is over 10 years old and has code from some of the projects I was doing while learning front-end dev. The comments I wrote in there killed me to read. Practically every block of code had a header like "// ---- Functions ----" or dumb stuff like that XD

  • @twiksify
    @twiksify Год назад

    Golang got a test suite which validates examples via code and expected output in a comment.
    These comments are later published in API docs along with the build

  • @KayOScode
    @KayOScode Год назад +1

    I write comments to describe why something needed to be done a specific way

  • @BFedie518
    @BFedie518 6 часов назад

    In my very first programming class, comments were 60% of your grade.
    I'm not kidding. We got docked for incorrect grammar, capitalization, punctuation, etc. A perfect program with no or "bad" comments got you a 40% on the assignment.

  • @austin4855
    @austin4855 Год назад

    I start with piles and piles of comments because my code usually starts out as a steaming pile. The first write is usually a single file in a single night with 4 cups of coffee, with 400 lines of code and 500 lines of comments. Then comes the first refactor, and the second refactor if it's real serious. Now I have well-structured, lightly commented self-documenting code. In the end I try to keep my comments to primarily just refs/links to outside resources like business rules the code is implementing.

  • @bimsherwood7006
    @bimsherwood7006 Год назад +1

    Naming things with the code is better than comments, because the compiler checks you. Actual comments can be plain wrong and you only figure it out after you've been confused. And that's the opposite of what the comment was for!

  • @ICEknightnine
    @ICEknightnine 11 месяцев назад +1

    This made me realize that I have a bunch of, what would be considered, redundant code but I don't want to remove it because the human language helps me understand at a glance what the code is meant to do even if that can be surmised by reading the code itself. It also helps with bugfixes if the code doesn't do what the comment says it's supposed to do, the original intent is preserved (though that means I need to update comments when making changes).

  • @AmirHosseinHonardust
    @AmirHosseinHonardust Год назад

    I usually do some kind of chapterization. Say I have a controller function which is long. I specifiy where I validate and parse the inputs, where I do bussiness logic related stuff, where I write the data into database and where I send a successfull response.
    I also do the day with types. Each type has a file, the file contains, type definition, type builder section, trait implementations, error type and...
    It helped my navigation a lot.

  • @gamerk316
    @gamerk316 8 месяцев назад +2

    Software Engineer speaking: I will ensure anyone who doesn't use/maintain clear and concise code comments will be assigned as far away from a keyboard as possible.

    • @shinobuoshino5066
      @shinobuoshino5066 5 месяцев назад

      ah yes, the software engineer who can't even read real code but has power to tell me how my code should be written is speaking...

    • @gamerk316
      @gamerk316 5 месяцев назад +1

      @@shinobuoshino5066 Yeah no, I've seen enough piss poor code written by EE's moonlighting as proper Software Engineers where I am tired of wasting time and effort trying to decipher what the hell you were attempting to accomplish.
      If you can't clearly document your code, I will ensure you never touch a keyboard again.

  • @simonfarre4907
    @simonfarre4907 Год назад +1

    I've seen Primagen confuse borrow checker with allocation & deallocation twice now. Though this time he's somewhat closer to the truth than the last time; because unique_ptrs *does* involve life time management. You can still share a unique_ptr via reference arbitrarily.

  • @adriankal
    @adriankal Год назад

    Closing li was required just to make html compatible with XML. In html it was and is required only in specific cases.

  • @DemonixTB
    @DemonixTB Год назад +3

    Comments need to exist to justify why something was done, so the same mistakes arent retreaded.
    but comments get stale, because they arent tested or statically checked. so always be careful when writing your comments to not make more assumptions then necessary.

    • @muhwyndham
      @muhwyndham 10 месяцев назад

      I prefer using test to document behavior.
      Hence justifying it by making the test fail if changes were made, and hopefully people read the test case name and understood the assignment

  • @mattmurphy7030
    @mattmurphy7030 Год назад +1

    I’m 40k lines into a game and so far the only real comment I’ve felt compelled to write is an outline of the pathfinding algorithm

  • @taco_engineer
    @taco_engineer Год назад +3

    Waiting for the Tom's a genius merch. There's your startup.

  • @rustmoody383
    @rustmoody383 Год назад +1

    Comments are great for “why”, often when I’m in position when I now that solution I posted might seems strange I want to message anyone who ever wants to edit it to double think it why it is that way. Also it is a great marker for fishy code, if something is commented - keep an eye on that one

  • @vladislavkaras491
    @vladislavkaras491 5 месяцев назад

    Thanks for the info!

  • @Davidlavieri
    @Davidlavieri Год назад +1

    I see myself all the time cleaning my phone screen due to some little black spot,but it's always the microphone green screen

  • @rhbvkleef
    @rhbvkleef 6 месяцев назад +1

    My rule for writing comments is that comments should describe why the code is there, not what it does.

  • @ShadowAid
    @ShadowAid Год назад +1

    I will only ask for comments to answer "why" but not "what". But having "why" comments saves people time later when needing to make changes.

  • @0e0
    @0e0 5 месяцев назад

    moving the condition to it's own function seems like best practice for me

  • @stanislawcronberg3271
    @stanislawcronberg3271 9 месяцев назад +6

    Watching ThePrimeagen instead of sitting down and coding is the real copium

    • @codegeek98
      @codegeek98 6 месяцев назад

      Sitting sagely listening to a RUclipsr listening to a RUclipsr talking about what not to do in code (my code has no bugs (because my code does not exist (yet

  • @rollin340
    @rollin340 7 месяцев назад

    Glad I'm not alone in hating unclosed tags. I also despise code where some have the curly brackets, and some don't. I always prefer with, but to do both is insane.

  • @khhnator
    @khhnator Год назад

    i ususaly have lots of comments, because before coding i usually write a pseudo algo of what i need to do so i can organize my thought... and i tend to just leave them

  • @johanrojassoderman5590
    @johanrojassoderman5590 Год назад

    4:18 ehehe, coming from C# I used to be, but since that seems to be the naming convention in the c++ standard library and the language in general, I just adopted it with the argument that consistency is less confusing.

  • @srijanraghavula
    @srijanraghavula 18 дней назад

    That line at the end of the video was awesome

  • @ideegeniali
    @ideegeniali 11 месяцев назад

    I wrote protocol specs documentation to talk to firmware in electronic boards i design and build. A client needed to use such boards. I sent protocol specs, he was having hard time. I sent well written, no comments, autoexplicative source code protocol parser class, and he immediately got it. He told me he apparently understands C++ way better than English and/or I can express myself more clearly in C++ than English!
    That's it: code IS its specs, in the most detailed description possible.

  • @lpprogrammingllc
    @lpprogrammingllc 7 месяцев назад +1

    If you are implementing something complex, especially if it is something that requires you, the programmer, to remind yourself how it works via stack overflow or wikipedia, you should comment it. At the least, you should include a link to the source of the algorithm. Then, if the algorithm has multiple steps in the wikipedia pseudocode or what not, you should note where you are starting each step. There is nothing worse than having to come back to some Vector3 trig functions because of some slight change in the program requirements, and then spending several hours re-learning something you haven't needed in a year.

  • @Zibblobet
    @Zibblobet 5 месяцев назад

    I lowercase the first letters of names of structs and classes but I also put all of them into their own namespace and if a data structure gets too large to necessitate a distinction then that struct/class is (probably) too large in scope

  • @Sameer.Trivedi
    @Sameer.Trivedi 11 месяцев назад

    How do you autoformat your code with vim? I can do = but that only indents stuff, what about spacing and all? I also found an autoformat plugin but it hasn't been updated in years.

  • @tsitsimelody5874
    @tsitsimelody5874 Год назад

    My guess is the optional types came from functional programming and haskell world, I remember learning them when i decided to learn functional programming

  • @Rockyzach88
    @Rockyzach88 Год назад +1

    For me it comes natural. I just put down what I think will be useful to know there. It's like writing notes on paper. I just write stuff that I think will be useful for me in the future. That can come in different forms.

  • @Mempler
    @Mempler Год назад +3

    0:30 i dont say what the code is doing. i say why it's doing. and sometimes use "why the fuck does this shit not work like that: but like this:

  • @hopelessdecoy
    @hopelessdecoy 4 месяца назад

    I use comments for 2 things:
    1. work in progress, so when I come back to it I have an idea of my line of thought and can finish the work
    2. To describe an entire script/set of functions, this is so someone doesn't have to read through the file to know the details of what it is accomplishing. This isn't always necessary but I think it is a good habit and really helps people still being on-boarded.

  • @mekelius
    @mekelius 6 месяцев назад +1

    Sometimes you have to use libraries that have such convoluted interfaces and unintuitive names your code will become unreadable just due to that. In some languages that includes the standard library..

    • @shinobuoshino5066
      @shinobuoshino5066 5 месяцев назад

      And having to read documentation won't help you, the problem is bad variables and functions, not lack of comments.

  • @L1Q
    @L1Q Год назад

    li closing is optional as per html spec, same as p, th, td, tr, head, html, body. They can't be nested so spec allows ommiting the closing tag.

    • @BusinessWolf1
      @BusinessWolf1 Год назад +1

      "just because you can, doesn't mean you should" - Footgun Avoider

  • @HerrDoktorWeberMD
    @HerrDoktorWeberMD 4 месяца назад

    I comment what a piece of code is doing all the time, but that's more about me learning and having not the best memory. So I can basically reference my own code like classroom notes

  • @d1ngd0
    @d1ngd0 Год назад +8

    Long live Tom. Tom is a genius!

  • @bababert8488
    @bababert8488 Год назад +1

    For the type not being in written in upper case, I feel like it's inferred conversion, that primitive types, stuff that takes less space then a pointer, should me lower case and everything else in upper case. Just like tsp and Tbsp.

  • @ttrev007
    @ttrev007 4 месяца назад

    I think it depends on the what the code is going to be used for. If you are a noob and learning, explaining what you are doing is helpful. If you are writing code that will be used by a novice or non programmer then the more description the better. In an academic setting were your code is likely going to be looked at by scientist who are weak on skill comments describing what it is doing is a good idea.

  • @wiczus6102
    @wiczus6102 Год назад

    More often then not there are multiple ways to understand what function is doing, a comment should clarify that. For instance GetDescendantsOfType should have a comment telling you 1. whether this is recursive 2. which types are okay. 3. What is the stop condition? Is there a situation where you don't want the recursion to continue?
    You don't want to sift through the code just to find these things out.

  • @telesniper2
    @telesniper2 4 месяца назад

    I remember one of my first programming projects at community college in like 2007. Now, this instructor was like a SUPER nice guy and really square like a Mr. Rodgers or Bob Ross or something. Well I was working on the project and got stuck with something and got SUPER frustrated and put in some VERY profane language in the comments at the time. I thought I had deleted all of them. When I finished it and turned it in, he handed it back to me with errors annotated as well as a comment "please refrain from leaving off colored comments in your code". I turned beet red

  • @mage3690
    @mage3690 Год назад

    I mean, you _can_ do an optional return in C if you do "void foo(args, &returnVal);" and squint hard enough. Wouldn't recommend, generally try to stick to returns because pointers are the literal devil, but it _can_ be done.

  • @DanKxxx
    @DanKxxx 2 дня назад

    I always try to comment on why things are done. If running a big chunk it's good for people to know a really quick summary so you don't have to read through it if you are not looking at that particular area.

  • @platypus001
    @platypus001 8 месяцев назад

    i use a plugin "comment anchors" to name and mark sections of code so i can quickly jump to whatever block i want

  • @sullivan3503
    @sullivan3503 Год назад +1

    Comments are meant to document intention. A well-placed paragraph can obviate the need to read and understand hundreds of lines of code.

  • @bearwolffish
    @bearwolffish Год назад +4

    I love when a dev explains what the code is (supposed to be) doing, makes it easier to audit the mistakes if the logic doesn't line up.
    Someone else being able to tell what your code is actually doing, and it doing what you think it does are two different things.

    • @Spencer-wc6ew
      @Spencer-wc6ew 9 месяцев назад +1

      Also not everyone thinks in the same way. Something may be so simple and obvious to you, but make no sense to another debeloper.

  • @gwaptiva
    @gwaptiva 6 месяцев назад

    As someone that first learnt SGML, you can stuff your new-fangled HTML

  • @TheSkepticSkwerl
    @TheSkepticSkwerl Год назад

    I have this one case that is extremely more performant by first converting the data to a hash table and then working with it and it's super wonky looking but it reduced the work time from 25 minutes to 45 seconds

  • @rothn2
    @rothn2 4 месяца назад

    We have a thing about this in our bathrooms at work sometimes!

  • @maxmustermann3938
    @maxmustermann3938 7 месяцев назад +1

    I agree... whether you comment "add 1 to x" or "compute the sum of the vector", the code already makes that obvious. But if you do some strange loop where you're adding +1 for indiscnernable reasons to some index and add a comment explaining how the data is layed out, that actually helps in understanding what's going on.
    Example I recently wrote, I had two loops, one going over odd indices, the other over even indices and they had a branch where some index was either i, i+1 or i-1 - you wouldn't know what the fuck is going on there without knowing the data layout, "positive" directions where stored at odd indices, with their corresponding negative direction stored +1 index away, and the algorithm implemented there required you to handle those separately and sometimes access those corresponding directions. It was all low-level CUDA code, so no super fancy abstractions. Comments are much more important when you really can't use abstractions like optional but *have* to work with just integers, low level data types and specific weird layouts and giving some weird meaning to specific values, like the -1 in the example. If you're implementing some very specific algorithm, a link to the paper or the name of that algorithm as a comment makes sense. Nobody knows what a fucking esoteric pull is supposed to be or how it works after all.