←back to thread

498 points azhenley | 1 comments | | HN request time: 0s | source
Show context
nixpulvis ◴[] No.45771961[source]
I try to keep deeper mutation to where it belongs, but I'll admit to shadowing variables pretty often.

If I have a `result` and I need to post-process it. I'm generally much happier doing `result = result.process()` rather than having something like `preresult`. Works nicely in cases where you end up moving it into a condition, or commenting it out to test an assumption while developing. If there's an obvious name for the intermediate result, I'll give it that, but I'm not over here naming things `result_without_processing`. You can read the code.

replies(5): >>45772035 #>>45772551 #>>45773468 #>>45775029 #>>45776914 #
MetaWhirledPeas ◴[] No.45775029[source]
> result.process()

What result? What process?

...says every person who has to read your code later.

replies(1): >>45776019 #
1. b_e_n_t_o_n ◴[] No.45776019[source]
I mean, it's probably pretty clear when you look at result's initial assignment...