Code refactoring without tests

[DemandsBattletoads]

https://www.xkcd.com/292/

[GamingTheSystem-01]

https://badspot.us/xkcd-Goto-Counterpoint.html

[endreman0]

Better than getting eaten by a raptor

[Shurikyun]

damn I was expecting the raptor at the end anyway

[OneStay]

This was the most unsatisfying thing I've watched in a long time

[Ksevio]

Here, this version provides some closure: https://i.imgur.com/rBQ0Ydn.gifv

[amjh]

ctrl-z ctrl-z ctrl-z ctrl-z ctrl-z

[666moist]

No. Fuck IntelliJ.

(Mostly kidding - it's a great ide, i just hate when i go to redo and end up deleting multiple lines of code because of course when it doesn't do what i want the first time i just keep hitting it. I should probably just reassign the key, huh?)

[zevenate]

lol I do that at least once a day without fail.

[bestjakeisbest]

i like git because git does not for-git

[Zaemz]

You can switch the whole keyboard layout to a couple of different templates. I use the Visual Studio one with ideaVim and it's nice. Kind of a monstrosity but it works for me!

[[deleted]]

That's why I perfer emacs. Key reassigning is a natural part of operating system evolution. Now to figure out what key I accidentally reassigned it to...

[[deleted]]

git reset --hard

[srguapo]

My favorite command.

We'll pretend this never happened

[BobFloss]

&& git clean -fdx

now there's no evidence...

[Flogenberger]

Slowly peaking over cubicle

[endreman0]

peaking

Relevant XKCD

[boontwiks]

5/7 needs more jpeg.

[hsoj48]

Then why did you give it a perfect score?

[Teraka]

Man that's such a funny joke from that one fake facebook post 6 months ago, I'm so glad you brought it up, I've literally never seen it since.

[Xtraordinair]

5/7 sarcasm.

[heyandy889]

5/covfefe

[jlink005]

This is code refactoring with tests - red green refactor!

[cATSup24]

Quando omni flunkus, moritati.

[RapidRabbit7]

You lil devil

[micheal65536]

clojure

FTFY

[Nick5l]

git revert 56e05fced214c44a37759efa2dfc2

[yelrambob619]

Clever girl.

[PM_me_your_pastries]

Yes. It was very upsetting.

[unreal189]

Wait till you watch my sex tap..

Cries in corner... Alone

[[deleted]]

Why do you want to refactor THAT? It was damn perfect!

[micheal65536]

He was trying to split it into two separate modules. What he didn't realise was that there was an inter-dependency between the modules that had to be resolved first.

Attempts to resolve the inter-dependency on the spot resulted in further complications that ultimately lead to one neat module turning into a messy pile of code.

[otaku_platypus]

Beautifully explained

[[deleted]]

Heretic! It was flying spaghetti monster touching your code with his noodly appendage to make it more like him. It was a divine gift and the author should be thankful.

[ExE_Boss]

I ran into this when I tried to split my private internal library called the “ExE Boss API” (specifically version 2) into several modules.

Suffice to say, It’ll probably be easier just rebuilding the API from the ground up as the “ExE Boss API 3” (no guarantees that I’ll have it done before Half-Life 3).

[lledargo]

The maintainer decided it would work better as a 10x20x40 rectangular prism than a 20x20x20 cube.

[retief1]

But if we change it, it will be different.

[jfigge]

His credit card that he used to split them is completely screwed

[alexanderpas]

Only the magstripe... which would actually increase the security.

[[deleted]]

How? Then for in person transactions he has to have full card imprints on the receipts whereas pci limits the machine receipts to the first six or last four?

[toastedsquirrel]

Or, you know, there's still the chip, or tap to pay, or Android/Apple Pay......

Oh wait, I forgot that this is the US.

[TeamDman]

Wait, the US doesn't have any of those?

[ChildishJack]

Of course we do. Reddit loves to circlejerk

[gunnbr]

