Programming as a career

public static final int EVENT_TYPE_OFFSET = 16;

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

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.

Because no one wants to be that guy that breaks things.

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

"Strike mine eyes from my face!"

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

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

this is me :(

Knew it!

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

Well said.

Kevlin Henney to thread

World’s Finest Commit Message:

Merge remote-tracking branch 'origin/master'

smh

SMH CARD

https://www.youtube.com/watch?v=X23v5_K7cXk

speaking of commit messages

https://www.twitter.com/gitlost

lol

always happy to see that account again

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.

Defensive Programming: C/D

Depends what it is… but I’d rather have good tests which catch any failures in most cases I would say.

iirc Go tries to push defensive programming as a default

iirc the main point of defensive programming is handle all the error conditions *first* then the code does what it’s supposed to

it’s smart imo, because happy path programming is terrible

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.

in short: trustno1

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

intant_update_user_permission = False
...
return intant_update_user_permission

how can you even c+p all that without noticing?

intant_karma.Said(“Knock You Out”)

hey my display name is relevant

> don't get fix

you never fix

who is screaming FIX at my code. i will never fix.

OMG we have a column in our DB called loyality_discount.

that’s right

I think about this Reddit thread a lot pic.twitter.com/w2QxYJ2kx6

— Sy Brand (@TartanLlama) August 20, 2020

when that thread was making the rounds I did a lot of mumbling “oh no”

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.

the comments thing has come to a head again after they refused to approve a pull request because of comments and *whitespace* in a component test! apparently the blank lines made him think it was a new Scenario even given the indentation.

so now we have a team meeting about it, but it'll be the usual steam-rollering, the alpha programmer forcing his views on everybody by dint of talking the most.

*sigh*

hopefully sanity prevailed. at least pretty much everybody thought that not approving changes because of comments was unreasonable.

That's a good sign!

I hope I never again have to work anywhere that doesn’t have strict autoformatting procedures in place.

even though i have very strong formatting preferences, i've found the best way to deal with the issue is to use an opinionated, non-configurable formatter as part of the build process. that way, everyone dislikes the formatting equally

We have flake8 for our Python stuff but not black, autoformatting would make me cwazy

my preferences > autoformatters > everyone else's preferences

i absolutely loathe black, but it's still better than the alternative on a shared repo, so i've learned to live with it.

is there a thread for just like general CS news and stuff? this seems pretty cool:

https://www.quantamagazine.org/computer-scientists-achieve-crown-jewel-of-cryptography-20201110/

my preferences > autoformatters > everyone else's preferences

cool link, brimstead!

Haha, yep, in our front-end JavaScript codebase Prettier is a godsend for being almost totally unconfigurable.

generate 84 files
commit to github
upload to server

repeat 40 times, changing one word each time.

(i have replaced step one with a sed script rather than manually editing the files but still...)