Don't Write Comments | Prime Reacts

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

[@XxDarkCinisterxX]

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

[@whig01]

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

[@MrHords]

Fuckin lol 😂 Tom is a genius

[@uchennaofoma4624]

Of course, Tom's a genius 😂😂😂

[@me-low-key]

Tom is a genius

[@user-gt2th3wz9c]

@@whig01 TOML is TOM's language?

[@farqueueman]

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

[@cmelgarejo]

Username checks out

[@farqueueman]

@@cmelgarejo how dare you!

[@QuantumImperfections]

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

[@MrLowbob]

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

[@ShrirajHegde]

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

[@thewizardsofthezoo5376]

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

[@ThePrimeTimeagen]

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

[@fulconandroadcone9488]

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

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]

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]

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.

[@kahnfatman]

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]

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.

[@tordjarv3802]

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]

respect

[@anmoon]

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

[@jayshartzer844]

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

[@notuxnobux]

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]

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

[@geekzombie8795]

@@mattmurphy7030Lel true

[@doughxDude87]

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

[@nerdycatgamer]

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

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

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]

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]

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

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]

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

Maintain them then?

[@omega_no_commentary]

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]

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]

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

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

[@QuantumImperfections]

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]

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

[@eyondev]

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

[@antdok9573]

This is why BDD test cases exist.

[@horriblyterrible1121]

Is this video made by Tom the Genius?

[@ThePrimeTimeagen]

Tom the gineous is such a genious

[@aurniox]

​@@ThePrimeTimeagenwow thats really geanious

[@zactron1997]

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]

Go!

[@KManAbout]

Doc strings are just comments

[@aoeu256]

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

[@aoeu256]

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

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.

[@ernststravoblofeld]

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.

[@julianelischer6961]

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]

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]

@@assetaden6662I believe that is called rotting documentation

[@turush4575]

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 =-=-= *KZbin 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 =-=-= *KZbin 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]

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 KZbin 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]

​@@mattmurphy7030 Totally agree! Thanks 👑

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

[@SimGunther]

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

[@Jabberwockybird]

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

[@sutsuj6437]

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]

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]

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]

why not moving the sections into their own functions?

[@ScibbieGames]

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

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

[@MilesFlavel]

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]

^-- this is actually really good observation

[@Ignas_]

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]

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]

thankyou. Some sane comment at last

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

[@sonofabippi]

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.

[@StdDev99]

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]

Exactly, well said.

[@mattmurphy7030]

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]

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]

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]

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

[@Jschmuck8987]

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]

It's because he sounds like a cartoon character.

[@desertfish74]

@@stefanms8803 this is exactly it

[@geoffreyzziwambazza7862]

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

[@rsdyeahh]

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

[@mattmurphy7030]

That’s cheating

[@okuno54]

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.

[@Qewbicle]

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.

[@salvadorroibon]

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

[@thom1218]

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]

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

[@thom1218]

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

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

[@Anri17_]

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]

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]

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

[@Anri17_]

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

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

[@spaceowl5957]

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.

[@TheNewton]

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]

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?

[@aonodensetsu]

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

[@luis10barbo]

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

[@jaredmistretta3018]

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

[@kebien6020]

@@jaredmistretta3018 Hi down, I'm up.

[@stanislawcronberg3271]

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

[@codegeek98]

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

[@woolfel]

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

[@ismaelgoldsteck5974]

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.

[@OhsoLosoo]

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]

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

[@OhsoLosoo]

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

[@conorstewart2214]

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]

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

[@jl6723]

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]

++ 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]

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

[@jaysistar2711]

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.

[@incremental_failure]

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]

me hates dynamic typing

[@incremental_failure]

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

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

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

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

[@kattankarl]

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.

[@JoshuaWise1994]

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]

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]

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

[@digitalnorth]

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

[@ThePrimeTimeagen]

time and time again, tom is a genius

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

[@dan-bz7dz]

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

[@epotnwarlock]

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

[@gamerk316]

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]

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]

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

[@jacksparr0w300]

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

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

