Whoa, xargs trick is neat!
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 14:10 (three years ago) link
I am old. I still use xargs -i and never switched to -I
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 14:13 (three years ago) link
Also can never remember that xargs idiom using the shell trick.
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 14:18 (three years ago) link
Ah, xargs -0 will keep the whitespace
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 14:22 (three years ago) link
Feel like comments should be judged on their practical effects rather than from some philosophical position. If it takes 2 minutes to figure out a chunk of code without comments and 10 seconds to figure out the code with comments, I don't care how "obvious" or "self documenting" the code itself is, the comments are good.
― Dan I., Tuesday, 21 July 2020 14:25 (three years ago) link
otm
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 14:28 (three years ago) link
I say this as person who long ago took a course - in Pascal! - nicknamed "Commenting For Credit" in which people were expected to do things like
i := I + 1; //increment I
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 14:32 (three years ago) link
(input wants to uppercase every i)
i := i + 1; //increment i again
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 14:33 (three years ago) link
Next topic: git commit messages.
//increment topic
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 14:36 (three years ago) link
ruby doesn't let you i++. spent a chunk of sunday looking at cryptic error message and not seeing the problem.
― koogs, Tuesday, 21 July 2020 14:44 (three years ago) link
Can we attach this to everyone's monitor?
― shout-out to his family (DJP), Tuesday, 21 July 2020 14:58 (three years ago) link
I think I only write comments that are at least 100 words long
― all cats are beautiful (silby), Tuesday, 21 July 2020 15:08 (three years ago) link
I'm the only one reading my code/commenting so too many of my comments are targeted at my osteoporosis of a memory, e.g.,
// The + 16 offset is event type, commenting so I don't forget it yet again// The + 16 offset is still event type, it appears I forgot it again
― like, I’m eating an elephant head (katherine), Tuesday, 21 July 2020 15:32 (three years ago) link
(4000 lines apart, to be fair, and not my code but decompiler output)
― like, I’m eating an elephant head (katherine), Tuesday, 21 July 2020 15:33 (three years ago) link
public static final int EVENT_TYPE_OFFSET = 16;
― koogs, Tuesday, 21 July 2020 15:35 (three years ago) link
trust me that's not feasible in this thing, it's certainly possible but this is 240K lines of code and I can't just ctrl-f
― like, I’m eating an elephant head (katherine), Tuesday, 21 July 2020 15:53 (three years ago) link
Btw, my posts from a few weeks ago was about colleagues who are part of the development process and are participating in alpha testing but seem to want to behave as if they are end users and are shocked - shocked! - when something breaks. Perhaps this is a touchy subject.
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 16:21 (three years ago) link
Because no one wants to be that guy that breaks things.
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 16:28 (three years ago) link
But only the other hand there is the short-term/local/pre-mature optimization of being fast on the send button with "It broke! Nothing should ever break!"
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 16:30 (three years ago) link
"Strike mine eyes from my face!"
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 16:31 (three years ago) link
"I have reported thy breaking to mine Lord of Unbreakability. My day's work is done, now I shall take leave and rest for the nonce."
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 16:34 (three years ago) link
this is me :(
― 𝔠𝔞𝔢𝔨 (caek), Tuesday, 21 July 2020 16:36 (three years ago) link
Knew it!
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 16:39 (three years ago) link
caek’s link is otm
I am in favor of breaking down functions to component parts when they’re overly long or complex. This is a necessity in code of any length, especially when it makes scope of variables clear and you’re doing a specific manipulation of data that is secondary to the purpose of what you’re writing. You don’t need inline parsing of object A into object B if what you’re actually doing is some sort of calculation.
The primary use of functions is to encapsulate code that could be reused somewhere else. If you can add a few comments and make it clear what you’re doing in the primary function, and you’re not reusing anything, just add comments.
Same for code paths that are going to end up calling all functions — why are all of them separate functions if you call every one of them 90% of the time? You’re just replacing comments with function names, and outside of the context of reading *all* of that code, I have no idea what replaceAllInstances does. Instances of.. the one argument to the function? It’s method signature as comment
― solo scampito (mh), Tuesday, 21 July 2020 18:27 (three years ago) link
Well said.
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 18:37 (three years ago) link
Kevlin Henney to thread
― Left Eye Frizzell (James Redd and the Blecchs), Tuesday, 21 July 2020 18:48 (three years ago) link
World’s Finest Commit Message:Merge remote-tracking branch 'origin/master'
― Time Will Show Leo Weiser (James Redd and the Blecchs), Tuesday, 28 July 2020 14:50 (three years ago) link
smh
― all cats are beautiful (silby), Tuesday, 28 July 2020 15:31 (three years ago) link
SMH CARD
― Time Will Show Leo Weiser (James Redd and the Blecchs), Tuesday, 28 July 2020 15:41 (three years ago) link
https://www.youtube.com/watch?v=X23v5_K7cXk
― Time Will Show Leo Weiser (James Redd and the Blecchs), Tuesday, 28 July 2020 18:29 (three years ago) link
speaking of commit messages
https://www.twitter.com/gitlost
― like, I’m eating an elephant head (katherine), Tuesday, 28 July 2020 19:06 (three years ago) link
lol
― shout-out to his family (DJP), Tuesday, 28 July 2020 19:08 (three years ago) link
always happy to see that account again
― all cats are beautiful (silby), Tuesday, 28 July 2020 20:08 (three years ago) link
this component test, two of the 'expected' fields are actually a comma separated list of additional tests and a parallel comma-separated list of the expected values for these tests.
― koogs, Friday, 31 July 2020 12:34 (three years ago) link
Defensive Programming: C/D
― Isinglass Ponys (James Redd and the Blecchs), Monday, 24 August 2020 15:36 (three years ago) link
Depends what it is… but I’d rather have good tests which catch any failures in most cases I would say.
― Chewshabadoo, Monday, 24 August 2020 16:26 (three years ago) link
iirc Go tries to push defensive programming as a defaultiirc the main point of defensive programming is handle all the error conditions *first* then the code does what it’s supposed toit’s smart imo, because happy path programming is terrible
― solo scampito (mh), Monday, 24 August 2020 17:32 (three years ago) link
worst mess up we've had here recently, one that made the papers, was component C relying on previous component B for validation only to then add component A as an input path and component A didn't bother.
― koogs, Monday, 24 August 2020 18:51 (three years ago) link
in short: trustno1
― koogs, Monday, 24 August 2020 18:57 (three years ago) link
Innocuous irrational anger: typos in method/var names that don't get fix and propagate throughout the system wherever the thing is called - check_intant_update_user_permission
― neith moon (ledge), Tuesday, 25 August 2020 10:31 (three years ago) link
intant_update_user_permission = False...return intant_update_user_permission
how can you even c+p all that without noticing?
― neith moon (ledge), Tuesday, 25 August 2020 10:33 (three years ago) link
intant_karma.Said(“Knock You Out”)
― Isinglass Ponys (James Redd and the Blecchs), Tuesday, 25 August 2020 10:57 (three years ago) link
hey my display name is relevant
― assert (MatthewK), Tuesday, 25 August 2020 11:14 (three years ago) link
> don't get fix
you never fix
― koogs, Tuesday, 25 August 2020 11:38 (three years ago) link
who is screaming FIX at my code. i will never fix.
― neith moon (ledge), Tuesday, 25 August 2020 12:45 (three years ago) link
OMG we have a column in our DB called loyality_discount.
― Ruth Bae Ginsburg (Leee), Tuesday, 25 August 2020 16:33 (three years ago) link
that’s right
― solo scampito (mh), Tuesday, 25 August 2020 16:42 (three years ago) link
I think about this Reddit thread a lot pic.twitter.com/w2QxYJ2kx6— Sy Brand (@TartanLlama) August 20, 2020
― koogs, Thursday, 27 August 2020 11:03 (three years ago) link
when that thread was making the rounds I did a lot of mumbling “oh no”
― solo scampito (mh), Thursday, 27 August 2020 13:23 (three years ago) link
i've only seen that one comment but a) wow at the inventiveness of people and b) fuck the inventiveness of people
you can use unicode letters (not punctuation) in java for variable names and i can see delta being useful for all the animations i write but fuck learning how to type it.
also, i think java 9 stopped the use of a single _ as a variable name, but two or more is fine.
― koogs, Thursday, 27 August 2020 13:41 (three years ago) link