Comment

Comment on Bill is a pro grammer

magic_lobster_party@kbin.social ⁨1⁩ ⁨year⁩ ago

I’ve seen code that look like this:

int delay = 15 * 60; // 10 minutes

Even if the comment was on the same line someone forgot to update it.

Better solution is to write (in C#):

TimeSpan delay = TimeSpan.FromMinutes(15)

Much more obvious what the code actually means.

Sort:hotnew top

18107@aussie.zone ⁨1⁩ ⁨year⁩ ago
A better comment would be delay in seconds as that is the one thing not obvious from glancing at the code.

source
- magic_lobster_party@kbin.social ⁨1⁩ ⁨year⁩ ago
  Or just name the variable delaySeconds if you really want to store it as an int. Bonus is that every use of the variable perfectly communicates what it is.
  
  source
CCatMan@lemmy.one ⁨1⁩ ⁨year⁩ ago
Is the better way is a runtime performance hit. Does the compiler optimize this?

source
- magic_lobster_party@kbin.social ⁨1⁩ ⁨year⁩ ago
  It’s probably a little bit slower, but there are other things more worth to optimize than to shave off a few microseconds from a 15 minute delay.
  
  source
  - CCatMan@lemmy.one ⁨1⁩ ⁨year⁩ ago
    Yeah, it adds up eventually when working with embedded platforms, but for PC stuff I agree.
    
    source
    magic_lobster_party@kbin.social ⁨1⁩ ⁨year⁩ ago
    If you’re working in embedded I guess you can probably make an inline function or a macro so it’s taken care of at compile time.
    
    source