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
  • НаукаНаука

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

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

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

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

      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 Год назад +747

    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 Год назад +438

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

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

      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 Год назад +141

    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  Год назад +29

      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 Год назад +87

    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

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

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

    • @nerdycatgamer
      @nerdycatgamer 6 месяцев назад +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 6 месяцев назад

      @@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

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

      @@doughxDude87 There is no "refactoring". Write the code correctly in the first place.

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

    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 Год назад +40

      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 9 месяцев назад +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?

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

    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 Год назад +9

      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.

    • @NDMAyy
      @NDMAyy 2 дня назад +1

      Good man, the takeaway lesson is to take onboard someone’s claim or opinion, but don’t believe anything until you’ve tested and confirmed for yourself. It can take a long time to do this so don’t try and pick a side until you’re satisfied with your understanding. Someone says comment intent, someone says don’t and make code clear to understand, maybe a one-size-fits-all is actually never the answer and the fact is it will always depend on situation and circumstances, keep doing more of what works best for you and do less of what doesn’t. It’s like mastering any skill, masters never stop honing their craft, don’t look for a finish line, rather enjoy the journey and recognise that when you hear opinions or blanket statements like this that one day you will have found your own foundations from experience that will no longer be shaken by others. Your journey will be yours and nobody else can tell you the right way to go, because nobody else can take your path except you.
      ☮️ ✌️ 💻 ❤

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

    Is this video made by Tom the Genius?

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

      Tom the gineous is such a genious

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

      ​@@ThePrimeTimeagenwow thats really geanious

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

    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 5 месяцев назад

      ​@@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 2 месяца назад +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 11 месяцев назад +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.

  • @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.

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

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

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

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

  • @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.

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

    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 6 месяцев назад

      @@assetaden6662I believe that is called rotting documentation

  • @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 Год назад +3

      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.

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

    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.

  • @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.

  • @thom1218
    @thom1218 8 месяцев назад +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.

  • @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 Год назад +1

      @@stefanms8803 this is exactly it

  • @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 6 месяцев назад +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.

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

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

  • @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

  • @kahnfatman
    @kahnfatman 8 месяцев назад +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 Месяц назад +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.

  • @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 Год назад +34

    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

  • @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 6 месяцев назад

      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?

  • @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.

  • @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 9 месяцев назад

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

    • @Anri17_
      @Anri17_ 9 месяцев назад +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 9 месяцев назад

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

    • @spaceowl5957
      @spaceowl5957 6 месяцев назад +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.

  • @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.

  • @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."

  • @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.

  • @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.

  • @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.

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

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

  • @dies200
    @dies200 5 месяцев назад +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

  • @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 6 месяцев назад

      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 6 месяцев назад

      @@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

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

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

  • @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.

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

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

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

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

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

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

    - Bjarne Stroustrup

  • @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

  • @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 ....

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

    Thanks for the info!

  • @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.

  • @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

  • @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.

  • @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

  • @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)

  • @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.

  • @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.

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

    Long live Tom. Tom is a genius!

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

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

  • @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.

  • @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

  • @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

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

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

  • @stanislawcronberg3271
    @stanislawcronberg3271 10 месяцев назад +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

  • @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

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

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

  • @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.

  • @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.

  • @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.

  • @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

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

    Tom is a genius

  • @ICEknightnine
    @ICEknightnine Год назад +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).

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

    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

  • @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.

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

    What theme was that when you showed the editor to do quick math example?

  • @BFedie518
    @BFedie518 13 дней назад

    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.

  • @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.

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

    That line at the end of the video was awesome

  • @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!

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

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

  • @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

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

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

  • @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.

  • @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

  • @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

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

    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.

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

    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

  • @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.

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

    What is the source of the video you are reacting to?
    Edit: Code Aesthetic

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

    Odd case, but what about comments for people who cant really code but get put on the project a couple years down the line. Would it be better to create a separate documentation? Or what approach would be better.

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

    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.

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

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

  • @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.

  • @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

  • @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:

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

    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.

  • @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.

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

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

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

    tom is a genius

  • @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.

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

    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

  • @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

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

    Code Aesthetic is always one step ahead of The Start Up

  • @Moosems-pk8vc
    @Moosems-pk8vc 2 месяца назад

    The ‘I use zig by the way’ at the beginning gave me the chills of ‘I use arch by the way’

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

    How will you add a counter for how many times someone has tried and failed to fix or refactor something?

  • @-vis-2492
    @-vis-2492 11 месяцев назад

    I am just wondering, If comments were not writent that bad how easy would it have been to train an AI to understand code ?