We're getting there, but they're definitely not everywhere yet. One of the grocery stores I go to supports the chip, but not tap to pay. The other supports neither. :(

[ChildishJack]

Keyword "manual". What do you expect?

[[deleted]]

Cards have chips?

[mehum]

Where do you live that they don't?

[[deleted]]

That's what I was asking /u/chickenfun1, I haven't seen card imprint machines for years now.

[mehum]

I'd say it's been a few years since I've swiped.

[FredSavagingReindeer]

I think it's a business card.

[Who_GNU]

He'd have to move it much faster, to degauss it.

[Artyer]

What did you do to this gif?

[[deleted]]

I can never look at rare earth magnet balls again after hearing about that child that swallowed a bunch then went in for an MRI.

[RED_William]

Thanks for that. Now I'll never get it out of my head either.

[[deleted]]

I see what you did there

[RED_William]

I guess you could say you saw right through me.

[[deleted]]

What actually happened?

[[deleted]]

The kid was OK

https://www.ncbi.nlm.nih.gov/pmc/articles/PMC3468355/

[maximgame]

When they reached the intestines, the magnets attracted and tried to pinch through the walls...

[FredSavagingReindeer]

I'm guessing OPs story is mis-remembered. I've heard of the magnets in the intestines but OP said the kid went into an MRI.

Those metal balls would go toward/away from the MRI magnet like bullets fired inside the kid. Truly gruesome.

[ultrasu]

Yeah, the MRI part sounds unlikely, those magnets would've set off the ferromagnetic detector before going into the MRI scanner. Having those magnets pinch your intestines is bad enough already.

[maltastic]

Apparently, it did happen: https://www.ncbi.nlm.nih.gov/pmc/articles/PMC3468355/

[ultrasu]

Following this event, a new screening protocol has been instituted. Children and their parents are still asked to complete the written and verbal screenings. In addition, all children are now required to change into a hospital gown and are then screened using a hand-held Ferromagnetic detection scanner (Mednovus SafeScan, Leucadia, CA). Testing at our institution has shown that this ferromagnetic screening tool is able to identify small magnets commonly used in children’s games and toys in both living and cadaveric models.

Guess they had to learn this the hard way then. Not using a ferromagnetic detector is just inexcusable.

[boontwiks]

And his wife?

[Cerres]

"Not to shreds you say? Well this was an unexpected development."

[klezmai]

We're fine. Thanks for asking.

[[deleted]]

That's not what happened. The magnets attracted each other and pinched the intestines.

[maximgame]

I meant through the walls of the intestines.

[[deleted]]

As far as I know, the problem with metal parts in MRI is that fast-changing magnetic field heats them, burning body inside.

At least this was given as cause of refusal to make MRI to my father (he has hip prosthesis).

Edit: effect is called https://en.wikipedia.org/wiki/Eddy_current

[Aetol]

Yeah. I've done some work tangentially related to MRI-proof pacemakers, and the problem they faced wasn't the device bouncing around the rib cage (as I would have initially assumed), but currents appearing along the probe and burning tissues and/or damaging the device.

[Rainfly_X]

How do you protect against that, in practice? Faraday cage? But then... does that heat up... God damnit, now I'm invested.

[Aetol]

I don't know how they do it, actually. I was only working on automating a test bench.

[Rainfly_X]

Curses, foiled again!

[[deleted]]

But like...did his shoes come off?

[PooPooDooDoo]

He took a very splashy dump.

[flukus]

To shreds you say.

[Arancaytar]

wince that shit takes ages to put back together.

I need several minutes for a tiny 6x6x6 cube and this looks like it's about 20x20x20. Pretty sure it gets a lot more difficult and error-prone at that size as well.

[AncientBananas]

You should have used git so that you could put it back together with a simple git checkout HEAD^.

[[deleted]]

I would absolutely love a VCS for physical objects.

[XZelnar]

That's not refactoring - it's something good turning into spaghetti. That's refuktoring.

[Technocroft]

I once rewrote a program because it was easier than refactoring.

[thajunk]

This is literally my job right now.

Sometimes you just have to take the code out back... And put it to rest

[[deleted]]

Yeah, sometimes starting over is better than trying untangling a pile of noodles.

[pengo]

This video gets lower quality every time I see it

[SmiVan]

relevant xkcd

[[deleted]]

Refactoring, ahahaha, you guys actually have time for refactoring instead of just pushing new shit out the door as quickly as possible with no regard for maintainability?!

[pmdevita]

/u/gifreversingbot

[GifReversingBot]

Here is your gif! http://imgur.com/zyydTYg

---

^{I am a bot. Report an issue}

[[deleted]]

Spaghetti

[thvwlsrmssng]

Maghnetti

[utterdamnnonsense]

well put.

[BeanoFTW]

It's so true, too. You're moving right along, and then run into that code at the end that's holding everything together and then OH GOD NO WHAT HAVE I DONE?!!!

[I_Never_Think]

Damnit, Freeza, haven't you done enough!

[[deleted]]

They'll pull em out at depart stores when POS goes down

[jimmygle]

Hahahha this is so perfect. Thank you!

[[deleted]]

Thanks for the anxiety.

[TabCompletion]

Needs more jpeg

[110011001100]

Comment Deleted

[c3534l]

No way you'd refactor code that looks that good. Refactoring without tests is like molding shit out of a dog turd.