[@dies200]

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

[@guavavodka]

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

[@vladislavkaras491]

Thanks for the info!

[@arjunsahlot]

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

[@isodoubIet]

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

[@karaloop9544]

Option/Maybe/Either: pretty sure it was Haskell again. It's wild how many programming concepts originated in that language.

[@nangld]

In my personal programming language, Symta, all typenames start with a lower-case letter, while all data variables with an upper-case letter. Yeah, I'm a Prolog fan.

[@trojan8550]

Comments never lie, they just don't tell the truth!

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

[@iFireender]

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]

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.

[@Flamewheel2001]

Tom’s a genius confirmed

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

[@Sameer.Trivedi]

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.

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

[@PieterJacob]

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.

[@platypus001]

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

[@taco_engineer]

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

[@Muaahaa]

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

[@mattmurphy7030]

Almost as bad as //increment the index

[@JoanGonzalezTrolloCat]

damn u didn't have to call me out like that

[@d1ngd0]

Long live Tom. Tom is a genius!

[@simonfarre4907]

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.

[@-vis-2492]

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

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

[@JakobRobert00]

3:40 in this scenario, I would argue that you should not name the variable "FIVE_MINUTES" because the variable name can become misleading just as the comment did. If you change 5 to a 3, you might forget to adjust the variable name.

[@Jabberwockybird]

Can you stop redundantly naming constants based on their expected value for FIVE MINUTES!?

[@0e0]

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

[@TheDXPower]

I work in hardware emulation... the domain essentially requires huge amounts of state and logic in the code to do its job (emulate hardware, which is inherently extremely stateful and extremely combinatorial). We try to manage this as best we can, but you still need that core logic *somewhere*, no amount of abstraction or naming functions will help you here. There's so much external knowledge to the code to make sense of variables and equations that comments are basically necessary to say why XYZ is done at point ABC.

[@shinobuoshino5066]

Your specific case is solved by vast documentation about how hardware works, said documentation has some form of naming convention that when followed, helps EVERYONE who read the documentation on how said hardware works to know what is where. Such code requires exactly 0 comments in regards to what is happening. For example gameboy manual references LD opcode, maybe you should name it LD in your code too and then everyone will know about what it means.

[@petar-boshnakov]

I always use the comments to set the conceptual idea of the code or logic that follows not the details of whats happening inside the code. So i read one line instead of 10 or who knows how many more(vomit)- like a function but at a higher abstraction. I'm lazy

[@Zibblobet]

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

[@DemonixTB]

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]

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

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

[@wade__]

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

[@BreathingCells]

"CodeAesthetics ... is by far one of my favorite, favorite KZbinrs" (1:45). Yes! With all of seven (7) videos, each one a masterpiece, we can only hope for more.

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

[@switchblade6226]

Tom is a genius

[@user-sv3dc5nz8w]

One of my favs: -- lock table events lock table events; Great comment.

[@HyperionStudiosDE]

I don't think that's even controversial. I've never met any professional who writes a significant amount of comments. They're just unnecessary most of the time.

[@bimsherwood7006]

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!

[@HerrDoktorWeberMD]

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

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

When Prime said "comments can do this" - such PHP vibe 😅

[@bababert8488]

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.

[@Mempler]

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:

[@JoshuaMiller-gr6sj]

I agree. Don't describe what it does but possibly why it exists.

[@rothn2]

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

[@legitjimmyjaylight8409]

Who needs optional anyway! A nice raw pointer can do that with null for unset! Or roll your own optional template class that wraps a raw pointer for fun!

[@Davidlavieri]

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

[@bearwolffish]

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]

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

[@adriankal]

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

[@ICEknightnine]

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).

[@mattmurphy7030]

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

[@ttrev007]

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.

[@vrumivruni]

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.

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

[@tieTYT]

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

[@FandangoJepZ]

We found some obfuscated code from the danish government at my job, and it basically had this structure: If LONG STRING VALIDATION: debugger else: debugger I wish I saw a comment for that code…

[@telesniper2]

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